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.

Sign up for GitHub

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

Jump to bottom

module name is not shown #3

Closed
smith opened this issue Oct 25, 2009 · 3 comments
Closed

module name is not shown #3

smith opened this issue Oct 25, 2009 · 3 comments

Comments

@smith
Copy link

smith commented Oct 25, 2009 •
edited by Krinkle
Loading

When modules are defined, their names do not appear (for me) next to the test names in the output.

The config object is defined on line 353, and config is assigned to in QUnit.init config.currentModule is set in the module function and accessed in the test function. It looks like a different config object is being referenced in each of the different functions.
@zhaoz
Copy link

zhaoz commented Oct 29, 2009

This commit seems to have fixed the issue, although it is untested with asyncTest stuff (which is when this issue was introduced).

http://github.com/zhaoz/qunit/commit/13eec8f1e6dbbde59aa07ee74dd25ca03ba80e87

@smith
Copy link
Author

smith commented Oct 29, 2009

This does work for me, though I haven't tested it with and asyncTests either.

@jeresig
Copy link
Contributor

jeresig commented Nov 5, 2009

I merged in zhaoz's commit - hopefully this fixes the problem.

@cowboy cowboy mentioned this issue May 7, 2012
Closed
vivin added a commit to vivin/qunit that referenced this issue Aug 16, 2012
@vivin
issue qunitjs#307
0508b89 
Approach qunitjs#3: Public "enum" used for assertion type. Removed some of the
tests because it seemed like they were more trouble than they were
worth. It does have a few tests that checks for the assertion type.
@vivin vivin mentioned this issue Aug 16, 2012
Closed
@Krinkle Krinkle mentioned this issue Mar 8, 2013
Closed
@smcclure15 smcclure15 mentioned this issue Dec 10, 2020
Merged
mgol pushed a commit to mgol/qunit that referenced this issue Mar 24, 2021
@prathe
QUnit: testrunner.js: Ensures equality of types (String, Boolean, Num…
be7f675 
…ber) declared with the 'new' prefix.

See comments qunitjs#3, qunitjs#4 and qunitjs#5 on http://philrathe.com/articles/equiv
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jeresig @smith @zhaoz