diff options
3 files changed, 8 insertions, 5 deletions
diff --git a/app/assets/javascripts/commit/pipelines/pipelines_bundle.js b/app/assets/javascripts/commit/pipelines/pipelines_bundle.js index 54b1ab9b449..687f09882a7 100644 --- a/app/assets/javascripts/commit/pipelines/pipelines_bundle.js +++ b/app/assets/javascripts/commit/pipelines/pipelines_bundle.js @@ -24,7 +24,9 @@ document.addEventListener('DOMContentLoaded', () => { if (event.detail.pipelines && event.detail.pipelines.count && event.detail.pipelines.count.all) { - document.querySelector('.js-pipelines-mr-count').textContent = event.detail.pipelines.count.all; + const badge = document.querySelector('.js-pipelines-mr-count'); + + badge.textContent = event.detail.pipelines.count.all; } }); diff --git a/app/assets/javascripts/commit/pipelines/pipelines_table.vue b/app/assets/javascripts/commit/pipelines/pipelines_table.vue index d6383a585ec..dd751ec97a8 100644 --- a/app/assets/javascripts/commit/pipelines/pipelines_table.vue +++ b/app/assets/javascripts/commit/pipelines/pipelines_table.vue @@ -63,7 +63,7 @@ }); // notifiy to update the count in tabs - if (this.$el.parentElement !== null) { + if (this.$el.parentElement) { this.$el.parentElement.dispatchEvent(updatePipelinesEvent); } }); diff --git a/spec/javascripts/commit/pipelines/pipelines_spec.js b/spec/javascripts/commit/pipelines/pipelines_spec.js index 8aba8b9e751..a34cadec0ab 100644 --- a/spec/javascripts/commit/pipelines/pipelines_spec.js +++ b/spec/javascripts/commit/pipelines/pipelines_spec.js @@ -86,7 +86,7 @@ describe('Pipelines table in Commits and Merge requests', () => { }); }); - describe('with a parent element', () => { + describe('pipeline badge counts', () => { const pipelinesResponse = (request, next) => { next(request.respondWith(JSON.stringify([pipeline]), { status: 200, @@ -102,10 +102,11 @@ describe('Pipelines table in Commits and Merge requests', () => { this.component.$destroy(); }); - it('should dispatch event', (done) => { + it('should receive update-pipelines-count event', (done) => { const element = document.createElement('div'); + document.body.appendChild(element); - $(element).on('update-pipelines-count', (event) => { + element.addEventListener('update-pipelines-count', (event) => { expect(event.detail.pipelines).toEqual([pipeline]); done(); }); |