Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change verification chains #115

Merged
merged 16 commits into from
Apr 25, 2022
Merged

Change verification chains #115

merged 16 commits into from
Apr 25, 2022

Conversation

bitPogo
Copy link
Owner

@bitPogo bitPogo commented Apr 25, 2022

📌 References

What does the PR achieve/Which part improves the PR?

This refactors the Assertion-/VerificationChains with the goal to decouple it from the current Expectation methods.
It also decouples both chains from Proxies.

🤔 DOD Checklist

  • My code passes the lint checks.
  • My changes are covered with proper tests.
  • All tests are pass.
  • My changes update the documentation.
  • My changes are reflected in the changelog.

@bitPogo bitPogo added this to the 0.2.0 milestone Apr 25, 2022
@bitPogo bitPogo self-assigned this Apr 25, 2022
@github-actions github-actions bot added documentation Improvements or additions to documentation examples kmock kmock-processor labels Apr 25, 2022
@github-actions
Copy link

github-actions bot commented Apr 25, 2022

Fails
🚫 This Pull-Request is way to big, please slice it into smaller pull-requests.
Warnings
⚠️ Changes should be reflected in the CHANGELOG.adoc

Generated by 🚫 Danger Kotlin against 9c5ce4c

@bitPogo bitPogo force-pushed the feature/change-verification-chains branch from 04fa544 to 9c5ce4c Compare April 25, 2022 22:57
@bitPogo bitPogo merged commit 1ed7cd5 into main Apr 25, 2022
@bitPogo bitPogo deleted the feature/change-verification-chains branch April 25, 2022 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples kmock kmock-processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant