search

ci-reporter / ci_reporter_minitest

Connects CI::Reporter to Minitest
MIT License
6 stars 7 forks source link

Replace result.class with result.klass #8

Open hakanai opened 1 year ago

hakanai commented 1 year ago

result.class returns the class of the result object, but what you really want is the name of the test class, which is in result.klass.

Currently, all test reports end up in the "Minitest::Result" suite.

I guess there should be a question about whether this breaks compatibility with some earlier version of minitest, whether that matters, and what to do if it does.

TastyPi commented 1 year ago

Fixes #9

@shepmaster can we get this merged please?