percy-capybara 2.4.0 → 2.4.1

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

@@ -6,26 +6,25 @@ require 'pathname'
 module Percy
   module Capybara
     module Loaders
-
       # Resource loader that looks for resources in the specified folder.
       class FilesystemLoader < BaseLoader
         SKIP_RESOURCE_EXTENSIONS = [
-          '.map',  # Ignore source maps.
-          '.gz',  # Ignore gzipped files.
-        ]
-        MAX_FILESIZE_BYTES = 15 * 1024**2  # 15 MB.
+          '.map', # Ignore source maps.
+          '.gz', # Ignore gzipped files.
+        ].freeze
+        MAX_FILESIZE_BYTES = 15 * 1024**2 # 15 MB.
 
         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] || ''
 
-          raise ArgumentError.new('assets_dir is required') if @assets_dir.nil? || @assets_dir == ''
-          if !Pathname.new(@assets_dir).absolute?
-            raise ArgumentError.new("assets_dir needs to be an absolute path. Received: #{@assets_dir}")
+          raise ArgumentError, 'assets_dir is required' if @assets_dir.nil? || @assets_dir == ''
+          unless Pathname.new(@assets_dir).absolute?
+            raise ArgumentError, "assets_dir needs to be an absolute path. Received: #{@assets_dir}"
           end
-          if !Dir.exist?(@assets_dir)
-            raise ArgumentError.new("assets_dir provided was not found. Received: #{@assets_dir}")
+          unless Dir.exist?(@assets_dir)
+            raise ArgumentError, "assets_dir provided was not found. Received: #{@assets_dir}"
           end
 
           super
@@ -39,7 +38,7 @@ module Percy
           resources = []
           Find.find(@assets_dir).each do |path|
             # Skip directories.
-            next if !FileTest.file?(path)
+            next unless FileTest.file?(path)
             # Skip certain extensions.
             next if SKIP_RESOURCE_EXTENSIONS.include?(File.extname(path))
             # Skip large files, these are hopefully downloads and not used in page rendering.

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

@@ -5,7 +5,6 @@ require 'uri'
 module Percy
   module Capybara
     module Loaders
-
       # Resource loader that uses the native Capybara browser interface to discover resources.
       # 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.
@@ -73,12 +72,13 @@ module Percy
           resource_urls = _evaluate_script(page, script)
 
           resource_urls.each do |url|
-            next if !_should_include_url?(url)
+            next unless _should_include_url?(url)
             response = _fetch_resource_url(url)
             _absolute_url_to_relative!(url, _current_host_port)
-            next if !response
+            next unless response
             resources << Percy::Client::Resource.new(
-              url, mimetype: 'text/css', content: response.body)
+              url, mimetype: 'text/css', content: response.body
+            )
           end
           resources
         end
@@ -92,7 +92,7 @@ module Percy
           # Find all image tags on the page.
           page.all('img').each do |image_element|
             srcs = []
-            srcs << image_element[:src] if !image_element[:src].nil?
+            srcs << image_element[:src] unless image_element[:src].nil?
 
             srcset_raw_urls = image_element[:srcset] || ''
             temp_urls = srcset_raw_urls.split(',')
@@ -152,7 +152,7 @@ module Percy
             # Skip duplicates.
             next if resources.find { |r| r.resource_url == resource_url }
 
-            next if !_should_include_url?(resource_url)
+            next unless _should_include_url?(resource_url)
 
             # Fetch the images.
             # TODO(fotinakis): this can be pretty inefficient for image-heavy pages because the
@@ -161,10 +161,11 @@ module Percy
             # development server, so it may not be so bad. Re-evaluate if this becomes an issue.
             response = _fetch_resource_url(resource_url)
             _absolute_url_to_relative!(resource_url, _current_host_port)
-            next if !response
+            next unless response
 
             resources << Percy::Client::Resource.new(
-              resource_url, mimetype: response.content_type, content: response.body)
+              resource_url, mimetype: response.content_type, content: response.body
+            )
           end
           resources
         end
@@ -173,8 +174,9 @@ module Percy
         # @private
         def _fetch_resource_url(url)
           response = Percy::Capybara::HttpFetcher.fetch(url)
-          if !response
-            STDERR.puts "[percy] Warning: failed to fetch page resource, this might be a bug: #{url}"
+          unless response
+            STDERR.puts '[percy] Warning: failed to fetch page resource, ' \
+              "this might be a bug: #{url}"
             return nil
           end
           response
@@ -216,15 +218,14 @@ module Percy
 
         # @private
         def _same_server?(url, host_port)
-          url.start_with?(host_port + "/") || url == host_port
+          url.start_with?(host_port + '/') || url == host_port
         end
 
         # @private
         def _absolute_url_to_relative!(url, host_port)
-          url.gsub!(host_port + '/','/') if url.start_with?(host_port + "/")
+          url.gsub!(host_port + '/', '/') if url.start_with?(host_port + '/')
         end
         private :_absolute_url_to_relative!
-
       end
     end
   end

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

@@ -14,10 +14,10 @@ module Percy
         attr_reader :sprockets_options
 
         SKIP_RESOURCE_EXTENSIONS = [
-          '.map',  # Ignore source maps.
-          '.gz',  # Ignore gzipped files.
-        ]
-        MAX_FILESIZE_BYTES = 15 * 1024**2  # 15 MB.
+          '.map', # Ignore source maps.
+          '.gz', # Ignore gzipped files.
+        ].freeze
+        MAX_FILESIZE_BYTES = 15 * 1024**2 # 15 MB.
 
         def initialize(options = {})
           @sprockets_environment = options[:sprockets_environment]
@@ -47,7 +47,7 @@ module Percy
 
             if defined?(ActionController)
               # Ask Rails where this asset is (this handles asset_hosts, digest paths, etc.).
-              resource_url = ActionController::Base.helpers.assets_dir(logical_path)
+              resource_url = ActionController::Base.helpers.asset_path(logical_path)
             else
               # TODO: more robust support for Sprockets usage outside Rails, ie Sinatra.
               # How do we find the correct path in that case?
@@ -66,7 +66,7 @@ module Percy
             public_path = _rails.public_path.to_s
             Find.find(public_path).each do |path|
               # Skip directories.
-              next if !FileTest.file?(path)
+              next unless FileTest.file?(path)
               # Skip certain extensions.
               next if SKIP_RESOURCE_EXTENSIONS.include?(File.extname(path))
               # Skip large files, these are hopefully downloads and not used in page rendering.

data/lib/percy/capybara/version.rb

@@ -1,5 +1,5 @@
 module Percy
   module Capybara
-    VERSION = '2.4.0'
+    VERSION = '2.4.1'.freeze
   end
 end

data/percy-capybara.gemspec

@@ -24,7 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'rspec', '~> 3.2'
   spec.add_development_dependency 'capybara', '~> 2.4'
-  spec.add_development_dependency 'capybara-webkit', '>= 1.6'
+  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.

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

@@ -1,10 +1,12 @@
 RSpec.describe Percy::Capybara::Client::Builds do
   let(:enabled) { true }
   let(:capybara_client) { Percy::Capybara::Client.new(enabled: enabled) }
-  let(:builds_api_url) { "https://percy.io/api/v1/repos/#{Percy::Client::Environment.repo}/builds/" }
+  let(:builds_api_url) do
+    "https://percy.io/api/v1/repos/#{Percy::Client::Environment.repo}/builds/"
+  end
 
   describe '#initialize_build', type: :feature, js: true do
-    before(:each) { setup_sprockets(capybara_client) }
+    before { setup_sprockets(capybara_client) }
 
     context 'percy is not enabled' do
       let(:enabled) { false }
@@ -31,7 +33,7 @@ RSpec.describe Percy::Capybara::Client::Builds do
           'id' => '123',
           'type' => 'builds',
           'relationships' => {
-            'self' => "/api/v1/snapshots/123",
+            'self' => '/api/v1/snapshots/123',
             'missing-resources' => {
               'data' => [
                 {
@@ -48,7 +50,7 @@ RSpec.describe Percy::Capybara::Client::Builds do
         .to_return(status: 201, body: mock_response.to_json)
 
       # Stub resource upload.
-      resources_stub = stub_request(:post, "https://percy.io/api/v1/builds/123/resources/")
+      resources_stub = stub_request(:post, 'https://percy.io/api/v1/builds/123/resources/')
         .to_return(status: 201, body: {success: true}.to_json)
       capybara_client.initialize_build
 
@@ -69,7 +71,7 @@ RSpec.describe Percy::Capybara::Client::Builds do
           'id' => '123',
           'type' => 'builds',
           'relationships' => {
-            'self' => "/api/v1/snapshots/123",
+            'self' => '/api/v1/snapshots/123',
             'missing-resources' => {
               'data' => [
                 {
@@ -147,7 +149,7 @@ RSpec.describe Percy::Capybara::Client::Builds do
     end
   end
   describe '#_upload_missing_build_resources', type: :feature, js: true do
-    before(:each) { setup_sprockets(capybara_client) }
+    before { setup_sprockets(capybara_client) }
 
     it 'returns 0 if there are no missing build resources to upload' do
       mock_response = {
@@ -161,8 +163,8 @@ RSpec.describe Percy::Capybara::Client::Builds do
       capybara_client.initialize_build
 
       loader = capybara_client.initialize_loader
-      expect(capybara_client.send(:_upload_missing_build_resources, loader.build_resources)).to eq(0)
+      result = capybara_client.send(:_upload_missing_build_resources, loader.build_resources)
+      expect(result).to eq(0)
     end
   end
-
 end

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

@@ -8,7 +8,6 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
 
   describe '#snapshot', type: :feature, js: true do
     context 'simple page with no resources' do
-      before(:each) { setup_sprockets(capybara_client) }
       let(:loader) { capybara_client.initialize_loader(page: page) }
       let(:build_resource_sha) { loader.build_resources.first.sha }
       let(:snapshot_resource_sha) { loader.snapshot_resources.first.sha }
@@ -18,7 +17,7 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
             'id' => '123',
             'type' => 'builds',
             'relationships' => {
-              'self' => "/api/v1/snapshots/123",
+              'self' => '/api/v1/snapshots/123',
               'missing-resources' => {},
             },
           },
@@ -30,7 +29,7 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
             'id' => '256',
             'type' => 'snapshots',
             'relationships' => {
-              'self' => "/api/v1/snapshots/123",
+              'self' => '/api/v1/snapshots/123',
               'missing-resources' => {
                 'data' => [
                   {
@@ -43,18 +42,20 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
           },
         }
       end
-      before :each do
+      before do
+        setup_sprockets(capybara_client)
+
         visit '/'
-        loader  # Force evaluation now.
+        loader # Force evaluation now.
         repo = Percy::Client::Environment.repo
         stub_request(:post, "https://percy.io/api/v1/repos/#{repo}/builds/")
           .to_return(status: 201, body: mock_build_response.to_json)
         stub_request(:post, 'https://percy.io/api/v1/builds/123/snapshots/')
           .to_return(status: 201, body: mock_snapshot_response.to_json)
-        stub_request(:post, "https://percy.io/api/v1/builds/123/resources/")
+        stub_request(:post, 'https://percy.io/api/v1/builds/123/resources/')
           .with(body: /#{snapshot_resource_sha}/)
           .to_return(status: 201, body: {success: true}.to_json)
-        stub_request(:post, "https://percy.io/api/v1/snapshots/256/finalize")
+        stub_request(:post, 'https://percy.io/api/v1/snapshots/256/finalize')
           .to_return(status: 200, body: {success: true}.to_json)
         capybara_client.initialize_build
       end
@@ -71,11 +72,12 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
       end
       it 'passes through options to the percy client if given' do
         expect(capybara_client.client).to receive(:create_snapshot)
-          .with(anything, anything, {name: 'foo', widths: [320, 1024], enable_javascript: true})
+          .with(anything, anything, name: 'foo', widths: [320, 1024], enable_javascript: true)
           .and_call_original
 
         result = capybara_client.snapshot(
-          page, name: 'foo', widths: [320, 1024], enable_javascript: true)
+          page, name: 'foo', widths: [320, 1024], enable_javascript: true
+        )
         expect(result).to eq(true)
         expect(capybara_client.failed?).to eq(false)
       end
@@ -83,7 +85,7 @@ RSpec.describe Percy::Capybara::Client::Snapshots, type: :feature do
         error = Percy::Client::BadRequestError.new(400, '', '', '', 'snapshot error msg')
         expect(capybara_client.client).to receive(:create_snapshot).and_raise(error)
         expect(capybara_client.snapshot(page)).to eq(nil)
-        expect(capybara_client.failed?).to eq(false)  # Build is not failed.
+        expect(capybara_client.failed?).to eq(false) # Build is not failed.
       end
       it 'safely handles connection errors' do
         expect(capybara_client.client).to receive(:create_snapshot)

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

@@ -6,8 +6,8 @@ RSpec.describe Percy::Capybara::Client do
   end
   describe '#enabled?' do
     context 'when required environment variables set' do
-      before(:each) { set_required_env_variables }
-      after(:each) { clear_percy_env_variables }
+      before { set_required_env_variables }
+      after { clear_percy_env_variables }
 
       it 'is true when PERCY_ENABLE is 1' do
         ENV['PERCY_ENABLE'] = '1'
@@ -28,7 +28,7 @@ RSpec.describe Percy::Capybara::Client do
     end
 
     context 'when required environment variables not set' do
-      before(:each) { clear_percy_env_variables }
+      before { clear_percy_env_variables }
 
       it 'is false' do
         ENV.delete 'PERCY_ENABLE'
@@ -131,7 +131,8 @@ RSpec.describe Percy::Capybara::Client do
 
     context 'when no configuration has been set' do
       it 'returns a NativeLoader' do
-        expect(capybara_client.initialize_loader.class).to eq(Percy::Capybara::Loaders::NativeLoader)
+        expect(capybara_client.initialize_loader.class)
+          .to eq(Percy::Capybara::Loaders::NativeLoader)
       end
     end
 

data/spec/lib/percy/capybara/{httpfetcher_spec.rb → http_fetcher_spec.rb}

@@ -4,7 +4,8 @@ RSpec.describe Percy::Capybara::HttpFetcher do
 
     # Slightly magical hash, just a SHA-256 sum of the image above.
     expect(Digest::SHA256.hexdigest(response.body)).to eq(
-      '4beb51550bef8e9e30d37ea8c13658e99bb01722062f218185e419af5ad93e13')
+      '4beb51550bef8e9e30d37ea8c13658e99bb01722062f218185e419af5ad93e13'
+    )
     expect(response.content_type).to eq('image/png')
   end
   it 'returns nil if fetch failed' do
@@ -14,4 +15,3 @@ RSpec.describe Percy::Capybara::HttpFetcher do
     # expect(Percy::Capybara::HttpFetcher.fetch('bad-url')).to be_nil
   end
 end
-

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

@@ -1,7 +1,7 @@
 IFRAME_PATH = File.expand_path('../../client/testdata/test-iframe.html', __FILE__)
 
 class RackAppWithIframe
-  def self.call(env)
+  def self.call(_env)
     [200, {}, File.read(IFRAME_PATH)]
   end
 end
@@ -75,8 +75,8 @@ RSpec.describe Percy::Capybara::Loaders::BaseLoader do
   end
 
   context 'Rack::Test', type: :feature do
-    before(:each) { Capybara.app = RackAppWithIframe }
-    after(:each) { Capybara.app = nil }
+    before { Capybara.app = RackAppWithIframe }
+    after { Capybara.app = nil }
 
     describe '#iframes_resources' do
       it 'is silently ignored' do
@@ -87,5 +87,3 @@ RSpec.describe Percy::Capybara::Loaders::BaseLoader do
     end
   end
 end
-
-

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

@@ -18,7 +18,7 @@ RSpec.describe Percy::Capybara::Loaders::FilesystemLoader do
       end
     end
     context 'assets_dir doesn\'t exist' do
-      let(:assets_dir) { File.expand_path("../../client/testdata-doesnt-exist", __FILE__) }
+      let(:assets_dir) { File.expand_path('../../client/testdata-doesnt-exist', __FILE__) }
       it 'raises an error' do
         expect { loader }.to raise_error(ArgumentError)
       end
@@ -42,9 +42,12 @@ RSpec.describe Percy::Capybara::Loaders::FilesystemLoader do
   describe '#build_resources' do
     context 'assets_dir including all test files' do
       it 'returns all included assets as resources' do
-        actual_paths = loader.build_resources.collect { |resource| resource.path.gsub(assets_dir, '') }
+        actual_paths = loader.build_resources.collect do |resource|
+          resource.path.gsub(assets_dir, '')
+        end
         expected_paths = [
-          '/assets/css/digested-f3420c6aee71c137a3ca39727052811bae84b2f37d898f4db242e20656a1579e.css',
+          '/assets/css/digested-f3420c6aee71c137a3ca39727052811ba' \
+            'e84b2f37d898f4db242e20656a1579e.css',
           '/css/base.css',
           '/css/digested.css',
           '/css/imports.css',
@@ -72,9 +75,10 @@ RSpec.describe Percy::Capybara::Loaders::FilesystemLoader do
         ]
         expect(actual_paths).to eq(expected_paths)
 
-        expected_urls = loader.build_resources.collect { |resource| resource.resource_url }
+        expected_urls = loader.build_resources.collect(&:resource_url)
         actual_urls = [
-          '/url-prefix/assets/css/digested-f3420c6aee71c137a3ca39727052811bae84b2f37d898f4db242e20656a1579e.css',
+          '/url-prefix/assets/css/digested-f3420c6aee71c137a3ca' \
+            '39727052811bae84b2f37d898f4db242e20656a1579e.css',
           '/url-prefix/css/base.css',
           '/url-prefix/css/digested.css',
           '/url-prefix/css/imports.css',

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

@@ -1,5 +1,5 @@
 RSpec.describe Percy::Capybara::Loaders::NativeLoader do
-  let(:fake_page) { OpenStruct.new(current_url: "http://localhost/foo")}
+  let(:fake_page) { OpenStruct.new(current_url: 'http://localhost/foo') }
   let(:loader) { described_class.new(page: fake_page) }
 
   describe '#build_resources' do
@@ -18,13 +18,13 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
       loader = described_class.new(page: page)
       resource_urls = loader.snapshot_resources.collect(&:resource_url)
       expect(resource_urls).to match_array([
-        "/test-css.html",
-        "/css/base.css",
-        "/css/imports.css",
-        "/css/level0-imports.css",
-        "/css/level1-imports.css",
-        "/css/level2-imports.css",
-        "/css/simple-imports.css",
+        '/test-css.html',
+        '/css/base.css',
+        '/css/imports.css',
+        '/css/level0-imports.css',
+        '/css/level1-imports.css',
+        '/css/level2-imports.css',
+        '/css/simple-imports.css',
       ])
     end
     it 'returns the root HTML and image resources' do
@@ -32,36 +32,36 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
       loader = described_class.new(page: page)
       resource_urls = loader.snapshot_resources.collect(&:resource_url)
       expect(resource_urls).to match_array([
-        "/test-images.html",
-        "/images/img-relative.png",
-        "/images/img-relative-to-root.png",
-        "/images/percy.svg",
-        "/images/srcset-base.png",
-        "/images/srcset-first.png",
-        "/images/srcset-second.png",
-        "/images/bg-relative.png",
-        "/images/bg-relative-to-root.png",
-        "/images/bg-stacked.png"
+        '/test-images.html',
+        '/images/img-relative.png',
+        '/images/img-relative-to-root.png',
+        '/images/percy.svg',
+        '/images/srcset-base.png',
+        '/images/srcset-first.png',
+        '/images/srcset-second.png',
+        '/images/bg-relative.png',
+        '/images/bg-relative-to-root.png',
+        '/images/bg-stacked.png',
       ])
     end
   end
-  describe "nonlocal.me", type: :feature, js: true do
-    before :each do
-      @orig_app_host = Capybara.app_host
+  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
-    after :each do
-      Capybara.app_host = @orig_app_host
+    after do
+      Capybara.app_host = orig_app_host
     end
     it 'returns the root HTML and image resources' do
       visit '/test-localtest-me-images.html'
       loader = described_class.new(page: page)
       resource_urls = loader.snapshot_resources.collect(&:resource_url)
       expect(resource_urls).to eq([
-        "/test-localtest-me-images.html",
-        "/images/img-relative.png"
+        '/test-localtest-me-images.html',
+        '/images/img-relative.png',
       ])
-      expect(loader.snapshot_resources.collect(&:is_root)).to eq([true,nil])
+      expect(loader.snapshot_resources.collect(&:is_root)).to eq([true, nil])
     end
   end
   describe '#_should_include_url?' do
@@ -97,19 +97,19 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
       expect(loader._should_include_url?('http://example.com/foo')).to eq(false)
       expect(loader._should_include_url?('https://example.com/foo')).to eq(false)
     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
+    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)
       end
-      it "returns false for different port" do
+      it 'returns false for different port' do
         expect(loader._should_include_url?('http://foo/')).to eq(false)
         expect(loader._should_include_url?('http://foo/bar')).to eq(false)
         expect(loader._should_include_url?('http://foo:1234/')).to eq(false)
         expect(loader._should_include_url?('http://foo:1234/bar')).to eq(false)
       end
-      it "returns false for different host" do
+      it 'returns false for different host' do
         expect(loader._should_include_url?('http://afoo:123/')).to eq(false)
         expect(loader._should_include_url?('http://afoo:123/bar')).to eq(false)
       end
@@ -146,7 +146,7 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
       expect(resource.sha).to eq(Digest::SHA256.hexdigest(resource.content))
 
       resource = find_resource(resources, '/css/level2-imports.css')
-      expect(resource.content).to include(".colored-by-level2-imports { color: red; }")
+      expect(resource.content).to include('.colored-by-level2-imports { color: red; }')
       expect(resource.sha).to eq(Digest::SHA256.hexdigest(resource.content))
 
       expect(resources.length).to eq(6)
@@ -239,15 +239,15 @@ RSpec.describe Percy::Capybara::Loaders::NativeLoader do
 
       resource_urls = resources.collect(&:resource_url)
       expect(resource_urls).to match_array([
-        "/images/img-relative.png",
-        "/images/img-relative-to-root.png",
-        "/images/percy.svg",
-        "/images/srcset-base.png",
-        "/images/srcset-first.png",
-        "/images/srcset-second.png",
-        "/images/bg-relative.png",
-        "/images/bg-relative-to-root.png",
-        "/images/bg-stacked.png"
+        '/images/img-relative.png',
+        '/images/img-relative-to-root.png',
+        '/images/percy.svg',
+        '/images/srcset-base.png',
+        '/images/srcset-first.png',
+        '/images/srcset-second.png',
+        '/images/bg-relative.png',
+        '/images/bg-relative-to-root.png',
+        '/images/bg-stacked.png',
       ])
       expect(resources.collect(&:is_root).uniq).to match_array([nil])
     end