summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorLin Jen-Shin <godfat@godfat.org>2016-06-28 18:14:21 +0800
committerLin Jen-Shin <godfat@godfat.org>2016-06-28 18:14:21 +0800
commitfe0c59d2e61238e1241be448a37be0e3e702a5ce (patch)
treea002bc7887880f291b2294ebce044c3527dc7df4 /spec
parentaa3a3fd12fc7152020ffbe54faa36a4f17eb94b2 (diff)
downloadgitlab-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.rb14
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