2017-08-17 22:00:37 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
feature 'Pages' do
|
|
|
|
given(:project) { create(:project) }
|
2017-08-17 22:00:37 +05:30
|
|
|
given(:user) { create(:user) }
|
|
|
|
given(:role) { :master }
|
|
|
|
|
|
|
|
background do
|
|
|
|
allow(Gitlab.config.pages).to receive(:enabled).and_return(true)
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
project.add_role(user, role)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
sign_in(user)
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'no pages deployed' do
|
|
|
|
scenario 'does not see anything to destroy' do
|
2017-09-10 17:25:29 +05:30
|
|
|
visit project_pages_path(project)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
expect(page).to have_content('Configure pages')
|
2017-08-17 22:00:37 +05:30
|
|
|
expect(page).not_to have_link('Remove pages')
|
|
|
|
expect(page).not_to have_text('Only the project owner can remove pages')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when user is the owner' do
|
|
|
|
background do
|
|
|
|
project.namespace.update(owner: user)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pages deployed' do
|
|
|
|
background do
|
|
|
|
allow_any_instance_of(Project).to receive(:pages_deployed?) { true }
|
|
|
|
end
|
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
scenario 'renders Access pages' do
|
2017-09-10 17:25:29 +05:30
|
|
|
visit project_pages_path(project)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
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
|
|
|
|
visit project_pages_path(project)
|
|
|
|
|
|
|
|
expect(page).to have_content('Support for domains and certificates is disabled')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-05-09 12:01:36 +05:30
|
|
|
context 'when pages are exposed on external HTTP address', :http_pages_enabled do
|
|
|
|
given(:project) { create(:project, pages_https_only: false) }
|
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
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'
|
|
|
|
|
|
|
|
expect(page).to have_content('my.test.domain.com')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'allows to add new domain' do
|
|
|
|
visit project_pages_path(project)
|
|
|
|
|
|
|
|
expect(page).to have_content('New Domain')
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'adds new domain'
|
|
|
|
|
|
|
|
context 'when project in group namespace' do
|
|
|
|
it_behaves_like 'adds new domain' do
|
|
|
|
let(:group) { create :group }
|
2018-05-09 12:01:36 +05:30
|
|
|
let(:project) { create(:project, namespace: group, pages_https_only: false) }
|
2018-03-27 19:54:05 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pages domain is added' do
|
|
|
|
before do
|
2018-05-09 12:01:36 +05:30
|
|
|
create(:pages_domain, project: project, domain: 'my.test.domain.com')
|
2018-03-27 19:54:05 +05:30
|
|
|
|
|
|
|
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')
|
|
|
|
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'
|
|
|
|
|
|
|
|
expect(page).to have_content('Domain has already been taken')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-05-09 12:01:36 +05:30
|
|
|
context 'when pages are exposed on external HTTPS address', :https_pages_enabled do
|
2018-03-27 19:54:05 +05:30
|
|
|
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-----
|
|
|
|
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-----
|
|
|
|
KEY
|
|
|
|
end
|
|
|
|
|
|
|
|
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'
|
|
|
|
|
|
|
|
expect(page).to have_content('my.test.domain.com')
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'updating the certificate for an existing domain' do
|
|
|
|
let!(:domain) do
|
2018-05-09 12:01:36 +05:30
|
|
|
create(:pages_domain, project: project)
|
2018-03-27 19:54:05 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
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'
|
|
|
|
|
|
|
|
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
|
|
|
|
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'
|
|
|
|
|
|
|
|
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
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'no pages deployed'
|
2018-03-27 19:54:05 +05:30
|
|
|
|
|
|
|
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')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pages are disabled' do
|
|
|
|
before do
|
|
|
|
allow(Gitlab.config.pages).to receive(:enabled).and_return(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not render "Pages" tab' do
|
|
|
|
visit edit_project_path(project)
|
|
|
|
|
|
|
|
page.within '.nav-sidebar' do
|
|
|
|
expect(page).not_to have_link('Pages')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the user is not the owner' do
|
|
|
|
context 'when pages deployed' do
|
|
|
|
background do
|
|
|
|
allow_any_instance_of(Project).to receive(:pages_deployed?) { true }
|
|
|
|
end
|
|
|
|
|
|
|
|
scenario 'sees "Only the project owner can remove pages" text' do
|
2017-09-10 17:25:29 +05:30
|
|
|
visit project_pages_path(project)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
expect(page).to have_text('Only the project owner can remove pages')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'no pages deployed'
|
|
|
|
end
|
2018-03-27 19:54:05 +05:30
|
|
|
|
2018-05-09 12:01:36 +05:30
|
|
|
describe 'HTTPS settings', :js, :https_pages_enabled do
|
|
|
|
background do
|
|
|
|
project.namespace.update(owner: user)
|
|
|
|
|
|
|
|
allow_any_instance_of(Project).to receive(:pages_deployed?) { true }
|
|
|
|
end
|
|
|
|
|
|
|
|
scenario 'tries to change the setting' do
|
|
|
|
visit project_pages_path(project)
|
|
|
|
expect(page).to have_content("Force domains with SSL certificates to use HTTPS")
|
|
|
|
|
|
|
|
uncheck :project_pages_https_only
|
|
|
|
|
|
|
|
click_button 'Save'
|
|
|
|
|
|
|
|
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') }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(Projects::UpdateService).to receive(:new).and_return(service)
|
|
|
|
allow(service).to receive(:execute).and_return(status: :error)
|
|
|
|
end
|
|
|
|
|
|
|
|
scenario 'tries to change the setting' do
|
|
|
|
visit project_pages_path(project)
|
|
|
|
|
|
|
|
uncheck :project_pages_https_only
|
|
|
|
|
|
|
|
click_button 'Save'
|
|
|
|
|
|
|
|
expect(page).to have_text('Something went wrong on our end')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'non-HTTPS domain exists' do
|
|
|
|
given(:project) { create(:project, pages_https_only: false) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
create(:pages_domain, :without_key, :without_certificate, project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
scenario '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')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'HTTPS pages are disabled', :https_pages_disabled do
|
|
|
|
scenario '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 domains with SSL certificates to use HTTPS')
|
|
|
|
expect(page).not_to have_button('Save')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
describe 'Remove page' do
|
|
|
|
context 'when user is the owner' do
|
|
|
|
let(:project) { create :project, :repository }
|
|
|
|
|
|
|
|
before do
|
|
|
|
project.namespace.update(owner: user)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pages are deployed' do
|
|
|
|
let(:pipeline) do
|
|
|
|
commit_sha = project.commit('HEAD').sha
|
|
|
|
|
|
|
|
project.pipelines.create(
|
|
|
|
ref: 'HEAD',
|
|
|
|
sha: commit_sha,
|
|
|
|
source: :push,
|
|
|
|
protected: false
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:ci_build) do
|
|
|
|
create(
|
|
|
|
:ci_build,
|
|
|
|
project: project,
|
|
|
|
pipeline: pipeline,
|
|
|
|
ref: 'HEAD',
|
|
|
|
legacy_artifacts_file: fixture_file_upload(Rails.root.join('spec/fixtures/pages.zip')),
|
|
|
|
legacy_artifacts_metadata: fixture_file_upload(Rails.root.join('spec/fixtures/pages.zip.meta'))
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
result = Projects::UpdatePagesService.new(project, ci_build).execute
|
|
|
|
expect(result[:status]).to eq(:success)
|
|
|
|
expect(project).to be_pages_deployed
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'removes the pages' do
|
|
|
|
visit project_pages_path(project)
|
|
|
|
|
|
|
|
expect(page).to have_link('Remove pages')
|
|
|
|
|
|
|
|
click_link 'Remove pages'
|
|
|
|
|
|
|
|
expect(project.pages_deployed?).to be_falsey
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|