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

refactor: remove Fluent Assertions from unit tests #33

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SebastienDegodez
Copy link
Member

@SebastienDegodez SebastienDegodez commented Jan 18, 2025

Pull Request

Proposed Changes

  • Updated all test methods to use Xunit's Assert class
  • Removed package dependency

resolves #32

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run dotnet test and ensure you have test coverage for the lines you are introducing
  • run dotnet husky run and fix any issues that you have introduced

Reviewer

  • Label as either feature, fix, documentation, enhancement, maintenance or breaking

- Updated all test methods to use Xunit's Assert class
- Removed package dependency
resolves #32

Signed-off-by: SebastienDegodez <[email protected]>
@SebastienDegodez SebastienDegodez marked this pull request as ready for review January 18, 2025 16:12
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.

Pin or remove fluentassertions
1 participant