From 81bff65fa8342b4e5d995825b1bb3266ff71d6ce Mon Sep 17 00:00:00 2001 From: Filipa Lacerda Date: Thu, 4 Oct 2018 09:27:56 +0100 Subject: [PATCH] Adds MR reference in job sidebar In the refactor the `!` reference was missed This commit adds it back and adds coverage for it. --- app/assets/javascripts/jobs/components/commit_block.vue | 2 +- spec/javascripts/jobs/components/commit_block_spec.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/jobs/components/commit_block.vue b/app/assets/javascripts/jobs/components/commit_block.vue index 39a4ff159e2..4b1788a1c16 100644 --- a/app/assets/javascripts/jobs/components/commit_block.vue +++ b/app/assets/javascripts/jobs/components/commit_block.vue @@ -46,7 +46,7 @@ v-if="mergeRequest" :href="mergeRequest.path" class="js-link-commit link-commit" - >{{ mergeRequest.iid }} + >!{{ mergeRequest.iid }}

diff --git a/spec/javascripts/jobs/components/commit_block_spec.js b/spec/javascripts/jobs/components/commit_block_spec.js index 61ee993f46a..0bcc4ff940f 100644 --- a/spec/javascripts/jobs/components/commit_block_spec.js +++ b/spec/javascripts/jobs/components/commit_block_spec.js @@ -56,7 +56,7 @@ describe('Commit block', () => { props.mergeRequest.path, ); expect(vm.$el.querySelector('.js-link-commit').textContent.trim()).toEqual( - props.mergeRequest.iid, + `!${props.mergeRequest.iid}`, ); }); });