diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2016-06-28 18:14:21 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2016-06-28 18:14:21 +0800 |
commit | fe0c59d2e61238e1241be448a37be0e3e702a5ce (patch) | |
tree | a002bc7887880f291b2294ebce044c3527dc7df4 /spec | |
parent | aa3a3fd12fc7152020ffbe54faa36a4f17eb94b2 (diff) | |
download | gitlab-ce-fe0c59d2e61238e1241be448a37be0e3e702a5ce.tar.gz |
Introduce ci_builds.artifacts_sizes as JSON:
We store the sizes as a hash from path to bytes like:
``` ruby
{'ci_artifacts.txt' => 27,
'other_artifacts_0.1.2/another-subdirectory/banana_sample.gif' =>
71759,
'other_artifacts_0.1.2/doc_sample.txt' => 1314,
'rails_sample.jpg' => 35255,
'tests_encoding/utf8 test dir ✓/regular_file_2' => 7}
```
So that it's easier to access than reading gzip file again.
Diffstat (limited to 'spec')
-rw-r--r-- | spec/requests/ci/api/builds_spec.rb | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/spec/requests/ci/api/builds_spec.rb b/spec/requests/ci/api/builds_spec.rb index ea984924701..fb164c221d0 100644 --- a/spec/requests/ci/api/builds_spec.rb +++ b/spec/requests/ci/api/builds_spec.rb @@ -309,7 +309,7 @@ describe Ci::API::API do shared_examples 'post artifact' do it 'updates successfully' do response_filename = - json_response["artifacts_file"]["filename"] + json_response['artifacts_file']['filename'] expect(response).to have_http_status(201) expect(response_filename).to eq(file_upload.original_filename) @@ -344,10 +344,14 @@ describe Ci::API::API do context 'should post artifacts file and metadata file' do let!(:artifacts) { file_upload } - let!(:metadata) { file_upload2 } + let!(:metadata) do + fixture_file_upload( + Rails.root + 'spec/fixtures/ci_build_artifacts_metadata.gz') + end let(:stored_artifacts_file) { build.reload.artifacts_file.file } let(:stored_metadata_file) { build.reload.artifacts_metadata.file } + let(:stored_artifacts_sizes) { build.reload.artifacts_sizes } before do post(post_url, post_data, headers_with_token) @@ -365,6 +369,12 @@ describe Ci::API::API do expect(response).to have_http_status(201) expect(stored_artifacts_file.original_filename).to eq(artifacts.original_filename) expect(stored_metadata_file.original_filename).to eq(metadata.original_filename) + expect(stored_artifacts_sizes).to eq( + 'ci_artifacts.txt' => 27, + 'other_artifacts_0.1.2/another-subdirectory/banana_sample.gif' => 71759, + 'other_artifacts_0.1.2/doc_sample.txt' => 1314, + 'rails_sample.jpg' => 35255, + 'tests_encoding/utf8 test dir ✓/regular_file_2' => 7) end end |