bigcommerce-oauth-api 1.1.5 → 1.2.0

data/lib/bigcommerce-oauth-api/configuration.rb

@@ -9,7 +9,10 @@ module BigcommerceOAuthAPI
         :adapter,
         :client_id,
         :access_token,
-        :format
+        :format,
+        # legacy authentication
+        :user_name,
+        :api_key
     ].freeze
 
     DEFAULT_STORE_HASH = nil
@@ -18,6 +21,8 @@ module BigcommerceOAuthAPI
     DEFAULT_ACCESS_TOKEN = nil
     DEFAULT_FORMAT = :json
     DEFAULT_ADAPTER = Faraday.default_adapter
+    DEFAULT_USER_NAME = nil
+    DEFAULT_API_KEY = nil
 
     attr_accessor *VALID_OPTIONS_KEYS
 
@@ -32,6 +37,8 @@ module BigcommerceOAuthAPI
       self.client_id = DEFAULT_CLIENT_ID
       self.access_token = DEFAULT_ACCESS_TOKEN
       self.adapter = DEFAULT_ADAPTER
+      self.user_name = DEFAULT_USER_NAME
+      self.api_key = DEFAULT_API_KEY
     end
 
     def configure

data/lib/bigcommerce-oauth-api/connection.rb

@@ -7,16 +7,24 @@ module BigcommerceOAuthAPI
 
     def connection
       options = {
-          :headers => {
-              'Accept' => "application/#{format}; charset=utf-8",
-              'X-Auth-Client' => client_id,
-              'X-Auth-Token' => access_token
-          },
-          :url => "#{endpoint}/#{store_hash}/v2/"
+        :headers => {
+          'Accept' => "application/#{format}; charset=utf-8"
+        }
       }
+      if is_legacy?
+        options[:url] = "#{endpoint}/api/v2/"
+      else
+        options[:headers]['X-Auth-Client'] = client_id
+        options[:headers]['X-Auth-Token'] = access_token
+        options[:url] = "#{endpoint}/#{store_hash}/v2/"
+      end
 
       Faraday::Connection.new(options) do |connection|
         connection.use Faraday::Request::UrlEncoded
+        if is_legacy?
+          connection.use Faraday::Request::BasicAuthentication, user_name, api_key
+        end
+
         case format.to_s.downcase
         when 'json' then connection.use Faraday::Response::ParseJson
         end

data/lib/bigcommerce-oauth-api/error.rb

@@ -53,5 +53,8 @@ module BigcommerceOAuthAPI
   # Raised if the client attempts to define an api method that already is defined elsewhere.
   # Specs will catch this type of error since it will be thrown upon initialization.
   class MethodAlreadyDefinedError < Error; end
+
+  # Raised if the client attempts to use a non-legacy api with a legacy configuration.
+  class NonLegacyApi < Error; end
 end
 

data/lib/bigcommerce-oauth-api/version.rb

@@ -1,3 +1,3 @@
 module BigcommerceOAuthAPI
-  VERSION = '1.1.5'.freeze
+  VERSION = '1.2.0'.freeze
 end

data/spec/bigcommerce_oauth_api/api_spec.rb

@@ -35,7 +35,9 @@ module BigcommerceOAuthAPI
               :client_id => 'c',
               :access_token => 'd',
               :adapter => 'e',
-              :store_hash => 'f'
+              :store_hash => 'f',
+              :user_name => 'g',
+              :api_key => 'h',
           }
         end
 
@@ -51,5 +53,62 @@ module BigcommerceOAuthAPI
         end
       end
     end
+
+    describe '#is_legacy?' do
+
+      after do
+        BigcommerceOAuthAPI.reset
+      end
+
+      context 'given the default configuration' do
+        it 'should return false' do
+          expect(described_class.new.is_legacy?).to eql(false)
+        end
+      end
+
+      context 'given a legacy configuration' do
+        describe 'with module configuration' do
+          before do
+            BigcommerceOAuthAPI.configure do |config|
+              [:endpoint, :user_name, :api_key].each do |key|
+                config.send("#{key}=", key)
+              end
+            end
+          end
+
+          it 'should return true' do
+            expect(described_class.new.is_legacy?).to eql(true)
+          end
+        end
+
+        describe 'with class configuration' do
+          it 'should return true' do
+            expect(described_class.new({ :endpoint => 'c', :user_name => 'd', :api_key => 'f' }).is_legacy?).to eql(true)
+          end
+        end
+      end
+
+      context 'given an oauth configuration' do
+        describe 'with module configuration' do
+          before do
+            BigcommerceOAuthAPI.configure do |config|
+              [:client_id, :access_token, :store_hash].each do |key|
+                config.send("#{key}=", key)
+              end
+            end
+          end
+
+          it 'should return true' do
+            expect(described_class.new.is_legacy?).to eql(false)
+          end
+        end
+
+        describe 'with class configuration' do
+          it 'should return true' do
+            expect(described_class.new({ :client_id => 'c', :access_token => 'd', :store_hash => 'f' }).is_legacy?).to eql(false)
+          end
+        end
+      end
+    end
   end
 end

data/spec/bigcommerce_oauth_api/client/client_module_spec.rb

@@ -7,116 +7,203 @@ describe BigcommerceOAuthAPI::Client do
   [
       { api_module: 'post', methods: [:all, :select, :create, :update, :delete], prefix_paths: 'blog', prefix_methods: 'blog'},
       { api_module: 'tag', methods: [:all], prefix_paths: 'blog', prefix_methods: 'blog'},
-      { api_module: 'brand', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'brand', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'category', methods: [:all, :select, :create, :update, :delete]},
       { api_module: 'customer', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'customer_group', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'country', scope: :self, methods: [:all, :select]},
-      { api_module: 'state', scope: :self, methods: [:all, :select]},
+      { api_module: 'country', scope: :self, methods: [:all, :select, :count]},
+      { api_module: 'order_status', methods: [:all, :select]},
       { api_module: 'order', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'method', methods: [:all], prefix_paths: 'payments', prefix_methods: 'payment'},
-      { api_module: 'option', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'option_set', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'option', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'option_set', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
-      { api_module: 'coupon', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'redirect', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'coupon', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'redirect', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'method', methods: [:all, :select], prefix_paths: 'shipping', prefix_methods: 'shipping'},
       { api_module: 'tax_class', scope: :self, methods: [:all, :select]},
-      { api_module: 'hook', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'hook', methods: [:all, :select, :create, :update, :delete], legacy: false}
   ]. each do |api_description|
     api_module = api_description[:api_module]
     api_module_pluralized = api_module.pluralize
     path_prefix = (api_description.has_key?(:prefix_paths) ? "#{api_description[:prefix_paths]}/" : nil)
     method_prefix = (api_description.has_key?(:prefix_methods) ? "#{api_description[:prefix_methods]}_" : nil)
+    has_legacy_support = (api_description.has_key?(:legacy) ? api_description[:legacy] : true)
 
-    before do
-      @client = BigcommerceOAuthAPI::Client.new(:store_hash => 'TEST_STORE',
-                                                :client_id => 'SECRET_ID',
-                                                :access_token => 'SECRET_TOKEN')
-    end
+    [:legacy, :oauth].each do |config_type|
+      context "given a #{config_type} configuration" do
+        before do
+          @client = case config_type
+                      when :legacy
+                        BigcommerceOAuthAPI::Client.new(:endpoint => 'http://example.bigcommerce.com',
+                                                        :user_name => 'USER',
+                                                        :api_key => 'API_KEY')
+                      when :oauth
+                        BigcommerceOAuthAPI::Client.new(:store_hash => 'TEST_STORE',
+                                                        :client_id => 'SECRET_ID',
+                                                        :access_token => 'SECRET_TOKEN')
+                    end
+        end
 
-    if api_description[:methods].include?(:all)
-      describe ".#{method_prefix}#{api_module_pluralized}" do
-        it "should get a list of #{api_module_pluralized}" do
-          stub_get(@client, "#{path_prefix}#{api_module_pluralized}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_module_pluralized}".to_sym)
-          expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:all)
+          describe ".#{method_prefix}#{api_module_pluralized}" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                expect { @client.send("#{method_prefix}#{api_module_pluralized}".to_sym) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "should get a list of #{api_module_pluralized}" do
+                stub_get(@client, "#{path_prefix}#{api_module_pluralized}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_module_pluralized}".to_sym)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}")).to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+
+            end
+          end
         end
-      end
-    end
 
-    if api_description[:methods].include?(:select)
-      describe ".#{method_prefix}#{api_module}" do
-        it "gets the #{api_module} with the given id" do
-          id = 10
-          stub_get(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_module}".to_sym, id)
-          expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:select)
+          describe ".#{method_prefix}#{api_module}" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                id = 10
+                expect { @client.send("#{method_prefix}#{api_module}".to_sym, id) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "gets the #{api_module} with the given id" do
+                id = 10
+                stub_get(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_module}".to_sym, id)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/#{id}")).to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            end
+          end
         end
-      end
-    end
 
-    if api_description[:methods].include?(:create)
-      describe ".create_#{method_prefix}#{api_module}" do
-        it "creates a #{api_module} with the given attributes" do
-          options = { name: 'A', description: 'B'}
-          stub_post(@client, "#{path_prefix}#{api_module_pluralized}").
-              to_return(:body => options.to_json, :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("create_#{method_prefix}#{api_module}".to_sym, options)
-          expect(a_post(@client, "#{path_prefix}#{api_module_pluralized}").
-                     with(:body => options,
-                          :headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:create)
+          describe ".create_#{method_prefix}#{api_module}" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                options = { name: 'A', description: 'B'}
+                expect { @client.send("create_#{method_prefix}#{api_module}".to_sym, options) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "creates a #{api_module} with the given attributes" do
+                options = { name: 'A', description: 'B'}
+                stub_post(@client, "#{path_prefix}#{api_module_pluralized}").
+                    to_return(:body => options.to_json, :headers => { :content_type => "application/#{@client.format}" })
+
+                @client.send("create_#{method_prefix}#{api_module}".to_sym, options)
+                if config_type == :legacy
+                  expect(a_post(@client, "#{path_prefix}#{api_module_pluralized}")).to have_been_made
+                else
+                  expect(a_post(@client, "#{path_prefix}#{api_module_pluralized}").
+                             with(:body => options,
+                                  :headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            end
+          end
         end
-      end
-    end
 
-    if api_description[:methods].include?(:update)
-      describe ".update_#{method_prefix}#{api_module}" do
-        it "update the attributes of the #{api_module} with the given id" do
-          id = 10
-          options = { name: 'A', description: 'B'}
-          stub_put(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-              with(:body => options).
-              to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("update_#{method_prefix}#{api_module}".to_sym, id, options)
-          expect(a_put(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-                     with(:body => options,
-                          :headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:update)
+          describe ".update_#{method_prefix}#{api_module}" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                id = 10
+                options = { name: 'A', description: 'B'}
+                expect { @client.send("update_#{method_prefix}#{api_module}".to_sym, id, options) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "update the attributes of the #{api_module} with the given id" do
+                id = 10
+                options = { name: 'A', description: 'B'}
+                stub_put(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                    with(:body => options).
+                    to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
+
+                @client.send("update_#{method_prefix}#{api_module}".to_sym, id, options)
+                if config_type == :legacy
+                  expect(a_put(@client, "#{path_prefix}#{api_module_pluralized}/#{id}")).
+                             to have_been_made
+                else
+                  expect(a_put(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                             with(:body => options,
+                                  :headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            end
+          end
         end
-      end
-    end
 
-    if api_description[:methods].include?(:delete)
-      describe ".delete_#{method_prefix}#{api_module}" do
-        it "deletes the #{api_module} with the given id" do
-          id = 10
-          stub_delete(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("delete_#{method_prefix}#{api_module}".to_sym, id)
-          expect(a_delete(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:delete)
+          describe ".delete_#{method_prefix}#{api_module}" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                id = 10
+                expect { @client.send("delete_#{method_prefix}#{api_module}".to_sym, id) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "deletes the #{api_module} with the given id" do
+                id = 10
+                stub_delete(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("delete_#{method_prefix}#{api_module}".to_sym, id)
+                if config_type == :legacy
+                  expect(a_delete(@client, "#{path_prefix}#{api_module_pluralized}/#{id}")).to have_been_made
+                else
+                  expect(a_delete(@client, "#{path_prefix}#{api_module_pluralized}/#{id}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            end
+          end
         end
-      end
-    end
 
-    if api_description[:methods].include?(:count)
-      describe ".#{method_prefix}#{api_module_pluralized}_count" do
-        it "returns the number of #{api_module_pluralized}" do
-          stub_get(@client, "#{path_prefix}#{api_module_pluralized}/count").
-              to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_module_pluralized}_count".to_sym)
-          expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/count").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if api_description[:methods].include?(:count)
+          describe ".#{method_prefix}#{api_module_pluralized}_count" do
+            if config_type == :legacy && !has_legacy_support
+              it 'should raise a non legacy api error' do
+                expect { @client.send("#{method_prefix}#{api_module_pluralized}_count".to_sym) }.
+                    to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+              end
+            else
+              it "returns the number of #{api_module_pluralized}" do
+                stub_get(@client, "#{path_prefix}#{api_module_pluralized}/count").
+                    to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_module_pluralized}_count".to_sym)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/count")).to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_module_pluralized}/count").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            end
+          end
         end
       end
     end

data/spec/bigcommerce_oauth_api/client/client_nested_module_spec.rb

@@ -5,21 +5,24 @@ require 'bigcommerce-oauth-api/client'
 
 describe BigcommerceOAuthAPI::Client do
   [
-      { api_module: 'product', api_parent_module: 'order', methods: [:all, :select]},
+      { api_module: 'product', api_parent_module: 'order', methods: [:all, :select, :count]},
       { api_module: 'shipping_address', api_parent_module: 'order', methods: [:all, :select]},
+      { api_module: 'coupon', api_parent_module: 'order', methods: [:all, :select]},
       { api_module: 'message', api_parent_module: 'order', methods: [:all, :select]},
       { api_module: 'shipment', api_parent_module: 'order', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'configurable_field', api_parent_module: 'product', methods: [:all, :select, :delete]},
+      { api_module: 'tax', api_parent_module: 'order', methods: [:all, :select]},
+      { api_module: 'configurable_field', api_parent_module: 'product', methods: [:all, :select, :delete, :count]},
       { api_module: 'custom_field', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
       { api_module: 'option', api_parent_module: 'option_set', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'value', api_parent_module: 'option_set', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'discount_rule', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'image', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'value', api_parent_module: 'option', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'discount_rule', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'image', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
       { api_module: 'option', api_parent_module: 'product', methods: [:all, :select]},
       { api_module: 'review', api_parent_module: 'product', methods: [:all]},
-      { api_module: 'rule', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'video', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
-      { api_module: 'sku', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete]},
+      { api_module: 'rule', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'video', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'sku', api_parent_module: 'product', methods: [:all, :select, :create, :update, :delete, :count]},
+      { api_module: 'state', api_parent_module: 'country', methods: [:all, :select, :count]},
       { api_module: 'address', api_parent_module: 'customer', methods: [:all, :select, :create, :update, :delete]}
   ]. each do |nested_module|
     api_parent_module = nested_module[:api_parent_module]
@@ -28,101 +31,202 @@ describe BigcommerceOAuthAPI::Client do
     api_module_pluralized = api_module.pluralize
     path_prefix = (nested_module.has_key?(:prefix_paths) ? "#{nested_module[:prefix_paths]}/" : nil)
     method_prefix = (nested_module.has_key?(:prefix_methods) ? "#{nested_module[:prefix_methods]}_" : nil)
+    has_legacy_support = (nested_module.has_key?(:legacy) ? nested_module[:legacy] : true)
 
-    before do
-      @client = BigcommerceOAuthAPI::Client.new(:store_hash => 'TEST_STORE',
-                                                :client_id => 'SECRET_ID',
-                                                :access_token => 'SECRET_TOKEN')
-    end
+    [:legacy, :oauth].each do |config_type|
+      context "given a #{config_type} configuration" do
+        before do
+          @client = case config_type
+                      when :legacy
+                        BigcommerceOAuthAPI::Client.new(:endpoint => 'http://example.bigcommerce.com',
+                                                        :user_name => 'USER',
+                                                        :api_key => 'API_KEY')
+                      when :oauth
+                        BigcommerceOAuthAPI::Client.new(:store_hash => 'TEST_STORE',
+                                                        :client_id => 'SECRET_ID',
+                                                        :access_token => 'SECRET_TOKEN')
+                    end
+        end
 
-    if nested_module[:methods].include?(:all)
-      describe ".#{method_prefix}#{api_parent_module}_#{api_module_pluralized}" do
-        it "should get a list of #{api_module_pluralized} for the given #{api_parent_module}" do
-          parent_id = 10
-          stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_parent_module}_#{api_module_pluralized}".to_sym, parent_id)
-          expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:all)
+          describe ".#{method_prefix}#{api_parent_module}_#{api_module_pluralized}" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     parent_id = 10
+            #     expect { @client.send("#{method_prefix}#{api_parent_module}_#{api_module_pluralized}".to_sym, parent_id) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "should get a list of #{api_module_pluralized} for the given #{api_parent_module}" do
+                parent_id = 10
+                stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_parent_module}_#{api_module_pluralized}".to_sym, parent_id)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}"))
+                      .to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
-      end
-    end
 
-    if nested_module[:methods].include?(:select)
-      describe ".#{method_prefix}#{api_parent_module}_#{api_module}" do
-        it "gets the #{api_module} with the given id for the given #{api_parent_module}" do
-          id = 10
-          parent_id = 5
-          stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id)
-          expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:select)
+          describe ".#{method_prefix}#{api_parent_module}_#{api_module}" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     id = 10
+            #     parent_id = 5
+            #     expect { @client.send("#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "gets the #{api_module} with the given id for the given #{api_parent_module}" do
+                id = 10
+                parent_id = 5
+                stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}"))
+                      .to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
-      end
-    end
 
-    if nested_module[:methods].include?(:create)
-      describe ".create_#{method_prefix}#{api_parent_module}_#{api_module}" do
-        it "creates a #{api_module} with the given attributes for the given #{api_parent_module}" do
-          options = { name: 'A', description: 'B'}
-          parent_id = 5
-          stub_post(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
-              to_return(:body => options.to_json, :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("create_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, options)
-          expect(a_post(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
-                     with(:body => options,
-                          :headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:create)
+          describe ".create_#{method_prefix}#{api_parent_module}_#{api_module}" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     options = { name: 'A', description: 'B'}
+            #     parent_id = 5
+            #     expect { @client.send("create_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, options) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "creates a #{api_module} with the given attributes for the given #{api_parent_module}" do
+                options = { name: 'A', description: 'B'}
+                parent_id = 5
+                stub_post(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
+                    to_return(:body => options.to_json, :headers => { :content_type => "application/#{@client.format}" })
+                @client.send("create_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, options)
+                if config_type == :legacy
+                  expect(a_post(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}"))
+                      .to have_been_made
+                else
+                  expect(a_post(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}").
+                             with(:body => options,
+                                  :headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
-      end
-    end
 
-    if nested_module[:methods].include?(:update)
-      describe ".update_#{method_prefix}#{api_parent_module}_#{api_module}" do
-        it "update the attributes of the #{api_module} with the given id for the #{api_parent_module}" do
-          id = 10
-          parent_id = 5
-          options = { name: 'A', description: 'B'}
-          stub_put(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-              with(:body => options).
-              to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("update_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id, options)
-          expect(a_put(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-                     with(:body => options,
-                          :headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:update)
+          describe ".update_#{method_prefix}#{api_parent_module}_#{api_module}" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     options = { name: 'A', description: 'B'}
+            #     id = 10
+            #     parent_id = 5
+            #     expect { @client.send("update_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id, options) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "update the attributes of the #{api_module} with the given id for the #{api_parent_module}" do
+                id = 10
+                parent_id = 5
+                options = { name: 'A', description: 'B'}
+                stub_put(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                    with(:body => options).
+                    to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
+                @client.send("update_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id, options)
+                if config_type == :legacy
+                  expect(a_put(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}"))
+                      .to have_been_made
+                else
+                  expect(a_put(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                             with(:body => options,
+                                  :headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
-      end
-    end
 
-    if nested_module[:methods].include?(:delete)
-      describe ".delete_#{method_prefix}#{api_parent_module}_#{api_module}" do
-        it "deletes the #{api_module} with the given id for the #{api_parent_module}" do
-          id = 10
-          parent_id = 5
-          stub_delete(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-              to_return(:headers => { :content_type => "application/#{@client.format}" })
-          @client.send("delete_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id)
-          expect(a_delete(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:delete)
+          describe ".delete_#{method_prefix}#{api_parent_module}_#{api_module}" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     id = 10
+            #     parent_id = 5
+            #     expect { @client.send("delete_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "deletes the #{api_module} with the given id for the #{api_parent_module}" do
+                id = 10
+                parent_id = 5
+                stub_delete(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                    to_return(:headers => { :content_type => "application/#{@client.format}" })
+                @client.send("delete_#{method_prefix}#{api_parent_module}_#{api_module}".to_sym, parent_id, id)
+                if config_type == :legacy
+                  expect(a_delete(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}"))
+                      .to have_been_made
+                else
+                  expect(a_delete(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/#{id}").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
-      end
-    end
 
-    if nested_module[:methods].include?(:count)
-      describe ".#{method_prefix}#{api_module_pluralized}_count" do
-        it "returns the number of #{api_module_pluralized} for the #{api_parent_module}" do
-          parent_id = 5
-          stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/count").
-              to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
-          @client.send("#{method_prefix}#{api_module_pluralized}_count".to_sym)
-          expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/count").
-                     with(:headers => {'X-Auth-Client' => 'SECRET_ID',
-                                       'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+        if nested_module[:methods].include?(:count)
+          describe ".#{method_prefix}#{api_parent_module}_#{api_module_pluralized}_count" do
+            # NOTICE! Currently all the nested APIs have legacy support
+            # if config_type == :legacy && !has_legacy_support
+            #   it 'should raise a non legacy api error' do
+            #     parent_id = 5
+            #     expect { @client.send("#{method_prefix}#{api_parent_module}_#{api_module_pluralized}_count".to_sym, parent_id) }.
+            #         to raise_error(BigcommerceOAuthAPI::NonLegacyApi)
+            #   end
+            # else
+              it "returns the number of #{api_module_pluralized} for the #{api_parent_module}" do
+                parent_id = 5
+                stub_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/count").
+                    to_return(:body => '', :headers => { :content_type => "application/#{@client.format}" })
+                @client.send("#{method_prefix}#{api_parent_module}_#{api_module_pluralized}_count".to_sym, parent_id)
+                if config_type == :legacy
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/count"))
+                      .to have_been_made
+                else
+                  expect(a_get(@client, "#{path_prefix}#{api_parent_module_pluralized}/#{parent_id}/#{api_module_pluralized}/count").
+                             with(:headers => {'X-Auth-Client' => 'SECRET_ID',
+                                               'X-Auth-Token' => 'SECRET_TOKEN'})).to have_been_made
+                end
+              end
+            # end
+          end
         end
       end
     end