codeclimate-services 1.11.2 → 1.11.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96ac3fb2929dbad8aadc4a01e46ae15dfdbd0f624da568dd171349b664cb9da2
-  data.tar.gz: de09b581fb3a7ec8c7d597d65f06d832acc8f00ecbbde05cc2a4a340ea6e75b2
+  metadata.gz: 9857632046cf9409d85f8ff2429efabf13c6eb5ab1300023d52407dc381deac0
+  data.tar.gz: 65fd4acdcc300cf00472ac8c8df740d0ec75dd382014a8963117680393629578
 SHA512:
-  metadata.gz: 9e75e38ea1dbcda5c960c5310a98ab6cade92b8ec51a54a3a4dfb43249ef04f35d318f2733f51e80920d337ed36c77cfbae14ca60ac5efbcafae68da6e3f1df5
-  data.tar.gz: 5ef15601e7864fd5a4287ac48658bd9e2e60847939e19c1a16f9bacc4836ee1d5eec9363b9648708edffdfa0678f662747e0b80b70792321572c52d0f3cc99ca
+  metadata.gz: 51e15698f1fbe51598067be6452db4b19ff6c4f913392ef172a772d58cb2c548805f569d34f11c74cb9ab486937cc75395104ffc082966b3a2edd7ee5866d832
+  data.tar.gz: 758fea2bc59978563b7a133affb1c8fd1d63d94250e88bbcabcd1d945a406c76845065abe7f249bd4ba33c4249b6620dd129d7ccfb8288a00297aa2309426b0a

data/lib/cc/pull_requests.rb

@@ -6,29 +6,15 @@ class CC::PullRequests < CC::Service
   end
 
   def receive_pull_request
-    setup_http
-    state = @payload["state"]
-
-    if %w[pending success failure skipped error].include?(state) && report_status?
-      send("update_status_#{state}")
-    else
-      @response = simple_failure("Unknown state")
-    end
-
-    response
+    receive_request(%w[pending success failure skipped error], :update_status)
   end
 
   def receive_pull_request_coverage
-    setup_http
-    state = @payload["state"]
-
-    if state == "success" && report_status?
-      update_coverage_status_success
-    else
-      @response = simple_failure("Unknown state")
-    end
+    receive_request("success", :update_coverage_status)
+  end
 
-    response
+  def receive_pull_request_diff_coverage
+    receive_request("skipped", :update_diff_coverage_status)
   end
 
   private
@@ -57,6 +43,10 @@ class CC::PullRequests < CC::Service
     raise NotImplementedError
   end
 
+  def update_diff_coverage_status_skipped
+    raise NotImplementedError
+  end
+
   def update_status_failure
     raise NotImplementedError
   end
@@ -91,6 +81,19 @@ class CC::PullRequests < CC::Service
     end
   end
 
+  def receive_request(*permitted_statuses, call_method)
+    setup_http
+    state = @payload["state"]
+
+    if permitted_statuses.flatten.include?(state) && report_status?
+      send(call_method.to_s + "_#{state}")
+    else
+      @response = simple_failure("Unknown state")
+    end
+
+    response
+  end
+
   def presenter
     CC::Service::PullRequestsPresenter.new(@payload)
   end

data/lib/cc/service.rb

@@ -36,6 +36,7 @@ module CC
       issue
       pull_request
       pull_request_coverage
+      pull_request_diff_coverage
       quality
       snapshot
       test

data/lib/cc/services/github_pull_requests.rb

@@ -71,6 +71,11 @@ class CC::Service::GitHubPullRequests < CC::PullRequests
     update_status("success", presenter.coverage_message, "#{config.context}/coverage")
   end
 
+  def update_diff_coverage_status_skipped
+    update_status("success", presenter.skipped_message, "#{config.context}/diff-coverage")
+    update_status("success", presenter.skipped_message, "#{config.context}/total-coverage")
+  end
+
   def update_status_failure
     update_status("failure", presenter.success_message)
   end

data/lib/cc/services/version.rb

@@ -1,5 +1,5 @@
 module CC
   module Services
-    VERSION = "1.11.2".freeze
+    VERSION = "1.11.3".freeze
   end
 end

data/spec/cc/pull_requests_spec.rb

@@ -0,0 +1,123 @@
+require "spec_helper"
+
+describe CC::PullRequests do
+  shared_examples "receive method" do
+    before do
+      allow(instance).to receive(:report_status?).and_return(true)
+      expect(instance).to receive(:setup_http)
+    end
+
+    context "when the status is valid" do
+      let(:instance) { CC::PullRequests.new({}, name: "test", state: payload_status) }
+      let(:response) do
+        {
+          ok: true,
+          state: 201,
+          message: "Success",
+        }
+      end
+
+      it "calls the corresponding method" do
+        expect(instance).to receive(expected_method_name) do
+          instance.instance_variable_set(:@response, response)
+        end
+        result = instance.send(method_to_call)
+
+        expect(result).to eq(response)
+      end
+
+      context "when report_status? is false" do
+        before { expect(instance).to receive(:report_status?).and_return(false) }
+
+        it "returns unknown status message" do
+          expect(instance).not_to receive(expected_method_name)
+          result = instance.send(method_to_call)
+
+          expect(result).to eq({ ok: false, message: "Unknown state" })
+        end
+      end
+    end
+
+    context "when the status is not valid" do
+      let(:instance) { CC::PullRequests.new({}, name: "test", status: "invalid_status") }
+
+      it "returns unknown status message" do
+        expect(instance).not_to receive(expected_method_name)
+        result = instance.send(method_to_call)
+
+        expect(result).to eq({ ok: false, message: "Unknown state" })
+      end
+    end
+  end
+
+  describe "#receive_test" do
+    let(:instance) { CC::PullRequests.new({}, name: "test") }
+
+    before do
+      expect(instance).to receive(:base_status_url) do |param|
+        "some_url" + param
+      end
+      expect(instance).to receive(:setup_http)
+    end
+
+    it "makes a raw http test post" do
+      expect_any_instance_of(CC::Service::HTTP).to receive(:raw_post).with(
+        "some_url" + ("0" * 40),
+        { state: "success" }.to_json
+      )
+
+      instance.receive_test
+    end
+
+    context "when raising an HTTPError" do
+      context "when message is equal to test_status_code" do
+        it "returns an ok message" do
+          expect(instance).to receive(:test_status_code) { 777 }
+          expect(instance).to receive(:raw_post).
+            and_raise(CC::Service::HTTPError.new("error", status: 777))
+
+          result = instance.receive_test
+          expect(result).to include(
+            ok: true,
+            status: 777,
+            message: "Access token is valid"
+          )
+        end
+      end
+
+      context "when message is different than test_status_code" do
+        it "raises the error" do
+          expect(instance).to receive(:test_status_code) { 777 }
+          expect(instance).to receive(:raw_post).
+            and_raise(CC::Service::HTTPError.new("error", status: 000))
+
+          expect { instance.receive_test }.to raise_error
+        end
+      end
+    end
+  end
+
+  describe "#receive_pull_request" do
+    let(:payload_status) { "skipped" }
+    let(:expected_method_name) { :update_status_skipped }
+    let(:method_to_call) { :receive_pull_request }
+
+    it_behaves_like "receive method"
+  end
+
+  describe "#receive_pull_request_coverage" do
+    let(:payload_status) { "success" }
+    let(:expected_method_name) { :update_coverage_status_success }
+    let(:method_to_call) { :receive_pull_request_coverage }
+
+    it_behaves_like "receive method"
+  end
+
+  describe "#receive_pull_request_diff_coverage" do
+    let(:payload_status) { "skipped" }
+    let(:expected_method_name) { :update_diff_coverage_status_skipped }
+    let(:method_to_call) { :receive_pull_request_diff_coverage }
+
+    it_behaves_like "receive method"
+  end
+end

data/spec/cc/service/github_pull_requests_spec.rb

@@ -82,6 +82,18 @@ describe CC::Service::GitHubPullRequests, type: :service do
       covered_percent_delta: 2.0)
   end
 
+  it "pull request diff coverage skipped" do
+    expect_status_update("pbrisbin/foo", "abc123", "state" => "success",
+      "description" => /skipped analysis/, "context" => /diff-coverage/)
+    expect_status_update("pbrisbin/foo", "abc123", "state" => "success",
+    "description" => /skipped analysis/, "context" => /total-coverage/)
+
+    receive_pull_request_diff_coverage({},
+      github_slug:     "pbrisbin/foo",
+      commit_sha:      "abc123",
+      state:           "skipped")
+  end
+
   it "pull request status test success" do
     http_stubs.post("/repos/pbrisbin/foo/statuses/#{"0" * 40}") { |_env| [422, {}, ""] }
 
@@ -213,6 +225,14 @@ describe CC::Service::GitHubPullRequests, type: :service do
     )
   end
 
+  def receive_pull_request_diff_coverage(config, event_data)
+    service_receive(
+      CC::Service::GitHubPullRequests,
+      { oauth_token: "123" }.merge(config),
+      { name: "pull_request_diff_coverage" }.merge(event_data),
+    )
+  end
+
   def receive_test(config, event_data = {})
     service_receive(
       CC::Service::GitHubPullRequests,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate-services
 version: !ruby/object:Gem::Version
-  version: 1.11.2
+  version: 1.11.3
 platform: ruby
 authors:
 - Bryan Helmkamp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-06-10 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -194,6 +194,7 @@ files:
 - spec/axiom/types/token_spec.rb
 - spec/cc/formatters/snapshot_formatter_spec.rb
 - spec/cc/presenters/pull_requests_presenter_spec.rb
+- spec/cc/pull_requests_spec.rb
 - spec/cc/resolve_spec.rb
 - spec/cc/service/asana_spec.rb
 - spec/cc/service/campfire_spec.rb
@@ -247,6 +248,7 @@ test_files:
 - spec/axiom/types/token_spec.rb
 - spec/cc/formatters/snapshot_formatter_spec.rb
 - spec/cc/presenters/pull_requests_presenter_spec.rb
+- spec/cc/pull_requests_spec.rb
 - spec/cc/resolve_spec.rb
 - spec/cc/service/asana_spec.rb
 - spec/cc/service/campfire_spec.rb