Merge branch '35853-pages-missing-dotfiles' into 'master'

Include a .hidden directory in pages artifact extraction tests See merge request gitlab-org/gitlab-ce!14852

Merge branch '35853-pages-missing-dotfiles' into 'master'
7a8b97e1 · Kamil Trzciński · a73e3295 · f1aabd89 · 7a8b97e1 · 7a8b97e1
Commit 7a8b97e1 authored Oct 16, 2017 by Kamil Trzciński
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

pages.tar.gz spec/fixtures/pages.tar.gz +0 -0

pages.zip spec/fixtures/pages.zip +0 -0

update_pages_service_spec.rb spec/services/projects/update_pages_service_spec.rb +5 -0

No files found.
--- a/spec/fixtures/pages.tar.gz
+++ b/spec/fixtures/pages.tar.gz
--- a/spec/fixtures/pages.zip
+++ b/spec/fixtures/pages.zip
--- a/spec/services/projects/update_pages_service_spec.rb
+++ b/spec/services/projects/update_pages_service_spec.rb
@@ -52,6 +52,11 @@ describe Projects::UpdatePagesService do
        expect(project.pages_deployed?).to be_falsey
        expect(execute).to eq(:success)
        expect(project.pages_deployed?).to be_truthy
+
+        # Check that all expected files are extracted
+        %w[index.html zero .hidden/file].each do |filename|
+          expect(File.exist?(File.join(project.public_pages_path, filename))).to be_truthy
+        end
      end

      it 'limits pages size' do