diff options
Diffstat (limited to 'spec/models/environment_status_spec.rb')
-rw-r--r-- | spec/models/environment_status_spec.rb | 76 |
1 files changed, 38 insertions, 38 deletions
diff --git a/spec/models/environment_status_spec.rb b/spec/models/environment_status_spec.rb index 9da16dea929..127e00b0581 100644 --- a/spec/models/environment_status_spec.rb +++ b/spec/models/environment_status_spec.rb @@ -1,4 +1,4 @@ -require 'spec_helper' +require "spec_helper" describe EnvironmentStatus do include ProjectForksHelper @@ -17,19 +17,19 @@ describe EnvironmentStatus do it { is_expected.to delegate_method(:deployed_at).to(:deployment) } it { is_expected.to delegate_method(:status).to(:deployment) } - describe '#project' do + describe "#project" do subject { environment_status.project } it { is_expected.to eq(project) } end - describe '#merge_request' do + describe "#merge_request" do subject { environment_status.merge_request } it { is_expected.to eq(merge_request) } end - describe '#deployment' do + describe "#deployment" do subject { environment_status.deployment } it { is_expected.to eq(deployment) } @@ -49,25 +49,25 @@ describe EnvironmentStatus do # # - source: /files\/(.+)/ # public: '\1' - describe '#changes' do - it 'contains only added and modified public pages' do + describe "#changes" do + it "contains only added and modified public pages" do expect(environment_status.changes).to contain_exactly( { - path: 'ruby-style-guide.html', - external_url: "#{environment.external_url}/ruby-style-guide.html" + path: "ruby-style-guide.html", + external_url: "#{environment.external_url}/ruby-style-guide.html", }, { - path: 'html/page.html', - external_url: "#{environment.external_url}/html/page.html" + path: "html/page.html", + external_url: "#{environment.external_url}/html/page.html", } ) end end - describe '.for_merge_request' do + describe ".for_merge_request" do let(:admin) { create(:admin) } let(:pipeline) { create(:ci_pipeline, sha: sha) } - it 'is based on merge_request.diff_head_sha' do + it "is based on merge_request.diff_head_sha" do expect(merge_request).to receive(:diff_head_sha) expect(merge_request).not_to receive(:merge_commit_sha) @@ -75,7 +75,7 @@ describe EnvironmentStatus do end end - describe '.after_merge_request' do + describe ".after_merge_request" do let(:admin) { create(:admin) } let(:pipeline) { create(:ci_pipeline, sha: sha) } @@ -83,7 +83,7 @@ describe EnvironmentStatus do merge_request.mark_as_merged! end - it 'is based on merge_request.merge_commit_sha' do + it "is based on merge_request.merge_commit_sha" do expect(merge_request).to receive(:merge_commit_sha) expect(merge_request).not_to receive(:diff_head_sha) @@ -91,14 +91,14 @@ describe EnvironmentStatus do end end - describe '.build_environments_status' do + describe ".build_environments_status" do subject { described_class.send(:build_environments_status, merge_request, user, pipeline) } let!(:build) { create(:ci_build, :deploy_to_production, pipeline: pipeline) } let(:environment) { build.deployment.environment } let(:user) { project.owner } - context 'when environment is created on a forked project' do + context "when environment is created on a forked project" do let(:project) { create(:project, :repository) } let(:forked) { fork_project(project, user, repository: true) } let(:sha) { forked.commit.sha } @@ -106,13 +106,13 @@ describe EnvironmentStatus do let(:merge_request) do create(:merge_request, - source_project: forked, - target_project: project, - target_branch: 'master', - head_pipeline: pipeline) + source_project: forked, + target_project: project, + target_branch: "master", + head_pipeline: pipeline) end - it 'returns environment status' do + it "returns environment status" do expect(subject.count).to eq(1) expect(subject[0].environment).to eq(environment) expect(subject[0].merge_request).to eq(merge_request) @@ -120,48 +120,48 @@ describe EnvironmentStatus do end end - context 'when environment is created on a target project' do + context "when environment is created on a target project" do let(:project) { create(:project, :repository) } let(:sha) { project.commit.sha } let(:pipeline) { create(:ci_pipeline, sha: sha, project: project) } let(:merge_request) do create(:merge_request, - source_project: project, - source_branch: 'feature', - target_project: project, - target_branch: 'master', - head_pipeline: pipeline) + source_project: project, + source_branch: "feature", + target_project: project, + target_branch: "master", + head_pipeline: pipeline) end - it 'returns environment status' do + it "returns environment status" do expect(subject.count).to eq(1) expect(subject[0].environment).to eq(environment) expect(subject[0].merge_request).to eq(merge_request) expect(subject[0].sha).to eq(sha) end - context 'when the build stops an environment' do + context "when the build stops an environment" do let!(:build) { create(:ci_build, :stop_review_app, pipeline: pipeline) } - it 'does not return environment status' do + it "does not return environment status" do expect(subject.count).to eq(0) end end - context 'when user does not have a permission to see the environment' do + context "when user does not have a permission to see the environment" do let(:user) { create(:user) } - it 'does not return environment status' do + it "does not return environment status" do expect(subject.count).to eq(0) end end - context 'when multiple deployments with the same SHA in different environments' do + context "when multiple deployments with the same SHA in different environments" do let(:pipeline2) { create(:ci_pipeline, sha: sha, project: project) } let!(:build2) { create(:ci_build, :start_review_app, pipeline: pipeline2) } - it 'returns deployments related to the head pipeline' do + it "returns deployments related to the head pipeline" do expect(subject.count).to eq(1) expect(subject[0].environment).to eq(environment) expect(subject[0].merge_request).to eq(merge_request) @@ -169,10 +169,10 @@ describe EnvironmentStatus do end end - context 'when multiple deployments in the same pipeline for the same environments' do + context "when multiple deployments in the same pipeline for the same environments" do let!(:build2) { create(:ci_build, :deploy_to_production, pipeline: pipeline) } - it 'returns unique entries' do + it "returns unique entries" do expect(subject.count).to eq(1) expect(subject[0].environment).to eq(environment) expect(subject[0].merge_request).to eq(merge_request) @@ -180,12 +180,12 @@ describe EnvironmentStatus do end end - context 'when environment is stopped' do + context "when environment is stopped" do before do environment.stop! end - it 'does not return environment status' do + it "does not return environment status" do expect(subject.count).to eq(0) end end |