diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-05-02 14:27:28 +0900 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2018-05-02 14:27:28 +0900 |
commit | 6ed91266d021c2058697d12a0123bec9b00548ad (patch) | |
tree | 4448df047d4e36ba1ced6f8614b19cb9c1339487 /spec/models/ci/build_trace_chunk_spec.rb | |
parent | 65c11f32612bd3fb65d82e8d3757ae5066b8a2bc (diff) | |
download | gitlab-ce-6ed91266d021c2058697d12a0123bec9b00548ad.tar.gz |
Skip truncate when offset == size. Fix static analysys.
Diffstat (limited to 'spec/models/ci/build_trace_chunk_spec.rb')
-rw-r--r-- | spec/models/ci/build_trace_chunk_spec.rb | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/spec/models/ci/build_trace_chunk_spec.rb b/spec/models/ci/build_trace_chunk_spec.rb index cab5db9ca06..b0ede29669e 100644 --- a/spec/models/ci/build_trace_chunk_spec.rb +++ b/spec/models/ci/build_trace_chunk_spec.rb @@ -345,7 +345,7 @@ describe Ci::BuildTraceChunk, :clean_gitlab_redis_shared_state do shared_examples_for 'deletes all build_trace_chunk and data in redis' do it do Gitlab::Redis::SharedState.with do |redis| - expect(redis.scan_each(match: "gitlab:ci:trace:?:chunks:?").to_a.count).to eq(3) + expect(redis.scan_each(match: "gitlab:ci:trace:?:chunks:?").to_a.size).to eq(3) end expect(described_class.count).to eq(3) @@ -355,14 +355,16 @@ describe Ci::BuildTraceChunk, :clean_gitlab_redis_shared_state do expect(described_class.count).to eq(0) Gitlab::Redis::SharedState.with do |redis| - expect(redis.scan_each(match: "gitlab:ci:trace:?:chunks:?").to_a.count).to eq(0) + expect(redis.scan_each(match: "gitlab:ci:trace:?:chunks:?").to_a.size).to eq(0) end end end - context 'when build is destroyed' do + context 'when traces are archived' do let(:subject) do - project.builds.destroy_all + project.builds.each do |build| + build.success! + end end it_behaves_like 'deletes all build_trace_chunk and data in redis' |