Skip to content
This repository was archived by the owner on Jul 6, 2021. It is now read-only.

Global and module hooks#76

Closed
leobalter wants to merge 7 commits into
qunitjs:masterfrom
leobalter:69-global-before-after-each
Closed

Global and module hooks#76
leobalter wants to merge 7 commits into
qunitjs:masterfrom
leobalter:69-global-before-after-each

Conversation

@leobalter

Copy link
Copy Markdown
Member

Won't fix #69 anymore
Fixes #70

Squash this on the module hooks commit, plz
@leobalter leobalter added this to the 1.16 milestone Sep 11, 2014
Comment thread entries/QUnit.config.xml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about "This global hook, if set, runs after all test blocks and their respective module's afterEach hook."

@leobalter

Copy link
Copy Markdown
Member Author

review round 2. start.

@jzaefferer

Copy link
Copy Markdown
Member

We still need an example for using the global hooks.

@leobalter

Copy link
Copy Markdown
Member Author

done

@jzaefferer

Copy link
Copy Markdown
Member

Looks good. I like the examples you're coming up with :-) Will also land with 1.16.

@jzaefferer

Copy link
Copy Markdown
Member

Since we're not landing the global hooks, can you remove/revert the QUnit.config changes? We still want the rest of this PR.

@leobalter

Copy link
Copy Markdown
Member Author

Done.

@jzaefferer

Copy link
Copy Markdown
Member

Looks good.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Document new setup/teardown renamed to beforeEach/afterEach Document universal tests beforeEach/afterEach

3 participants