You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All unit tests of exercise "all-your-base" that expect an ArgumentException should expect it with Assert.ThrowsAny(Of ArgumentException) instead of Assert.Throws(Of ArgumentException) which then would allow more accurate sub-exceptions like ArgumentOutOfRangeException to be thrown.
The text was updated successfully, but these errors were encountered:
That is certainly a valid option. Another would be to change the expected exception to the "proper" exception (e.g. ArgumentOutOfRangeException). I'd be in favor of the latter option, as it better teaches the student which exception is appropriate in which situation. Would you be willing to submit a PR?
All unit tests of exercise "all-your-base" that expect an
ArgumentException
should expect it withAssert.ThrowsAny(Of ArgumentException)
instead ofAssert.Throws(Of ArgumentException)
which then would allow more accurate sub-exceptions likeArgumentOutOfRangeException
to be thrown.The text was updated successfully, but these errors were encountered: