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

Group tests #936

Merged
merged 3 commits into from
Oct 26, 2015
Merged

Group tests #936

merged 3 commits into from
Oct 26, 2015

Conversation

jrjohnson
Copy link
Member

This is my response to our failing tests. Hopefully it is temporary and phantom js 2 will become more stable. I created 2 test groups (itga, itgb, itgc) - stands for Ilios Test Group - and placed our tests in one of those groups. Even if tests fail they should be easier and faster to restart.

The grouping is done by appending the test group name to the test description and then using the testem filter option to only run those that match the group name.

I added a grep test as well which checks to see that the test-groups helper is imported into each test file. This is the best I could come up with for a way to check that each set of tests was in at least one group.

Tests are placed into group a.b.c so they can be split up and run in
smaller batches.  Hopefully this will make our CI builds more reliable
and quick.
Since our tests are in groups now we can’t send test coverage anymore.

Combined the unit tests in group A with the scss lint tests since that
will all run fast.
Use grep to search all of the test files and discover any tests that
did not import the test-groups helper.
@thecoolestguy thecoolestguy self-assigned this Oct 26, 2015
thecoolestguy added a commit that referenced this pull request Oct 26, 2015
@thecoolestguy thecoolestguy merged commit 7df5c3b into ilios:master Oct 26, 2015
@jrjohnson jrjohnson deleted the group-tests branch November 10, 2015 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants