diff options
| author | Filipa Lacerda <filipa@gitlab.com> | 2018-10-23 11:01:54 +0100 |
|---|---|---|
| committer | Filipa Lacerda <filipa@gitlab.com> | 2018-10-23 16:02:36 +0100 |
| commit | 20d3c6eba73a0170bc1ab53afb5a198879ed156a (patch) | |
| tree | 7ce2e9162d15f48f2b857703d3c4c452ad95166a /spec | |
| parent | 1ad3ce921fdba0533da0d4bd826e5ff6abdc78b2 (diff) | |
| download | gitlab-ce-20d3c6eba73a0170bc1ab53afb5a198879ed156a.tar.gz | |
Removes escape from job item53013-duplicate-escape
Tooltips aren't using v-html anymore thus causing
escape not to be needed
Diffstat (limited to 'spec')
| -rw-r--r-- | spec/features/projects/jobs_spec.rb | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/spec/features/projects/jobs_spec.rb b/spec/features/projects/jobs_spec.rb index 1ea8a640e17..c3902ecdd17 100644 --- a/spec/features/projects/jobs_spec.rb +++ b/spec/features/projects/jobs_spec.rb @@ -151,9 +151,8 @@ describe 'Jobs', :clean_gitlab_redis_shared_state do end it 'renders escaped tooltip name' do - page.within('aside.right-sidebar') do - expect(find('.active.build-job a')['data-original-title']).to eq('<img src=x onerror=alert(document.domain)> - passed') - end + page.find('.active.build-job a').hover + expect(page).to have_content('<img src=x onerror=alert(document.domain)> - passed') end end |