accesslint-ci 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3840fab63725b20542c997c0cd062b3103f2846c
-  data.tar.gz: 3917337c92e34239f5b96fd067a50bdf40f370b8
+  metadata.gz: 2e7a0f2982e8d6d21d2f016a2b3fdca3f217838d
+  data.tar.gz: 788d55d96d361b8d24ba533b6d35252a8929c5cb
 SHA512:
-  metadata.gz: 8f59a26bb277105362edf16ad7469fc02cd5ad66825c2f012a92ea57397767ff749af95f1959ed84fdb335a00c0d7d22a862ba3b39edd5bd5fa7d86aff2f5ca3
-  data.tar.gz: d69831972181023f729bb2d9615ae6caac33c9446fda4145ffdb3c34a7f28060589f246a41506f74e02ed998fe4638ca13c74f534a12d5fe33ee67c49849c2bd
+  metadata.gz: dc96189566149bc5842b68d03c68a2cd4473f075eb028c51503d687797cdab6c66809c59bc0d5331de05f55cfbdfaf8166a7b282c5309ed7b8888f15c51828d3
+  data.tar.gz: fbc563fe7369de94472b4361317de5b33eaeb7837b4102a5aa749dc9ec364353016d10d117973fc1162465a5da000d39eb4079de29b266a77b7cb88ef79cb89d

data/accesslint-ci.gemspec

@@ -19,13 +19,13 @@ Gem::Specification.new do |spec|
   spec.executables   = %w[accesslint-ci]
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.12"
-  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 "bundler", "~> 1.14"
+  spec.add_development_dependency "climate_control", "~> 0.1.0"
+  spec.add_development_dependency "dotenv", "~> 2.2.0"
+  spec.add_development_dependency "pry-byebug", "~> 3.4.2"
+  spec.add_development_dependency "rake", "~> 12.0.0"
+  spec.add_development_dependency "rspec", "~> 3.5.0"
 
-  spec.add_dependency "rest-client", "~> 2.0"
-  spec.add_dependency "thor", "~> 0.19"
+  spec.add_dependency "rest-client", "~> 2.0.1"
+  spec.add_dependency "thor", "~> 0.19.4"
 end

data/lib/accesslint/ci/cli.rb

@@ -9,34 +9,81 @@ module Accesslint
       option :"compare", type: :string
       option :"outfile", type: :string
       def scan(host)
-        current = Scanner.perform(host: host).split("\n")
+        @host = host
 
-        if !options[:"skip-ci"]
-          existing = []
+        if skip_ci?
+          puts current_errors
+          return
+        elsif pr? && changes?
+          save_diff
+          post_comment
+        end
+      end
 
-          if ENV.fetch("CIRCLE_BRANCH") != "master"
-            if options[:compare]
-              existing = ReadAccesslintLog.perform(options[:compare])
-            elsif !options[:hosted]
-              existing = LogManager.get.split("\n")
-            end
+      no_commands do
+        attr_reader :host
+
+        def skip_ci?
+          options[:"skip-ci"]
+        end
 
-            diff = current - existing
+        def pr?
+          ENV.fetch("CIRCLE_BRANCH") != "master"
+        end
 
-            puts diff
+        def changes?
+          new_diff.any?
+        end
 
-            if diff.any?
-              outfile = options[:outfile] || "tmp/accesslint.diff"
+        def new_diff
+          new_errors - existing_diff
+        end
 
-              File.open(outfile, "w") do |file|
-                file.write(diff.join("\n"))
-              end
+        def new_errors
+          current_errors - baseline_errors
+        end
 
-              Commenter.perform(diff)
-            end
+        def current_errors
+          @current_errors ||= Scanner.perform(host: host).split("\n")
+        end
+
+        def baseline_errors
+          if baseline_file
+            @baseline_errors ||= ReadAccesslintLog.perform(baseline_file)
+          else
+            []
           end
-        else
-          puts current
+        end
+
+        def baseline_file
+          options[:base]
+        end
+
+        def existing_diff
+          if previous_diff_file
+            @existing_diff ||= ReadAccesslintLog.perform(previous_diff_file)
+          else
+            @existing_diff ||= LogManager.get.split("\n")
+          end
+        end
+
+        def previous_diff_file
+          options[:compare]
+        end
+
+        def save_diff
+          WriteAccesslintLog.perform(
+            file_name: new_diff_file,
+            contents: new_diff.join("\n"),
+          )
+        end
+
+        def new_diff_file
+          options[:outfile] || previous_diff_file
+        end
+
+        def post_comment
+          Commenter.perform(new_diff)
         end
       end
     end
@@ -50,5 +97,15 @@ module Accesslint
         end
       end
     end
+
+    class WriteAccesslintLog
+      def self.perform(file_name:, contents:)
+        if File.exist?(file_name)
+          File.open(file_name, "w") do |file|
+            file.write(contents)
+          end
+        end
+      end
+    end
   end
 end

data/lib/accesslint/ci/version.rb

@@ -1,5 +1,5 @@
 module Accesslint
   module Ci
-    VERSION = "0.3.0"
+    VERSION = "0.3.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: accesslint-ci
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Cameron Cundiff
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-15 00:00:00.000000000 Z
+date: 2017-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,112 +16,112 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.12'
+        version: '1.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.12'
+        version: '1.14'
 - !ruby/object:Gem::Dependency
   name: climate_control
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.3
+        version: 0.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.3
+        version: 0.1.0
 - !ruby/object:Gem::Dependency
   name: dotenv
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.1
+        version: 2.2.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.1
+        version: 2.2.0
 - !ruby/object:Gem::Dependency
   name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 3.4.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 3.4.2
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.0.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.5.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: 3.5.0
 - !ruby/object:Gem::Dependency
   name: rest-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 2.0.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 2.0.1
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: 0.19.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: 0.19.4
 description: accesslint-ci runs accessibility tests in CI and comments on corresponding
   GitHub pull requests
 email:
@@ -172,7 +172,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 CI builds