Skip to content

Commit

Permalink
Tests updated to reflect namespace on links
Browse files Browse the repository at this point in the history
  • Loading branch information
philrenaud committed May 13, 2022
1 parent 93b9c39 commit 1efdb97
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 6 deletions.
2 changes: 1 addition & 1 deletion ui/tests/acceptance/allocation-detail-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,7 @@ module('Acceptance | allocation detail (preemptions)', function (hooks) {
await Allocation.preempter.visitJob();
assert.equal(
currentURL(),
`/jobs/${preempterJob.id}`,
`/jobs/${preempterJob.id}@default`,
'Clicking the preempter job link navigates to the preempter job page'
);

Expand Down
4 changes: 2 additions & 2 deletions ui/tests/acceptance/jobs-list-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ module('Acceptance | jobs list', function (hooks) {

assert.equal(jobRow.name, job.name, 'Name');
assert.notOk(jobRow.hasNamespace);
assert.equal(jobRow.link, `/ui/jobs/${job.id}`, 'Detail Link');
assert.equal(jobRow.link, `/ui/jobs/${job.id}@default`, 'Detail Link');
assert.equal(jobRow.status, job.status, 'Status');
assert.equal(jobRow.type, typeForJob(job), 'Type');
assert.equal(jobRow.priority, job.priority, 'Priority');
Expand All @@ -78,7 +78,7 @@ module('Acceptance | jobs list', function (hooks) {
await JobsList.visit();
await JobsList.jobs.objectAt(0).clickName();

assert.equal(currentURL(), `/jobs/${job.id}`);
assert.equal(currentURL(), `/jobs/${job.id}@default`);
});

test('the new job button transitions to the new job page', async function (assert) {
Expand Down
6 changes: 5 additions & 1 deletion ui/tests/acceptance/regions-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,11 @@ module('Acceptance | regions (only one)', function (hooks) {

const jobId = JobsList.jobs.objectAt(0).id;
await JobsList.jobs.objectAt(0).clickRow();
assert.equal(currentURL(), `/jobs/${jobId}`, 'No region query param');
assert.equal(
currentURL(),
`/jobs/${jobId}@default`,
'No region query param'
);

await ClientsList.visit();
assert.equal(currentURL(), '/clients', 'No region query param');
Expand Down
2 changes: 1 addition & 1 deletion ui/tests/acceptance/task-detail-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ module('Acceptance | task detail', function (hooks) {
await Layout.breadcrumbFor('jobs.job.index').visit();
assert.equal(
currentURL(),
`/jobs/${job.id}`,
`/jobs/${job.id}@default`,
'Job breadcrumb links correctly'
);

Expand Down
2 changes: 1 addition & 1 deletion ui/tests/acceptance/topology-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ module('Acceptance | topology', function (hooks) {
await reset();

await Topology.allocInfoPanel.visitJob();
assert.equal(currentURL(), `/jobs/${job.id}`);
assert.equal(currentURL(), `/jobs/${job.id}@default`);

await reset();

Expand Down

0 comments on commit 1efdb97

Please sign in to comment.