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

Remove moq #3018

Merged
merged 4 commits into from
Aug 11, 2023
Merged

Remove moq #3018

merged 4 commits into from
Aug 11, 2023

Conversation

NPadrutt
Copy link
Contributor

@NPadrutt NPadrutt commented Aug 11, 2023

Issue: #

PR Type

What kind of change does this PR introduce?

  • Refactoring (no functional changes, no api changes)

What is the current behavior?

We started moving to NSubstitute a long time ago. Moq was left in one class.

What is the new behavior?

NSubstitute is the only mocking library left.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code on Windows
  • Tested code on Android
  • Tested code on iOS
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Contains NO breaking changes

Other information

@NPadrutt NPadrutt added this pull request to the merge queue Aug 11, 2023
Merged via the queue into MoneyFox:master with commit 27226e9 Aug 11, 2023
3 checks passed
@NPadrutt NPadrutt deleted the remove-moq branch August 11, 2023 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant