yotpo 0.0.3 → 0.0.5

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDQ2MjFmYTQ4ZTQ4Zjc0ZDU5ZDUwYTQyM2YzZjdmZTIyMjQzNzg1OA==
+    ZDZiMDVjNWFmNjgxMGMzZmE0M2IyMjM2ZTZjNTI3NGQxZDA3ZTFlMg==
   data.tar.gz: !binary |-
-    ZDk4ODZhYTJlYjE2NGI3YmI0MTQ1OGE3ZTc2NzM3NzhlZTUwNzg4YQ==
+    NWVjYWY5MzJmOGIwZTA5ZWE1ZjRkZWE4Y2RlYTM1ZGU2MjAxZjJiOA==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NzhkMjJiMWFhNDZlMDY4Y2UzNjE1N2Y1ZWFhYThhYThmMmE1YjAwOTU3MjU5
-    N2UwNjRmYmY3Y2Y3YTk4NjlhZDM4MjFiZDhlN2Q1NmQzOGE2YWYwMGU1NDEx
-    ZjM0ZjI3NTI3Mjg4YmE5NjM2OTI2NjQxNTcxMjZjNjdlMTVmY2E=
+    YTllNmE1MzYxZTc0Nzc4ZTY2ZGI4MjMyNDA3NDVmYzdiNzgwOWYxNjYzZWU4
+    ZWY0Y2U2ODhmNzEyYzNjNDdiYTNhMmU1NmYxYzJmMTA4MmEyZTc3MjA3MDIw
+    YjBjYjIxMThjNDM2MTBjNmQzMDRkNDgwYTQ1N2RhMGE5ZjdmZWE=
   data.tar.gz: !binary |-
-    NWI2YTU3NmQ0NDI3MTFiZmY0OGM5YWQ2ZjZkYjI5ZDVlZmM1ZjE1N2Q5NTMw
-    ZDIzNTAzZjZjOWU0MzEwMDFiOGI4ZGM0OWU3YTA5NmRiZDYyM2Y2OTIyNGEw
-    MDNhMmE1YjAwZjdkMDUyZGEyM2YzZGVhYzc1ODNkZTFmNjFiNWY=
+    NjMxZWVlZmUxNzRlMTJiNzhmYTQ0ZDlmM2U1MjA4YjBiODM0NTljYzk3Mzgw
+    MTE3MjhkZjcxN2E5ZmEwZjAyNGQzZGZmNjBlZTE2ZDQzMTcwMjc5NTI2NDU2
+    ZDM2MTFiMTBiODBiOTZmZDRiODdiMWQ4MGZmNDlmNjY2YzY3MjI=

data/Gemfile

@@ -10,6 +10,7 @@ group :test do
   gem 'simplecov', :require => false
   gem 'webmock'
   gem 'ffaker'
+  gem 'vcr'
 end
 
 # Specify your gem's dependencies in yotpo-api-connector.gemspec

data/lib/yotpo/api/user.rb

@@ -34,14 +34,14 @@ module Yotpo
     # Retrieves an oauth bearer token from Yotpo
     #
     # @param params [Hash] the request details
-    # @option params [String] :client_id the app key received at the registration
-    # @option params [String] :client_secret app secret received at the registration
+    # @option params [String] :app_key the app key received at the registration
+    # @option params [String] :secret app secret received at the registration
     # @return [Hash] that includes access_token and the token_type
     def get_oauth_token(params)
       request = {
           client_id: params[:app_key],
           client_secret: params[:secret],
-          grant_type: "client_credentials"
+          grant_type: 'client_credentials'
       }
 
       response = post('/oauth/token', request)
@@ -58,7 +58,7 @@ module Yotpo
           app_key: params[:app_key],
           secret: params[:secret]
       }
-      get('/users/b2blogin', request)
+      get('/users/b2blogin.json', request)
     end
   end
 end

data/lib/yotpo/client.rb

@@ -41,6 +41,7 @@ module Yotpo
     # @param url [String] the relative path in the Yotpo API
     # @param params [Hash] the url params that should be passed in the request
     def get(url, params = {})
+      params = params.inject({}){|memo,(k,v)| memo[k.to_s] = v; memo}
       preform(url, :get, params: params) do
         return connection.get(url, params)
       end
@@ -52,6 +53,7 @@ module Yotpo
     # @param url [String] the relative path in the Yotpo API
     # @param params [Hash] the body of the request
     def post(url, params)
+      params = convert_hash_keys(params)
       preform(url, :post, params: params) do
         return connection.post(url, params)
       end
@@ -63,6 +65,7 @@ module Yotpo
     # @param url [String] the relative path in the Yotpo API
     # @param params [Hash] the body of the request
     def put(url, params)
+      params = convert_hash_keys(params)
       preform(url, :put, params: params) do
         return connection.put(url, params)
       end
@@ -130,5 +133,16 @@ module Yotpo
         conn.adapter :typhoeus
       end
     end
+    private
+    def convert_hash_keys(value)
+      case value
+        when Array
+          value.map { |v| convert_hash_keys(v) }
+        when Hash
+          Hash[value.map { |k, v| [k.to_s, convert_hash_keys(v)] }]
+        else
+          value
+      end
+    end
   end
 end

data/lib/yotpo/version.rb

@@ -1,3 +1,3 @@
 module Yotpo
-  VERSION = '0.0.3'
+  VERSION = '0.0.5'
 end

data/spec/api/account_platform_spec.rb

@@ -9,13 +9,12 @@ describe Yotpo::AccountPlatform do
           plan_name: 'test plan',
           platform_type_id: 1,
           deleted: false,
-          utoken: 'asdeuh1di1udifn1309fn09',
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_post("/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/account_platform").
-          to_return(:status => 200, :body => fixture('new_account_platform.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-
-      @response = Yotpo.create_account_platform(create_account_platform_request)
+      VCR.use_cassette('create_account_platform') do
+        @response = Yotpo.create_account_platform(create_account_platform_request)
+      end
     end
 
     subject { @response.body.account_platform }
@@ -23,7 +22,6 @@ describe Yotpo::AccountPlatform do
     it { should respond_to :id }
     it { should respond_to :shop_token }
     it { should respond_to :shop_domain }
-    it { should respond_to :shop_api_url }
     it { should respond_to :plan_name }
     it { should respond_to :platform_type }
     it { should respond_to :deleted }

data/spec/api/account_spec.rb

@@ -9,12 +9,12 @@ describe Yotpo::Account do
           minisite_subdomain: Faker::Internet.domain_name,
           minisite_cname: Faker::Internet.domain_name,
           minisite_subdomain_active: [true, false].sample,
-          utoken: '12ei1rr1i3ufn23itfoijver903',
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_put('/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG').
-          to_return(:status => 200, :body => fixture('update_account.json'), :headers => {})
-      @response = Yotpo.update_account(account_data)
+      VCR.use_cassette('update_account') do
+        @response = Yotpo.update_account(account_data)
+      end
     end
     subject { @response.body.app }
     it { should be_a ::Hashie::Rash }
@@ -25,14 +25,13 @@ describe Yotpo::Account do
   describe '#check_minisite_subdomain' do
     before(:all) do
       sub_domain_data = {
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
           subdomain: 'shalom1',
-          utoken: '12ei1rr1i3ufn23itfoijver903'
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_get('/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/subomain_check/shalom1?utoken=12ei1rr1i3ufn23itfoijver903').
-          to_return(:status => 200, :body => fixture('check_subdomain.json'), :headers => {})
-
-      @response = Yotpo.check_minisite_subdomain(sub_domain_data)
+      VCR.use_cassette('check_minisite_subdomain') do
+        @response = Yotpo.check_minisite_subdomain(sub_domain_data)
+      end
     end
     subject { @response.body.subdomain }
     it { should be_a ::Hashie::Rash }

data/spec/api/product_spec.rb

@@ -4,14 +4,12 @@ describe Yotpo::Product do
   describe '#get_all_bottom_lines' do
     before(:all) do
       get_app_bottom_lines_params = {
-          utoken: 'asdeuh1di1udifn1309fn09',
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'
-
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_get("/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/bottom_lines?utoken=asdeuh1di1udifn1309fn09").
-          to_return(:status => 200, :body => fixture('get_all_bottom_lines.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-
-      @response = Yotpo.get_all_bottom_lines(get_app_bottom_lines_params)
+      VCR.use_cassette('check_minisite_subdomain') do
+        @response = Yotpo.get_all_bottom_lines(get_app_bottom_lines_params)
+      end
     end
 
     subject { @response.body.bottomlines[0] }
@@ -24,14 +22,12 @@ describe Yotpo::Product do
   describe '#get_product_bottom_line' do
     before(:all) do
       get_bottom_line_params = {
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
-          product_id: 'A12'
+          product_id: 'D-4771',
+          app_key: @app_key
       }
-
-      stub_get("/products/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/A12/bottomline").
-          to_return(:status => 200, :body => fixture('get_product_bottom_line.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-
-      @response = Yotpo.get_product_bottom_line(get_bottom_line_params)
+      VCR.use_cassette('get_product_bottom_line') do
+        @response = Yotpo.get_product_bottom_line(get_bottom_line_params)
+      end
     end
 
     subject { @response.body.bottomline }

data/spec/api/purchase_spec.rb

@@ -16,13 +16,13 @@ describe Yotpo::Purchase do
                   description: 'this is the description of a product'
               }
           ],
-          utoken: 'asdeuh1di1udifn1309fn09',
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_post("/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/purchases").
-          to_return(:status => 200, :body => fixture('new_purchase.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
+      VCR.use_cassette('create_new_purchase') do
+        @response = Yotpo.create_new_purchase(create_new_purchase_request)
+      end
 
-      @response = Yotpo.create_new_purchase(create_new_purchase_request)
     end
 
     subject { @response.body }
@@ -34,8 +34,6 @@ describe Yotpo::Purchase do
   describe '#create_new_purchases' do
     before(:all) do
       create_new_purchase_request = {
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
-          utoken: 'asdeuh1di1udifn1309fn09',
           orders: [
               {
                   email: Faker::Internet.email,
@@ -53,12 +51,13 @@ describe Yotpo::Purchase do
 
 
               }
-          ]
+          ],
+        utoken: @utoken,
+        app_key: @app_key
       }
-      stub_post("/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/purchases/mass_create").
-          to_return(:status => 200, :body => fixture('new_purchase.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-
-      @response = Yotpo.create_new_purchases(create_new_purchase_request)
+      VCR.use_cassette('create_new_purchases') do
+        @response = Yotpo.create_new_purchases(create_new_purchase_request)
+      end
     end
 
     subject { @response.body }
@@ -70,13 +69,12 @@ describe Yotpo::Purchase do
   describe '#get_purchases' do
     before(:all) do
       get_purchases_request = {
-          utoken: 'asdeuh1di1udifn1309fn09',
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'
+          utoken: @utoken,
+          app_key: @app_key
       }
-      stub_get('/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/purchases?count=10&page=1&utoken=asdeuh1di1udifn1309fn09').
-          to_return(:status => 200, :body => fixture('get_list_of_purchases.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-
-      @response = Yotpo.get_purchases(get_purchases_request)
+      VCR.use_cassette('get_purchases') do
+        @response = Yotpo.get_purchases(get_purchases_request)
+      end
     end
 
     subject { @response.body.purchases[0] }

data/spec/api/reminder_spec.rb

@@ -5,14 +5,13 @@ describe Yotpo::Reminder do
     describe '#send_test_reminder' do
       before(:all) do
         request ={
-            utoken: 'asdeuh1di1udifn1309fn09',
-            app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
-            email: 'vlad@yotpo.com'
+            email: 'vlad@yotpo.com',
+            utoken: @utoken,
+            app_key: @app_key
         }
-        stub_post('/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/reminders/send_test_email').with(:body => Oj.dump({utoken: 'asdeuh1di1udifn1309fn09',
-                                                                                                                     email: 'vlad@yotpo.com'})).
-            to_return(:body => fixture('send_test_email.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-        @response = Yotpo.send_test_reminder(request)
+        VCR.use_cassette('send_test_reminder') do
+          @response = Yotpo.send_test_reminder(request)
+        end
       end
       subject { @response.body }
       it { should be_a ::Hashie::Rash }

data/spec/api/review_spec.rb

@@ -5,22 +5,22 @@ describe Yotpo::Review do
   describe '#create_review' do
     before(:all) do
       anonymous_review = {
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
-          sku: Faker::Product.model,
+          product_id: 'D-4771',
           domain: Faker::Internet.domain_name,
           product_title: Faker::Product.product,
           product_description: Faker::Lorem.paragraph(3),
           product_url: Faker::Internet.http_url,
           product_image_url: 'https://www.google.com/images/srpr/logo4w.png',
-          display_name: Faker::Internet.user_name,
-          email: Faker::Internet.email,
-          review_content: Faker::Lorem.paragraph(3),
+          user_display_name: Faker::Internet.user_name,
+          user_email: Faker::Internet.email,
+          review_body: Faker::Lorem.paragraph(3),
           review_title: Faker::Lorem.sentence(5),
-          review_score: [0, 0.5, 1, 1.5, 2, 2.5, 3, 3.5, 4, 4.5, 5].sample
+          review_score: [0, 0.5, 1, 1.5, 2, 2.5, 3, 3.5, 4, 4.5, 5].sample,
+          app_key: @app_key
       }
-      stub_get('/reviews/dynamic_create').with(:query => hash_including({:appkey => 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG'})).to_return(:body => fixture('new_review.json'),
-                                                                                                                                             :headers => {:content_type => 'application/json; charset=utf-8'})
-      @response = Yotpo.create_review(anonymous_review)
+      VCR.use_cassette('create_review') do
+        @response = Yotpo.create_review(anonymous_review)
+      end
     end
 
     subject { @response.body.reviews[0] }
@@ -36,12 +36,12 @@ describe Yotpo::Review do
       get_reviews_params = {
           page: 1,
           count: 5,
-          app_key: 'nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG',
-          product_id: 'A12'
+          app_key: @app_key,
+          product_id: 'D-4771'
       }
-      stub_get("/products/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/A12/reviews?count=5&page=1").
-          to_return(:status => 200, :body => fixture('get_product_reviews.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-      @response = Yotpo.get_product_reviews(get_reviews_params)
+      VCR.use_cassette('get_product_reviews') do
+        @response = Yotpo.get_product_reviews(get_reviews_params)
+      end
     end
 
     subject { @response.body.reviews[0] }

data/spec/api/user_spec.rb

@@ -15,8 +15,10 @@ describe Yotpo::User do
           callback_url: Faker::Internet.http_url,
           url: Faker::Internet.http_url
       }
-      stub_post('/users').with(:body => Oj.dump({:user => @user_info})).to_return(:body => fixture('new_user.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-      @response = Yotpo.create_user(@user_info)
+
+      VCR.use_cassette('create_user') do
+        @response = Yotpo.create_user(@user_info)
+      end
     end
     subject { @response.body }
     it { should be_a ::Hashie::Rash }
@@ -29,14 +31,13 @@ describe Yotpo::User do
   describe '#get_oauth_token' do
     before(:all) do
       oauth_request = {
-          app_key: 'a3lmMnC3u4SNmz0ZcHf3lODeIYM9LEQwtTWXRdDP',
-          secret: 'NumuadvlCGOTwnCCvY5BRAhGib1LTCFptYxfvebm',
+          app_key: @app_key,
+          secret: @secret,
           grant_type: "client_credentials"
       }
-      stub_request(:post, "https://api.yotpo.com/oauth/token").
-          with(:body => "{\":client_id\":\"a3lmMnC3u4SNmz0ZcHf3lODeIYM9LEQwtTWXRdDP\",\":client_secret\":\"NumuadvlCGOTwnCCvY5BRAhGib1LTCFptYxfvebm\",\":grant_type\":\"client_credentials\"}").
-          to_return(:status => 200, :body => fixture('bearer_token.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-      @response = Yotpo.get_oauth_token(oauth_request)
+      VCR.use_cassette('get_oauth_token') do
+        @response = Yotpo.get_oauth_token(oauth_request)
+      end
     end
     subject { @response.body }
     it { should be_a ::Hashie::Rash }
@@ -47,11 +48,12 @@ describe Yotpo::User do
   describe '#get_login_url' do
     before(:all) do
       request = {
-          app_key: 'a3lmMnC3u4SNmz0ZcHf3lODeIYM9LEQwtTWXRdDP',
-          secret: 'NumuadvlCGOTwnCCvY5BRAhGib1LTCFptYxfvebm'
+          app_key: @app_key,
+          secret: @secret
       }
-      stub_get('/users/b2blogin').with(:query => hash_including(request)).to_return(:body => fixture('get_login_url.json'), :headers => {:content_type => 'application/json; charset=utf-8'})
-      @response = Yotpo.get_login_url(request)
+      VCR.use_cassette('get_login_url') do
+        @response = Yotpo.get_login_url(request)
+      end
     end
     subject { @response.body }
     it { should be_a ::Hashie::Rash }

data/spec/cassettes/check_minisite_subdomain.yml

@@ -0,0 +1,99 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://api.yotpo.com/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/bottom_lines?utoken=mz3AdCKGsoxJB00u0gmq2usGE3HI8HDHFZC5dlLl
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      User-Agent:
+      - Faraday v0.8.7
+      Yotpo-Api-Connector:
+      - 0.0.4
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - nginx/1.2.1
+      Date:
+      - Thu, 23 May 2013 12:01:38 GMT
+      Content-Type:
+      - application/json; charset=utf-8
+      Transfer-Encoding:
+      - chunked
+      Connection:
+      - keep-alive
+      Status:
+      - 200 OK
+      X-Ua-Compatible:
+      - IE=Edge,chrome=1
+      Etag:
+      - ! '"1098f768b6bbfd4e53e90ac3390a772e"'
+      Cache-Control:
+      - max-age=0, private, must-revalidate
+      X-Request-Id:
+      - 33f3945461b4c4062073959cad0f556b
+      X-Runtime:
+      - '0.061216'
+      Strict-Transport-Security:
+      - max-age=2592000; includeSubdomains
+      P3p:
+      - policyref="/w3c/p3p.xml", CP="IDC DSP COR ADM DEVi TAIi PSA PSD IVAi IVDi
+        CONi HIS OUR IND CNT", CP="CAO PSA OUR"
+    body:
+      encoding: US-ASCII
+      string: ! '{"status":{"code":200,"message":"OK"},"response":{"bottomlines":[{"domain_key":"D-4771","product_score":"3.0","total_reviews":100},{"domain_key":"S5","product_score":"4.5","total_reviews":2},{"domain_key":"123","product_score":"5.0","total_reviews":4}]}}'
+    http_version: 
+  recorded_at: Thu, 23 May 2013 12:01:38 GMT
+- request:
+    method: get
+    uri: https://api.yotpo.com/apps/nNgGNA54ETOqaXQ7hRZymxqdtwwetJKDVs0v8qGG/subomain_check/shalom1?utoken=mz3AdCKGsoxJB00u0gmq2usGE3HI8HDHFZC5dlLl
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      User-Agent:
+      - Faraday v0.8.7
+      Yotpo-Api-Connector:
+      - 0.0.4
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - nginx/1.2.1
+      Date:
+      - Thu, 23 May 2013 12:05:12 GMT
+      Content-Type:
+      - application/json; charset=utf-8
+      Transfer-Encoding:
+      - chunked
+      Connection:
+      - keep-alive
+      Status:
+      - 200 OK
+      X-Ua-Compatible:
+      - IE=Edge,chrome=1
+      Etag:
+      - ! '"9c50c01193944b740719a84607eeaaa0"'
+      Cache-Control:
+      - max-age=0, private, must-revalidate
+      X-Request-Id:
+      - 162be4b9069fb1b3b59a5d48d88bce0c
+      X-Runtime:
+      - '0.091567'
+      Strict-Transport-Security:
+      - max-age=2592000; includeSubdomains
+      P3p:
+      - policyref="/w3c/p3p.xml", CP="IDC DSP COR ADM DEVi TAIi PSA PSD IVAi IVDi
+        CONi HIS OUR IND CNT", CP="CAO PSA OUR"
+    body:
+      encoding: US-ASCII
+      string: ! '{"status":{"code":200,"message":"OK"},"response":{"subdomain":{"available":"true","subdomain":"shalom1"}}}'
+    http_version: 
+  recorded_at: Thu, 23 May 2013 12:05:12 GMT
+recorded_with: VCR 2.2.5