codeclimate-services 1.6.1 → 1.7.0

data/test/presenters/pull_requests_presenter_test.rb

@@ -1,70 +0,0 @@
-require "helper"
-require "cc/presenters/pull_requests_presenter"
-
-class TestPullRequestsPresenter < CC::Service::TestCase
-  def test_message_singular
-    assert_equal(
-      "Code Climate found 1 new issue and 1 fixed issue.",
-      build_presenter("fixed" => 1, "new" => 1).success_message
-    )
-  end
-
-  def test_message_plural
-    assert_equal(
-      "Code Climate found 2 new issues and 1 fixed issue.",
-      build_presenter("fixed" => 1, "new" => 2).success_message
-    )
-  end
-
-  def test_message_only_fixed
-    assert_equal(
-      "Code Climate found 1 fixed issue.",
-      build_presenter("fixed" => 1, "new" => 0).success_message
-    )
-  end
-
-  def test_message_only_new
-    assert_equal(
-      "Code Climate found 3 new issues.",
-      build_presenter("fixed" => 0, "new" => 3).success_message
-    )
-  end
-
-  def test_message_no_new_or_fixed
-    assert_equal(
-      "Code Climate didn't find any new or fixed issues.",
-      build_presenter("fixed" => 0, "new" => 0).success_message
-    )
-  end
-
-  def test_message_coverage_same
-    assert_equal(
-      "85% test coverage",
-      build_presenter({}, "covered_percent" => 85, "covered_percent_delta" => 0).coverage_message
-    )
-  end
-
-  def test_message_coverage_up
-    assert_equal(
-      "85.5% test coverage (+2.46%)",
-      build_presenter({}, "covered_percent" => 85.5, "covered_percent_delta" => 2.4567).coverage_message
-    )
-  end
-
-  def test_message_coverage_down
-    assert_equal(
-      "85.35% test coverage (-3%)",
-      build_presenter({}, "covered_percent" => 85.348, "covered_percent_delta" => -3.0).coverage_message
-    )
-  end
-
-private
-
-  def build_payload(issue_counts)
-    { "issue_comparison_counts" => issue_counts }
-  end
-
-  def build_presenter(issue_counts, payload = {})
-    CC::Service::PullRequestsPresenter.new(build_payload(issue_counts).merge(payload))
-  end
-end

data/test/service_test.rb

@@ -1,84 +0,0 @@
-require File.expand_path("../helper", __FILE__)
-
-class TestService < CC::Service::TestCase
-  def test_validates_events
-    assert_raises(ArgumentError) do
-      CC::Service.new(:foo, {}, {}, {})
-    end
-  end
-
-  def test_default_path_to_ca_file
-    s = CC::Service.new({}, {name: "test"})
-    assert_equal(File.expand_path("../../config/cacert.pem", __FILE__), s.ca_file)
-    assert File.exist?(s.ca_file)
-  end
-
-  def test_custom_path_to_ca_file
-    ENV["CODECLIMATE_CA_FILE"] = "/tmp/cacert.pem"
-    s = CC::Service.new({}, {name: "test"})
-    assert_equal("/tmp/cacert.pem", s.ca_file)
-  ensure
-    ENV.delete("CODECLIMATE_CA_FILE")
-  end
-
-  def test_nothing_has_a_handler
-    service = CC::Service.new({}, {name: "test"})
-
-    result = service.receive
-
-    assert_equal false, result[:ok]
-    assert_true true, result[:ignored]
-    assert_equal "No service handler found", result[:message]
-  end
-
-  def test_post_success
-    stub_http("/my/test/url", [200, {}, '{"ok": true, "thing": "123"}'])
-
-    response = service_post("/my/test/url", {token: "1234"}.to_json, {}) do |inner_response|
-      body = JSON.parse(inner_response.body)
-      { thing: body["thing"] }
-    end
-
-    assert_true response[:ok]
-    assert_equal '{"token":"1234"}', response[:params]
-    assert_equal "/my/test/url", response[:endpoint_url]
-    assert_equal 200, response[:status]
-  end
-
-  def test_post_redirect_success
-    stub_http("/my/test/url", [307, {"Location" => "/my/redirect/url"}, '{"ok": false, "redirect": true}'])
-    stub_http("/my/redirect/url", [200, {}, '{"ok": true, "thing": "123"}'])
-
-    response = service_post_with_redirects("/my/test/url", {token: "1234"}.to_json, {}) do |inner_response|
-      body = JSON.parse(inner_response.body)
-      { thing: body["thing"] }
-    end
-
-    assert_true response[:ok]
-    assert_equal '{"token":"1234"}', response[:params]
-    assert_equal "/my/test/url", response[:endpoint_url]
-    assert_equal 200, response[:status]
-  end
-
-  def test_post_http_failure
-    stub_http("/my/wrong/url", [404, {}, ""])
-
-    assert_raises(CC::Service::HTTPError) do
-      service_post("/my/wrong/url", {token: "1234"}.to_json, {})
-    end
-  end
-
-  def test_post_some_other_failure
-    stub_http("/my/wrong/url"){ raise ArgumentError.new("lol") }
-
-    assert_raises(ArgumentError) do
-      service_post("/my/wrong/url", {token: "1234"}.to_json, {})
-    end
-  end
-
-  def test_services
-    services = CC::Service.services
-
-    assert !services.include?(CC::PullRequests)
-  end
-end

data/test/stash_pull_requests_test.rb

@@ -1,146 +0,0 @@
-require File.expand_path('../helper', __FILE__)
-
-class TestStashPullRequests < CC::Service::TestCase
-  def test_receive_test
-    @stubs.get "/rest/api/1.0/users" do
-      [200, {}, "{ 'values': [] }"]
-    end
-
-    response = receive_test
-
-    assert_equal({ ok: true, message: "Test succeeded" }, response)
-  end
-
-  def test_failed_receive_test
-    @stubs.get "/rest/api/1.0/users" do
-      [401, {}, ""]
-    end
-
-    response = receive_test
-
-    assert_equal({ ok: false, message: "API request unsuccessful (401)" }, response)
-  end
-
-  def test_pull_request_status_pending
-    expect_status_update("abc123", {
-      "state" => "INPROGRESS",
-      "description" => /is analyzing/,
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      state: "pending",
-    )
-  end
-
-  def test_pull_request_status_success_detailed
-    expect_status_update("abc123", {
-      "state"       => "SUCCESSFUL",
-      "description" => "Code Climate found 2 new issues and 1 fixed issue.",
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      state: "success",
-    )
-  end
-
-  def test_pull_request_status_failure
-    expect_status_update("abc123", {
-      "state"       => "FAILED",
-      "description" => "Code Climate found 2 new issues and 1 fixed issue.",
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      state: "failure"
-    )
-  end
-
-  def test_pull_request_status_error
-    expect_status_update("abc123", {
-      "state" => "FAILED",
-      "description" => "Code Climate encountered an error attempting to analyze this pull request."
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      state: "error"
-    )
-  end
-
-  def test_pull_request_status_error_message_provided
-    message = "Everything broke."
-
-    expect_status_update("abc123", {
-      "state" => "FAILED",
-      "description" => message
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      message: message,
-      state: "error"
-    )
-  end
-
-  def test_pull_request_status_skipped
-    expect_status_update("abc123", {
-      "state" => "SUCCESSFUL",
-      "description" => "Code Climate has skipped analysis of this commit."
-    })
-
-    receive_pull_request(
-      commit_sha: "abc123",
-      state: "skipped",
-    )
-  end
-
-  def test_failed_receive_pull_request
-    commit_sha = "abc123"
-
-    @stubs.post("/rest/build-status/1.0/commits/#{commit_sha}") do
-      [401, {}, ""]
-    end
-
-    assert_raises(CC::Service::HTTPError) do
-      receive_pull_request(
-        commit_sha: "abc123",
-        state: "success",
-      )
-    end
-  end
-
-  private
-
-  def expect_status_update(commit_sha, params)
-    @stubs.post("/rest/build-status/1.0/commits/#{commit_sha}") do |env|
-      body = JSON.parse(env[:body])
-
-      params.each do |k, v|
-        assert v === body[k],
-          "Unexpected value for #{k}. #{v.inspect} !== #{body[k].inspect}"
-      end
-    end
-  end
-
-  def default_config
-    { domain: "example.com", username: "zaphod", password: "g4rgl3bl4st3r" }
-  end
-
-  def receive_pull_request(event_data, config = {})
-    receive(
-      CC::Service::StashPullRequests,
-      default_config.merge(config),
-      { name: "pull_request", issue_comparison_counts: {'fixed' => 1, 'new' => 2} }.merge(event_data)
-    )
-  end
-
-  def receive_test(config = {}, event_data = {})
-    receive(
-      CC::Service::StashPullRequests,
-      default_config.merge(config),
-      { name: "test" }.merge(event_data)
-    )
-  end
-end

data/test/with_metrics_test.rb

@@ -1,19 +0,0 @@
-# encoding: UTF-8
-
-require File.expand_path('../helper', __FILE__)
-
-class WithMetrics < CC::Service::TestCase
-
-  class FakeInvocation
-    def call
-      raise CC::Service::HTTPError.new("Whoa", {})
-    end
-  end
-
-  def test_statsd_error_key
-    statsd = Object.new
-    statsd.stubs(:timing)
-    statsd.expects("increment").with("services.errors.githubpullrequests.cc-service-http_error")
-    CC::Service::Invocation::WithMetrics.new(FakeInvocation.new, statsd, "githubpullrequests").call rescue CC::Service::HTTPError
-  end
-end