Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QUnit.done should provide aggregated results #472

Closed
jzaefferer opened this issue Sep 26, 2013 · 8 comments
Closed

QUnit.done should provide aggregated results #472

jzaefferer opened this issue Sep 26, 2013 · 8 comments

Comments

@jzaefferer
Copy link
Member

Similar to #351

Any test runner that doesn't need live updating shouldn't have to aggregate results manually.

@JamesMGreene
Copy link
Member

How granular/verbose are you thinking we should make it?

@Krinkle
Copy link
Member

Krinkle commented Sep 30, 2013

See #405 (comment).

@jzaefferer
Copy link
Member Author

@JamesMGreene @leobalter same here, pre-2.0.

@JamesMGreene
Copy link
Member

I would again ask:

How granular/verbose are you thinking we should make it?

For example, unlike #351, I would expect that we would not include the assertions on the done event details... or would we? What would our data object look like?

Happy to implement if we can get the data details hammered out.

@jzaefferer jzaefferer modified the milestones: pre-2.0, JS Reporter Sep 13, 2014
@jzaefferer
Copy link
Member Author

Basic structure could look like this:

QUnit.on( "runEnd", function( details ) {
    details.tests.forEach(function( test ) {
        // Access test.name, test.moduleName, test.assertions
        // assertions is the same as in testDone
    });
});

@jzaefferer
Copy link
Member Author

Here's a usage example that could be simplified by implementing this ticket: https://github.com/axemclion/grunt-saucelabs#test-result-details-with-qunit

@leobalter
Copy link
Member

@jzaefferer I'll try to address this issue on #882

@leobalter
Copy link
Member

similarly to #351, I believe this is a wontfix for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants