Commit 5186f6e3 authored by Robert Speicher's avatar Robert Speicher

Merge branch 'default-scope-on-builds-page' into 'master'

Fix a description for default scope on builds

Should be "**All* builds (from this project)" for the {project,admin}
builds page without scope.

Fix a regression on !2243 (#4240).

See merge request !4118
parents 850234eb 18fdbf0a
...@@ -54,6 +54,7 @@ v 8.7.5 ...@@ -54,6 +54,7 @@ v 8.7.5
- Fix relative links in wiki pages. !4050 - Fix relative links in wiki pages. !4050
- Fix always showing build notification message when switching between merge requests !4086 - Fix always showing build notification message when switching between merge requests !4086
- Fix an issue when filtering merge requests with more than one label. !3886 - Fix an issue when filtering merge requests with more than one label. !3886
- Fix short note for the default scope on build page (Takuya Noguchi)
v 8.7.4 v 8.7.4
- Links for Redmine issue references are generated correctly again !4048 (Benedikt Huss) - Links for Redmine issue references are generated correctly again !4048 (Benedikt Huss)
......
...@@ -20,7 +20,7 @@ ...@@ -20,7 +20,7 @@
= link_to 'Cancel all', cancel_all_admin_builds_path, data: { confirm: 'Are you sure?' }, class: 'btn btn-danger', method: :post = link_to 'Cancel all', cancel_all_admin_builds_path, data: { confirm: 'Are you sure?' }, class: 'btn btn-danger', method: :post
.row-content-block.second-block .row-content-block.second-block
#{(@scope || 'running').capitalize} builds #{(@scope || 'all').capitalize} builds
%ul.content-list %ul.content-list
- if @builds.blank? - if @builds.blank?
......
...@@ -36,7 +36,7 @@ ...@@ -36,7 +36,7 @@
%span CI Lint %span CI Lint
.row-content-block .row-content-block
#{(@scope || 'running').capitalize} builds from this project #{(@scope || 'all').capitalize} builds from this project
%ul.content-list %ul.content-list
- if @builds.blank? - if @builds.blank?
......
...@@ -19,6 +19,7 @@ describe 'Admin Builds' do ...@@ -19,6 +19,7 @@ describe 'Admin Builds' do
visit admin_builds_path visit admin_builds_path
expect(page).to have_selector('.nav-links li.active', text: 'All') expect(page).to have_selector('.nav-links li.active', text: 'All')
expect(page).to have_selector('.row-content-block', text: 'All builds')
expect(page.all('.build-link').size).to eq(4) expect(page.all('.build-link').size).to eq(4)
expect(page).to have_link 'Cancel all' expect(page).to have_link 'Cancel all'
end end
......
...@@ -43,6 +43,7 @@ describe "Builds" do ...@@ -43,6 +43,7 @@ describe "Builds" do
end end
it { expect(page).to have_selector('.nav-links li.active', text: 'All') } it { expect(page).to have_selector('.nav-links li.active', text: 'All') }
it { expect(page).to have_selector('.row-content-block', text: 'All builds from this project') }
it { expect(page).to have_content @build.short_sha } it { expect(page).to have_content @build.short_sha }
it { expect(page).to have_content @build.ref } it { expect(page).to have_content @build.ref }
it { expect(page).to have_content @build.name } it { expect(page).to have_content @build.name }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment