qunitjs / js-reporters

📋 Common Reporter Interface (CRI) for JavaScript testing frameworks.
MIT License
60 stars 18 forks source link

Fix Jasmine Adapter #63

Closed flore77 closed 8 years ago

flore77 commented 8 years ago

After the new testing we need to fix some issues in the adapter.

Issues

[1] Objects are emitted are then mutated. Emitted and mutated.

[2] Global tests have as suite name Jasmine__TopLevel__Suite, they should have undefined.

[3] Skipped tests as a Number for their runtime, it must be undefined.

[4] Replace and test Jasmine specific errors.

[5] Nested suite execution order.

Solutions

jzaefferer commented 8 years ago

On [2], removing the suite name where needed should be easy enough, right?

The rest looks good.

flore77 commented 8 years ago

On [2], removing the suite name where needed should be easy enough, right?

Yes, but I didn't know what to write :laughing: the solution will come out from the implementation, then I will fill in with a link to the line that takes care of that.