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
TestCase::new() and TestCase::Skip do not make a lot of sense when creating the two possible variants of a Test case. Furthermore, does it make sense to keep a builder pattern for non-skipped test cases? If the answer is yes, does it make sense to simply return Skip cases even when calling builder functions, effectively making the call a no-op? Should we panic!() to indicate that the user might be doing something unexpected or silently ignore that as we currently do?
The text was updated successfully, but these errors were encountered:
TestCase::new()
andTestCase::Skip
do not make a lot of sense when creating the two possible variants of a Test case. Furthermore, does it make sense to keep a builder pattern for non-skipped test cases? If the answer is yes, does it make sense to simply returnSkip
cases even when calling builder functions, effectively making the call a no-op? Should wepanic!()
to indicate that the user might be doing something unexpected or silently ignore that as we currently do?The text was updated successfully, but these errors were encountered: