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

Enable Testing with Ember Exam #7963

Merged
merged 4 commits into from
Jul 15, 2024
Merged

Conversation

jrjohnson
Copy link
Member

Splits the tests into chunks and launches a bunch of browsers to run
them. I've defaulted this to 8 for running tests on a local machine, but
only 3 for github where there are less cores available.

Splits the tests into chunks and launches a bunch of browsers to run
them. I've defaulted this to 8 for running tests on a local machine, but
only 3 for github where there are less cores available.
These oks are not testing anything, we wanted equal. For some reason
this broke in ember-exam which I guess is a good thing.
Without this we need to define how many tests to split apart, adding
this is needed to evenly split tests.
This makes it possible to reply the test order locally and will be very
useful if we have any flaky tests.
@jrjohnson jrjohnson marked this pull request as ready for review July 14, 2024 06:07
@@ -74,7 +74,7 @@ module('Integration | Component | learnergroup-selection-cohort-manager', functi
assert.expect(2);
this.set('filter', '');
this.set('remove', (learnerGroup) => {
assert.ok(this.secondLevelLearnerGroup1, learnerGroup);
assert.deepEqual(this.secondLevelLearnerGroup1, learnerGroup);
Copy link
Member

Choose a reason for hiding this comment

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

nice.

@stopfstedt stopfstedt merged commit 6f61add into ilios:master Jul 15, 2024
35 checks passed
@jrjohnson jrjohnson deleted the exam-split branch July 15, 2024 17:48
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