percy-capybara 2.4.5 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e561d868086de68aa397b3e538011fc091dda1b6
-  data.tar.gz: 60546d75deeba4888104d845e044dda785618ffe
+  metadata.gz: a93e613a082603c5384a75f77067ba5f9c54d610
+  data.tar.gz: 78c5eaf278c3216124c5f1f0e1bd8d8bfc9face1
 SHA512:
-  metadata.gz: 24fc1254f9ab3c787044f1a56197cb45c58e94c1444b8e256a5c00fb3262dab1d8ed9b37d05495c0266b8f2df4a0509b47499162e1c4e2bc820c2143b839424d
-  data.tar.gz: e061a97734fceac8e4cb050cbdc492797dc02157496b871ada5d7fd60157f37005cc21eb0cc154441057ddac40cf1f6e7bde940add9c21e1c614c16f3c3daba0
+  metadata.gz: 206802cab6fac788341736da0cd30ff3775d88b4e87d498126fb826025924d94b2e06135bc6d895cecc49e9bd01e850b0f5f993b3b51e933fbfc2747944f0689
+  data.tar.gz: ab66d080a77fbd298288bafe18ca27014f66c62e5e57cc5274209f298962a1e8a202c84bf796c48565c2e6f74d831e759f83895e50fe6f15d634715483b5bff8

data/lib/percy/capybara/anywhere.rb

@@ -0,0 +1,37 @@
+require 'percy/capybara'
+require 'capybara/poltergeist'
+
+module Percy
+  module Capybara
+    # Simple block runner for self-contained Capybara tests.
+    #
+    # Requires:
+    #   - poltergeist gem (which requires phantomjs)
+    # Usage:
+    #   Percy::Capybara::Anywhere.run(SERVER, ASSETS_DIR, ASSETS_BASE_URL) do |page|
+    #     page.visit('/')
+    #     Percy::Capybara.snapshot(page, name: 'main page')
+    #   end
+    module Anywhere
+      def self.run(server, assets_dir, assets_base_url = nil)
+        unless ENV['PERCY_PROJECT'] && ENV['PERCY_TOKEN']
+          raise 'Whoops! You need to setup the PERCY_PROJECT and PERCY_TOKEN environment variables.'
+        end
+
+        ::Capybara.run_server = false
+        ::Capybara.app_host = server
+        page = ::Capybara::Session.new(:poltergeist)
+
+        Percy::Capybara.use_loader(:filesystem, assets_dir: assets_dir, base_url: assets_base_url)
+        build = Percy::Capybara.initialize_build
+
+        yield(page)
+
+        Percy::Capybara.finalize_build
+        puts
+        puts 'Done! Percy snapshots are now processing...'
+        puts "--> #{build['data']['attributes']['web-url']}"
+      end
+    end
+  end
+end

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

@@ -1,3 +1,5 @@
+require 'percy/capybara'
+
 module Percy
   module Capybara
     module Loaders

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

@@ -17,7 +17,7 @@ module Percy
         def initialize(options = {})
           # @assets_dir should point to a _compiled_ static assets directory, not source assets.
           @assets_dir = options[:assets_dir]
-          @base_url = options[:base_url] || ''
+          @base_url = options[:base_url] || '/'
 
           raise ArgumentError, 'assets_dir is required' if @assets_dir.nil? || @assets_dir == ''
           unless Pathname.new(@assets_dir).absolute?
@@ -45,7 +45,7 @@ module Percy
             next if File.size(path) > MAX_FILESIZE_BYTES
 
             # Replace the assets_dir with the base_url to generate the resource_url
-            resource_url = path.sub(@assets_dir, @base_url)
+            resource_url = @base_url.chomp('/') + path.sub(@assets_dir, '')
 
             sha = Digest::SHA256.hexdigest(File.read(path))
             resources << Percy::Client::Resource.new(resource_url, sha: sha, path: path)

data/lib/percy/capybara/version.rb

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

data/spec/lib/percy/capybara/client/builds_spec.rb

@@ -10,6 +10,7 @@ RSpec.describe Percy::Capybara::Client::Builds do
 
     context 'percy is not enabled' do
       let(:enabled) { false }
+
       it 'returns nil if not enabled' do
         expect(capybara_client.initialize_build).to be_nil
       end

data/spec/lib/percy/capybara/client/snapshots_spec.rb

@@ -42,6 +42,7 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
           },
         }
       end
+
       before do
         setup_sprockets(capybara_client)
 

data/spec/lib/percy/capybara/client_spec.rb

@@ -47,6 +47,7 @@ RSpec.describe Percy::Capybara::Client do
   end
   describe '#rescue_connection_failures' do
     let(:capybara_client) { Percy::Capybara::Client.new(enabled: true) }
+
     it 'returns block result on success' do
       result = capybara_client.rescue_connection_failures do
         true

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

@@ -1,24 +1,27 @@
 RSpec.describe Percy::Capybara::Loaders::FilesystemLoader do
   let(:fake_page) { OpenStruct.new(current_url: 'http://localhost/foo') }
   let(:assets_dir) { File.expand_path('../../client/testdata', __FILE__) }
-  let(:base_url) { '/url-prefix' }
+  let(:base_url) { '/url-prefix/' }
   let(:loader) { described_class.new(base_url: base_url, assets_dir: assets_dir, page: fake_page) }
 
   describe 'initialize' do
     context 'assets_dir not specified' do
       let(:assets_dir) { nil }
+
       it 'raises an error' do
         expect { loader }.to raise_error(ArgumentError)
       end
     end
     context 'assets_dir is not an absolute path' do
       let(:assets_dir) { '../../client/testdata' }
+
       it 'raises an error' do
         expect { loader }.to raise_error(ArgumentError)
       end
     end
     context 'assets_dir doesn\'t exist' do
       let(:assets_dir) { File.expand_path('../../client/testdata-doesnt-exist', __FILE__) }
+
       it 'raises an error' do
         expect { loader }.to raise_error(ArgumentError)
       end
@@ -110,9 +113,27 @@ RSpec.describe Percy::Capybara::Loaders::FilesystemLoader do
         ]
         expect(actual_urls).to match_array(expected_urls)
       end
+      it 'works with different base_url configs' do
+        loader = described_class.new(base_url: '/url-prefix/', assets_dir: assets_dir)
+        expected_urls = loader.build_resources.collect(&:resource_url)
+        expect(expected_urls).to include('/url-prefix/css/font.css')
+
+        loader = described_class.new(base_url: '/url-prefix', assets_dir: assets_dir)
+        expected_urls = loader.build_resources.collect(&:resource_url)
+        expect(expected_urls).to include('/url-prefix/css/font.css')
+
+        loader = described_class.new(base_url: '/', assets_dir: assets_dir)
+        expected_urls = loader.build_resources.collect(&:resource_url)
+        expect(expected_urls).to include('/css/font.css')
+
+        loader = described_class.new(base_url: '', assets_dir: assets_dir)
+        expected_urls = loader.build_resources.collect(&:resource_url)
+        expect(expected_urls).to include('/css/font.css')
+      end
     end
     context 'assets_dir with only skippable resources' do
       let(:assets_dir) { File.expand_path('../../client/testdata/assets/images', __FILE__) }
+
       it 'returns an empty list' do
         expect(loader.build_resources).to eq([])
       end

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

@@ -64,6 +64,7 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
   end
   describe 'nonlocal.me', type: :feature, js: true do
     let(:orig_app_host) { Capybara.app_host }
+
     before do
       Capybara.app_host = Capybara.app_host.gsub('http://localhost:', 'http://localtest.me:')
     end
@@ -113,6 +114,7 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
 
     context 'when loader is initialised with asset hostnames' do
       let(:asset_hostnames) { ['dev.local'] }
+
       context 'with the same port' do
         it 'returns in accordance with asset_hostnames' do
           expect(loader._should_include_url?('http://dev.local/')).to eq(true)
@@ -136,6 +138,7 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
     end
     context 'for nonlocal hosts' do
       let(:fake_page) { OpenStruct.new(current_url: 'http://foo:123/') }
+
       it 'returns true for the same host port' do
         expect(loader._should_include_url?('http://foo:123/')).to eq(true)
         expect(loader._should_include_url?('http://foo:123/bar')).to eq(true)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: percy-capybara
 version: !ruby/object:Gem::Version
-  version: 2.4.5
+  version: 2.5.0
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-22 00:00:00.000000000 Z
+date: 2017-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: percy-client
@@ -190,6 +190,7 @@ files:
 - README.md
 - Rakefile
 - lib/percy/capybara.rb
+- lib/percy/capybara/anywhere.rb
 - lib/percy/capybara/client.rb
 - lib/percy/capybara/client/builds.rb
 - lib/percy/capybara/client/snapshots.rb
@@ -262,7 +263,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Percy::Capybara