percy-capybara 2.5.0 → 2.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a93e613a082603c5384a75f77067ba5f9c54d610
-  data.tar.gz: 78c5eaf278c3216124c5f1f0e1bd8d8bfc9face1
+  metadata.gz: 6ba34ea074b4869f59bc97456d10244725ab00c4
+  data.tar.gz: c582901df940381843c536adae6b12cf925de398
 SHA512:
-  metadata.gz: 206802cab6fac788341736da0cd30ff3775d88b4e87d498126fb826025924d94b2e06135bc6d895cecc49e9bd01e850b0f5f993b3b51e933fbfc2747944f0689
-  data.tar.gz: ab66d080a77fbd298288bafe18ca27014f66c62e5e57cc5274209f298962a1e8a202c84bf796c48565c2e6f74d831e759f83895e50fe6f15d634715483b5bff8
+  metadata.gz: 8a6b638286d1e7be3567b2e72c8a3ae8ba24e0da8f0cb3491d6caf49b1a1f22424db100ed1d168b335aa86bde8f05b5dc4c7f5f607e55360ba68b3e277b6090d
+  data.tar.gz: 551f496366ba9956bb969d1b9d400cace00bcb87faa62059bb298077d445793e9ffd51216563cc5c97c68dd669871e7d271535cc8d72b84ce4cf8e7d3ad6ac10

data/.rubocop.yml

@@ -58,6 +58,32 @@ Style/TrailingCommaInLiteral:
 RSpec/MessageSpies:
   EnforcedStyle: receive
 
+Style/FileName:
+  Exclude:
+    - 'Gemfile'
+    - 'Guardfile'
+    - 'percy-capybara.gemspec'
+
+Style/EmptyLineAfterMagicComment:
+  Exclude:
+    - 'percy-capybara.gemspec'
+
+Style/PercentLiteralDelimiters:
+  Exclude:
+    - 'percy-capybara.gemspec'
+
+Style/UnneededPercentQ:
+  Exclude:
+    - 'percy-capybara.gemspec'
+
+Style/RegexpLiteral:
+  Exclude:
+    - 'percy-capybara.gemspec'
+
+Style/SymbolArray:
+  Exclude:
+    - 'spec/spec_helper.rb'
+
 # Will be able to do this in >= v1.11
 # RSpec/DescribedClass:
 #   EnforcedStyle: explicit

data/Gemfile

@@ -9,6 +9,6 @@ gem 'guard-rspec', require: false
 # gem 'percy-client', path: '~/src/percy-client'
 
 group :test do
-  gem 'rubocop'
+  gem 'rubocop', '~> 0.48.0'
   gem 'rubocop-rspec'
 end

data/lib/percy/capybara/httpfetcher.rb

@@ -19,7 +19,7 @@ module Percy
         # influenced by any HTTP middleware/restrictions. This helps us avoid causing lots of
         # problems for people using gems like VCR/WebMock. We also disable certificate checking
         # because, as odd as that is, it's the default state for Selenium Firefox and others.
-        output = `curl --insecure -v -o #{temppath} "#{url.shellescape}" 2>&1`
+        output = `curl --compressed --insecure -v -o #{temppath} "#{url.shellescape}" 2>&1`
         content_type = output.match(/< Content-Type:(.*)/i)
         content_type = content_type[1].strip if content_type
 

data/lib/percy/capybara/loaders/native_loader.rb

@@ -9,7 +9,7 @@ module Percy
       # This loader uses JavaScript to discover page resources, so specs must be tagged with
       # "js: true" because the default Rack::Test driver does not support executing JavaScript.
       class NativeLoader < BaseLoader # rubocop:disable ClassLength
-        PATH_REGEX = /\A\/[^\\s\"']*/
+        PATH_REGEX = %r{\A/[^\\s\"']*}
         DATA_URL_REGEX = /\Adata:/
         LOCAL_HOSTNAMES = [
           'localhost',

data/lib/percy/capybara/loaders/sprockets_loader.rb

@@ -93,16 +93,20 @@ module Percy
         end
 
         def _asset_logical_paths
-          # Re-implement the same technique that "rake assets:precompile" uses to generate the
-          # list of asset paths to include in compiled assets. https://goo.gl/sy2R4z
-          # We can't just use environment.each_logical_path without any filters, because then
-          # we will attempt to compile assets before they're rendered (such as _mixins.css).
-          precompile_list = sprockets_options.precompile
-          logical_paths = sprockets_environment.each_logical_path(*precompile_list).to_a
-          logical_paths += precompile_list.flatten.select do |filename|
-            Pathname.new(filename).absolute? if filename.is_a?(String)
+          if _rails && _rails.application.respond_to?(:precompiled_assets)
+            _rails.application.precompiled_assets
+          else
+            # Re-implement the same technique that "rake assets:precompile" uses to generate the
+            # list of asset paths to include in compiled assets. https://goo.gl/sy2R4z
+            # We can't just use environment.each_logical_path without any filters, because then
+            # we will attempt to compile assets before they're rendered (such as _mixins.css).
+            precompile_list = sprockets_options.precompile
+            logical_paths = sprockets_environment.each_logical_path(*precompile_list).to_a
+            logical_paths += precompile_list.flatten.select do |filename|
+              Pathname.new(filename).absolute? if filename.is_a?(String)
+            end
+            logical_paths.uniq
           end
-          logical_paths.uniq
         end
       end
     end

data/lib/percy/capybara/version.rb

@@ -1,5 +1,5 @@
 module Percy
   module Capybara
-    VERSION = '2.5.0'.freeze
+    VERSION = '2.5.1'.freeze
   end
 end

data/percy-capybara.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'capybara-webkit', '>= 1.6', '< 1.12.0' # 1.12.0 breaks specs
   spec.add_development_dependency 'selenium-webdriver'
   spec.add_development_dependency 'webmock', '~> 1'
-  spec.add_development_dependency 'mime-types', '< 3'  # For Ruby 1.9 testing support.
+  spec.add_development_dependency 'mime-types', '< 3' # For Ruby 1.9 testing support.
   spec.add_development_dependency 'faraday', '>= 0.8'
   spec.add_development_dependency 'sprockets', '>= 3.2.0'
 end

data/spec/lib/percy/capybara/loaders/native_loader_spec.rb

@@ -225,7 +225,7 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
       content = File.read(path)
       # In Ruby 1.9.3 the SVG mimetype is not registered so our mini ruby webserver doesn't serve
       # the correct content type. Allow either to work here so we can test older Rubies fully.
-      expect(resource.mimetype).to match(/image\/svg\+xml|application\/octet-stream/)
+      expect(resource.mimetype).to match(%r{image/svg\+xml|application/octet-stream})
       expected_sha = Digest::SHA256.hexdigest(content)
       expect(Digest::SHA256.hexdigest(resource.content)).to eq(expected_sha)
       expect(resource.sha).to eq(expected_sha)

data/spec/lib/percy/capybara/loaders/sprockets_loader_spec.rb

@@ -24,7 +24,7 @@ RSpec.describe Percy::Capybara::Loaders::SprocketsLoader do
   let(:sprockets_options) do
     options = double('options')
     # Set specific files we want to compile. In normal use, this would be all asset files.
-    precompile_list = [/(?:\/|\\|\A)(base|digested)\.(css|js)$|\.map|\.png/]
+    precompile_list = [%r{(?:/|\\|\A)(base|digested)\.(css|js)$|\.map|\.png}]
     allow(options).to receive(:precompile).and_return(precompile_list)
     allow(options).to receive(:digest).and_return(digest_enabled)
     options
@@ -77,6 +77,7 @@ RSpec.describe Percy::Capybara::Loaders::SprocketsLoader do
         # Pretend like we're in a Rails app right now, all we care about is Rails.public_path.
         rails_double = double('Rails')
         # Pretend like the entire testdata directory is the public/ folder.
+        expect(rails_double).to receive(:application).and_return(nil)
         expect(rails_double).to receive(:public_path).and_return(environment.root + '/public')
         expect(loader).to receive(:_rails).at_least(:once).and_return(rails_double)
       end

data/spec/support/test_helpers.rb

@@ -40,7 +40,7 @@ module TestHelpers
     environment.append_path '.'
 
     sprockets_options = double('sprockets_options')
-    allow(sprockets_options).to receive(:precompile).and_return([/(?:\/|\\|\A)base\.(css|js)$/])
+    allow(sprockets_options).to receive(:precompile).and_return([%r{(?:/|\\|\A)base\.(css|js)$}])
     allow(sprockets_options).to receive(:digest).and_return(false)
 
     capybara_client.sprockets_environment = environment

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: percy-capybara
 version: !ruby/object:Gem::Version
-  version: 2.5.0
+  version: 2.5.1
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-25 00:00:00.000000000 Z
+date: 2017-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: percy-client