SWI-12283 Allow Setting Custom Token URL (#223)

* SWI-12283 Allow Setting Custom Token URL

* update unit tests to not hit live oauth url

Author: ckoegel

custom_templates/configuration.mustache

@@ -65,6 +65,10 @@ module {{moduleName}}
     # Defines the access token (Bearer) used with OAuth2.
     attr_accessor :access_token
 
+    # Defines the URL used to obtain access tokens (Bearer) used with OAuth2.
+    # @return [String]
+    attr_accessor :access_token_url
+
     # Defines a Proc used to fetch or refresh access tokens (Bearer) used with OAuth2.
     # Overrides the access_token if set
     # @return [Proc]
@@ -175,16 +179,16 @@ module {{moduleName}}
       {{#authMethods}}
       {{#isOAuth}}
       @access_token_expiration = nil
+      @access_token_url = '{{tokenUrl}}'
       @access_token_getter = Proc.new {
         access_token_valid = @access_token && (@access_token_expiration.nil? || @access_token_expiration > Time.now + 60)
         next @access_token if access_token_valid
         next unless @client_id && @client_secret
 
         puts "Refreshing access token..." if @debugging
         # obtain new access token using client credentials
-        token_url = '{{tokenUrl}}'
         auth_header = 'Basic ' + ["#{@client_id}:#{@client_secret}"].pack('m').delete("\r\n")
-        conn = Faraday.new(url: token_url) do |faraday|
+        conn = Faraday.new(url: @access_token_url) do |faraday|
           faraday.request :url_encoded
           faraday.adapter Faraday.default_adapter
         end

lib/bandwidth-sdk/configuration.rb

@@ -72,6 +72,10 @@ class Configuration
     # Defines the access token (Bearer) used with OAuth2.
     attr_accessor :access_token
 
+    # Defines the URL used to obtain access tokens (Bearer) used with OAuth2.
+    # @return [String]
+    attr_accessor :access_token_url
+
     # Defines a Proc used to fetch or refresh access tokens (Bearer) used with OAuth2.
     # Overrides the access_token if set
     # @return [Proc]
@@ -193,16 +197,16 @@ def initialize
       @force_ending_format = false
       @logger = defined?(Rails) ? Rails.logger : Logger.new(STDOUT)
       @access_token_expiration = nil
+      @access_token_url = 'https://api.bandwidth.com/api/v1/oauth2/token'
       @access_token_getter = Proc.new {
         access_token_valid = @access_token && (@access_token_expiration.nil? || @access_token_expiration > Time.now + 60)
         next @access_token if access_token_valid
         next unless @client_id && @client_secret
 
         puts "Refreshing access token..." if @debugging
         # obtain new access token using client credentials
-        token_url = 'https://api.bandwidth.com/api/v1/oauth2/token'
         auth_header = 'Basic ' + ["#{@client_id}:#{@client_secret}"].pack('m').delete("\r\n")
-        conn = Faraday.new(url: token_url) do |faraday|
+        conn = Faraday.new(url: @access_token_url) do |faraday|
           faraday.request :url_encoded
           faraday.adapter Faraday.default_adapter
         end

spec/unit/api/calls_api_spec.rb

@@ -5,8 +5,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end

spec/unit/api/conferences_api_spec.rb

@@ -7,8 +7,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.return_binary_data = true
       config.host = '127.0.0.1:4010'

spec/unit/api/endpoints_api_spec.rb

@@ -5,8 +5,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end

spec/unit/api/media_api_spec.rb

@@ -7,8 +7,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.return_binary_data = true
       config.host = '127.0.0.1:4010'

spec/unit/api/messages_api_spec.rb

@@ -3,8 +3,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end

spec/unit/api/mfa_api_spec.rb

@@ -6,8 +6,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end

spec/unit/api/multi_channel_api_spec.rb

@@ -3,8 +3,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end

spec/unit/api/phone_number_lookup_api_spec.rb

@@ -7,8 +7,7 @@
   before(:all) do
     Bandwidth.configure do |config|
       config.debugging = true
-      config.username = BW_USERNAME
-      config.password = BW_PASSWORD
+      config.access_token = 'abcd1234'
       config.ignore_operation_servers = true
       config.host = '127.0.0.1:4010'
     end