diff options
Diffstat (limited to 'spec/features/projects/pages_spec.rb')
-rw-r--r-- | spec/features/projects/pages_spec.rb | 230 |
1 files changed, 115 insertions, 115 deletions
diff --git a/spec/features/projects/pages_spec.rb b/spec/features/projects/pages_spec.rb index f564ae34f11..30670acec1b 100644 --- a/spec/features/projects/pages_spec.rb +++ b/spec/features/projects/pages_spec.rb @@ -1,6 +1,6 @@ -require 'spec_helper' +require "spec_helper" -describe 'Pages' do +describe "Pages" do let(:project) { create(:project) } let(:user) { create(:user) } let(:role) { :maintainer } @@ -13,157 +13,157 @@ describe 'Pages' do sign_in(user) end - context 'when user is the owner' do + context "when user is the owner" do before do project.namespace.update(owner: user) end - context 'when pages deployed' do + context "when pages deployed" do before do allow_any_instance_of(Project).to receive(:pages_deployed?) { true } end - it 'renders Access pages' do + it "renders Access pages" do visit project_pages_path(project) - expect(page).to have_content('Access pages') + expect(page).to have_content("Access pages") end - context 'when support for external domains is disabled' do - it 'renders message that support is disabled' do + context "when support for external domains is disabled" do + it "renders message that support is disabled" do visit project_pages_path(project) - expect(page).to have_content('Support for domains and certificates is disabled') + expect(page).to have_content("Support for domains and certificates is disabled") end end - context 'when pages are exposed on external HTTP address', :http_pages_enabled do + context "when pages are exposed on external HTTP address", :http_pages_enabled do let(:project) { create(:project, pages_https_only: false) } - shared_examples 'adds new domain' do - it 'adds new domain' do + shared_examples "adds new domain" do + it "adds new domain" do visit new_project_pages_domain_path(project) - fill_in 'Domain', with: 'my.test.domain.com' - click_button 'Create New Domain' + fill_in "Domain", with: "my.test.domain.com" + click_button "Create New Domain" - expect(page).to have_content('my.test.domain.com') + expect(page).to have_content("my.test.domain.com") end end - it 'allows to add new domain' do + it "allows to add new domain" do visit project_pages_path(project) - expect(page).to have_content('New Domain') + expect(page).to have_content("New Domain") end - it_behaves_like 'adds new domain' + it_behaves_like "adds new domain" - context 'when project in group namespace' do - it_behaves_like 'adds new domain' do + context "when project in group namespace" do + it_behaves_like "adds new domain" do let(:group) { create :group } let(:project) { create(:project, namespace: group, pages_https_only: false) } end end - context 'when pages domain is added' do + context "when pages domain is added" do before do - create(:pages_domain, project: project, domain: 'my.test.domain.com') + create(:pages_domain, project: project, domain: "my.test.domain.com") visit new_project_pages_domain_path(project) end - it 'renders certificates is disabled' do - expect(page).to have_content('Support for custom certificates is disabled') + it "renders certificates is disabled" do + expect(page).to have_content("Support for custom certificates is disabled") end - it 'does not adds new domain and renders error message' do - fill_in 'Domain', with: 'my.test.domain.com' - click_button 'Create New Domain' + it "does not adds new domain and renders error message" do + fill_in "Domain", with: "my.test.domain.com" + click_button "Create New Domain" - expect(page).to have_content('Domain has already been taken') + expect(page).to have_content("Domain has already been taken") end end end - context 'when pages are exposed on external HTTPS address', :https_pages_enabled do + context "when pages are exposed on external HTTPS address", :https_pages_enabled do let(:certificate_pem) do <<~PEM - -----BEGIN CERTIFICATE----- - MIICGzCCAYSgAwIBAgIBATANBgkqhkiG9w0BAQUFADAbMRkwFwYDVQQDExB0ZXN0 - LWNlcnRpZmljYXRlMB4XDTE2MDIxMjE0MzIwMFoXDTIwMDQxMjE0MzIwMFowGzEZ - MBcGA1UEAxMQdGVzdC1jZXJ0aWZpY2F0ZTCBnzANBgkqhkiG9w0BAQEFAAOBjQAw - gYkCgYEApL4J9L0ZxFJ1hI1LPIflAlAGvm6ZEvoT4qKU5Xf2JgU7/2geNR1qlNFa - SvCc08Knupp5yTgmvyK/Xi09U0N82vvp4Zvr/diSc4A/RA6Mta6egLySNT438kdT - nY2tR5feoTLwQpX0t4IMlwGQGT5h6Of2fKmDxzuwuyffcIHqLdsCAwEAAaNvMG0w - DAYDVR0TAQH/BAIwADAdBgNVHQ4EFgQUxl9WSxBprB0z0ibJs3rXEk0+95AwCwYD - VR0PBAQDAgXgMBEGCWCGSAGG+EIBAQQEAwIGQDAeBglghkgBhvhCAQ0EERYPeGNh - IGNlcnRpZmljYXRlMA0GCSqGSIb3DQEBBQUAA4GBAGC4T8SlFHK0yPSa+idGLQFQ - joZp2JHYvNlTPkRJ/J4TcXxBTJmArcQgTIuNoBtC+0A/SwdK4MfTCUY4vNWNdese - 5A4K65Nb7Oh1AdQieTBHNXXCdyFsva9/ScfQGEl7p55a52jOPs0StPd7g64uvjlg - YHi2yesCrOvVXt+lgPTd - -----END CERTIFICATE----- + -----BEGIN CERTIFICATE----- + MIICGzCCAYSgAwIBAgIBATANBgkqhkiG9w0BAQUFADAbMRkwFwYDVQQDExB0ZXN0 + LWNlcnRpZmljYXRlMB4XDTE2MDIxMjE0MzIwMFoXDTIwMDQxMjE0MzIwMFowGzEZ + MBcGA1UEAxMQdGVzdC1jZXJ0aWZpY2F0ZTCBnzANBgkqhkiG9w0BAQEFAAOBjQAw + gYkCgYEApL4J9L0ZxFJ1hI1LPIflAlAGvm6ZEvoT4qKU5Xf2JgU7/2geNR1qlNFa + SvCc08Knupp5yTgmvyK/Xi09U0N82vvp4Zvr/diSc4A/RA6Mta6egLySNT438kdT + nY2tR5feoTLwQpX0t4IMlwGQGT5h6Of2fKmDxzuwuyffcIHqLdsCAwEAAaNvMG0w + DAYDVR0TAQH/BAIwADAdBgNVHQ4EFgQUxl9WSxBprB0z0ibJs3rXEk0+95AwCwYD + VR0PBAQDAgXgMBEGCWCGSAGG+EIBAQQEAwIGQDAeBglghkgBhvhCAQ0EERYPeGNh + IGNlcnRpZmljYXRlMA0GCSqGSIb3DQEBBQUAA4GBAGC4T8SlFHK0yPSa+idGLQFQ + joZp2JHYvNlTPkRJ/J4TcXxBTJmArcQgTIuNoBtC+0A/SwdK4MfTCUY4vNWNdese + 5A4K65Nb7Oh1AdQieTBHNXXCdyFsva9/ScfQGEl7p55a52jOPs0StPd7g64uvjlg + YHi2yesCrOvVXt+lgPTd + -----END CERTIFICATE----- PEM end let(:certificate_key) do <<~KEY - -----BEGIN PRIVATE KEY----- - MIICdgIBADANBgkqhkiG9w0BAQEFAASCAmAwggJcAgEAAoGBAKS+CfS9GcRSdYSN - SzyH5QJQBr5umRL6E+KilOV39iYFO/9oHjUdapTRWkrwnNPCp7qaeck4Jr8iv14t - PVNDfNr76eGb6/3YknOAP0QOjLWunoC8kjU+N/JHU52NrUeX3qEy8EKV9LeCDJcB - kBk+Yejn9nypg8c7sLsn33CB6i3bAgMBAAECgYA2D26w80T7WZvazYr86BNMePpd - j2mIAqx32KZHzt/lhh40J/SRtX9+Kl0Y7nBoRR5Ja9u/HkAIxNxLiUjwg9r6cpg/ - uITEF5nMt7lAk391BuI+7VOZZGbJDsq2ulPd6lO+C8Kq/PI/e4kXcIjeH6KwQsuR - 5vrXfBZ3sQfflaiN4QJBANBt8JY2LIGQF8o89qwUpRL5vbnKQ4IzZ5+TOl4RLR7O - AQpJ81tGuINghO7aunctb6rrcKJrxmEH1whzComybrMCQQDKV49nOBudRBAIgG4K - EnLzsRKISUHMZSJiYTYnablof8cKw1JaQduw7zgrUlLwnroSaAGX88+Jw1f5n2Lh - Vlg5AkBDdUGnrDLtYBCDEQYZHblrkc7ZAeCllDOWjxUV+uMqlCv8A4Ey6omvY57C - m6I8DkWVAQx8VPtozhvHjUw80rZHAkB55HWHAM3h13axKG0htCt7klhPsZHpx6MH - EPjGlXIT+aW2XiPmK3ZlCDcWIenE+lmtbOpI159Wpk8BGXs/s/xBAkEAlAY3ymgx - 63BDJEwvOb2IaP8lDDxNsXx9XJNVvQbv5n15vNsLHbjslHfAhAbxnLQ1fLhUPqSi - nNp/xedE1YxutQ== - -----END PRIVATE KEY----- + -----BEGIN PRIVATE KEY----- + MIICdgIBADANBgkqhkiG9w0BAQEFAASCAmAwggJcAgEAAoGBAKS+CfS9GcRSdYSN + SzyH5QJQBr5umRL6E+KilOV39iYFO/9oHjUdapTRWkrwnNPCp7qaeck4Jr8iv14t + PVNDfNr76eGb6/3YknOAP0QOjLWunoC8kjU+N/JHU52NrUeX3qEy8EKV9LeCDJcB + kBk+Yejn9nypg8c7sLsn33CB6i3bAgMBAAECgYA2D26w80T7WZvazYr86BNMePpd + j2mIAqx32KZHzt/lhh40J/SRtX9+Kl0Y7nBoRR5Ja9u/HkAIxNxLiUjwg9r6cpg/ + uITEF5nMt7lAk391BuI+7VOZZGbJDsq2ulPd6lO+C8Kq/PI/e4kXcIjeH6KwQsuR + 5vrXfBZ3sQfflaiN4QJBANBt8JY2LIGQF8o89qwUpRL5vbnKQ4IzZ5+TOl4RLR7O + AQpJ81tGuINghO7aunctb6rrcKJrxmEH1whzComybrMCQQDKV49nOBudRBAIgG4K + EnLzsRKISUHMZSJiYTYnablof8cKw1JaQduw7zgrUlLwnroSaAGX88+Jw1f5n2Lh + Vlg5AkBDdUGnrDLtYBCDEQYZHblrkc7ZAeCllDOWjxUV+uMqlCv8A4Ey6omvY57C + m6I8DkWVAQx8VPtozhvHjUw80rZHAkB55HWHAM3h13axKG0htCt7klhPsZHpx6MH + EPjGlXIT+aW2XiPmK3ZlCDcWIenE+lmtbOpI159Wpk8BGXs/s/xBAkEAlAY3ymgx + 63BDJEwvOb2IaP8lDDxNsXx9XJNVvQbv5n15vNsLHbjslHfAhAbxnLQ1fLhUPqSi + nNp/xedE1YxutQ== + -----END PRIVATE KEY----- KEY end - it 'adds new domain with certificate' do + it "adds new domain with certificate" do visit new_project_pages_domain_path(project) - fill_in 'Domain', with: 'my.test.domain.com' - fill_in 'Certificate (PEM)', with: certificate_pem - fill_in 'Key (PEM)', with: certificate_key - click_button 'Create New Domain' + fill_in "Domain", with: "my.test.domain.com" + fill_in "Certificate (PEM)", with: certificate_pem + fill_in "Key (PEM)", with: certificate_key + click_button "Create New Domain" - expect(page).to have_content('my.test.domain.com') + expect(page).to have_content("my.test.domain.com") end - describe 'updating the certificate for an existing domain' do + describe "updating the certificate for an existing domain" do let!(:domain) do create(:pages_domain, project: project) end - it 'allows the certificate to be updated' do + it "allows the certificate to be updated" do visit project_pages_path(project) - within('#content-body') { click_link 'Details' } - click_link 'Edit' - click_button 'Save Changes' + within("#content-body") { click_link "Details" } + click_link "Edit" + click_button "Save Changes" - expect(page).to have_content('Domain was updated') + expect(page).to have_content("Domain was updated") end - context 'when the certificate is invalid' do - it 'tells the user what the problem is' do + context "when the certificate is invalid" do + it "tells the user what the problem is" do visit project_pages_path(project) - within('#content-body') { click_link 'Details' } - click_link 'Edit' - fill_in 'Certificate (PEM)', with: 'invalid data' - click_button 'Save Changes' + within("#content-body") { click_link "Details" } + click_link "Edit" + fill_in "Certificate (PEM)", with: "invalid data" + click_button "Save Changes" - expect(page).to have_content('Certificate must be a valid PEM certificate') - expect(page).to have_content('Certificate misses intermediates') + expect(page).to have_content("Certificate must be a valid PEM certificate") + expect(page).to have_content("Certificate misses intermediates") expect(page).to have_content("Key doesn't match the certificate") end end @@ -171,23 +171,23 @@ describe 'Pages' do end end - it 'does not see anything to destroy' do + it "does not see anything to destroy" do visit project_pages_path(project) - expect(page).to have_content('Configure pages') - expect(page).not_to have_link('Remove pages') + expect(page).to have_content("Configure pages") + expect(page).not_to have_link("Remove pages") end - describe 'project settings page' do + describe "project settings page" do it 'renders "Pages" tab' do visit edit_project_path(project) - page.within '.nav-sidebar' do - expect(page).to have_link('Pages') + page.within ".nav-sidebar" do + expect(page).to have_link("Pages") end end - context 'when pages are disabled' do + context "when pages are disabled" do before do allow(Gitlab.config.pages).to receive(:enabled).and_return(false) end @@ -195,87 +195,87 @@ describe 'Pages' do it 'does not render "Pages" tab' do visit edit_project_path(project) - page.within '.nav-sidebar' do - expect(page).not_to have_link('Pages') + page.within ".nav-sidebar" do + expect(page).not_to have_link("Pages") end end end end end - describe 'HTTPS settings', :js, :https_pages_enabled do + describe "HTTPS settings", :js, :https_pages_enabled do before do project.namespace.update(owner: user) allow_any_instance_of(Project).to receive(:pages_deployed?) { true } end - it 'tries to change the setting' do + it "tries to change the setting" do visit project_pages_path(project) expect(page).to have_content("Force HTTPS (requires valid certificates)") uncheck :project_pages_https_only - click_button 'Save' + click_button "Save" - expect(page).to have_text('Your changes have been saved') - expect(page).not_to have_checked_field('project_pages_https_only') + expect(page).to have_text("Your changes have been saved") + expect(page).not_to have_checked_field("project_pages_https_only") end - context 'setting could not be updated' do - let(:service) { instance_double('Projects::UpdateService') } + context "setting could not be updated" do + let(:service) { instance_double("Projects::UpdateService") } before do allow(Projects::UpdateService).to receive(:new).and_return(service) allow(service).to receive(:execute).and_return(status: :error) end - it 'tries to change the setting' do + it "tries to change the setting" do visit project_pages_path(project) uncheck :project_pages_https_only - click_button 'Save' + click_button "Save" - expect(page).to have_text('Something went wrong on our end') + expect(page).to have_text("Something went wrong on our end") end end - context 'non-HTTPS domain exists' do + context "non-HTTPS domain exists" do let(:project) { create(:project, pages_https_only: false) } before do create(:pages_domain, :without_key, :without_certificate, project: project) end - it 'the setting is disabled' do + it "the setting is disabled" do visit project_pages_path(project) expect(page).to have_field(:project_pages_https_only, disabled: true) - expect(page).not_to have_button('Save') + expect(page).not_to have_button("Save") end end - context 'HTTPS pages are disabled', :https_pages_disabled do - it 'the setting is unavailable' do + context "HTTPS pages are disabled", :https_pages_disabled do + it "the setting is unavailable" do visit project_pages_path(project) expect(page).not_to have_field(:project_pages_https_only) - expect(page).not_to have_content('Force HTTPS (requires valid certificates)') - expect(page).not_to have_button('Save') + expect(page).not_to have_content("Force HTTPS (requires valid certificates)") + expect(page).not_to have_button("Save") end end end - describe 'Remove page' do + describe "Remove page" do let(:project) { create :project, :repository } - context 'when pages are deployed' do + context "when pages are deployed" do let(:pipeline) do - commit_sha = project.commit('HEAD').sha + commit_sha = project.commit("HEAD").sha project.ci_pipelines.create( - ref: 'HEAD', + ref: "HEAD", sha: commit_sha, source: :push, protected: false @@ -287,9 +287,9 @@ describe 'Pages' do :ci_build, project: project, pipeline: pipeline, - ref: 'HEAD', - legacy_artifacts_file: fixture_file_upload(File.join('spec/fixtures/pages.zip')), - legacy_artifacts_metadata: fixture_file_upload(File.join('spec/fixtures/pages.zip.meta')) + ref: "HEAD", + legacy_artifacts_file: fixture_file_upload(File.join("spec/fixtures/pages.zip")), + legacy_artifacts_metadata: fixture_file_upload(File.join("spec/fixtures/pages.zip.meta")) ) end @@ -299,12 +299,12 @@ describe 'Pages' do expect(project).to be_pages_deployed end - it 'removes the pages' do + it "removes the pages" do visit project_pages_path(project) - expect(page).to have_link('Remove pages') + expect(page).to have_link("Remove pages") - click_link 'Remove pages' + click_link "Remove pages" expect(project.pages_deployed?).to be_falsey end |