summaryrefslogtreecommitdiff
path: root/qa/spec
diff options
context:
space:
mode:
authorGrzegorz Bizon <grzesiek.bizon@gmail.com>2017-03-13 13:54:33 +0100
committerGrzegorz Bizon <grzesiek.bizon@gmail.com>2017-03-13 13:54:33 +0100
commit0731365b9de9411a937bd786d6e3fd7e10099800 (patch)
treeec4d8356380c2fc898847f0125a12c965e52f7dc /qa/spec
parentcbba91840d7c640f94e604575ce018a44f9937d1 (diff)
downloadgitlab-ce-0731365b9de9411a937bd786d6e3fd7e10099800.tar.gz
Add GitLab QA CE strategy and simplify inflectorfeature/gb/gitlab-qa-integration-tests
Diffstat (limited to 'qa/spec')
-rw-r--r--qa/spec/runtime/release_spec.rb53
1 files changed, 14 insertions, 39 deletions
diff --git a/qa/spec/runtime/release_spec.rb b/qa/spec/runtime/release_spec.rb
index 5d12315be5b..e6b5a8dc315 100644
--- a/qa/spec/runtime/release_spec.rb
+++ b/qa/spec/runtime/release_spec.rb
@@ -1,33 +1,25 @@
describe QA::Runtime::Release do
context 'when release version has extension strategy' do
- subject { described_class.new('VER') }
- let(:strategy) { spy('VER::Strategy') }
+ let(:strategy) { spy('strategy') }
before do
- stub_const('QA::VER::Strategy', strategy)
+ stub_const('QA::CE::Strategy', strategy)
+ stub_const('QA::EE::Strategy', strategy)
end
- describe '#has_strategy?' do
- it 'return true' do
- expect(subject.has_strategy?).to be true
+ describe '#version' do
+ it 'return either CE or EE version' do
+ expect(subject.version).to eq(:CE).or eq(:EE)
end
end
describe '#strategy' do
it 'return the strategy constant' do
- expect(subject.strategy).to eq QA::VER::Strategy
+ expect(subject.strategy).to eq strategy
end
end
describe 'delegated class methods' do
- before do
- allow_any_instance_of(described_class)
- .to receive(:has_strategy?).and_return(true)
-
- allow_any_instance_of(described_class)
- .to receive(:strategy).and_return(strategy)
- end
-
it 'delegates all calls to strategy class' do
described_class.some_method(1, 2)
@@ -38,37 +30,20 @@ describe QA::Runtime::Release do
end
context 'when release version does not have extension strategy' do
- subject { described_class.new('NOVER') }
-
- describe '#has_strategy?' do
- it 'returns false' do
- expect(subject.has_strategy?).to be false
- end
+ before do
+ allow_any_instance_of(described_class)
+ .to receive(:version).and_return('something')
end
describe '#strategy' do
it 'raises error' do
- expect { subject.strategy }.to raise_error(NameError)
+ expect { subject.strategy }.to raise_error(LoadError)
end
end
- describe 'does not delegate class methods' do
- before do
- allow_any_instance_of(described_class)
- .to receive(:has_strategy?).and_return(false)
- end
-
- it 'behaves like a null object and does nothing' do
- expect { described_class.some_method(2, 3) }.not_to raise_error
- end
-
- it 'returns nil' do
- expect(described_class.something).to be_nil
- end
-
- it 'does not delegate to strategy object' do
- expect_any_instance_of(described_class)
- .not_to receive(:strategy)
+ describe 'delegated class methods' do
+ it 'raises error' do
+ expect { described_class.some_method(2, 3) }.to raise_error(LoadError)
end
end
end