2019-10-12 21:52:04 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2018-10-15 14:42:47 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2020-07-28 23:09:34 +05:30
|
|
|
RSpec.describe Gitlab::Ci::Trace::ChunkedIO, :clean_gitlab_redis_cache do
|
2018-10-15 14:42:47 +05:30
|
|
|
include ChunkedIOHelpers
|
|
|
|
|
2020-04-08 14:13:33 +05:30
|
|
|
let_it_be(:build) { create(:ci_build, :running) }
|
2021-06-08 01:23:25 +05:30
|
|
|
|
2018-10-15 14:42:47 +05:30
|
|
|
let(:chunked_io) { described_class.new(build) }
|
|
|
|
|
|
|
|
before do
|
2021-03-11 19:13:27 +05:30
|
|
|
stub_feature_flags(ci_enable_live_trace: true, gitlab_ci_trace_read_consistency: true)
|
2018-10-15 14:42:47 +05:30
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#initialize" do
|
2018-10-15 14:42:47 +05:30
|
|
|
context 'when a chunk exists' do
|
|
|
|
before do
|
|
|
|
build.trace.set('ABC')
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect(chunked_io.size).to eq(3) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when two chunks exist' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(4)
|
|
|
|
build.trace.set('ABCDEF')
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect(chunked_io.size).to eq(6) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when no chunks exists' do
|
|
|
|
it { expect(chunked_io.size).to eq(0) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#seek" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.seek(pos, where) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when moves pos to end of the file' do
|
|
|
|
let(:pos) { 0 }
|
|
|
|
let(:where) { IO::SEEK_END }
|
|
|
|
|
|
|
|
it { is_expected.to eq(sample_trace_raw.bytesize) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when moves pos to middle of the file' do
|
|
|
|
let(:pos) { sample_trace_raw.bytesize / 2 }
|
|
|
|
let(:where) { IO::SEEK_SET }
|
|
|
|
|
|
|
|
it { is_expected.to eq(pos) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when moves pos around' do
|
|
|
|
it 'matches the result' do
|
|
|
|
expect(chunked_io.seek(0)).to eq(0)
|
|
|
|
expect(chunked_io.seek(100, IO::SEEK_CUR)).to eq(100)
|
|
|
|
expect { chunked_io.seek(sample_trace_raw.bytesize + 1, IO::SEEK_CUR) }
|
|
|
|
.to raise_error('new position is outside of file')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#eof?" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.eof? }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when current pos is at end of the file' do
|
|
|
|
before do
|
|
|
|
chunked_io.seek(sample_trace_raw.bytesize, IO::SEEK_SET)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to be_truthy }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when current pos is not at end of the file' do
|
|
|
|
before do
|
|
|
|
chunked_io.seek(0, IO::SEEK_SET)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to be_falsey }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#each_line" do
|
2018-10-15 14:42:47 +05:30
|
|
|
let(:string_io) { StringIO.new(sample_trace_raw) }
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'yields lines' do
|
|
|
|
expect { |b| chunked_io.each_line(&b) }
|
|
|
|
.to yield_successive_args(*string_io.each_line.to_a)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'calls get_chunk only once' do
|
2020-01-01 13:55:28 +05:30
|
|
|
expect_next_instance_of(Gitlab::Ci::Trace::ChunkedIO) do |instance|
|
|
|
|
expect(instance).to receive(:current_chunk).once.and_call_original
|
|
|
|
end
|
2018-10-15 14:42:47 +05:30
|
|
|
|
|
|
|
chunked_io.each_line { |line| }
|
|
|
|
end
|
|
|
|
end
|
2019-02-15 15:39:39 +05:30
|
|
|
|
|
|
|
context 'when buffer consist of many empty lines' do
|
|
|
|
let(:sample_trace_raw) { Array.new(10, " ").join("\n") }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'yields lines' do
|
|
|
|
expect { |b| chunked_io.each_line(&b) }
|
|
|
|
.to yield_successive_args(*string_io.each_line.to_a)
|
|
|
|
end
|
|
|
|
end
|
2018-10-15 14:42:47 +05:30
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#read" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.read(length) }
|
|
|
|
|
|
|
|
context 'when read the whole size' do
|
|
|
|
let(:length) { nil }
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to eq(sample_trace_raw) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to eq(sample_trace_raw) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-02-15 15:39:39 +05:30
|
|
|
context 'when chunk is missing data' do
|
|
|
|
let(:length) { nil }
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_buffer_size(1024)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
|
|
|
|
# make second chunk to not have data
|
|
|
|
build.trace_chunks.second.append('', 0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'raises an error' do
|
|
|
|
expect { subject }.to raise_error described_class::FailedToGetChunkError
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-10-15 14:42:47 +05:30
|
|
|
context 'when read only first 100 bytes' do
|
|
|
|
let(:length) { 100 }
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to eq(sample_trace_raw.byteslice(0, length))
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to eq(sample_trace_raw.byteslice(0, length))
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when tries to read oversize' do
|
|
|
|
let(:length) { sample_trace_raw.bytesize + 1000 }
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to eq(sample_trace_raw)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to eq(sample_trace_raw)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when tries to read 0 bytes' do
|
|
|
|
let(:length) { 0 }
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to be_empty
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads a trace' do
|
|
|
|
is_expected.to be_empty
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#readline" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.readline }
|
|
|
|
|
|
|
|
let(:string_io) { StringIO.new(sample_trace_raw) }
|
|
|
|
|
|
|
|
shared_examples 'all line matching' do
|
|
|
|
it do
|
|
|
|
(0...sample_trace_raw.lines.count).each do
|
|
|
|
expect(chunked_io.readline).to eq(string_io.readline)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'all line matching'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'all line matching'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pos is at middle of the file' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
|
|
|
|
chunked_io.seek(chunked_io.size / 2)
|
|
|
|
string_io.seek(string_io.size / 2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'reads from pos' do
|
|
|
|
expect(chunked_io.readline).to eq(string_io.readline)
|
|
|
|
end
|
|
|
|
end
|
2019-02-15 15:39:39 +05:30
|
|
|
|
|
|
|
context 'when chunk is missing data' do
|
|
|
|
let(:length) { nil }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
|
|
|
|
# make first chunk to have invalid data
|
|
|
|
build.trace_chunks.first.append('data', 0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'raises an error' do
|
|
|
|
expect { subject }.to raise_error described_class::FailedToGetChunkError
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when utf-8 is being used' do
|
2019-10-12 21:52:04 +05:30
|
|
|
let(:sample_trace_raw) { sample_trace_raw_utf8.dup.force_encoding(Encoding::BINARY) }
|
2019-02-15 15:39:39 +05:30
|
|
|
let(:sample_trace_raw_utf8) { "😺\n😺\n😺\n😺" }
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_buffer_size(3) # the utf-8 character has 4 bytes
|
|
|
|
|
|
|
|
build.trace.set(sample_trace_raw_utf8)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'has known length' do
|
|
|
|
expect(sample_trace_raw_utf8.bytesize).to eq(4 * 4 + 3 * 1)
|
|
|
|
expect(sample_trace_raw.bytesize).to eq(4 * 4 + 3 * 1)
|
|
|
|
expect(chunked_io.size).to eq(4 * 4 + 3 * 1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'all line matching'
|
|
|
|
end
|
2018-10-15 14:42:47 +05:30
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#write" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.write(data) }
|
|
|
|
|
|
|
|
let(:data) { sample_trace_raw }
|
|
|
|
|
|
|
|
context 'when data does not exist' do
|
|
|
|
shared_examples 'writes a trace' do
|
|
|
|
it do
|
|
|
|
is_expected.to eq(data.bytesize)
|
|
|
|
|
|
|
|
chunked_io.seek(0, IO::SEEK_SET)
|
|
|
|
expect(chunked_io.read).to eq(data)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(data.bytesize / 2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'writes a trace'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(data.bytesize * 2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'writes a trace'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when data already exists' do
|
|
|
|
let(:exist_data) { 'exist data' }
|
|
|
|
|
|
|
|
shared_examples 'appends a trace' do
|
|
|
|
it do
|
|
|
|
chunked_io.seek(0, IO::SEEK_END)
|
|
|
|
is_expected.to eq(data.bytesize)
|
|
|
|
|
|
|
|
chunked_io.seek(0, IO::SEEK_SET)
|
|
|
|
expect(chunked_io.read).to eq(exist_data + data)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(exist_data)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'appends a trace'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(exist_data)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'appends a trace'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#truncate" do
|
2018-10-15 14:42:47 +05:30
|
|
|
let(:offset) { 10 }
|
|
|
|
|
|
|
|
context 'when data does not exist' do
|
|
|
|
shared_examples 'truncates a trace' do
|
|
|
|
it do
|
|
|
|
chunked_io.truncate(offset)
|
|
|
|
|
|
|
|
chunked_io.seek(0, IO::SEEK_SET)
|
|
|
|
expect(chunked_io.read).to eq(sample_trace_raw.byteslice(0, offset))
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is smaller than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize / 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'truncates a trace'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when buffer size is larger than file size' do
|
|
|
|
before do
|
|
|
|
stub_buffer_size(sample_trace_raw.bytesize * 2)
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'truncates a trace'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-03-13 15:44:24 +05:30
|
|
|
describe "#destroy!" do
|
2018-10-15 14:42:47 +05:30
|
|
|
subject { chunked_io.destroy! }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build.trace.set(sample_trace_raw)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'deletes' do
|
|
|
|
expect { subject }.to change { chunked_io.size }
|
|
|
|
.from(sample_trace_raw.bytesize).to(0)
|
|
|
|
|
|
|
|
expect(Ci::BuildTraceChunk.where(build: build).count).to eq(0)
|
|
|
|
end
|
2019-10-12 21:52:04 +05:30
|
|
|
|
|
|
|
context 'when the job does not have archived trace' do
|
|
|
|
it 'leaves a message in sidekiq log' do
|
|
|
|
expect(Sidekiq.logger).to receive(:warn).with(
|
|
|
|
message: 'The job does not have archived trace but going to be destroyed.',
|
|
|
|
job_id: build.id).and_call_original
|
|
|
|
|
|
|
|
subject
|
|
|
|
end
|
|
|
|
end
|
2018-10-15 14:42:47 +05:30
|
|
|
end
|
|
|
|
end
|