-
Notifications
You must be signed in to change notification settings - Fork 652
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
Warning stops test execution #43
Comments
I don't think the warn is anything to do with that, because that's just being emitted when I test plusOrMinus in |
Ah the issue is just that there is an ignored test in ConfigTest. It's there just to make sure the dsl doesn't change and for a visible check. I'm not sure how to check that an ignored test didn't actually run. |
How does JUnit gets the number of tests? Maybe ignored tests can be subtracted there. |
That's certainly easy enough to do, but then it doesn't appear in the list. On 28 June 2016 at 10:59, Christian Helmbold notifications@github.com
|
Agreed. I'll figure it out while you're doing the config stuff. On 28 June 2016 at 11:40, Christian Helmbold notifications@github.com
|
Sorted |
fix the following bug - the test below was passing: ``` "fail to detect count mismatch" { sequenceOf("apple", "apple", "orange") should containAllInAnyOrder( sequenceOf("apple", "orange", "orange") ) } ``` * also fix two tests named "for subset, same count with nulls" - they were not validating correct behavior. * also provide more details about what exactly did not match, such as: ``` Sequence should contain the values of [1, 1, 2] in any order, but was [1, 2, 2]. Count Mismatches: For 1: expected count: <2>, but was: <1> For 2: expected count: <1>, but was: <2> ```
If I run all tests of the KotlinTest project, the execution (in IntelliJ 2016.1.3) stops with 109 of 110 tests. It appears the message:
The text was updated successfully, but these errors were encountered: