search

sbt / sbt-jupiter-interface

Implementation of sbt's test interface for JUnit 5's Jupiter module
Apache License 2.0
33 stars 19 forks source link

Missing class names in test reports #125

Open jodersky opened 1 week ago

jodersky commented 1 week ago

After merging #87, class names are no longer part of reported test names.

E.g. instead of printing foo.Bar#testMethod(), now the reporter only prints #testMethod()

I narrowed down this change in behavior to https://github.com/sbt/sbt-jupiter-interface/pull/87/commits/7adbdd0ac272b26fa2f7219d708e51983d3110ce, which removes the need to skip the first element of a test identifier as is still done (https://github.com/sbt/sbt-jupiter-interface/blob/main/src/library/src/main/java/com/github/sbt/junit/jupiter/internal/Configuration.java#L321)

Strangely however, the tests were updated to exclude the classnames. https://github.com/sbt/sbt-jupiter-interface/pull/87/commits/f14d04cabdc3ee1a3dc95204d32fd29a00b8d943

jodersky commented 1 week ago

If I remove the skip(1), the behavior seems correct again, however the formatting then breaks for tests with an @Suite annotation