RuntimeException after minimal source change about scalatest HOT 5 CLOSED

Hi Tobias,

Sorry I only today noticed this issue. The problem is actually that you put your assertion in a "scope" not a "test". This is unfortunately not a type error, and unfortunately the error message you get in the exception doesn't seem to give much clue. I think what I'll do for the next release is try and give a better hint in the error message. I'll also look at ways to make this kind of error a compile-time error.

Anyway, in a FreeSpec "scopes" are introduced with a dash (-) and "tests" are introduced with "in". Scopes are a way to group tests. Although the bodies of the tests are registered at construction time for later execution, the bodies of scopes are evaluated at construction time. That means in this case your assertions execute at construction time, and when they fail, it means ScalaTest can't instantiate the Suite and so the run aborts. To fix it you can just change the "-" to an "in".

from scalatest.

Thanks for you explanation. Unfortunately, I'm not quite sure I understand it. You said, the assert is not in a "test", but actually, it is surrounded by a "string" in { .. } construct. Also, the assert will not throw an exception in the case the code is right, but still I see this issue.

Why I can get rid of the exception, when I delete the toString() override in the class under test?

Also, if I remove the third of these test cases (constructed with the mergeTest function, so my understanding is, it is a "test") all other tests run well. Even if I now modify the class under test so that it produces incorrect results, I still get a proper test run outlining the successful ran and the failed tests.

from scalatest.

Sorry, I didn't notice the "in". This one is back in action.

from scalatest.

Ah, duplicate test name exception was being thrown!

from scalatest.

Thank you Bill for taking the time (live @ ScalaDays) to look into this!

(The strange three cases from the original issue (overridden toString, removed feature, removed test case) all influenced the titles of the tests and thus implicitly avoided the duplication.)

from scalatest.