qunitjs / js-reporters

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

Automated test infrastructure #23

Closed fcarstens closed 9 years ago

fcarstens commented 9 years ago

This PR tries to set up a better test infrastructure and fixes a lot of bugs on the way. I accidentally included the qunit_umd_issue and issue_19 branches, hope that's okay. :smiley:

fcarstens commented 9 years ago

I adressed all issues mentioned above :smiley:

jzaefferer commented 9 years ago

I've now reviewed TestReporter properly. Let me know when you've updated this PR.

fcarstens commented 9 years ago

All fixed.

jzaefferer commented 9 years ago

This looks good to land - can you squash the whole thing into one commit with a proper commit message? Speaking of, we could add commitplease.

fcarstens commented 9 years ago

All done! :smiley:

jzaefferer commented 9 years ago

When merging a squashed commit like that, use "Closes #xxx" to reference the PR and automatically close it.