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

Add new warning status and result state #1912

Closed
CharliePoole opened this issue Nov 26, 2016 · 1 comment
Closed

Add new warning status and result state #1912

CharliePoole opened this issue Nov 26, 2016 · 1 comment

Comments

@CharliePoole
Copy link
Member

CharliePoole commented Nov 26, 2016

This issue is preliminary to all other work on the Warning-level Assertions Epic.

To do, while making sure all tests continue to pass:

  • Create a warning TestStatus
  • Create a warning ResultState
  • Ensure that all code that switches on either TestStatus or ResultState takes the new values into account.
  • Redefine the Ignored result state as a warning and change logic around ignored tests
  • Create and Test Assert.Warn
  • Create and Test Warn.If and Warn.Unless (same overloads as Assume.That)
  • Ensure that warnings are all saved in the <assertions> element, with status Warning.
@CharliePoole
Copy link
Member Author

Blocked while waiting for a decision on the issue of redefining ResultStatus.Ignore as a type of warning. The other issues under the Warning Assert epic all depend on this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants