omniauth-bookingsync 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bae9c4da86dca7f00aaad6b49d0e0186238f7aa7
-  data.tar.gz: 79f7916e128c0a8caacf5d6db9aabb61fcd2bc7d
+  metadata.gz: 96471ecd45f00803267b9e01d06f950b00f9284d
+  data.tar.gz: cb7ea15ba8ee59443dbe1163f27ae5942dad07dc
 SHA512:
-  metadata.gz: f9252f455c913e3ef984340dfa1b57f31dfc13d201717006148e8e05fedb57cf9b952dee7cdeedcfd9e1a2108baf43ce802956546552e4033959cb7eb16e2b98
-  data.tar.gz: 7627343b15c0af7b6f07ccbb44b72ea218ac34adb2385f4955fa37ed27a1a4ed0856179cc7a16925da4b758ccbd449bf70e8bf990e1fd28d04f42a0a7bd612f5
+  metadata.gz: 5845860e19f5bc0d8bd3d2952d0e7f0e1cf896de35ca33b381c86a4c3a52c92990699cd6a74923569ed69e5092d81b8be55d5d0a0ecc3afa72a8c3f089c3cbfa
+  data.tar.gz: 6a6f5109ce17516e0545b4bfa8a5ac8ce3346653c3030dddb159f3a7b2cf93731e8d43e39de469d53f7f891f6f0331371914630bd10c42c4e69caea0d24bf168

data/.rspec

@@ -1 +1 @@
---colour
+--color

data/.travis.yml

@@ -1,10 +1,15 @@
-sudo: false
-language: ruby
+sudo: required
+dist: trusty
 before_install:
   - gem install bundler
-  - gem update bundler
+  - bundle install --jobs=3 --retry=3
+  - bundle binstub rake
 rvm:
-  - 2.2.5
-  - 2.3.1
+  - 2.2.6
+  - 2.3.3
+  - 2.4.0
   - ruby-head
-  - jruby-head
+  - jruby
+gemfile:
+  - gemfiles/omniauth-oauth2-1.3.gemfile
+  - gemfiles/omniauth-oauth2-1.4.gemfile

data/Appraisals

@@ -0,0 +1,7 @@
+appraise "omniauth-oauth2-1.3" do
+  gem "omniauth-oauth2", "~> 1.3.1"
+end
+
+appraise "omniauth-oauth2-1.4" do
+  gem "omniauth-oauth2", "~> 1.4.0"
+end

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 #master
 
+# 0.5.0 - 2017-04-22
+
+* Add support for omniauth-oauth2 1.4
+* Update gems dependencies
+* Add Ruby 2.4 to Travis
+* Add account auto-selection during the authorization process
+
+# 0.4.0
+
+* Update Readme
 * bump oauth2 to 1.2
 * add Ruby 2.3.1 to travis, drop Ruby 2.0 and 2.1 (end of maintenance)
 * Update example application to work over SSL

data/Gemfile.lock

@@ -1,46 +1,51 @@
 PATH
   remote: .
   specs:
-    omniauth-bookingsync (0.4.0)
-      oauth2 (~> 1.2.0)
-      omniauth (~> 1.1)
-      omniauth-oauth2 (< 1.4)
+    omniauth-bookingsync (0.5.0)
+      oauth2 (~> 1.3.0)
+      omniauth (~> 1.6)
+      omniauth-oauth2 (<= 1.4)
 
 GEM
   remote: https://rubygems.org/
   specs:
+    appraisal (2.2.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
     ast (2.3.0)
-    diff-lcs (1.2.5)
-    faraday (0.9.2)
+    diff-lcs (1.3)
+    faraday (0.11.0)
       multipart-post (>= 1.2, < 3)
-    hashie (3.4.4)
-    jwt (1.5.4)
+    hashie (3.5.5)
+    jwt (1.5.6)
     multi_json (1.12.1)
-    multi_xml (0.5.5)
+    multi_xml (0.6.0)
     multipart-post (2.0.0)
-    oauth2 (1.2.0)
-      faraday (>= 0.8, < 0.10)
+    oauth2 (1.3.1)
+      faraday (>= 0.8, < 0.12)
       jwt (~> 1.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
       rack (>= 1.2, < 3)
-    omniauth (1.3.1)
-      hashie (>= 1.2, < 4)
-      rack (>= 1.0, < 3)
-    omniauth-oauth2 (1.3.1)
+    omniauth (1.6.1)
+      hashie (>= 3.4.6, < 3.6.0)
+      rack (>= 1.6.2, < 3)
+    omniauth-oauth2 (1.4.0)
       oauth2 (~> 1.0)
       omniauth (~> 1.2)
-    parser (2.3.1.2)
+    parser (2.4.0.0)
       ast (~> 2.2)
     powerpack (0.1.1)
     rack (2.0.1)
-    rainbow (2.1.0)
-    rake (11.2.2)
+    rainbow (2.2.2)
+      rake
+    rake (12.0.0)
     rspec (3.5.0)
       rspec-core (~> 3.5.0)
       rspec-expectations (~> 3.5.0)
       rspec-mocks (~> 3.5.0)
-    rspec-core (3.5.2)
+    rspec-core (3.5.4)
       rspec-support (~> 3.5.0)
     rspec-expectations (3.5.0)
       diff-lcs (>= 1.2.0, < 2.0)
@@ -49,23 +54,25 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.5.0)
     rspec-support (3.5.0)
-    rubocop (0.42.0)
-      parser (>= 2.3.1.1, < 3.0)
+    rubocop (0.48.1)
+      parser (>= 2.3.3.1, < 3.0)
       powerpack (~> 0.1)
       rainbow (>= 1.99.1, < 3.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (~> 1.0, >= 1.0.1)
     ruby-progressbar (1.8.1)
-    unicode-display_width (1.1.0)
+    thor (0.19.4)
+    unicode-display_width (1.2.1)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
+  appraisal
   omniauth-bookingsync!
   rake
   rspec
   rubocop
 
 BUNDLED WITH
-   1.12.5
+   1.14.6

data/README.md

@@ -1,4 +1,5 @@
-[![Build Status](https://travis-ci.org/BookingSync/omniauth-bookingsync.svg?branch=master)](https://travis-ci.org/BookingSync/omniauth-bookingsync)
+[![CI Build
+Status](https://secure.travis-ci.org/BookingSync/omniauth-bookingsync.png)](http://travis-ci.org/BookingSync/omniauth-bookingsync)
 
 # OmniAuth BookingSync
 
@@ -36,7 +37,7 @@ this will skip the account selection process and show the accept/deny page.
 
 ## Supported Rubies
 
-OmniAuth BookingSync is tested under 2.2, 2.3, Ruby-head, JRuby.
+OmniAuth BookingSync is tested under 2.2, 2.3, 2.4, Ruby-head, JRuby.
 
 [![CI Build
 Status](https://secure.travis-ci.org/BookingSync/omniauth-bookingsync.png)](http://travis-ci.org/BookingSync/omniauth-bookingsync)

data/example/Gemfile

@@ -1,3 +1,5 @@
+source 'https://rubygems.org'
+
 gem "sinatra"
 gem "omniauth-bookingsync"
 gem "json"

data/example/Gemfile.lock

@@ -1,24 +1,25 @@
 GEM
+  remote: https://rubygems.org/
   specs:
     faraday (0.9.2)
       multipart-post (>= 1.2, < 3)
-    hashie (3.4.3)
+    hashie (3.5.5)
     json (1.8.3)
-    jwt (1.5.4)
-    multi_json (1.11.2)
-    multi_xml (0.5.5)
+    jwt (1.5.6)
+    multi_json (1.12.1)
+    multi_xml (0.6.0)
     multipart-post (2.0.0)
-    oauth2 (1.0.0)
+    oauth2 (1.2.0)
       faraday (>= 0.8, < 0.10)
       jwt (~> 1.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
-      rack (~> 1.2)
-    omniauth (1.3.1)
-      hashie (>= 1.2, < 4)
-      rack (>= 1.0, < 3)
-    omniauth-bookingsync (0.3.1)
-      oauth2 (= 1.0)
+      rack (>= 1.2, < 3)
+    omniauth (1.6.1)
+      hashie (>= 3.4.6, < 3.6.0)
+      rack (>= 1.6.2, < 3)
+    omniauth-bookingsync (0.4.0)
+      oauth2 (~> 1.2.0)
       omniauth (~> 1.1)
       omniauth-oauth2 (< 1.4)
     omniauth-oauth2 (1.3.1)
@@ -42,4 +43,4 @@ DEPENDENCIES
   sinatra
 
 BUNDLED WITH
-   1.12.5
+   1.14.6

data/lib/omniauth-bookingsync.rb

@@ -1,2 +1,3 @@
 require "omniauth/bookingsync/version"
+require "omniauth/bookingsync/identifier"
 require "omniauth/strategies/bookingsync"

data/lib/omniauth/bookingsync/identifier.rb

@@ -0,0 +1,20 @@
+module OmniAuth
+  module BookingSync
+    class Identifier
+      attr_reader :identifier
+
+      def initialize(identifier)
+        @identifier = identifier
+      end
+
+      def value
+        casted_value = identifier.to_i
+        casted_value > 0 ? casted_value : nil
+      end
+
+      def valid?
+        value != nil
+      end
+    end
+  end
+end

data/lib/omniauth/bookingsync/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module BookingSync
-    VERSION = "0.4.0"
+    VERSION = "0.5.0"
   end
 end

data/lib/omniauth/strategies/bookingsync.rb

@@ -21,11 +21,23 @@ module OmniAuth
 
       def authorize_params
         super.tap do |params|
-          if request.params["account_id"]
-            params[:account_id] = request.params["account_id"]
+          bookingsync_account_id = OmniAuth::BookingSync::Identifier.new(
+            request.params["_bookingsync_account_id"])
+          account_id = OmniAuth::BookingSync::Identifier.new(request.params["account_id"])
+
+          if bookingsync_account_id.valid?
+            params[:account_id] = bookingsync_account_id.value
+          elsif account_id.valid?
+            params[:account_id] = account_id.value
           end
         end
       end
+
+      # Fixes regression in omniauth-oauth2 v1.4.0 by
+      # https://github.com/intridea/omniauth-oauth2/commit/85fdbe117c2a4400d001a6368cc359d88f40abc7
+      def callback_url
+        options[:redirect_uri] || (full_host + script_name + callback_path)
+      end
     end
   end
 end

data/omniauth-bookingsync.gemspec

@@ -11,18 +11,17 @@ Gem::Specification.new do |gem|
   gem.summary = "An OmniAuth 1.0 strategy for BookingSync OAuth2 identification."
   gem.description = "An OmniAuth 1.0 strategy for BookingSync OAuth2 identification."
 
-  gem.executables = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  gem.executables = `git ls-files -- exe/*`.split("\n").map { |f| File.basename(f) }
   gem.files = `git ls-files`.split("\n")
   gem.test_files = `git ls-files -- {test,spec,features}/*`.split("\n")
   gem.require_paths = ["lib"]
 
-  gem.add_dependency "omniauth", "~> 1.1"
-  # this constraint is to change in 1.4 breaking redirect url generation
-  # https://github.com/intridea/omniauth-oauth2/issues/81
-  gem.add_dependency "omniauth-oauth2", "< 1.4"
-  gem.add_dependency "oauth2", "~> 1.2.0"
+  gem.add_dependency "omniauth", "~> 1.6"
+  gem.add_dependency "omniauth-oauth2", "<= 1.4"
+  gem.add_dependency "oauth2", "~> 1.3.0"
 
   gem.add_development_dependency "rspec"
   gem.add_development_dependency "rake"
   gem.add_development_dependency "rubocop"
+  gem.add_development_dependency "appraisal"
 end

data/spec/omniauth/bookingsync/identifier_spec.rb

@@ -0,0 +1,39 @@
+require "spec_helper"
+
+describe OmniAuth::BookingSync::Identifier do
+  describe "#value" do
+    it "returns a number" do
+      expect(OmniAuth::BookingSync::Identifier.new("1").value).to eq 1
+    end
+
+    it "returns nil if blank" do
+      expect(OmniAuth::BookingSync::Identifier.new("").value).to be_nil
+    end
+
+    it "returns nil if non numeric characters given" do
+      expect(OmniAuth::BookingSync::Identifier.new("a").value).to be_nil
+    end
+
+    it "returns nil if negative number" do
+      expect(OmniAuth::BookingSync::Identifier.new("-1").value).to be_nil
+    end
+  end
+
+  describe "#valid?" do
+    it "returns true if a number" do
+      expect(OmniAuth::BookingSync::Identifier.new("1")).to be_valid
+    end
+
+    it "returns false if blank" do
+      expect(OmniAuth::BookingSync::Identifier.new("")).not_to be_valid
+    end
+
+    it "returns false if non numeric characters given" do
+      expect(OmniAuth::BookingSync::Identifier.new("a")).not_to be_valid
+    end
+
+    it "returns false if negative number" do
+      expect(OmniAuth::BookingSync::Identifier.new("-1")).not_to be_valid
+    end
+  end
+end

data/spec/omniauth/strategies/bookingsync_spec.rb

@@ -1,15 +1,18 @@
 require "spec_helper"
-require "omniauth-bookingsync"
 
 describe OmniAuth::Strategies::BookingSync do
   before do
     OmniAuth.config.test_mode = true
   end
 
+  after do
+    OmniAuth.config.test_mode = false
+  end
+
   let(:request) { double("Request", params: {}, cookies: {}, env: {}) }
   let(:options) { {} }
 
-  subject do
+  let(:strategy) do
     OmniAuth::Strategies::BookingSync.new(nil, options).tap do |strategy|
       allow(strategy).to receive(:request) { request }
     end
@@ -17,64 +20,121 @@ describe OmniAuth::Strategies::BookingSync do
 
   it_should_behave_like "an oauth2 strategy"
 
-  describe '#client' do
-    it "has the correct site" do
-      expect(subject.client.site).to eq("https://www.bookingsync.com/")
+  describe "#client" do
+    it "returns the correct site" do
+      expect(strategy.client.site).to eq "https://www.bookingsync.com/"
     end
 
-    it "has the correct authorization url" do
-      expect(subject.client.options[:authorize_url]).to eq("/oauth/authorize")
+    it "returns the correct authorization url" do
+      expect(strategy.client.options[:authorize_url]).to eq "/oauth/authorize"
     end
 
-    it "has the correct token url" do
-      expect(subject.client.options[:token_url]).to eq("/oauth/token")
+    it "returns the correct token url" do
+      expect(strategy.client.options[:token_url]).to eq "/oauth/token"
     end
   end
 
-  describe '#callback_path' do
-    it "has the correct callback path" do
-      expect(subject.callback_path).to eq("/auth/bookingsync/callback")
+  describe "#callback_path" do
+    it "returns the correct callback path" do
+      expect(strategy.callback_path).to eq("/auth/bookingsync/callback")
     end
   end
 
-  describe '#authorize_params' do
+  describe "#authorize_params" do
     it "passes account_id from request params" do
       request.params["account_id"] = "123"
-      expect(subject.authorize_params[:account_id]).to eq("123")
+      expect(strategy.authorize_params[:account_id]).to eq 123
+    end
+
+    it "passes _bookingsync_account_id from request params" do
+      request.params["_bookingsync_account_id"] = "123"
+      expect(strategy.authorize_params[:account_id]).to eq 123
+    end
+
+    it "passes _bookingsync_account_id and account_id from request params, " +
+    "it use _bookingsync_account_id" do
+      request.params["_bookingsync_account_id"] = "123"
+      request.params["account_id"] = "456"
+      expect(strategy.authorize_params[:account_id]).to eq 123
+    end
+
+    it "ignores blank params" do
+      request.params["_bookingsync_account_id"] = ""
+      request.params["account_id"] = "456"
+      expect(strategy.authorize_params[:account_id]).to eq 456
+    end
+
+    it "ignores non numeric params" do
+      request.params["_bookingsync_account_id"] = "a"
+      request.params["account_id"] = "b"
+      expect(strategy.authorize_params[:account_id]).to be_nil
     end
   end
 
-  describe '#raw_info' do
+  describe "#raw_info" do
     let(:body) { '{"accounts": [{"id": 1}]}' }
     let(:response) { double("response", headers: headers, status: 200, body: body) }
 
     before do
-      allow(subject).to receive(:access_token).and_return(double)
-      expect(subject.access_token).to receive(:get).with("/api/v3/accounts")
+      allow(strategy).to receive(:access_token).and_return(double)
+      expect(strategy.access_token).to receive(:get).with("/api/v3/accounts")
         .and_return(OAuth2::Response.new(response))
     end
 
     context "when Content-Type not supported" do
       let(:headers) { { "Content-Type" => "application/foo-bar" } }
 
-      it "blows up with NoMethodError" do
-        expect { subject.raw_info }.to raise_error(NoMethodError)
+      it "raises a NoMethodError error" do
+        expect { strategy.raw_info }.to raise_error(NoMethodError)
       end
     end
 
-    context "when Content-Type supported" do
+    context "when using JSONAPI Content-Type" do
       let(:headers) { { "Content-Type" => "application/vnd.api+json" } }
 
-      it "fetches account info from api v3" do
-        expect(subject.raw_info).to eq("id" => 1)
+      it "returns account info from api v3" do
+        expect(strategy.raw_info).to eq({ "id" => 1 })
       end
     end
 
-    context "when Content-Type supported" do
+    context "when using JSON Content-Type" do
       let(:headers) { { "Content-Type" => "application/json" } }
 
-      it "fetches account info from api v3" do
-        expect(subject.raw_info).to eq("id" => 1)
+      it "returns account info from api v3" do
+        expect(strategy.raw_info).to eq({ "id" => 1 })
+      end
+    end
+  end
+
+  describe "#callback_url" do
+    let(:url_base) { "https://auth.example.com" }
+
+    before do
+      allow(request).to receive(:scheme).and_return("https")
+      allow(request).to receive(:url).and_return("#{url_base}/some/page?foo=bar")
+
+      allow(strategy).to receive(:script_name).and_return("") # as not to depend on Rack env
+      allow(strategy).to receive(:query_string).and_return("?foo=bar")
+    end
+
+    it "returns default callback url (omitting querystring)" do
+      expect(strategy.callback_url).to eq "#{url_base}/auth/bookingsync/callback"
+    end
+
+    context "with custom callback path" do
+      let(:options) { { callback_path: "/auth/bookingsync/done" } }
+
+      it "returns default callback url (omitting querystring)" do
+        expect(strategy.callback_url).to eq "#{url_base}/auth/bookingsync/done"
+      end
+    end
+
+    context "with custom callback url" do
+      let(:url) { "https://auth.myapp.com/auth/bookingsync/callback" }
+      let(:options) { { redirect_uri: url } }
+
+      it "returns url from redirect_uri option" do
+        expect(strategy.callback_url).to eq url
       end
     end
   end

data/spec/spec_helper.rb

@@ -1,5 +1,7 @@
 require "bundler/setup"
 require "rspec"
+require "omniauth-bookingsync"
+
 Dir[File.expand_path("../support/**/*", __FILE__)].each { |f| require f }
 
 RSpec.configure do |_config|

data/spec/support/shared_examples.rb

@@ -1,23 +1,23 @@
 shared_examples "an oauth2 strategy" do
-  describe '#client' do
+  describe "#client" do
     let(:options) do
       { client_options: { "authorize_url" => "https://example.com" } }
     end
 
     it "is initialized with symbolized client_options" do
-      expect(subject.client.options[:authorize_url]).to eq "https://example.com"
+      expect(strategy.client.options[:authorize_url]).to eq "https://example.com"
     end
   end
 
-  describe '#authorize_params' do
+  describe "#authorize_params" do
     context "when passed as authorize_params" do
       let(:options) do
         { authorize_params: { foo: "bar", baz: "zip" } }
       end
 
       it "includes any authorize params passed in the :authorize_params option" do
-        expect(subject.authorize_params["foo"]).to eq("bar")
-        expect(subject.authorize_params["baz"]).to eq("zip")
+        expect(strategy.authorize_params["foo"]).to eq("bar")
+        expect(strategy.authorize_params["baz"]).to eq("zip")
       end
     end
 
@@ -27,21 +27,21 @@ shared_examples "an oauth2 strategy" do
       end
 
       it "includes top-level options that are marked as :authorize_options" do
-        expect(subject.authorize_params["scope"]).to eq("bar")
-        expect(subject.authorize_params["foo"]).to eq("baz")
+        expect(strategy.authorize_params["scope"]).to eq("bar")
+        expect(strategy.authorize_params["foo"]).to eq("baz")
       end
     end
   end
 
-  describe '#token_params' do
+  describe "#token_params" do
     context "when passed as token_params" do
       let(:options) do
         { token_params: { foo: "bar", baz: "zip" } }
       end
 
       it "includes any token params passed in the :token_params option" do
-        expect(subject.token_params["foo"]).to eq("bar")
-        expect(subject.token_params["baz"]).to eq("zip")
+        expect(strategy.token_params["foo"]).to eq("bar")
+        expect(strategy.token_params["baz"]).to eq("zip")
       end
     end
 
@@ -51,8 +51,8 @@ shared_examples "an oauth2 strategy" do
       end
 
       it "includes top-level options that are marked as :token_options" do
-        expect(subject.token_params["scope"]).to eq("bar")
-        expect(subject.token_params["foo"]).to eq("baz")
+        expect(strategy.token_params["scope"]).to eq("bar")
+        expect(strategy.token_params["foo"]).to eq("baz")
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-bookingsync
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Sebastien Grosjean
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-22 00:00:00.000000000 Z
+date: 2017-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth
@@ -16,26 +16,26 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.6'
 - !ruby/object:Gem::Dependency
   name: omniauth-oauth2
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "<="
       - !ruby/object:Gem::Version
         version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "<="
       - !ruby/object:Gem::Version
         version: '1.4'
 - !ruby/object:Gem::Dependency
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +94,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: An OmniAuth 1.0 strategy for BookingSync OAuth2 identification.
 email:
 - dev@bookingsync.com
@@ -106,6 +120,7 @@ files:
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
 - ".travis.yml"
+- Appraisals
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
@@ -118,9 +133,11 @@ files:
 - example/bookingsync_oauth_helper.rb
 - example/config.ru
 - lib/omniauth-bookingsync.rb
+- lib/omniauth/bookingsync/identifier.rb
 - lib/omniauth/bookingsync/version.rb
 - lib/omniauth/strategies/bookingsync.rb
 - omniauth-bookingsync.gemspec
+- spec/omniauth/bookingsync/identifier_spec.rb
 - spec/omniauth/strategies/bookingsync_spec.rb
 - spec/spec_helper.rb
 - spec/support/shared_examples.rb
@@ -148,6 +165,8 @@ signing_key:
 specification_version: 4
 summary: An OmniAuth 1.0 strategy for BookingSync OAuth2 identification.
 test_files:
+- spec/omniauth/bookingsync/identifier_spec.rb
 - spec/omniauth/strategies/bookingsync_spec.rb
 - spec/spec_helper.rb
 - spec/support/shared_examples.rb
+has_rdoc: