accesslint-ci 0.1.10 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 32d1a4462f8e21525f57f7b101b9cb3d7d7c2dd5
-  data.tar.gz: f6f6de0991859d1bcb9d0c786b68374e79f8b5ce
+  metadata.gz: 0f1db830fb2a79fb494a3989ca2617fd4754f05c
+  data.tar.gz: 58d1cea942713bae8067dd9925ea0285baef9a97
 SHA512:
-  metadata.gz: 8e4219c9c8cc64ef1cedaa6f586e77480806870a4279f82549e518c7d0e0f99faa3efdbeed18aa432fff358675c8b18a3f52fd49b7c26c8799bae17c28f257a7
-  data.tar.gz: cb4b977cc80bcb05d06add9ac2396af0239840f02f177e8e8d3ec3edfb595a74af9693efbc28da3438e5ba39f1089ee250d3bcb8d5714ffac31f51125dffcc64
+  metadata.gz: e88421d424996769be09d4dc30aeb9c13b0547a957bb01d61385279381dfce3b9aad93ca472b22e411c1ff5ed9eaca6ff9fdeb63542d76838b634789da4c3570
+  data.tar.gz: a795fdf4c7a53781d362b1506eb93c25b9f697d2f38bfbc318228bc8f0278dd0361cd57af82cbb95177cd0fe12213167954f2f095cd2641e8656b26775288aa8

data/.env.test

@@ -2,4 +2,5 @@ CIRCLE_BRANCH=example-branch
 CIRCLE_PROJECT_REPONAME=accesslint-ci
 CIRCLE_PROJECT_USERNAME=accesslint
 CIRCLE_TOKEN=ABC123
-ACCESSLINT_GITHUB_TOKEN=ABC456
+ACCESSLINT_API_TOKEN=ABC456
+ACCESSLINT_GITHUB_USER=example-user

data/README.md

@@ -11,10 +11,7 @@ the pull request that initiated the build in CircleCI.
 
 ## Installation
 
-### Command line (without CircleCI and GitHub)
-
-1. `gem install accesslint-ci`
-1. `accesslint-ci scan --skip-ci <url>`
+1. Authenticate with GitHub at https://accesslint.com to get an API token.
 
 ### Circle CI
 
@@ -33,7 +30,8 @@ general:
 machine:
   environment:
     CIRCLE_TOKEN: <CircleCI API token>
-    ACCESSLINT_GITHUB_TOKEN: <GitHub Personal Access Token>
+    ACCESSLINT_TOKEN: <API token from https://accesslint.com>
+    ACCESSLINT_GITHUB_USER: <GitHub user authenticated at https://accesslint.com>
   node:
     version: 6.1.0
 

data/accesslint-ci.gemspec

@@ -20,11 +20,12 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_development_dependency "bundler", "~> 1.12"
-  spec.add_development_dependency "pry-byebug"
+  spec.add_development_dependency "climate_control", "~> 0.0.3"
+  spec.add_development_dependency "dotenv", "~> 2.1.1"
+  spec.add_development_dependency "pry-byebug", "~> 3.3.0"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "dotenv"
 
-  spec.add_dependency "rest-client"
-  spec.add_dependency "thor"
+  spec.add_dependency "rest-client", "~> 2.0"
+  spec.add_dependency "thor", "~> 0.19"
 end

data/lib/accesslint/ci/commenter.rb

@@ -3,79 +3,64 @@ require "rest-client"
 module Accesslint
   module Ci
     class Commenter
+      class CommenterError < StandardError; end
+
       def self.perform(*args)
         new(*args).perform
       end
 
-      def initialize(diff)
-        @diff = diff
+      def initialize(errors)
+        @errors = errors
       end
 
       def perform
-        RestClient.post(github_uri, payload)
+        RestClient.post(accesslint_service_url, body: payload)
+      rescue CommenterError => e
+        puts e.message
       end
 
       private
 
-      attr_reader :diff
-
-      def github_uri
-        @github_uri ||= "#{github_host}/issues/#{pull_request_number}/comments"
-      end
-
-      def github_host
-        URI.join(
-          "https://#{auth}@api.github.com/",
-          "repos/",
-          project_path,
-        )
+      attr_reader :errors
+
+      def accesslint_service_url
+        @accesslint_service_url ||= URI(
+          File.join([
+            "https://#{authentication}@accesslint.com/api/v1/projects/",
+            project_path,
+            "pulls",
+            pull_request_number,
+            "comments",
+          ])
+        ).to_s
       end
 
-      def auth
-        "accesslint-ci:#{ENV.fetch('ACCESSLINT_GITHUB_TOKEN')}"
+      def authentication
+        "#{ENV.fetch('ACCESSLINT_GITHUB_USER')}:#{ENV.fetch('ACCESSLINT_API_TOKEN')}"
       end
 
       def pull_request_number
         if ENV["CI_PULL_REQUEST"]
           ENV.fetch("CI_PULL_REQUEST").match(/(\d+)/)[0]
         else
-          pull_requests[0].fetch("number")
+          raise CommenterError.new("Failed to comment: missing CI_PULL_REQUEST.")
         end
       end
 
-      def pull_requests
-        @prs ||= JSON.parse(
-          RestClient.get(
-            "#{github_host}/pulls?head=#{pull_request_head}"
-          )
-        )
-      end
-
-      def pull_request_head
-        "#{ENV.fetch('CIRCLE_PROJECT_USERNAME')}:#{ENV.fetch('CIRCLE_BRANCH')}"
-      end
-
       def payload
-        {
-          body: message,
-        }.to_json
+        { errors: errors }.to_json
       end
 
-      def message
-        "Found #{diff.count} new accessibility issues: \n```\n#{snippet}\n```"
-      end
-
-      def snippet
-        diff.join("\n")
+      def github_user
+        ENV.fetch("CIRCLE_PROJECT_USERNAME")
       end
 
       def project_path
         [
-          ENV.fetch("CIRCLE_PROJECT_USERNAME"),
+          github_user,
           ENV.fetch("CIRCLE_PROJECT_REPONAME"),
         ].join("/")
       end
     end
   end
 end
-

data/lib/accesslint/ci/log_manager.rb

@@ -33,7 +33,9 @@ module Accesslint
       end
 
       def artifacts
-        @artifacts ||= JSON.parse(RestClient.get("#{artifacts_uri}?#{query}"))
+        @artifacts ||= JSON.parse(
+          RestClient.get("#{artifacts_uri}?#{query}")
+        )
       rescue RestClient::NotFound
         raise MissingArtifactError.new("No existing artifacts.")
       end

data/lib/accesslint/ci/version.rb

@@ -1,5 +1,5 @@
 module Accesslint
   module Ci
-    VERSION = "0.1.10"
+    VERSION = "0.2.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: accesslint-ci
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.2.1
 platform: ruby
 authors:
 - Cameron Cundiff
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-27 00:00:00.000000000 Z
+date: 2016-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -25,89 +25,103 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.12'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: climate_control
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.3
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: dotenv
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 2.1.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 2.1.1
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.3.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.3.0
 - !ruby/object:Gem::Dependency
-  name: dotenv
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rest-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.19'
 description: accesslint-ci runs accessibility tests in CircleCI and comments on corresponding
   GitHub pull requests
 email:
@@ -159,7 +173,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: Run accessibility tests in CircleCI builds