codeclimate-services 1.6.1 → 1.7.0

data/test/axiom/types/password_test.rb

@@ -1,22 +0,0 @@
-require File.expand_path('../../../helper', __FILE__)
-
-class Axiom::Types::PasswordTest < CC::Service::TestCase
-  class TestConfiguration < CC::Service::Config
-    attribute :password_attribute, Password
-    attribute :string_attribute, String
-  end
-
-  def test_password_type_inference
-    assert_equal(
-      Axiom::Types::Password,
-      TestConfiguration.attribute_set[:password_attribute].type
-    )
-  end
-
-  def test_string_type_inference
-    assert_equal(
-      Axiom::Types::String,
-      TestConfiguration.attribute_set[:string_attribute].type
-    )
-  end
-end

data/test/formatters/snapshot_formatter_test.rb

@@ -1,47 +0,0 @@
-require "helper"
-
-class TestSnapshotFormatter < Test::Unit::TestCase
-  def described_class
-    CC::Formatters::SnapshotFormatter::Base
-  end
-
-  def test_quality_alert_with_new_constants
-    f = described_class.new({"new_constants" => [{"to" => {"rating" => "D"}}], "changed_constants" => []})
-    refute_nil f.alert_constants_payload
-  end
-
-  def test_quality_alert_with_decreased_constants
-    f = described_class.new({"new_constants" => [],
-                             "changed_constants" => [{"to" => {"rating" => "D"}, "from" => {"rating" => "A"}}]
-    })
-    refute_nil f.alert_constants_payload
-  end
-
-  def test_quality_improvements_with_better_ratings
-    f = described_class.new({"new_constants" => [],
-                             "changed_constants" => [{"to" => {"rating" => "A"}, "from" => {"rating" => "D"}}]
-    })
-    refute_nil f.improved_constants_payload
-  end
-
-  def test_nothing_set_without_changes
-    f = described_class.new({"new_constants" => [], "changed_constants" => []})
-    assert_nil f.alert_constants_payload
-    assert_nil f.improved_constants_payload
-  end
-
-  def test_snapshot_formatter_test_with_relaxed_constraints
-    f = CC::Formatters::SnapshotFormatter::Sample.new({
-       "new_constants" => [{"name" => "foo", "to" => {"rating" => "A"}}, {"name" => "bar", "to" => {"rating" => "A"}}],
-       "changed_constants" => [
-         {"from" => {"rating" => "B"}, "to" => {"rating" => "C"}},
-         {"from" => {"rating" => "D"}, "to" => {"rating" => "D"}},
-         {"from" => {"rating" => "D"}, "to" => {"rating" => "D"}},
-         {"from" => {"rating" => "A"}, "to" => {"rating" => "B"}},
-         {"from" => {"rating" => "C"}, "to" => {"rating" => "B"}}
-       ]})
-
-    refute_nil f.alert_constants_payload
-    refute_nil f.improved_constants_payload
-  end
-end

data/test/github_pull_requests_test.rb

@@ -1,217 +0,0 @@
-require File.expand_path('../helper', __FILE__)
-
-class TestGitHubPullRequests < CC::Service::TestCase
-  def test_pull_request_status_pending
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "pending",
-      "description" => /is analyzing/,
-    })
-
-    receive_pull_request({}, {
-      github_slug: "pbrisbin/foo",
-      commit_sha:  "abc123",
-      state:       "pending",
-    })
-  end
-
-  def test_pull_request_status_success_detailed
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "success",
-      "description" => "Code Climate found 2 new issues and 1 fixed issue.",
-    })
-
-    receive_pull_request(
-      {},
-      {
-        github_slug: "pbrisbin/foo",
-        commit_sha:  "abc123",
-        state:       "success"
-      }
-    )
-  end
-
-  def test_pull_request_status_failure
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "failure",
-      "description" => "Code Climate found 2 new issues and 1 fixed issue.",
-    })
-
-    receive_pull_request(
-      {},
-      {
-        github_slug: "pbrisbin/foo",
-        commit_sha:  "abc123",
-        state:       "failure"
-      }
-    )
-  end
-
-  def test_pull_request_status_success_generic
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "success",
-      "description" => /found 2 new issues and 1 fixed issue/,
-    })
-
-    receive_pull_request({}, {
-      github_slug: "pbrisbin/foo",
-      commit_sha:  "abc123",
-      state:       "success",
-    })
-  end
-
-  def test_pull_request_status_error
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "error",
-      "description" => "Code Climate encountered an error attempting to analyze this pull request.",
-    })
-
-    receive_pull_request({}, {
-      github_slug: "pbrisbin/foo",
-      commit_sha:  "abc123",
-      state:       "error",
-      message:     nil,
-    })
-  end
-
-  def test_pull_request_status_error_message_provided
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "error",
-      "description" => "descriptive message",
-    })
-
-    receive_pull_request({}, {
-      github_slug: "pbrisbin/foo",
-      commit_sha:  "abc123",
-      state:       "error",
-      message:     "descriptive message",
-    })
-  end
-
-  def test_pull_request_status_skipped
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "success",
-      "description" => /skipped analysis/,
-    })
-
-    receive_pull_request({}, {
-      github_slug: "pbrisbin/foo",
-      commit_sha:  "abc123",
-      state:       "skipped",
-    })
-  end
-
-  def test_pull_request_coverage_status
-    expect_status_update("pbrisbin/foo", "abc123", {
-      "state"       => "success",
-      "description" => "87% test coverage (+2%)",
-    })
-
-    receive_pull_request_coverage({},
-      github_slug:     "pbrisbin/foo",
-      commit_sha:      "abc123",
-      state:           "success",
-      covered_percent: 87,
-      covered_percent_delta: 2.0,
-    )
-  end
-
-  def test_pull_request_status_test_success
-    @stubs.post("/repos/pbrisbin/foo/statuses/#{"0" * 40}") { |env| [422, {}, ""] }
-
-    assert receive_test({}, { github_slug: "pbrisbin/foo" })[:ok], "Expected test of pull request to be true"
-  end
-
-  def test_pull_request_status_test_doesnt_blow_up_when_unused_keys_present_in_config
-    @stubs.post("/repos/pbrisbin/foo/statuses/#{"0" * 40}") { |env| [422, {}, ""] }
-
-    assert receive_test({ wild_flamingo: true }, { github_slug: "pbrisbin/foo" })[:ok], "Expected test of pull request to be true"
-  end
-
-  def test_pull_request_status_test_failure
-    @stubs.post("/repos/pbrisbin/foo/statuses/#{"0" * 40}") { |env| [401, {}, ""] }
-
-    assert_raises(CC::Service::HTTPError) do
-      receive_test({}, { github_slug: "pbrisbin/foo" })
-    end
-  end
-
-  def test_pull_request_unknown_state
-    response = receive_pull_request({}, { state: "unknown" })
-
-    assert_equal({ ok: false, message: "Unknown state" }, response)
-  end
-
-  def test_different_base_url
-    @stubs.post("/repos/pbrisbin/foo/statuses/#{"0" * 40}") do |env|
-      assert env[:url].to_s == "http://example.com/repos/pbrisbin/foo/statuses/#{"0" * 40}"
-      [422, { "x-oauth-scopes" => "gist, user, repo" }, ""]
-    end
-
-    assert receive_test({ base_url: "http://example.com" }, { github_slug: "pbrisbin/foo" })[:ok], "Expected test of pull request to be true"
-  end
-
-  def test_default_context
-    expect_status_update("gordondiggs/ellis", "abc123", {
-      "context" => "codeclimate",
-      "state" => "pending",
-    })
-
-    receive_pull_request({}, {
-      github_slug: "gordondiggs/ellis",
-      commit_sha:  "abc123",
-      state:       "pending",
-    })
-  end
-
-  def test_different_context
-    expect_status_update("gordondiggs/ellis", "abc123", {
-      "context" => "sup",
-      "state" => "pending",
-    })
-
-    receive_pull_request({ context: "sup" }, {
-      github_slug: "gordondiggs/ellis",
-      commit_sha:  "abc123",
-      state:       "pending",
-    })
-  end
-
-private
-
-  def expect_status_update(repo, commit_sha, params)
-    @stubs.post "repos/#{repo}/statuses/#{commit_sha}" do |env|
-      assert_equal "token 123", env[:request_headers]["Authorization"]
-
-      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 receive_pull_request(config, event_data)
-    receive(
-      CC::Service::GitHubPullRequests,
-      { oauth_token: "123" }.merge(config),
-      { name: "pull_request", issue_comparison_counts: {'fixed' => 1, 'new' => 2} }.merge(event_data)
-    )
-  end
-
-  def receive_pull_request_coverage(config, event_data)
-    receive(
-      CC::Service::GitHubPullRequests,
-      { oauth_token: "123" }.merge(config),
-      { name: "pull_request_coverage", issue_comparison_counts: {'fixed' => 1, 'new' => 2} }.merge(event_data)
-    )
-  end
-
-  def receive_test(config, event_data = {})
-    receive(
-      CC::Service::GitHubPullRequests,
-      { oauth_token: "123" }.merge(config),
-      { name: "test", issue_comparison_counts: {'fixed' => 1, 'new' => 2} }.merge(event_data)
-    )
-  end
-end

data/test/helper.rb

@@ -1,58 +0,0 @@
-require 'test/unit'
-require 'mocha/test_unit'
-require 'pp'
-
-require "codeclimate-test-reporter"
-CodeClimate::TestReporter.start
-
-cwd = File.expand_path(File.dirname(__FILE__))
-require "#{cwd}/../config/load"
-require "#{cwd}/fixtures"
-Dir["#{cwd}/support/*.rb"].sort.each do |helper|
-  require helper
-end
-CC::Service.load_services
-
-
-class CC::Service::TestCase < Test::Unit::TestCase
-  def setup
-    @stubs = Faraday::Adapter::Test::Stubs.new
-
-    I18n.enforce_available_locales = true
-  end
-
-  def teardown
-    @stubs.verify_stubbed_calls
-  end
-
-  def service(klass, data, payload)
-    service = klass.new(data, payload)
-    service.http :adapter => [:test, @stubs]
-    service
-  end
-
-  def receive(*args)
-    service(*args).receive
-  end
-
-  def service_post(*args)
-    service(
-      CC::Service,
-      { data: "my data" },
-      event(:quality, to: "D", from: "C")
-    ).service_post(*args)
-  end
-
-  def service_post_with_redirects(*args)
-    service(
-      CC::Service,
-      { data: "my data" },
-      event(:quality, to: "D", from: "C")
-    ).service_post_with_redirects(*args)
-  end
-
-  def stub_http(url, response = nil, &block)
-    block ||= lambda{|*args| response }
-    @stubs.post(url, &block)
-  end
-end

data/test/invocation_error_handling_test.rb

@@ -1,51 +0,0 @@
-require File.expand_path('../helper', __FILE__)
-
-class InvocationErrorHandling < CC::Service::TestCase
-  def test_success_returns_upstream_result
-    handler = CC::Service::Invocation::WithErrorHandling.new(
-      lambda { :success },
-      FakeLogger.new,
-      "not important"
-    )
-
-    assert_equal :success, handler.call
-  end
-
-  def test_http_errors_return_relevant_data
-    logger = FakeLogger.new
-    env = {
-      status: 401,
-      params: "params",
-      url: "url"
-    }
-
-    handler = CC::Service::Invocation::WithErrorHandling.new(
-      lambda { raise CC::Service::HTTPError.new("foo", env) },
-      logger,
-      "prefix"
-    )
-
-    result = handler.call
-    assert_equal false, result[:ok]
-    assert_equal 401, result[:status]
-    assert_equal "params", result[:params]
-    assert_equal "url", result[:endpoint_url]
-    assert_equal "foo", result[:message]
-    assert_equal "Exception invoking service: [prefix] (CC::Service::HTTPError) foo. Response: <nil>", result[:log_message]
-  end
-
-  def test_error_returns_a_hash_with_explanations
-    logger = FakeLogger.new
-
-    handler = CC::Service::Invocation::WithErrorHandling.new(
-      lambda { raise ArgumentError.new("lol") },
-      logger,
-      "prefix"
-    )
-
-    result = handler.call
-    assert_equal false, result[:ok]
-    assert_equal "lol", result[:message]
-    assert_equal "Exception invoking service: [prefix] (ArgumentError) lol", result[:log_message]
-  end
-end

data/test/invocation_return_values_test.rb

@@ -1,21 +0,0 @@
-require File.expand_path('../helper', __FILE__)
-
-class InvocationReturnValuesTest < CC::Service::TestCase
-  def test_success_returns_upstream_result
-    handler = CC::Service::Invocation::WithReturnValues.new(
-      lambda { :return_value },
-      "error message"
-    )
-
-    assert_equal :return_value, handler.call
-  end
-
-  def test_empty_results_returns_hash
-    handler = CC::Service::Invocation::WithReturnValues.new(
-      lambda { nil },
-      "error message"
-    )
-
-    assert_equal( {ok: false, message: "error message"}, handler.call )
-  end
-end

data/test/pivotal_tracker_test.rb

@@ -1,73 +0,0 @@
-require File.expand_path('../helper', __FILE__)
-
-class TestPivotalTracker < CC::Service::TestCase
-  def test_quality
-    response = assert_pivotal_receives(
-      event(:quality, to: "D", from: "C"),
-      "Refactor User from a D on Code Climate",
-      "https://codeclimate.com/repos/1/feed"
-    )
-    assert_equal "123", response[:id]
-    assert_equal "http://pivotaltracker.com/n/projects/123/stories/123",
-      response[:url]
-  end
-
-  def test_vulnerability
-    assert_pivotal_receives(
-      event(:vulnerability, vulnerabilities: [{
-        "warning_type" => "critical",
-        "location" => "app/user.rb line 120"
-      }]),
-      "New critical issue found in app/user.rb line 120",
-      "A critical vulnerability was found by Code Climate in app/user.rb line 120.\n\nhttps://codeclimate.com/repos/1/feed"
-    )
-  end
-
-  def test_issue
-    payload = {
-      issue: {
-        "check_name" => "Style/LongLine",
-        "description" => "Line is too long [1000/80]"
-      },
-      constant_name: "foo.rb",
-      details_url: "http://example.com/repos/id/foo.rb#issue_123"
-    }
-
-    assert_pivotal_receives(
-      event(:issue, payload),
-      "Fix \"Style/LongLine\" issue in foo.rb",
-      "Line is too long [1000/80]\n\nhttp://example.com/repos/id/foo.rb#issue_123"
-    )
-  end
-
-  def test_receive_test
-    @stubs.post 'services/v3/projects/123/stories' do |env|
-      [200, {}, '<story><id>123</id><url>http://foo.bar</url></story>']
-    end
-
-    response = receive_event(name: "test")
-
-    assert_equal "Ticket <a href='http://foo.bar'>123</a> created.", response[:message]
-  end
-
-  private
-
-  def assert_pivotal_receives(event_data, name, description)
-    @stubs.post 'services/v3/projects/123/stories' do |env|
-      body = Hash[URI.decode_www_form(env[:body])]
-      assert_equal "token", env[:request_headers]["X-TrackerToken"]
-      assert_equal name, body["story[name]"]
-      assert_equal description, body["story[description]"]
-      [200, {}, '<doc><story><id>123</id><url>http://pivotaltracker.com/n/projects/123/stories/123</url></story></doc>']
-    end
-    receive_event(event_data)
-  end
-
-  def receive_event(event_data = nil)
-    receive(
-      CC::Service::PivotalTracker,
-      { api_token: "token", project_id: "123" },
-      event_data || event(:quality, from: "C", to: "D")
-    )
-  end
-end