balihoo_lpc_client 0.5.1 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 449cbc54003a199e7fa1d809a3cdd2fec2ff464f
-  data.tar.gz: 4e00cabd5c9c3533e818900d792dd780c30fb322
+  metadata.gz: a419f5a1be223fef3953c4d503c46ca1efc38f7e
+  data.tar.gz: 0b8cdec43c0f6c5548824557760e3d4f0b212c54
 SHA512:
-  metadata.gz: 2421d99c26f469f280b9ac6ed1ae0aed8af52186e00edc739305b665d4bb1e09775487a6a5e3090be8409ea3c61fb0e25e6d787f1e27c970d853f886836da1e0
-  data.tar.gz: 36cb2d5b69cbff4eaefabd2048e0876bb1b3a4b199c7b7fd9749e5360876ad492d34155c7be316d67a65028369eff7d90ea02264137923290a847028fa7d7de5
+  metadata.gz: 645048672a5b8baa72902b266866787ffe6db5822b478680d3563af2b924fb235426dac7f7bf3933ada5a5df7d36ad9130b6f3f1fcd1cdc1a791b518cd44d8e2
+  data.tar.gz: de75e508e3f92fd324d9625f16552aee8c73de5b92a3cf5dc98ccd2f78d7293957a6accc3e71d9db91d0e73bc833fe72cc662c33a06ad41cfda2e554824d34b3

data/README.md

@@ -40,8 +40,17 @@ opts = {
   location_key: 'location_key', # optional more below
   user_id: 'brand_key', # currently not used, Balihoo suggests setting same as brand_key
   group_id: 'brand_key' # currently not used, Balihoo suggests setting same as brand_key
+  api_base: 'http://baseurl' # defaults to https://bac.dev.balihoo-cloud.com
+  api_version: 'v1.0' # defaults to v1.0
 }
 config = BalihooLpcClient::Configuration.create(opts)
+
+# Blocks are also supported with the create method
+
+config = BalihooLpcClient::Configuration.create(some_opts) do |c|
+  c.api_base = 'https://bac.dev.balihoo-cloud.com'
+  ...
+end
 ```
 
 The `location_key` is an optional parameter. When not given the api will require
@@ -59,7 +68,9 @@ api.authenticate!
 ```
 
 The `authenticate!` method will call out to Balihoo and set the `client_id` and
-`client_api_key` on the config object for you.
+`client_api_key` on the config object for you. **If an endpoint method is called
+without calling `authenticate!` first a `BalihooLpcClient::NotAuthenticatedError`
+will be raised.**
 
 ```ruby
 config.client_id # => <client_id from Balihoo>

data/lib/balihoo_lpc_client/api.rb

@@ -34,15 +34,24 @@ module BalihooLpcClient
     private
 
     def validate_params_and_fetch!(**args)
-      locations = args[:params][:locations]
+      validate_locations_key locations: args[:params][:locations]
+      authenticated?
 
+      fetch(**args)
+    end
+
+    def validate_locations_key(locations:)
       if config.location_key.nil? && locations.nil?
         raise ApiOptionError, 'must give params[:locations] since no location_key given'
       elsif config.location_key.nil? && !locations.is_a?(Array)
         raise ApiOptionError, 'locations must be an array'
       end
+    end
 
-      fetch(**args)
+    def authenticated?
+      if config.client_id.nil? || config.client_api_key.nil?
+        raise NotAuthenticatedError, 'must call authenticate! before any endpoint'
+      end
     end
 
     def fetch(**args)

data/lib/balihoo_lpc_client/request/authentication.rb

@@ -25,7 +25,7 @@ module BalihooLpcClient
         auth = Response::Authentication.new(data)
         config.client_id = auth.client_id
         config.client_api_key = auth.client_api_key
-        data
+        auth
       end
     end
   end

data/lib/balihoo_lpc_client/version.rb

@@ -1,3 +1,3 @@
 module BalihooLpcClient
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 end

data/lib/balihoo_lpc_client.rb

@@ -24,4 +24,5 @@ module BalihooLpcClient
   class BalihooLpcError < StandardError; end
   class AuthenticationError < BalihooLpcError; end
   class ApiOptionError < BalihooLpcError; end
+  class NotAuthenticatedError < BalihooLpcError; end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: balihoo_lpc_client
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
 platform: ruby
 authors:
 - JD Guzman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-03-07 00:00:00.000000000 Z
+date: 2016-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty