percy-client 1.3.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 83fb35f9fc640e3977279f706cfd9f89fc0f9fa0
-  data.tar.gz: f10670c00a19ed150df06ef41c679a62f909e7e3
+  metadata.gz: 91dde3564fe8e518402afa26ee2f52d5690ae688
+  data.tar.gz: 4653d42a6fe35bbf4f55e97a4b181ced166b63b8
 SHA512:
-  metadata.gz: 1f69fdabc1807da61eaed7cd76b354c1acb21af65a89c68a38c41ad287862d291e0df4bbb7c5510140638243c21a058669d08a4732f9a006aa745d6ff07e740e
-  data.tar.gz: 0252cd9c150a9fc37d5095bbe5e400a8b30f56289c502643712e794a20f69d851eb336f4dde434465499f8268e9de00e69272f14188b330276657cdfd7fc4f6d
+  metadata.gz: 503bad70215a73b1c78ca2aa617c29f1a84baff8b6d5adee4270ed34885a3ad31b333775062045479e7588d580a9d1db34b6856b0b39f0d59509740057a851de
+  data.tar.gz: 30a2bbcc431217f32c1afd707440641fe166a9d81951680e507c2dcf778a3f27820f4cd82a2213dbd97401ed04c1cb186e0213255df67d1f1d2bf13b843cbbd0

data/.travis.yml

@@ -1,7 +1,11 @@
 language: ruby
+sudo: false
+cache: bundler
 rvm:
   - 1.9.3
   - 2.1.1
   - 2.2.2
   - ruby-head
+before_install:
+  - gem update bundler
 script: bundle exec rspec

data/lib/percy.rb

@@ -41,6 +41,7 @@ module Percy
     client.respond_to?(method_name, include_private) || super
   end if RUBY_VERSION < '1.9'
 
+  # @private
   def self.method_missing(method_name, *args, &block)
     return super if !client.respond_to?(method_name)
     client.send(method_name, *args, &block)

data/lib/percy/client.rb

@@ -32,6 +32,19 @@ module Percy
       end
     end
 
+    class ClientError < HttpError; end  # 4xx;
+    class BadRequestError < ClientError; end  # 400.
+    class UnauthorizedError < ClientError; end  # 401.
+    class PaymentRequiredError < ClientError; end  # 402.
+    class ForbiddenError < ClientError; end  # 403.
+    class NotFoundError < ClientError; end  # 404.
+    class ConflictError < ClientError; end  # 409.
+
+    class ServerError < HttpError; end  # 5xx.
+    class InternalServerError < ServerError; end  # 500.
+    class BadGatewayError < ServerError; end  # 502.
+    class ServiceUnavailableError < ServerError; end  # 503.
+
     attr_reader :config
 
     def initialize(options = {})

data/lib/percy/client/connection.rb

@@ -19,12 +19,32 @@ module Percy
         CLIENT_ERROR_STATUS_RANGE = 400...600
 
         def on_complete(env)
+          error_class = nil
           case env[:status]
-          when CLIENT_ERROR_STATUS_RANGE
-            raise Percy::Client::HttpError.new(
-              env.status, env.method.upcase, env.url, env.body,
-              "Got #{env.status} (#{env.method.upcase} #{env.url}):\n#{env.body}")
+          when 400
+            error_class = Percy::Client::BadRequestError
+          when 401
+            error_class = Percy::Client::UnauthorizedError
+          when 402
+            error_class = Percy::Client::PaymentRequiredError
+          when 403
+            error_class = Percy::Client::ForbiddenError
+          when 404
+            error_class = Percy::Client::NotFoundError
+          when 409
+            error_class = Percy::Client::ConflictError
+          when 500
+            error_class = Percy::Client::InternalServerError
+          when 502
+            error_class = Percy::Client::BadGatewayError
+          when 503
+            error_class = Percy::Client::ServiceUnavailableError
+          when CLIENT_ERROR_STATUS_RANGE  # Catchall.
+            error_class = Percy::Client::HttpError
           end
+          raise error_class.new(
+              env.status, env.method.upcase, env.url, env.body,
+              "Got #{env.status} (#{env.method.upcase} #{env.url}):\n#{env.body}") if error_class
         end
       end
 
@@ -41,8 +61,8 @@ module Percy
         @connection
       end
 
-      def get(path)
-        retries = 3
+      def get(path, options = {})
+        retries = options[:retries] || 3
         begin
           response = connection.get do |request|
             request.url(path)
@@ -63,8 +83,8 @@ module Percy
         JSON.parse(response.body)
       end
 
-      def post(path, data)
-        retries = 3
+      def post(path, data, options = {})
+        retries = options[:retries] || 3
         begin
           response = connection.post do |request|
             request.url(path)
@@ -75,9 +95,8 @@ module Percy
           raise Percy::Client::TimeoutError
         rescue Faraday::ConnectionFailed
           raise Percy::Client::ConnectionFailed
-        rescue Percy::Client::HttpError => e
-          # Retry on 502 errors.
-          if e.status == 502 && (retries -= 1) >= 0
+        rescue Percy::Client::ServerError => e
+          if (retries -= 1) >= 0
             sleep(rand(1..3))
             retry
           end

data/lib/percy/client/resources.rb

@@ -65,7 +65,7 @@ module Percy
         }
         begin
           post("#{config.api_url}/builds/#{build_id}/resources/", data)
-        rescue Percy::Client::HttpError => e
+        rescue Percy::Client::ConflictError => e
           raise e if e.status != 409
           STDERR.puts "[percy] Warning: unnecessary resource reuploaded with SHA-256: #{sha}"
         end

data/lib/percy/client/snapshots.rb

@@ -6,14 +6,15 @@ module Percy
           raise ArgumentError.new(
             'resources argument must be an iterable of Percy::Client::Resource objects')
         end
-        name = options[:name]
-        enable_javascript = options[:enable_javascript]
+
+        widths = options[:widths] || config.default_widths
         data = {
           'data' => {
             'type' => 'snapshots',
             'attributes' => {
-              'name' => name,
-              'enable-javascript' => enable_javascript,
+              'name' => options[:name],
+              'enable-javascript' => options[:enable_javascript],
+              'widths' => widths,
             },
             'relationships' => {
               'resources' => {

data/lib/percy/client/version.rb

@@ -1,5 +1,5 @@
 module Percy
   class Client
-    VERSION = '1.3.0'
+    VERSION = '1.4.0'
   end
 end

data/lib/percy/config.rb

@@ -4,11 +4,18 @@ module Percy
     #   @return [String] Percy repo access token.
     # @!attribute api_url
     #   @return [String] Base URL for API requests. Default: https://percy.io/api/v1/
+    # @!attribute debug
+    #   @return [Boolean] Whether or not to enable debug logging.
+    # @!attribute repo
+    #   @return [String] Git repo name.
+    # @!attribute default_widths
+    #   @return [Array] List of default widths for snapshot rendering unless overridden.
 
     attr_accessor :access_token
     attr_accessor :api_url
     attr_accessor :debug
     attr_accessor :repo
+    attr_accessor :default_widths
 
     # List of configurable keys for {Percy::Client}
     # @return [Array] Option keys.
@@ -18,6 +25,7 @@ module Percy
         :api_url,
         :debug,
         :repo,
+        :default_widths,
       ]
     end
 
@@ -36,5 +44,10 @@ module Percy
     def repo
       @repo ||= Percy::Client::Environment.repo
     end
+
+    # List of default widths sent for every snapshot, unless overridden on a per-snapshot basis.
+    def default_widths
+      @default_widths ||= []
+    end
   end
 end

data/spec/cassettes/Percy_Client_Snapshots/_create_snapshot/overrides_default_widths_if_given.yml

@@ -0,0 +1,120 @@
+---
+http_interactions:
+- request:
+    method: post
+    uri: http://localhost:3000/api/v1/repos/fotinakis/percy-examples/builds/
+    body:
+      encoding: UTF-8
+      string: '{"data":{"type":"builds","attributes":{"commit-sha":"fc4d2c2e6b55d995a005dba2d071e2f2fca5e04b","commit-branch":"master","commit-committed-at":"2015-06-09
+        23:22:31 -0700","commit-author-name":"<COMMIT_AUTHOR_NAME>","commit-author-email":"<COMMIT_AUTHOR_EMAIL>","commit-committer-name":"<COMMIT_AUTHOR_NAME>","commit-committer-email":"<COMMIT_AUTHOR_EMAIL>","commit-message":"Fix
+        test environment issue.","pull-request-number":null}}}'
+    headers:
+      User-Agent:
+      - Faraday v0.9.1
+      Accept:
+      - "*/*"
+      Date:
+      - Thu, 11 Jun 2015 00:20:37 GMT
+      Content-Type:
+      - application/vnd.api+json
+      Authorization:
+      - Token token="<FILTERED_PERCY_TOKEN>"
+  response:
+    status:
+      code: 201
+      message: Created
+    headers:
+      Date:
+      - Thu, 11 Jun 2015 00:20:37 GMT
+      Status:
+      - 201 Created
+      Connection:
+      - close
+      X-Frame-Options:
+      - SAMEORIGIN
+      X-Xss-Protection:
+      - 1; mode=block
+      X-Content-Type-Options:
+      - nosniff
+      Access-Control-Allow-Origin:
+      - "*"
+      Access-Control-Allow-Methods:
+      - GET, POST, PUT, PATCH, DELETE, OPTIONS
+      Access-Control-Allow-Headers:
+      - Authorization, Content-Type
+      Cache-Control:
+      - no-cache, no-store, max-age=0, must-revalidate
+      Expires:
+      - Thu, 01 Jan 1970 00:00:00 GMT
+      Content-Type:
+      - application/json; charset=utf-8
+      X-Request-Id:
+      - 998e5483-89b7-4118-aa43-63883455d31b
+      X-Runtime:
+      - '0.042738'
+      Transfer-Encoding:
+      - chunked
+    body:
+      encoding: UTF-8
+      string: '{"data":{"id":"28","type":"builds","attributes":{"build-number":28,"state":"pending","is-pull-request":false,"pull-request-number":0,"pull-request-title":null,"approved-at":null,"created-at":"2015-06-11T00:20:37.863Z","updated-at":"2015-06-11T00:20:37.863Z"},"links":{"self":"/api/v1/builds/28"},"relationships":{"commit":{"links":{"self":"/api/v1/builds/28/relationships/commit","related":"/api/v1/builds/28/commit"},"data":{"type":"commits","id":"2"}},"repo":{"links":{"self":"/api/v1/builds/28/relationships/repo","related":"/api/v1/builds/28/repo"}},"base-build":{"links":{"self":"/api/v1/builds/28/relationships/base-build","related":"/api/v1/builds/28/base-build"},"data":{"type":"builds","id":"22"}},"approved-by":{"links":{"self":"/api/v1/builds/28/relationships/approved-by","related":"/api/v1/builds/28/approved-by"}},"snapshots":{"links":{"self":"/api/v1/builds/28/relationships/snapshots","related":"/api/v1/builds/28/snapshots"}},"comparisons":{"links":{"self":"/api/v1/builds/28/relationships/comparisons","related":"/api/v1/builds/28/comparisons"}},"missing-resources":{"links":{"self":"/api/v1/builds/28/relationships/missing-resources","related":"/api/v1/builds/28/missing-resources"}}},"meta":{"finalize-link":"/api/v1/builds/28/finalize","approve-link":"/api/v1/builds/28/approve"}},"included":[{"id":"2","type":"commits","attributes":{"sha":"fc4d2c2e6b55d995a005dba2d071e2f2fca5e04b","branch":"master","message":"Fix
+        test environment issue.","committed-at":"2015-06-09 23:22:31 -0700","author-name":"<COMMIT_AUTHOR_NAME>","committer-name":"<COMMIT_AUTHOR_NAME>","created-at":"2015-06-11T00:15:24.000Z","updated-at":"2015-06-11T00:15:24.000Z"},"links":{"self":"/api/v1/commits/2"}},{"id":"22","type":"builds","attributes":{"build-number":22,"state":"finished","is-pull-request":false,"pull-request-number":0,"pull-request-title":null,"approved-at":null,"created-at":"2015-06-11T00:17:46.000Z","updated-at":"2015-06-11T00:17:46.000Z"},"links":{"self":"/api/v1/builds/22"},"relationships":{"commit":{"links":{"self":"/api/v1/builds/22/relationships/commit","related":"/api/v1/builds/22/commit"},"data":{"type":"commits","id":"2"}},"repo":{"links":{"self":"/api/v1/builds/22/relationships/repo","related":"/api/v1/builds/22/repo"}},"base-build":{"links":{"self":"/api/v1/builds/22/relationships/base-build","related":"/api/v1/builds/22/base-build"}},"approved-by":{"links":{"self":"/api/v1/builds/22/relationships/approved-by","related":"/api/v1/builds/22/approved-by"}},"snapshots":{"links":{"self":"/api/v1/builds/22/relationships/snapshots","related":"/api/v1/builds/22/snapshots"}},"comparisons":{"links":{"self":"/api/v1/builds/22/relationships/comparisons","related":"/api/v1/builds/22/comparisons"}},"missing-resources":{"links":{"self":"/api/v1/builds/22/relationships/missing-resources","related":"/api/v1/builds/22/missing-resources"}}},"meta":{"finalize-link":"/api/v1/builds/22/finalize","approve-link":"/api/v1/builds/22/approve"}}]}'
+    http_version: 
+  recorded_at: Thu, 11 Jun 2015 00:20:37 GMT
+- request:
+    method: post
+    uri: http://localhost:3000/api/v1/builds/28/snapshots/
+    body:
+      encoding: UTF-8
+      string: '{"data":{"type":"snapshots","attributes":{"name":"homepage"},"relationships":{"resources":{"data":[{"type":"resources","id":"8d68af40ce7ee4591a7df49d1a40db8bf0b6535a37c896eda504f963c622535a","attributes":{"resource-url":"/foo/test.html","mimetype":null,"is-root":true}},{"type":"resources","id":"8d68af40ce7ee4591a7df49d1a40db8bf0b6535a37c896eda504f963c622535a","attributes":{"resource-url":"/css/test.css","mimetype":null,"is-root":null}}]}}}}'
+    headers:
+      User-Agent:
+      - Faraday v0.9.1
+      Accept:
+      - "*/*"
+      Date:
+      - Thu, 11 Jun 2015 00:20:37 GMT
+      Content-Type:
+      - application/vnd.api+json
+      Authorization:
+      - Token token="<FILTERED_PERCY_TOKEN>"
+  response:
+    status:
+      code: 201
+      message: Created
+    headers:
+      Date:
+      - Thu, 11 Jun 2015 00:20:38 GMT
+      Status:
+      - 201 Created
+      Connection:
+      - close
+      X-Frame-Options:
+      - SAMEORIGIN
+      X-Xss-Protection:
+      - 1; mode=block
+      X-Content-Type-Options:
+      - nosniff
+      Access-Control-Allow-Origin:
+      - "*"
+      Access-Control-Allow-Methods:
+      - GET, POST, PUT, PATCH, DELETE, OPTIONS
+      Access-Control-Allow-Headers:
+      - Authorization, Content-Type
+      Cache-Control:
+      - no-cache, no-store, max-age=0, must-revalidate
+      Expires:
+      - Thu, 01 Jan 1970 00:00:00 GMT
+      Content-Type:
+      - application/json; charset=utf-8
+      X-Request-Id:
+      - 54eed09b-c2ca-4378-a52b-5d5003f6f7ec
+      X-Runtime:
+      - '0.212265'
+      Transfer-Encoding:
+      - chunked
+    body:
+      encoding: UTF-8
+      string: '{"data":{"id":"1110","type":"snapshots","attributes":{"name":"homepage","created-at":"2015-06-11T00:20:38.085Z","updated-at":"2015-06-11T00:20:38.085Z","finished-processing-at":null},"links":{"self":"/api/v1/snapshots/1110"},"relationships":{"build":{"links":{"self":"/api/v1/snapshots/1110/relationships/build","related":"/api/v1/snapshots/1110/build"}},"screenshots":{"links":{"self":"/api/v1/snapshots/1110/relationships/screenshots","related":"/api/v1/snapshots/1110/screenshots"}},"missing-resources":{"links":{"self":"/api/v1/snapshots/1110/relationships/missing-resources","related":"/api/v1/snapshots/1110/missing-resources"},"data":[{"type":"resources","id":"8d68af40ce7ee4591a7df49d1a40db8bf0b6535a37c896eda504f963c622535a"}]}}},"included":[{"id":"8d68af40ce7ee4591a7df49d1a40db8bf0b6535a37c896eda504f963c622535a","type":"resources","attributes":{},"links":{"self":"/api/v1/resources/8d68af40ce7ee4591a7df49d1a40db8bf0b6535a37c896eda504f963c622535a"}}]}'
+    http_version: 
+  recorded_at: Thu, 11 Jun 2015 00:20:38 GMT
+recorded_with: VCR 2.9.3

data/spec/lib/percy/client/connection_spec.rb

@@ -35,7 +35,7 @@ RSpec.describe Percy::Client::Connection do
         .to_return(body: {foo: true}.to_json, status: 502).times(3)
       expect do
         Percy.client.get("#{Percy.config.api_url}/test")
-      end.to raise_error(Percy::Client::HttpError)
+      end.to raise_error(Percy::Client::BadGatewayError)
     end
   end
   describe '#post' do
@@ -56,9 +56,48 @@ RSpec.describe Percy::Client::Connection do
         Percy.client.post("#{Percy.config.api_url}/test", {})
       end.to raise_error(Percy::Client::ConnectionFailed)
     end
-    it 'retries on 502 errors' do
+    it 'raises custom error classes for some HTTP errors' do
       stub_request(:post, "#{Percy.config.api_url}/test")
-        .to_return(body: {foo: true}.to_json, status: 502)
+        .to_return(body: {foo: true}.to_json, status: 400)
+        .then.to_return(body: {foo: true}.to_json, status: 401)
+        .then.to_return(body: {foo: true}.to_json, status: 402)
+        .then.to_return(body: {foo: true}.to_json, status: 403)
+        .then.to_return(body: {foo: true}.to_json, status: 404)
+        .then.to_return(body: {foo: true}.to_json, status: 409)
+        .then.to_return(body: {foo: true}.to_json, status: 500)
+        .then.to_return(body: {foo: true}.to_json, status: 502)
+        .then.to_return(body: {foo: true}.to_json, status: 503)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::BadRequestError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::UnauthorizedError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::PaymentRequiredError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::ForbiddenError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::NotFoundError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::ConflictError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::InternalServerError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::BadGatewayError)
+      expect do
+        Percy.client.post("#{Percy.config.api_url}/test", {}, retries: 0)
+      end.to raise_error(Percy::Client::ServiceUnavailableError)
+    end
+    it 'retries on server errors' do
+      stub_request(:post, "#{Percy.config.api_url}/test")
+        .to_return(body: {foo: true}.to_json, status: 500)
         .then.to_return(body: {foo: true}.to_json, status: 200)
 
       data = Percy.client.post("#{Percy.config.api_url}/test", {})
@@ -69,7 +108,7 @@ RSpec.describe Percy::Client::Connection do
         .to_return(body: {foo: true}.to_json, status: 502).times(3)
       expect do
         Percy.client.post("#{Percy.config.api_url}/test", {})
-      end.to raise_error(Percy::Client::HttpError)
+      end.to raise_error(Percy::Client::BadGatewayError)
     end
   end
 end

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

@@ -8,6 +8,46 @@ RSpec.describe Percy::Client::Snapshots, :vcr do
       resources = []
       resources << Percy::Client::Resource.new('/foo/test.html', sha: sha, is_root: true)
       resources << Percy::Client::Resource.new('/css/test.css', sha: sha)
+
+      # Whitebox test to catch POST data that is sent but is not returned in the API response.
+      expect_any_instance_of(Percy::Client).to \
+        receive(:post)
+        .with(/snapshots\/$/, {
+          'data' => {
+            'type' => 'snapshots',
+            'attributes' => {
+              'name' => 'homepage',
+              'enable-javascript' => true,
+              'widths' => Percy.config.default_widths,
+            },
+            'relationships' => {
+              'resources' => {
+                'data' => [
+                  {
+                    'type' => 'resources',
+                    'id' => kind_of(String),
+                    'attributes' => {
+                      'resource-url' => '/foo/test.html',
+                      'mimetype' => nil,
+                      'is-root' => true,
+                    },
+                  },
+                  {
+                    'type' => 'resources',
+                    'id' => kind_of(String),
+                    'attributes' => {
+                      'resource-url' => '/css/test.css',
+                      'mimetype' => nil,
+                      'is-root' => nil,
+                    },
+                  },
+                ],
+              },
+            },
+          },
+        })
+        .and_call_original
+
       snapshot = Percy.create_snapshot(
         build['data']['id'],
         resources,
@@ -21,6 +61,64 @@ RSpec.describe Percy::Client::Snapshots, :vcr do
       expect(snapshot['data']['attributes']['name']).to eq('homepage')
       expect(snapshot['data']['relationships']['missing-resources']).to be
     end
+    it 'overrides default_widths if given' do
+      build = Percy.create_build('fotinakis/percy-examples')
+      resources = []
+      resources << Percy::Client::Resource.new('/foo/test.html', sha: sha, is_root: true)
+      resources << Percy::Client::Resource.new('/css/test.css', sha: sha)
+
+      # Whitebox test to catch POST data that is sent but is not returned in the API response.
+      expect_any_instance_of(Percy::Client).to \
+        receive(:post)
+        .with(/snapshots\/$/, {
+          'data' => {
+            'type' => 'snapshots',
+            'attributes' => {
+              'name' => 'homepage',
+              'enable-javascript' => nil,
+              'widths' => [320, 1280],
+            },
+            'relationships' => {
+              'resources' => {
+                'data' => [
+                  {
+                    'type' => 'resources',
+                    'id' => kind_of(String),
+                    'attributes' => {
+                      'resource-url' => '/foo/test.html',
+                      'mimetype' => nil,
+                      'is-root' => true,
+                    },
+                  },
+                  {
+                    'type' => 'resources',
+                    'id' => kind_of(String),
+                    'attributes' => {
+                      'resource-url' => '/css/test.css',
+                      'mimetype' => nil,
+                      'is-root' => nil,
+                    },
+                  },
+                ],
+              },
+            },
+          },
+        })
+        .and_call_original
+
+      snapshot = Percy.create_snapshot(
+        build['data']['id'],
+        resources,
+        name: 'homepage',
+        widths: [320, 1280],
+      )
+
+      expect(snapshot['data']).to be
+      expect(snapshot['data']['id']).to be
+      expect(snapshot['data']['type']).to eq('snapshots')
+      expect(snapshot['data']['attributes']['name']).to eq('homepage')
+      expect(snapshot['data']['relationships']['missing-resources']).to be
+    end
     it 'fails if no resources are given' do
       build = Percy.create_build('fotinakis/percy-examples')
       expect do

data/spec/lib/percy/client_spec.rb

@@ -1,19 +1,9 @@
 RSpec.describe Percy::Client do
-  describe '#config' do
+  describe 'config' do
     it 'returns the config object given when initialized' do
       config = Percy::Config.new
       client = Percy::Client.new(config: config)
       expect(client.config).to eq(config)
-      expect(client.config.keys).to eq([
-        :access_token,
-        :api_url,
-        :debug,
-        :repo,
-      ])
-      expect(client.config.access_token).to be_nil
-      expect(client.config.api_url).to eq(ENV['PERCY_API'])
-      expect(client.config.debug).to eq(false)
-      expect(client.config.repo).to eq('percy/percy-client')
     end
   end
 end

data/spec/lib/percy/config_spec.rb

@@ -0,0 +1,18 @@
+RSpec.describe Percy::Config do
+  let(:config) { described_class.new }
+
+  it 'returns the correct defaults' do
+    expect(config.keys).to eq([
+      :access_token,
+      :api_url,
+      :debug,
+      :repo,
+      :default_widths,
+    ])
+    expect(config.access_token).to be_nil
+    expect(config.api_url).to eq(ENV['PERCY_API'])
+    expect(config.debug).to eq(false)
+    expect(config.repo).to eq('percy/percy-client')
+    expect(config.default_widths).to eq([])
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: percy-client
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-17 00:00:00.000000000 Z
+date: 2016-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -140,6 +140,7 @@ files:
 - spec/cassettes/Percy_Client_Resources/_upload_resource/returns_true_with_success.yml
 - spec/cassettes/Percy_Client_Snapshots/_create_snapshot/creates_a_snapshot.yml
 - spec/cassettes/Percy_Client_Snapshots/_create_snapshot/fails_if_no_resources_are_given.yml
+- spec/cassettes/Percy_Client_Snapshots/_create_snapshot/overrides_default_widths_if_given.yml
 - spec/cassettes/Percy_Client_Snapshots/_finalize_snapshot/finalizes_a_snapshot.yml
 - spec/lib/percy/client/builds_spec.rb
 - spec/lib/percy/client/connection_spec.rb
@@ -147,6 +148,7 @@ files:
 - spec/lib/percy/client/resources_spec.rb
 - spec/lib/percy/client/snapshots_spec.rb
 - spec/lib/percy/client_spec.rb
+- spec/lib/percy/config_spec.rb
 - spec/lib/percy_spec.rb
 - spec/spec_helper.rb
 - spec/support/vcr_setup.rb
@@ -182,6 +184,7 @@ test_files:
 - spec/cassettes/Percy_Client_Resources/_upload_resource/returns_true_with_success.yml
 - spec/cassettes/Percy_Client_Snapshots/_create_snapshot/creates_a_snapshot.yml
 - spec/cassettes/Percy_Client_Snapshots/_create_snapshot/fails_if_no_resources_are_given.yml
+- spec/cassettes/Percy_Client_Snapshots/_create_snapshot/overrides_default_widths_if_given.yml
 - spec/cassettes/Percy_Client_Snapshots/_finalize_snapshot/finalizes_a_snapshot.yml
 - spec/lib/percy/client/builds_spec.rb
 - spec/lib/percy/client/connection_spec.rb
@@ -189,6 +192,7 @@ test_files:
 - spec/lib/percy/client/resources_spec.rb
 - spec/lib/percy/client/snapshots_spec.rb
 - spec/lib/percy/client_spec.rb
+- spec/lib/percy/config_spec.rb
 - spec/lib/percy_spec.rb
 - spec/spec_helper.rb
 - spec/support/vcr_setup.rb