summaryrefslogtreecommitdiff
path: root/spec/models/build_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/models/build_spec.rb')
-rw-r--r--spec/models/build_spec.rb88
1 files changed, 44 insertions, 44 deletions
diff --git a/spec/models/build_spec.rb b/spec/models/build_spec.rb
index 04f0f0f..cb080d2 100644
--- a/spec/models/build_spec.rb
+++ b/spec/models/build_spec.rb
@@ -30,14 +30,14 @@ describe Build do
let(:commit) { FactoryGirl.create :commit, project: project }
let(:build) { FactoryGirl.create :build, commit: commit }
- it { should belong_to(:commit) }
- it { should validate_presence_of :status }
+ it { is_expected.to belong_to(:commit) }
+ it { is_expected.to validate_presence_of :status }
- it { should respond_to :success? }
- it { should respond_to :failed? }
- it { should respond_to :running? }
- it { should respond_to :pending? }
- it { should respond_to :trace_html }
+ it { is_expected.to respond_to :success? }
+ it { is_expected.to respond_to :failed? }
+ it { is_expected.to respond_to :running? }
+ it { is_expected.to respond_to :pending? }
+ it { is_expected.to respond_to :trace_html }
describe '.first_pending' do
let(:first) { FactoryGirl.create :build, commit: commit, status: 'pending', created_at: Date.yesterday }
@@ -45,8 +45,8 @@ describe Build do
before { first; second }
subject { Build.first_pending }
- it { should be_a(Build) }
- it('returns with the first pending build') { should eq(first) }
+ it { is_expected.to be_a(Build) }
+ it('returns with the first pending build') { is_expected.to eq(first) }
end
describe '.create_from' do
@@ -69,14 +69,14 @@ describe Build do
context 'without started_at' do
before { build.started_at = nil }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
%w(running success failed).each do |status|
context "if build status is #{status}" do
before { build.status = status }
- it { should be_truthy }
+ it { is_expected.to be_truthy }
end
end
@@ -84,7 +84,7 @@ describe Build do
context "if build status is #{status}" do
before { build.status = status }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
end
end
@@ -96,7 +96,7 @@ describe Build do
context "if build.status is #{state}" do
before { build.status = state }
- it { should be_truthy }
+ it { is_expected.to be_truthy }
end
end
@@ -104,7 +104,7 @@ describe Build do
context "if build.status is #{state}" do
before { build.status = state }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
end
end
@@ -116,7 +116,7 @@ describe Build do
context "if build.status is #{state}" do
before { build.status = state }
- it { should be_truthy }
+ it { is_expected.to be_truthy }
end
end
@@ -124,7 +124,7 @@ describe Build do
context "if build.status is #{state}" do
before { build.status = state }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
end
end
@@ -138,13 +138,13 @@ describe Build do
context 'and build.status is success' do
before { build.status = 'success' }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
context 'and build.status is failed' do
before { build.status = 'failed' }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
end
@@ -154,13 +154,13 @@ describe Build do
context 'and build.status is success' do
before { build.status = 'success' }
- it { should be_falsey }
+ it { is_expected.to be_falsey }
end
context 'and build.status is failed' do
before { build.status = 'failed' }
- it { should be_truthy }
+ it { is_expected.to be_truthy }
end
end
end
@@ -168,13 +168,13 @@ describe Build do
describe '#trace' do
subject { build.trace_html }
- it { should be_empty }
+ it { is_expected.to be_empty }
context 'if build.trace contains text' do
let(:text) { 'example output' }
before { build.trace = text }
- it { should include(text) }
+ it { is_expected.to include(text) }
it { expect(subject.length).to be >= text.length }
end
end
@@ -182,13 +182,13 @@ describe Build do
describe '#timeout' do
subject { build.timeout }
- it { should eq(commit.project.timeout) }
+ it { is_expected.to eq(commit.project.timeout) }
end
describe '#duration' do
subject { build.duration }
- it { should eq(120.0) }
+ it { is_expected.to eq(120.0) }
context 'if the building process has not started yet' do
before do
@@ -196,7 +196,7 @@ describe Build do
build.finished_at = nil
end
- it { should be_nil }
+ it { is_expected.to be_nil }
end
context 'if the building process has started' do
@@ -205,8 +205,8 @@ describe Build do
build.finished_at = nil
end
- it { should be_a(Float) }
- it { should > 0.0 }
+ it { is_expected.to be_a(Float) }
+ it { is_expected.to be > 0.0 }
end
end
@@ -221,86 +221,86 @@ describe Build do
}
subject { build.options }
- it { should eq(options) }
+ it { is_expected.to eq(options) }
end
describe '#ref' do
subject { build.ref }
- it { should eq(commit.ref) }
+ it { is_expected.to eq(commit.ref) }
end
describe '#sha' do
subject { build.sha }
- it { should eq(commit.sha) }
+ it { is_expected.to eq(commit.sha) }
end
describe '#short_sha' do
subject { build.short_sha }
- it { should eq(commit.short_sha) }
+ it { is_expected.to eq(commit.short_sha) }
end
describe '#before_sha' do
subject { build.before_sha }
- it { should eq(commit.before_sha) }
+ it { is_expected.to eq(commit.before_sha) }
end
describe '#allow_git_fetch' do
subject { build.allow_git_fetch }
- it { should eq(project.allow_git_fetch) }
+ it { is_expected.to eq(project.allow_git_fetch) }
end
describe '#project' do
subject { build.project }
- it { should eq(commit.project) }
+ it { is_expected.to eq(commit.project) }
end
describe '#project_id' do
subject { build.project_id }
- it { should eq(commit.project_id) }
+ it { is_expected.to eq(commit.project_id) }
end
describe '#project_name' do
subject { build.project_name }
- it { should eq(project.name) }
+ it { is_expected.to eq(project.name) }
end
describe '#repo_url' do
subject { build.repo_url }
- it { should eq(project.repo_url_with_auth) }
+ it { is_expected.to eq(project.repo_url_with_auth) }
end
describe '#extract_coverage' do
context 'valid content & regex' do
subject { build.extract_coverage('Coverage 1033 / 1051 LOC (98.29%) covered', '\(\d+.\d+\%\) covered') }
- it { should eq(98.29) }
+ it { is_expected.to eq(98.29) }
end
context 'valid content & bad regex' do
subject { build.extract_coverage('Coverage 1033 / 1051 LOC (98.29%) covered', 'very covered') }
- it { should be_nil }
+ it { is_expected.to be_nil }
end
context 'no coverage content & regex' do
subject { build.extract_coverage('No coverage for today :sad:', '\(\d+.\d+\%\) covered') }
- it { should be_nil }
+ it { is_expected.to be_nil }
end
context 'multiple results in content & regex' do
subject { build.extract_coverage(' (98.39%) covered. (98.29%) covered', '\(\d+.\d+\%\) covered') }
- it { should eq(98.29) }
+ it { is_expected.to eq(98.29) }
end
end
@@ -314,7 +314,7 @@ describe Build do
]
}
- it { should eq(variables) }
+ it { is_expected.to eq(variables) }
context 'and secure variables' do
let(:secure_variables) {
@@ -327,7 +327,7 @@ describe Build do
build.project.variables << Variable.new(key: 'SECRET_KEY', value: 'secret_value')
end
- it { should eq(variables + secure_variables) }
+ it { is_expected.to eq(variables + secure_variables) }
context 'and trigger variables' do
let(:trigger) { FactoryGirl.create :trigger, project: project }
@@ -342,7 +342,7 @@ describe Build do
build.trigger_request = trigger_request
end
- it { should eq(variables + secure_variables + trigger_variables) }
+ it { is_expected.to eq(variables + secure_variables + trigger_variables) }
end
end
end