Commit e02ca5fd authored by Stefan Hanreich's avatar Stefan Hanreich

Updated spec for build to include new ref_slug invariants

parent 39b2d730
...@@ -1033,19 +1033,27 @@ describe Ci::Build, :models do ...@@ -1033,19 +1033,27 @@ describe Ci::Build, :models do
end end
describe '#ref_slug' do describe '#ref_slug' do
let(:build) { build(:ci_build, ref: "'100%") }
subject { build.ref_slug }
it { is_expected.not_to start_with('-') }
it { is_expected.not_to end_with('-') }
{ {
'master' => 'master', 'master' => 'master',
'1-foo' => '1-foo', '1-foo' => '1-foo',
'fix/1-foo' => 'fix-1-foo', 'fix/1-foo' => 'fix-1-foo',
'fix-1-foo' => 'fix-1-foo', 'fix-1-foo' => 'fix-1-foo',
'a' * 63 => 'a' * 63, 'a' * 63 => 'a' * 63,
'a' * 64 => 'a' * 63, 'a' * 64 => 'a' * 63,
'FOO' => 'foo' 'FOO' => 'foo',
'-' + 'a' * 61 + '-' => 'a' * 61,
'a' * 62 + ' ' => 'a' * 62,
}.each do |ref, slug| }.each do |ref, slug|
it "transforms #{ref} to #{slug}" do it "transforms #{ref} to #{slug}" do
build.ref = ref build.ref = ref
expect(build.ref_slug).to eq(slug) is_expected.to eq(slug)
end end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment