diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2015-09-28 15:18:25 +0200 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2015-09-29 10:47:03 +0200 |
commit | 8f8efcfa00c40beacae9886d7be51e82e7a67989 (patch) | |
tree | 2afcfcf856bdc8a538383ee910e3c086b9b71902 /spec | |
parent | 2c1f7ccac8180a49e45fb3cf79e03318420d1037 (diff) | |
download | gitlab-ce-8f8efcfa00c40beacae9886d7be51e82e7a67989.tar.gz |
Fix tests
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/ci/project_spec.rb | 11 | ||||
-rw-r--r-- | spec/requests/ci/builds_spec.rb | 2 | ||||
-rw-r--r-- | spec/requests/ci/commits_spec.rb | 2 |
3 files changed, 6 insertions, 9 deletions
diff --git a/spec/models/ci/project_spec.rb b/spec/models/ci/project_spec.rb index b83eccb94e2..fe12160659c 100644 --- a/spec/models/ci/project_spec.rb +++ b/spec/models/ci/project_spec.rb @@ -28,9 +28,8 @@ require 'spec_helper' describe Ci::Project do - let(:gl_project) { FactoryGirl.create :empty_project, gitlab_ci_project: project } - let (:gl_project) { } - subject { FactoryGirl.build :ci_project } + let(:gl_project) { FactoryGirl.create :empty_project } + subject { FactoryGirl.create :ci_project, gl_project: gl_project } it { is_expected.to have_many(:runner_projects) } it { is_expected.to have_many(:runners) } @@ -40,9 +39,7 @@ describe Ci::Project do it { is_expected.to have_many(:triggers) } it { is_expected.to have_many(:services) } - it { is_expected.to validate_presence_of :name } it { is_expected.to validate_presence_of :timeout } - it { is_expected.to validate_presence_of :default_ref } describe 'before_validation' do it 'should set an random token if none provided' do @@ -78,7 +75,7 @@ describe Ci::Project do it 'returns ordered list of commits' do commit1 = FactoryGirl.create :ci_commit, committed_at: 1.hour.ago, gl_project: project commit2 = FactoryGirl.create :ci_commit, committed_at: 2.hour.ago, gl_project: project - expect(project.commits).to eq([commit2.project, commit1.project]) + expect(project.commits).to eq([commit2, commit1]) end it 'returns commits ordered by committed_at and id, with nulls last' do @@ -86,7 +83,7 @@ describe Ci::Project do commit2 = FactoryGirl.create :ci_commit, committed_at: nil, gl_project: project commit3 = FactoryGirl.create :ci_commit, committed_at: 2.hour.ago, gl_project: project commit4 = FactoryGirl.create :ci_commit, committed_at: nil, gl_project: project - expect(project.commits).to eq([commit2.project, commit4.project, commit3.project, commit1.project]) + expect(project.commits).to eq([commit2, commit4, commit3, commit1]) end end diff --git a/spec/requests/ci/builds_spec.rb b/spec/requests/ci/builds_spec.rb index 32ca52686a9..f68116c52aa 100644 --- a/spec/requests/ci/builds_spec.rb +++ b/spec/requests/ci/builds_spec.rb @@ -8,7 +8,7 @@ describe "Builds" do describe "GET /:project/builds/:id/status.json" do before do - get status_ci_project_build_path(@project, @build), format: :json + get status_ci_project_build_path(@commit.project, @build), format: :json end it { expect(response.status).to eq(200) } diff --git a/spec/requests/ci/commits_spec.rb b/spec/requests/ci/commits_spec.rb index 4d7f132023b..3ab8c915dfd 100644 --- a/spec/requests/ci/commits_spec.rb +++ b/spec/requests/ci/commits_spec.rb @@ -7,7 +7,7 @@ describe "Commits" do describe "GET /:project/refs/:ref_name/commits/:id/status.json" do before do - get status_ci_project_ref_commits_path(@project, @commit.ref, @commit.sha), format: :json + get status_ci_project_ref_commits_path(@commit.project, @commit.ref, @commit.sha), format: :json end it { expect(response.status).to eq(200) } |