-
-
Notifications
You must be signed in to change notification settings - Fork 16
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 unit test assertions #357
Comments
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
I wish to take up this issue if it's still current. |
Do not bother with refactoring the |
Hello @ljdarj!! I just wanted to touch base with you and see if you still plan on working with this issue. I noticed that no pull request has been created for this and it seems like there has not been any activity for about 3 weeks. Thanks for any updates. 😀 |
Just a quick side note @ljdarj. If you do still plan on doing this, that is completely fine. I just needed to know which release to slot this in for. We can release it at a later time. |
Sorry I wasn't here, life got in the way. I intend to do it by the first week of January at the latest. |
Not a problem at all. 😀 I understand how life can get in the way. Especially as we approach the end of the year and the holidays. Thanks for updating me!! |
@ljdarj Happy New Year!! I hope you're doing well. I am following up on this issue. If you're not able to work on it, that's completely understandable. Please let myself or @CalvinWilkinson know and we can take care of it. Thanks and have a great rest of your week! |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale due to the lack of activity for 60 days. The issue will be closed after 7 days if no further activity occurs. Thank you for your contributions. |
This stale issue has been closed due to a lack of activity. |
Complete The Item Below
Description
Refactor all of the unit test assertions to use the FluentAssertions library.
This will greatly cleanup the unit testing side of things and make the creation of unit tests easier.
Acceptance Criteria
- 55 linesEnsureThatTests.cs
SystemDisplayTests.cs
- 359 linesInvalidInputExceptionTests.cs
- 50 linesNoKeyboardExceptionTests.cs
- 50 linesNoMouseExceptionTests.cs
- 50 linesToDo Items
Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels
🐛bug
🧨breaking changes
✨new feature
workflow
🗒️documentation/code
📝documentation/product
Priority Type Labels
low priority
medium priority
high priority
Code of Conduct
The text was updated successfully, but these errors were encountered: