Refactor spec to not use truthy or falsey
This commit is contained in:
parent
76469b5082
commit
c7e8f5c613
1 changed files with 58 additions and 58 deletions
|
@ -4,66 +4,66 @@ describe Gitlab::GitRefValidator do
|
||||||
using RSpec::Parameterized::TableSyntax
|
using RSpec::Parameterized::TableSyntax
|
||||||
|
|
||||||
context '.validate' do
|
context '.validate' do
|
||||||
it { expect(described_class.validate('feature/new')).to be_truthy }
|
it { expect(described_class.validate('feature/new')).to be true }
|
||||||
it { expect(described_class.validate('implement_@all')).to be_truthy }
|
it { expect(described_class.validate('implement_@all')).to be true }
|
||||||
it { expect(described_class.validate('my_new_feature')).to be_truthy }
|
it { expect(described_class.validate('my_new_feature')).to be true }
|
||||||
it { expect(described_class.validate('my-branch')).to be_truthy }
|
it { expect(described_class.validate('my-branch')).to be true }
|
||||||
it { expect(described_class.validate('#1')).to be_truthy }
|
it { expect(described_class.validate('#1')).to be true }
|
||||||
it { expect(described_class.validate('feature/refs/heads/foo')).to be_truthy }
|
it { expect(described_class.validate('feature/refs/heads/foo')).to be true }
|
||||||
it { expect(described_class.validate('feature/~new/')).to be_falsey }
|
it { expect(described_class.validate('feature/~new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/^new/')).to be_falsey }
|
it { expect(described_class.validate('feature/^new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/:new/')).to be_falsey }
|
it { expect(described_class.validate('feature/:new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/?new/')).to be_falsey }
|
it { expect(described_class.validate('feature/?new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/*new/')).to be_falsey }
|
it { expect(described_class.validate('feature/*new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/[new/')).to be_falsey }
|
it { expect(described_class.validate('feature/[new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/new/')).to be_falsey }
|
it { expect(described_class.validate('feature/new/')).to be false }
|
||||||
it { expect(described_class.validate('feature/new.')).to be_falsey }
|
it { expect(described_class.validate('feature/new.')).to be false }
|
||||||
it { expect(described_class.validate('feature\@{')).to be_falsey }
|
it { expect(described_class.validate('feature\@{')).to be false }
|
||||||
it { expect(described_class.validate('feature\new')).to be_falsey }
|
it { expect(described_class.validate('feature\new')).to be false }
|
||||||
it { expect(described_class.validate('feature//new')).to be_falsey }
|
it { expect(described_class.validate('feature//new')).to be false }
|
||||||
it { expect(described_class.validate('feature new')).to be_falsey }
|
it { expect(described_class.validate('feature new')).to be false }
|
||||||
it { expect(described_class.validate('refs/heads/')).to be_falsey }
|
it { expect(described_class.validate('refs/heads/')).to be false }
|
||||||
it { expect(described_class.validate('refs/remotes/')).to be_falsey }
|
it { expect(described_class.validate('refs/remotes/')).to be false }
|
||||||
it { expect(described_class.validate('refs/heads/feature')).to be_falsey }
|
it { expect(described_class.validate('refs/heads/feature')).to be false }
|
||||||
it { expect(described_class.validate('refs/remotes/origin')).to be_falsey }
|
it { expect(described_class.validate('refs/remotes/origin')).to be false }
|
||||||
it { expect(described_class.validate('-')).to be_falsey }
|
it { expect(described_class.validate('-')).to be false }
|
||||||
it { expect(described_class.validate('-branch')).to be_falsey }
|
it { expect(described_class.validate('-branch')).to be false }
|
||||||
it { expect(described_class.validate('+foo:bar')).to be_falsey }
|
it { expect(described_class.validate('+foo:bar')).to be false }
|
||||||
it { expect(described_class.validate('foo:bar')).to be_falsey }
|
it { expect(described_class.validate('foo:bar')).to be false }
|
||||||
it { expect(described_class.validate('.tag')).to be_falsey }
|
it { expect(described_class.validate('.tag')).to be false }
|
||||||
it { expect(described_class.validate('my branch')).to be_falsey }
|
it { expect(described_class.validate('my branch')).to be false }
|
||||||
it { expect(described_class.validate("\xA0\u0000\xB0")).to be_falsey }
|
it { expect(described_class.validate("\xA0\u0000\xB0")).to be false }
|
||||||
end
|
end
|
||||||
|
|
||||||
context '.validate_merge_request_branch' do
|
context '.validate_merge_request_branch' do
|
||||||
it { expect(described_class.validate_merge_request_branch('HEAD')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('HEAD')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/new')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('feature/new')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('implement_@all')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('implement_@all')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('my_new_feature')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('my_new_feature')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('my-branch')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('my-branch')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('#1')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('#1')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/refs/heads/foo')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('feature/refs/heads/foo')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/~new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/~new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/^new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/^new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/:new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/:new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/?new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/?new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/*new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/*new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/[new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/[new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/new/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/new/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature/new.')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature/new.')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature\@{')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature\@{')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature\new')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature\new')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature//new')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature//new')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('feature new')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('feature new')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('refs/heads/master')).to be_truthy }
|
it { expect(described_class.validate_merge_request_branch('refs/heads/master')).to be true }
|
||||||
it { expect(described_class.validate_merge_request_branch('refs/heads/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('refs/heads/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('refs/remotes/')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('refs/remotes/')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('-')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('-')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('-branch')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('-branch')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('+foo:bar')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('+foo:bar')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('foo:bar')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('foo:bar')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('.tag')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('.tag')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch('my branch')).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch('my branch')).to be false }
|
||||||
it { expect(described_class.validate_merge_request_branch("\xA0\u0000\xB0")).to be_falsey }
|
it { expect(described_class.validate_merge_request_branch("\xA0\u0000\xB0")).to be false }
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue