percy-capybara 0.4.12 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f0b53ea049a7cd739dd11324879b1d408b3e7e26
-  data.tar.gz: 1f4cec79e9afe311ea950875effc68a50e4f09bd
+  metadata.gz: 4c2619a9098477489f443eed78751adf81401c66
+  data.tar.gz: 0c41b94845f31ddd1fe133c49749cd2bd76b36af
 SHA512:
-  metadata.gz: 35561bd7f8d6dde6ea3b7fbd38f89405f939d02cacd900deb55edd7593a23b5d91296d2ff3ffa40f94df72861000b885298a4618906796d6a399be276b1ae6aa
-  data.tar.gz: b1b58fbcf54f31d651ce15e629fbd777a5b268d4c00066cf37f8fc9439dea2f42e0f1a493ebf1b3726169de35c2bb7ac6ca56e155a2c1b2974ab7032198c6a57
+  metadata.gz: 0dc1613a5a0614815620e004be594d50a833e0ffc2dda8633b9c61ab41a750062fc88f4aea5e7e81b0ba408060637eff0fc28bfd1f20d0727e863a7b1190b78a
+  data.tar.gz: 7f58f3e80ceffcb84feea0403f0f93bfe6d94e66cfbfeb15dd152ba03ebc82907783bb9c8495a6ed289bf0d8509405ae5119a05d2813155d1817314f0933cea4

data/lib/percy/capybara/client.rb

@@ -40,10 +40,12 @@ module Percy
 
       def initialize_loader(options = {})
         if sprockets_environment && sprockets_options
+          Percy.logger.debug { 'Using sprockets_loader to discover assets.' }
           options[:sprockets_environment] = sprockets_environment
           options[:sprockets_options] = sprockets_options
           Percy::Capybara::Loaders::SprocketsLoader.new(options)
         else
+          Percy.logger.debug { 'Using native_loader to discover assets (slower).' }
           Percy::Capybara::Loaders::NativeLoader.new(options)
         end
       end

data/lib/percy/capybara/client/builds.rb

@@ -5,6 +5,7 @@ module Percy
         def current_build(options = {})
           return if !enabled?  # Silently skip if the client is disabled.
           @current_build ||= client.create_build(client.config.repo, options)
+          @current_build
         end
         alias_method :initialize_build, :current_build
 

data/lib/percy/capybara/client/snapshots.rb

@@ -3,6 +3,7 @@ require 'faraday'
 require 'httpclient'
 require 'digest'
 require 'uri'
+require 'time'
 require 'pathname'
 
 module Percy
@@ -18,27 +19,44 @@ module Percy
         #   entirely identify the current state.
         def snapshot(page, options = {})
           return if !enabled?  # Silently skip if the client is disabled.
+
           name = options[:name]
           loader = initialize_loader(page: page)
 
+          Percy.logger.debug { "Snapshot started (name: #{name.inspect})" }
+
           # If this is the first snapshot, create the build and upload build resources.
           if !build_initialized?
+            start = Time.now
             build_resources = loader.build_resources
+            if Percy.config.debug
+              build_resources.each do |build_resource|
+                Percy.logger.debug { "Build resource: #{build_resource.resource_url}" }
+              end
+            end
+            Percy.logger.debug { "All build resources loaded (#{Time.now - start}s)" }
             initialize_build(resources: build_resources)
             upload_missing_build_resources(build_resources)
           end
 
+          start = Time.now
           current_build_id = current_build['data']['id']
           resources = loader.snapshot_resources
           resource_map = {}
-          resources.each { |r| resource_map[r.sha] = r }
+          resources.each do |r|
+            resource_map[r.sha] = r
+            Percy.logger.debug { "Snapshot resource: #{r.resource_url}" }
+          end
+          Percy.logger.debug { "All snapshot resources loaded (#{Time.now - start}s)" }
 
           # Create the snapshot and upload any missing snapshot resources.
+          start = Time.now
           snapshot = client.create_snapshot(current_build_id, resources, name: name)
           snapshot['data']['relationships']['missing-resources']['data'].each do |missing_resource|
             sha = missing_resource['id']
             client.upload_resource(current_build_id, resource_map[sha].content)
           end
+          Percy.logger.debug { "All snapshot resources uploaded (#{Time.now - start}s)" }
 
           # Finalize the snapshot.
           client.finalize_snapshot(snapshot['data']['id'])

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

@@ -148,6 +148,9 @@ module Percy
             # will have no effect.
             resource_url = URI.join(page.current_url, image_url).to_s
 
+            # Skip duplicates.
+            next if resources.find { |r| r.resource_url == resource_url }
+
             next if !_should_include_url?(resource_url)
 
             # Fetch the images.

data/lib/percy/capybara/version.rb

@@ -1,5 +1,5 @@
 module Percy
   module Capybara
-    VERSION = '0.4.12'
+    VERSION = '0.5.0'
   end
 end

data/percy-capybara.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'percy-client', '>= 0.4.1'
+  spec.add_dependency 'percy-client', '>= 0.8.0'
 
   spec.add_development_dependency 'bundler', '~> 1.7'
   spec.add_development_dependency 'rake', '~> 10.0'

data/spec/spec_helper.rb

@@ -47,6 +47,9 @@ RSpec.configure do |config|
     WebMock.disable_net_connect!(allow_localhost: true, allow: [/i.imgur.com/])
   end
 
+  # Cover all debug messages by outputting them in this gem's tests.
+  Percy.config.debug = true
+
   # Start a temp webserver that serves the testdata directory.
   # You can test this server manually by running:
   # ruby -run -e httpd spec/lib/percy/capybara/client/testdata/ -p 9090

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: percy-capybara
 version: !ruby/object:Gem::Version
-  version: 0.4.12
+  version: 0.5.0
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-23 00:00:00.000000000 Z
+date: 2015-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: percy-client
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.4.1
+        version: 0.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.4.1
+        version: 0.8.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement