From a7028df82837d8d0e3ef8644c17b90da7a1bb085 Mon Sep 17 00:00:00 2001 From: kialam Date: Wed, 17 Oct 2018 16:40:01 -0400 Subject: [PATCH] Fix one failing unit test. --- .../client/features/output/details.component.js | 4 ++++ .../components/job-details-split-jobs.unit.js | 16 ++++++++-------- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/awx/ui/client/features/output/details.component.js b/awx/ui/client/features/output/details.component.js index 93f405d2a0..5d0ef52338 100644 --- a/awx/ui/client/features/output/details.component.js +++ b/awx/ui/client/features/output/details.component.js @@ -139,6 +139,10 @@ function getSliceJobDetails () { const number = resource.model.get('job_slice_number'); + if (!number) { + return null; + } + const label = strings.get('labels.SLICE_JOB'); const offset = `${number}/${count}`; const tooltip = strings.get('tooltips.SLICE_JOB_DETAILS'); diff --git a/awx/ui/test/unit/components/job-details-split-jobs.unit.js b/awx/ui/test/unit/components/job-details-split-jobs.unit.js index d6f7b0f23a..39099969aa 100644 --- a/awx/ui/test/unit/components/job-details-split-jobs.unit.js +++ b/awx/ui/test/unit/components/job-details-split-jobs.unit.js @@ -125,10 +125,10 @@ describe('View: Job Details', () => { job_slice_count: 2, job_slice_number: 2, labels: { - SPLIT_JOB: null + SLICE_JOB: null }, tooltips: { - SPLIT_JOB_DETAILS: null + SLICE_JOB_DETAILS: null } }; JobDetails.$onInit(); @@ -140,10 +140,10 @@ describe('View: Job Details', () => { job_slice_count: null, job_slice_number: 2, labels: { - SPLIT_JOB: 'foo' + SLICE_JOB: 'foo' }, tooltips: { - SPLIT_JOB_DETAILS: 'bar' + SLICE_JOB_DETAILS: 'bar' } }; JobDetails.$onInit(); @@ -155,10 +155,10 @@ describe('View: Job Details', () => { job_slice_count: 2, job_slice_number: null, labels: { - SPLIT_JOB: 'foo' + SLICE_JOB: 'foo' }, tooltips: { - SPLIT_JOB_DETAILS: 'bar' + SLICE_JOB_DETAILS: 'bar' } }; JobDetails.$onInit(); @@ -170,10 +170,10 @@ describe('View: Job Details', () => { job_slice_count: 1, job_slice_number: null, labels: { - SPLIT_JOB: 'foo' + SLICE_JOB: 'foo' }, tooltips: { - SPLIT_JOB_DETAILS: 'bar' + SLICE_JOB_DETAILS: 'bar' } }; JobDetails.$onInit();