Merge pull request #87 from Jesus/expired-access-token

Implement ExpiredAccessTokenError

Author: Jesus

lib/dropbox_api.rb

@@ -91,6 +91,7 @@
 require 'dropbox_api/errors/create_folder_error'
 require 'dropbox_api/errors/create_shared_link_with_settings_error'
 require 'dropbox_api/errors/download_error'
+require 'dropbox_api/errors/expired_access_token_error'
 require 'dropbox_api/errors/sharing_file_access_error'
 require 'dropbox_api/errors/sharing_user_error'
 require 'dropbox_api/errors/file_member_action_error'

lib/dropbox_api/endpoints/base.rb

@@ -24,6 +24,11 @@ def process_response(raw_response)
         # Status code 409 is "Endpoint-specific error". We need to look at
         # the response body to build an exception.
         build_result(raw_response.env[:api_result])
+      when 401
+        raise DropboxApi::Errors::ExpiredAccessTokenError.build(
+          raw_response.env[:api_result]['error_summary'],
+          raw_response.env[:api_result]['error']
+        )
       when 429
         error = DropboxApi::Errors::TooManyRequestsError.build(
           raw_response.env[:api_result]['error_summary'],

lib/dropbox_api/errors/expired_access_token_error.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+module DropboxApi::Errors
+  class ExpiredAccessTokenError < BasicError
+  end
+end

spec/client_spec.rb

@@ -33,5 +33,13 @@ module DropboxApi
         Faraday::Adapter::NetHttpPersistent
       ))
     end
+
+    it 'will raise on 401 if there is no refresh token', cassette: 'client/raise_on_401' do
+      client = Client.new("MOCK_EXPIRED_AUTHORIZATION_BEARER")
+
+      expect do
+        client.list_folder ""
+      end.to raise_error(DropboxApi::Errors::ExpiredAccessTokenError)
+    end
   end
 end