diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
commit | 48aff82709769b098321c738f3444b9bdaa694c6 (patch) | |
tree | e00c7c43e2d9b603a5a6af576b1685e400410dee /spec/lib/gitlab/repository_size_checker_spec.rb | |
parent | 879f5329ee916a948223f8f43d77fba4da6cd028 (diff) | |
download | gitlab-ce-48aff82709769b098321c738f3444b9bdaa694c6.tar.gz |
Add latest changes from gitlab-org/gitlab@13-5-stable-eev13.5.0-rc42
Diffstat (limited to 'spec/lib/gitlab/repository_size_checker_spec.rb')
-rw-r--r-- | spec/lib/gitlab/repository_size_checker_spec.rb | 59 |
1 files changed, 11 insertions, 48 deletions
diff --git a/spec/lib/gitlab/repository_size_checker_spec.rb b/spec/lib/gitlab/repository_size_checker_spec.rb index 9b2c02b1190..bd030d81d97 100644 --- a/spec/lib/gitlab/repository_size_checker_spec.rb +++ b/spec/lib/gitlab/repository_size_checker_spec.rb @@ -3,14 +3,16 @@ require 'spec_helper' RSpec.describe Gitlab::RepositorySizeChecker do + let_it_be(:namespace) { nil } let(:current_size) { 0 } let(:limit) { 50 } let(:enabled) { true } subject do described_class.new( - current_size_proc: -> { current_size }, - limit: limit, + current_size_proc: -> { current_size.megabytes }, + limit: limit.megabytes, + namespace: namespace, enabled: enabled ) end @@ -18,7 +20,7 @@ RSpec.describe Gitlab::RepositorySizeChecker do describe '#enabled?' do context 'when enabled' do it 'returns true' do - expect(subject.enabled?).to be_truthy + expect(subject.enabled?).to eq(true) end end @@ -26,7 +28,7 @@ RSpec.describe Gitlab::RepositorySizeChecker do let(:limit) { 0 } it 'returns false' do - expect(subject.enabled?).to be_falsey + expect(subject.enabled?).to eq(false) end end end @@ -35,59 +37,20 @@ RSpec.describe Gitlab::RepositorySizeChecker do let(:current_size) { 49 } it 'returns true when changes go over' do - expect(subject.changes_will_exceed_size_limit?(2)).to be_truthy + expect(subject.changes_will_exceed_size_limit?(2.megabytes)).to eq(true) end it 'returns false when changes do not go over' do - expect(subject.changes_will_exceed_size_limit?(1)).to be_falsey + expect(subject.changes_will_exceed_size_limit?(1.megabytes)).to eq(false) end end describe '#above_size_limit?' do - context 'when size is above the limit' do - let(:current_size) { 100 } - - it 'returns true' do - expect(subject.above_size_limit?).to be_truthy - end - end - - it 'returns false when not over the limit' do - expect(subject.above_size_limit?).to be_falsey - end + include_examples 'checker size above limit' + include_examples 'checker size not over limit' end describe '#exceeded_size' do - context 'when current size is below or equal to the limit' do - let(:current_size) { 50 } - - it 'returns zero' do - expect(subject.exceeded_size).to eq(0) - end - end - - context 'when current size is over the limit' do - let(:current_size) { 51 } - - it 'returns zero' do - expect(subject.exceeded_size).to eq(1) - end - end - - context 'when change size will be over the limit' do - let(:current_size) { 50 } - - it 'returns zero' do - expect(subject.exceeded_size(1)).to eq(1) - end - end - - context 'when change size will not be over the limit' do - let(:current_size) { 49 } - - it 'returns zero' do - expect(subject.exceeded_size(1)).to eq(0) - end - end + include_examples 'checker size exceeded' end end |