Commit cfffc9ef authored by Grzegorz Bizon's avatar Grzegorz Bizon

Update build specs for artifacts browser support

parent e5d2a81b
...@@ -394,21 +394,20 @@ describe Ci::Build, models: true do ...@@ -394,21 +394,20 @@ describe Ci::Build, models: true do
describe :artifacts_browser_supported? do describe :artifacts_browser_supported? do
subject { build.artifacts_browser_supported? } subject { build.artifacts_browser_supported? }
before do context 'artifacts metadata does not exist' do
file = fixture_file_upload(archive_file, archive_type)
build.update_attributes(artifacts_file: file)
end
context 'artifacts archive is not a zip file' do
let(:archive_file) { Rails.root + 'spec/fixtures/banana_sample.gif' }
let(:archive_type) { 'image/gif' }
it { is_expected.to be_falsy } it { is_expected.to be_falsy }
end end
context 'artifacts archive is a zip file' do context 'artifacts archive is a zip file and metadata exists' do
let(:archive_file) { Rails.root + 'spec/fixtures/ci_build_artifacts.zip' } before do
let(:archive_type) { 'application/zip' } fixture_dir = Rails.root + 'spec/fixtures/'
archive = fixture_file_upload(fixture_dir + 'ci_build_artifacts.zip',
'application/zip')
metadata = fixture_file_upload(fixture_dir + 'ci_build_artifacts_metadata.gz',
'application/x-gzip')
build.update_attributes(artifacts_file: archive)
build.update_attributes(artifacts_metadata: metadata)
end
it { is_expected.to be_truthy } it { is_expected.to be_truthy }
end end
......
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