aws_agcod_2 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57228cfced3a4ce27d86778e5458cb28c6dd00490880ae22db9ef8c2b1ca456e
-  data.tar.gz: f06da4395c979dba879ef0aaced9bbdd6ad79958b3514340d6ccbed0bfec74f0
+  metadata.gz: 1d03083227c8e83a78a5bd5080293fd21682bfd9d94d979ab6465f9848f18b3a
+  data.tar.gz: 41d35ecbde14d73906af7ab4e08c5c00c1c2768f49bd82ed4cb56e3d13aaeea2
 SHA512:
-  metadata.gz: 7384eddccb0d643c91eb8e6c00ea515333b7a9a602b46200f2bda20d1d5464e49c02dbc36b3d650de43ffc27aa9a5fc6bc35349a6ae6b1007f92400545e3b12a
-  data.tar.gz: 7971db886d28511ba4818d283b8c3245ac0cf14f850d14ea58539e8b574293ae4d7b75454d0661574556bd6b7348a83dfe8d11bec623071e7efa0c187aef5a79
+  metadata.gz: 19b706ad529e1578c7116051b54a90d1432de36ea1ffcf442e67eaa2dc5c549e5de05248925d9e046888f7d3884deccbdfcecae995b9e976af5b2255dbd39870
+  data.tar.gz: 0d0c070d32a953dd436ffc50e60f96a159415e961cd68ff4650c528265416df84d9c0c1d6052ab2f14a23f5c9937c7b1afde4621084b2bd4727068306512c6f4

data/README.md

@@ -1,7 +1,7 @@
 # AGCOD
 
 [![Build Status](https://travis-ci.org/compwron/aws_agcod.svg?branch=master)](https://travis-ci.org/compwron/aws_agcod)
-[![Gem Version](https://badge.fury.io/rb/aws_agcod.svg)](http://badge.fury.io/rb/aws_agcod)
+[![Gem Version](https://badge.fury.io/rb/aws_agcod_2.svg)](http://badge.fury.io/rb/aws_agcod_2)
 
 Amazon Gift Code On Demand (AGCOD) API v2 implementation for distributing Amazon gift cards (gift codes) instantly in any denomination.
 
@@ -10,7 +10,7 @@ Amazon Gift Code On Demand (AGCOD) API v2 implementation for distributing Amazon
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'aws_agcod'
+gem 'aws_agcod_2' # https://rubygems.org/gems/aws_agcod_2
 ```
 
 And then execute:

data/lib/aws_agcod.rb

@@ -1,8 +1,8 @@
-require "aws_agcod/version"
-require "aws_agcod/config"
-require "aws_agcod/create_gift_card"
-require "aws_agcod/cancel_gift_card"
-require "aws_agcod/gift_card_activity_list"
+require 'aws_agcod/version'
+require 'aws_agcod/config'
+require 'aws_agcod/create_gift_card'
+require 'aws_agcod/cancel_gift_card'
+require 'aws_agcod/gift_card_activity_list'
 
 module AGCOD
   def self.configure(&block)

data/lib/aws_agcod/cancel_gift_card.rb

@@ -1,4 +1,4 @@
-require "aws_agcod/request"
+require 'aws_agcod/request'
 
 module AGCOD
   class CancelGiftCard
@@ -7,9 +7,9 @@ module AGCOD
     def_delegators :@response, :status, :success?, :error_message
 
     def initialize(httpable, request_id, gc_id)
-      @response = Request.new(httpable,"CancelGiftCard",
-        "creationRequestId" => request_id,
-        "gcId" => gc_id
+      @response = Request.new(httpable,'CancelGiftCard',
+        'creationRequestId' => request_id,
+        'gcId' => gc_id
       ).response
     end
   end

data/lib/aws_agcod/config.rb

@@ -8,14 +8,14 @@ module AGCOD
                   :production
 
     URI = {
-      sandbox: "https://agcod-v2-gamma.amazon.com",
-      production: "https://agcod-v2.amazon.com"
+      sandbox: 'https://agcod-v2-gamma.amazon.com',
+      production: 'https://agcod-v2.amazon.com'
     }
 
     def initialize
       # API defaults
       @production = false
-      @region = "us-east-1"
+      @region = 'us-east-1'
     end
 
     def uri

data/lib/aws_agcod/create_gift_card.rb

@@ -1,4 +1,4 @@
-require "aws_agcod/request"
+require 'aws_agcod/request'
 
 module AGCOD
   class CreateGiftCardError < StandardError; end
@@ -10,34 +10,34 @@ module AGCOD
 
     def_delegators :@response, :status, :success?, :error_message
 
-    def initialize(httpable, request_id, amount, currency = "USD")
+    def initialize(httpable, request_id, amount, currency = 'USD')
       unless CURRENCIES.include?(currency.to_s)
         raise CreateGiftCardError, "Currency #{currency} not supported, available types are #{CURRENCIES.join(", ")}"
       end
 
-      @response = Request.new(httpable, "CreateGiftCard",
-        "creationRequestId" => request_id,
-        "value" => {
-          "currencyCode" => currency,
-          "amount" => amount
+      @response = Request.new(httpable, 'CreateGiftCard',
+        'creationRequestId' => request_id,
+        'value' => {
+          'currencyCode' => currency,
+          'amount' => amount
         }
       ).response
     end
 
     def claim_code
-      @response.payload["gcClaimCode"]
+      @response.payload['gcClaimCode']
     end
 
     def expiration_date
-      @expiration_date ||= Time.parse @response.payload["gcExpirationDate"]
+      @expiration_date ||= Time.parse @response.payload['gcExpirationDate']
     end
 
     def gc_id
-      @response.payload["gcId"]
+      @response.payload['gcId']
     end
 
     def request_id
-      @response.payload["creationRequestId"]
+      @response.payload['creationRequestId']
     end
   end
 end

data/lib/aws_agcod/gift_card_activity_list.rb

@@ -1,4 +1,4 @@
-require "aws_agcod/request"
+require 'aws_agcod/request'
 
 module AGCOD
   class GiftCardActivityListError < StandardError; end
@@ -9,19 +9,19 @@ module AGCOD
 
     def initialize(payload)
       @payload = payload
-      @status = payload["activityStatus"]
-      @created_at = payload["activityTime"]
-      @type = payload["activityType"]
-      @card_number = payload["cardNumber"]
-      @amount = payload["cardValue"]["amount"] if payload["cardValue"]
-      @error_code = payload["failureCode"]
-      @gc_id = payload["gcId"]
-      @partner_id = payload["partnerId"]
-      @request_id = payload["requestId"]
+      @status = payload['activityStatus']
+      @created_at = payload['activityTime']
+      @type = payload['activityType']
+      @card_number = payload['cardNumber']
+      @amount = payload['cardValue']['amount'] if payload['cardValue']
+      @error_code = payload['failureCode']
+      @gc_id = payload['gcId']
+      @partner_id = payload['partnerId']
+      @request_id = payload['requestId']
     end
 
     def is_real?
-      @payload["isRealOp"] == "true"
+      @payload['isRealOp'] == 'true'
     end
   end
 
@@ -29,25 +29,25 @@ module AGCOD
     extend Forwardable
 
     LIMIT = 1000 # limit per request
-    TIME_FORMAT = "%Y-%m-%dT%H:%M:%SZ"
+    TIME_FORMAT = '%Y-%m-%dT%H:%M:%SZ'
 
     def_delegators :@response, :success?, :error_message
 
     def initialize(httpable, request_id, start_time, end_time, page = 1, per_page = 100, show_no_ops = false)
       raise GiftCardActivityListError, "Only #{LIMIT} records allowed per request." if per_page > LIMIT
 
-      @response = Request.new(httpable,"GetGiftCardActivityPage",
-        "requestId" => request_id,
-        "utcStartDate" => start_time.strftime(TIME_FORMAT),
-        "utcEndDate" => end_time.strftime(TIME_FORMAT),
-        "pageIndex" => (page - 1) * per_page,
-        "pageSize" => per_page,
-        "showNoOps" => show_no_ops
+      @response = Request.new(httpable,'GetGiftCardActivityPage',
+        'requestId' => request_id,
+        'utcStartDate' => start_time.strftime(TIME_FORMAT),
+        'utcEndDate' => end_time.strftime(TIME_FORMAT),
+        'pageIndex' => (page - 1) * per_page,
+        'pageSize' => per_page,
+        'showNoOps' => show_no_ops
       ).response
     end
 
     def results
-      @response.payload["cardActivityList"].map { |payload| GiftCardActivity.new(payload) }
+      @response.payload['cardActivityList'].map { |payload| GiftCardActivity.new(payload) }
     end
   end
 end

data/lib/aws_agcod/request.rb

@@ -1,11 +1,11 @@
-require "aws_agcod/signature"
-require "aws_agcod/response"
-require "http"
-require "yaml"
+require 'aws_agcod/signature'
+require 'aws_agcod/response'
+require 'http'
+require 'yaml'
 
 module AGCOD
   class Request
-    TIME_FORMAT = "%Y%m%dT%H%M%SZ"
+    TIME_FORMAT = '%Y%m%dT%H%M%SZ'
     MOCK_REQUEST_IDS = %w(F0000 F2005)
 
     attr_reader :response
@@ -23,12 +23,12 @@ module AGCOD
       time = Time.now.utc
 
       headers = {
-        "content-type" => "application/json",
-        "x-amz-date" => time.strftime(TIME_FORMAT),
-        "accept" => "application/json",
-        "host" => uri.host,
+        'content-type' => 'application/json',
+        'x-amz-date' => time.strftime(TIME_FORMAT),
+        'accept' => 'application/json',
+        'host' => uri.host,
         "x-amz-target" => "com.amazonaws.agcod.AGCODService.#{@action}",
-        "date" => time.to_s
+        'date' => time.to_s
       }
 
       Signature.new(AGCOD.config).sign(uri, headers, body)
@@ -40,19 +40,19 @@ module AGCOD
 
     def body
       @body ||= @params.merge(
-        "partnerId" => AGCOD.config.partner_id
+        'partnerId' => AGCOD.config.partner_id
       ).to_json
     end
 
     def sanitized_params(params)
       # Prefix partner_id when it's not given as part of request_id for creationRequestId and it's not a mock request_id
-      if params["creationRequestId"] && !(params["creationRequestId"] =~ /#{AGCOD.config.partner_id}/) && !(MOCK_REQUEST_IDS.member?(params["creationRequestId"]))
-        params["creationRequestId"] = "#{AGCOD.config.partner_id}#{params["creationRequestId"]}"
+      if params['creationRequestId'] && !(params['creationRequestId'] =~ /#{AGCOD.config.partner_id}/) && !(MOCK_REQUEST_IDS.member?(params["creationRequestId"]))
+        params['creationRequestId'] = "#{AGCOD.config.partner_id}#{params['creationRequestId']}"
       end
 
       # Remove partner_id when it's prefixed in requestId
-      if params["requestId"] && !!(params["requestId"] =~ /^#{AGCOD.config.partner_id}/)
-        params["requestId"].sub!(/^#{AGCOD.config.partner_id}/, "")
+      if params['requestId'] && !!(params['requestId'] =~ /^#{AGCOD.config.partner_id}/)
+        params['requestId'].sub!(/^#{AGCOD.config.partner_id}/, '')
       end
 
       params

data/lib/aws_agcod/response.rb

@@ -11,21 +11,21 @@ module AGCOD
       # SUCCESS -- Operation succeeded
       # FAILURE -- Operation failed
       # RESEND -- A temporary/recoverable system failure that can be resolved by the partner retrying the request
-      @status = if payload["status"]
-        payload["status"]
-      elsif payload["agcodResponse"]
-        payload["agcodResponse"]["status"]
+      @status = if payload['status']
+        payload['status']
+      elsif payload['agcodResponse']
+        payload['agcodResponse']['status']
       else
-        "FAILURE"
+        'FAILURE'
       end
     end
 
     def success?
-      status == "SUCCESS"
+      status == 'SUCCESS'
     end
 
     def error_message
-      "#{payload["errorCode"]} #{payload["errorType"]} - #{payload["message"]}"
+      "#{payload['errorCode']} #{payload['errorType']} - #{payload['message']}"
     end
   end
 end

data/lib/aws_agcod/signature.rb

@@ -2,13 +2,13 @@
 # this class generates signed headers for making proper request to AGCOD service.
 #
 # Based on https://github.com/ifeelgoods/aws4/blob/master/lib/aws4/signer.rb
-require "openssl"
-require "uri"
-require "pathname"
+require 'openssl'
+require 'uri'
+require 'pathname'
 
 module AGCOD
   class Signature
-    SERVICE = "AGCODService"
+    SERVICE = 'AGCODService'
 
     def initialize(credentials)
       @access_key = credentials.access_key
@@ -16,14 +16,14 @@ module AGCOD
       @region = credentials.region || DEFAULT_REGION
     end
 
-    def sign(uri, headers, body = "")
+    def sign(uri, headers, body = '')
       @uri = uri
       @headers = headers
       @body = body
-      @date = headers["x-amz-date"]
+      @date = headers['x-amz-date']
 
       signed_headers = headers.dup
-      signed_headers["Authorization"] = authorization
+      signed_headers['Authorization'] = authorization
 
       signed_headers
     end
@@ -35,22 +35,22 @@ module AGCOD
         "AWS4-HMAC-SHA256 Credential=#{@access_key}/#{credential_string}",
         "SignedHeaders=#{@headers.keys.map(&:downcase).sort.join(";")}",
         "Signature=#{signature}"
-      ].join(", ")
+      ].join(', ')
     end
 
     # Reference http://docs.aws.amazon.com/general/latest/gr/sigv4-calculate-signature.html
     def signature
-      k_date = hmac("AWS4" + @secret_key, @date[0, 8])
+      k_date = hmac('AWS4' + @secret_key, @date[0, 8])
       k_region = hmac(k_date, @region)
       k_service = hmac(k_region, SERVICE)
-      k_credentials = hmac(k_service, "aws4_request")
+      k_credentials = hmac(k_service, 'aws4_request')
       hexhmac(k_credentials, string_to_sign)
     end
 
     # Reference http://docs.aws.amazon.com/general/latest/gr/sigv4-create-string-to-sign.html
     def string_to_sign
       @string_to_sign ||= [
-        "AWS4-HMAC-SHA256", # Algorithm
+        'AWS4-HMAC-SHA256', # Algorithm
         @date, # RequestDate
         credential_string, # CredentialScope
         hexdigest(canonical_request) # HashedCanonicalRequest
@@ -58,17 +58,17 @@ module AGCOD
     end
 
     def credential_string
-      @credential_string ||= [@date[0, 8], @region, SERVICE, "aws4_request"].join("/")
+      @credential_string ||= [@date[0, 8], @region, SERVICE, 'aws4_request'].join('/')
     end
 
     # Reference http://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html
     def canonical_request
       @canonical_request ||= [
-        "POST", # HTTPRequestMethod
+        'POST', # HTTPRequestMethod
         Pathname.new(@uri.path).cleanpath.to_s, # CanonicalURI
         @uri.query, # CanonicalQueryString
         @headers.sort.map { |k, v| [k.downcase, v.strip].join(":") }.join("\n") + "\n", # CanonicalHeaders
-        @headers.sort.map { |k, v| k.downcase }.join(";"), # SignedHeaders
+        @headers.sort.map { |k, v| k.downcase }.join(';'), # SignedHeaders
         hexdigest(@body) # HexEncode(Hash(RequestPayload))
       ].join("\n")
     end
@@ -85,11 +85,11 @@ module AGCOD
     # is a mechanism for calculating a message authentication code
     # involving a hash function in combination with a secret key
     def hmac(key, value)
-      OpenSSL::HMAC.digest(OpenSSL::Digest.new("sha256"), key, value)
+      OpenSSL::HMAC.digest(OpenSSL::Digest.new('sha256'), key, value)
     end
 
     def hexhmac(key, value)
-      OpenSSL::HMAC.hexdigest(OpenSSL::Digest.new("sha256"), key, value)
+      OpenSSL::HMAC.hexdigest(OpenSSL::Digest.new('sha256'), key, value)
     end
   end
 end

data/lib/aws_agcod/version.rb

@@ -1,3 +1,3 @@
 module AGCOD
-  VERSION = "1.1.0"
+  VERSION = '1.1.1'
 end

data/spec/aws_agcod/cancel_gift_card_spec.rb

@@ -1,8 +1,8 @@
-require "spec_helper"
-require "aws_agcod/cancel_gift_card"
+require 'spec_helper'
+require 'aws_agcod/cancel_gift_card'
 
 describe AGCOD::CancelGiftCard do
-  let(:partner_id) { "Testa" }
+  let(:partner_id) { 'Testa' }
   let(:response) { spy }
   let(:httpable) { HTTP }
 
@@ -12,25 +12,25 @@ describe AGCOD::CancelGiftCard do
     end
   end
 
-  context ".new" do
-    let(:request_id) { "test1" }
-    let(:gc_id) { "FOO" }
+  context '.new' do
+    let(:request_id) { 'test1' }
+    let(:gc_id) { 'FOO' }
 
-    it "makes cancel request" do
+    it 'makes cancel request' do
       expect(AGCOD::Request).to receive(:new) do |_, action, params|
-        expect(action).to eq("CancelGiftCard")
-        expect(params["creationRequestId"]).to eq(request_id)
-        expect(params["gcId"]).to eq(gc_id)
+        expect(action).to eq('CancelGiftCard')
+        expect(params['creationRequestId']).to eq(request_id)
+        expect(params['gcId']).to eq(gc_id)
       end.and_return(response)
 
       AGCOD::CancelGiftCard.new(httpable, request_id, gc_id)
     end
   end
 
-  shared_context "request with response" do
-    let(:gc_id) { "BAR" }
-    let(:creation_request_id) { "BAZ" }
-    let(:status) { "SUCCESS" }
+  shared_context 'request with response' do
+    let(:gc_id) { 'BAR' }
+    let(:creation_request_id) { 'BAZ' }
+    let(:status) { 'SUCCESS' }
     let(:request) { AGCOD::CancelGiftCard.new(httpable, creation_request_id, gc_id) }
 
     before do
@@ -39,10 +39,10 @@ describe AGCOD::CancelGiftCard do
     end
   end
 
-  context "#status" do
-    include_context "request with response"
+  context '#status' do
+    include_context 'request with response'
 
-    it "returns the response status" do
+    it 'returns the response status' do
       expect(request.status).to eq(status)
     end
   end

data/spec/aws_agcod/config_spec.rb

@@ -1,41 +1,41 @@
-require "spec_helper"
-require "aws_agcod/config"
+require 'spec_helper'
+require 'aws_agcod/config'
 
 describe AGCOD::Config do
   let(:config) { AGCOD::Config.new }
 
-  context ".new" do
-    it "sets default uri and region" do
+  context '.new' do
+    it 'sets default uri and region' do
       expect(config.uri).not_to be_nil
       expect(config.region).not_to be_nil
       expect(config.production).to eq(false)
     end
   end
 
-  context "#uri" do
-    context "when uri is set" do
+  context '#uri' do
+    context 'when uri is set' do
       before do
-        config.uri = "https://custom-uri.example.com"
+        config.uri = 'https://custom-uri.example.com'
       end
 
-      it "returns the custom uri" do
-        expect(config.uri).to eq("https://custom-uri.example.com")
+      it 'returns the custom uri' do
+        expect(config.uri).to eq('https://custom-uri.example.com')
       end
     end
 
-    context "when uri is not set" do
-      context "when production is enabled" do
+    context 'when uri is not set' do
+      context 'when production is enabled' do
         before do
           config.production = true
         end
 
-        it "returns the production uri" do
+        it 'returns the production uri' do
           expect(config.uri).to eq(AGCOD::Config::URI[:production])
         end
       end
 
-      context "when production is disabled" do
-        it "returns the sandbox uri" do
+      context 'when production is disabled' do
+        it 'returns the sandbox uri' do
           expect(config.uri).to eq(AGCOD::Config::URI[:sandbox])
         end
       end

data/spec/aws_agcod/create_gift_card_spec.rb

@@ -1,9 +1,9 @@
-require "spec_helper"
-require "aws_agcod/create_gift_card"
+require 'spec_helper'
+require 'aws_agcod/create_gift_card'
 
 describe AGCOD::CreateGiftCard do
-  let(:partner_id) { "Testa" }
-  let(:request_id) { "test1" }
+  let(:partner_id) { 'Testa' }
+  let(:request_id) { 'test1' }
   let(:amount) { 10 }
   let(:currency) { AGCOD::CreateGiftCard::CURRENCIES.first }
   let(:response) { spy }
@@ -15,16 +15,16 @@ describe AGCOD::CreateGiftCard do
     end
   end
 
-  context ".new" do
-    context "when currency available" do
-      it "makes create request" do
+  context '.new' do
+    context 'when currency available' do
+      it 'makes create request' do
         expect(AGCOD::Request).to receive(:new) do |httpable, action, params|
           expect(httpable).to eq(HTTP)
-          expect(action).to eq("CreateGiftCard")
-          expect(params["creationRequestId"]).to eq(request_id)
-          expect(params["value"]).to eq(
-            "currencyCode" => currency,
-            "amount" => amount
+          expect(action).to eq('CreateGiftCard')
+          expect(params['creationRequestId']).to eq(request_id)
+          expect(params['value']).to eq(
+            'currencyCode' => currency,
+            'amount' => amount
           )
         end.and_return(response)
 
@@ -32,10 +32,10 @@ describe AGCOD::CreateGiftCard do
       end
     end
 
-    context "when currency not available" do
-      let(:currency) { "NOTEXIST" }
+    context 'when currency not available' do
+      let(:currency) { 'NOTEXIST' }
 
-      it "raises error" do
+      it 'raises error' do
         expect {
           AGCOD::CreateGiftCard.new(httpable, request_id, amount, currency)
         }.to raise_error(
@@ -46,13 +46,13 @@ describe AGCOD::CreateGiftCard do
     end
   end
 
-  shared_context "request with response" do
-    let(:claim_code) { "FOO" }
-    let(:expiration_date) { "Wed Mar 12 22:59:59 UTC 2025" }
-    let(:gc_id) { "BAR" }
-    let(:creation_request_id) { "BAZ" }
-    let(:status) { "SUCCESS" }
-    let(:payload) { {"gcClaimCode" => claim_code, "gcId" => gc_id, "creationRequestId" => creation_request_id, "gcExpirationDate" => expiration_date} }
+  shared_context 'request with response' do
+    let(:claim_code) { 'FOO' }
+    let(:expiration_date) { 'Wed Mar 12 22:59:59 UTC 2025' }
+    let(:gc_id) { 'BAR' }
+    let(:creation_request_id) { 'BAZ' }
+    let(:status) { 'SUCCESS' }
+    let(:payload) { {'gcClaimCode' => claim_code, 'gcId' => gc_id, 'creationRequestId' => creation_request_id, 'gcExpirationDate' => expiration_date} }
     let(:request) { AGCOD::CreateGiftCard.new(httpable, request_id, amount, currency) }
 
     before do
@@ -62,42 +62,42 @@ describe AGCOD::CreateGiftCard do
     end
   end
 
-  context "#claim_code" do
-    include_context "request with response"
+  context '#claim_code' do
+    include_context 'request with response'
 
-    it "returns claim_code" do
+    it 'returns claim_code' do
       expect(request.claim_code).to eq(claim_code)
     end
   end
 
-  context "#expiration_date" do
-    include_context "request with response"
+  context '#expiration_date' do
+    include_context 'request with response'
 
-    it "returns expiration_date" do
+    it 'returns expiration_date' do
       expect(request.expiration_date).to eq(Time.parse expiration_date)
     end
   end
 
-  context "#gc_id" do
-    include_context "request with response"
+  context '#gc_id' do
+    include_context 'request with response'
 
-    it "returns gc_id" do
+    it 'returns gc_id' do
       expect(request.gc_id).to eq(gc_id)
     end
   end
 
-  context "#request_id" do
-    include_context "request with response"
+  context '#request_id' do
+    include_context 'request with response'
 
-    it "returns creation request_id" do
+    it 'returns creation request_id' do
       expect(request.request_id).to eq(creation_request_id)
     end
   end
 
-  context "#status" do
-    include_context "request with response"
+  context '#status' do
+    include_context 'request with response'
 
-    it "returns the response status" do
+    it 'returns the response status' do
       expect(request.status).to eq(status)
     end
   end

data/spec/aws_agcod/get_available_funds_spec.rb

@@ -17,20 +17,20 @@ describe AGCOD::GetAvailableFunds do
       }),
     }),
   )
-  let(:payload) { { "availableFunds" => { "amount" => amount, "currencyCode" => currency_code }, "status" => status, "timestamp" => timestamp } }
+  let(:payload) { { 'availableFunds' => { 'amount' => amount, 'currencyCode' => currency_code }, 'status' => status, 'timestamp' => timestamp } }
 
   let(:request_response) {
     OpenStruct.new(response:
       OpenStruct.new(payload:
         {
-          "availableFunds" => [
+          'availableFunds' => [
             {
-              "amount" => amount,
-              "currencyCode" => currency_code
+              'amount' => amount,
+              'currencyCode' => currency_code
             }
           ],
-          "status" => status,
-          "timestamp" => timestamp
+          'status' => status,
+          'timestamp' => timestamp
         }
       )
     )

data/spec/aws_agcod/gift_card_activity_list_spec.rb

@@ -1,11 +1,11 @@
-require "spec_helper"
-require "aws_agcod/gift_card_activity_list"
+require 'spec_helper'
+require 'aws_agcod/gift_card_activity_list'
 
 describe AGCOD::GiftCardActivityList do
-  let(:partner_id) { "Testa" }
-  let(:request_id) { "test1" }
-  let(:start_time) { double("start_time") }
-  let(:end_time) { double("end_time") }
+  let(:partner_id) { 'Testa' }
+  let(:request_id) { 'test1' }
+  let(:start_time) { double('start_time') }
+  let(:end_time) { double('end_time') }
   let(:page) { 1 }
   let(:per_page) { AGCOD::GiftCardActivityList::LIMIT }
   let(:show_no_ops) { true }
@@ -18,8 +18,8 @@ describe AGCOD::GiftCardActivityList do
     end
   end
 
-  context ".new" do
-    it "makes request" do
+  context '.new' do
+    it 'makes request' do
       expect(start_time).to receive(:strftime).with(
         AGCOD::GiftCardActivityList::TIME_FORMAT
       ).and_return(start_time)
@@ -29,22 +29,22 @@ describe AGCOD::GiftCardActivityList do
       ).and_return(end_time)
 
       expect(AGCOD::Request).to receive(:new) do |_, action, params|
-        expect(action).to eq("GetGiftCardActivityPage")
-        expect(params["requestId"]).to eq(request_id)
-        expect(params["utcStartDate"]).to eq(start_time)
-        expect(params["utcEndDate"]).to eq(end_time)
-        expect(params["pageIndex"]).to eq((page - 1) * per_page)
-        expect(params["pageSize"]).to eq(per_page)
-        expect(params["showNoOps"]).to eq(show_no_ops)
+        expect(action).to eq('GetGiftCardActivityPage')
+        expect(params['requestId']).to eq(request_id)
+        expect(params['utcStartDate']).to eq(start_time)
+        expect(params['utcEndDate']).to eq(end_time)
+        expect(params['pageIndex']).to eq((page - 1) * per_page)
+        expect(params['pageSize']).to eq(per_page)
+        expect(params['showNoOps']).to eq(show_no_ops)
       end.and_return(response)
 
       AGCOD::GiftCardActivityList.new(httpable, request_id, start_time, end_time, page, per_page, show_no_ops)
     end
 
-    context "when request per_page reaches limit" do
+    context 'when request per_page reaches limit' do
       let(:per_page) { AGCOD::GiftCardActivityList::LIMIT + 1 }
 
-      it "raises error" do
+      it 'raises error' do
         expect {
           AGCOD::GiftCardActivityList.new(httpable, request_id, start_time, end_time, page, per_page, show_no_ops)
         }.to raise_error(
@@ -55,8 +55,8 @@ describe AGCOD::GiftCardActivityList do
     end
   end
 
-  context "#results" do
-    let(:payload) { { "cardActivityList" => [spy] } }
+  context '#results' do
+    let(:payload) { { 'cardActivityList' => [spy] } }
     let(:request) { AGCOD::GiftCardActivityList.new(httpable, request_id, start_time, end_time, page, per_page, show_no_ops) }
 
     before do
@@ -66,7 +66,7 @@ describe AGCOD::GiftCardActivityList do
       allow(response).to receive(:payload) { payload }
     end
 
-    it "returns GiftCardActivity instances" do
+    it 'returns GiftCardActivity instances' do
       request.results.each do |item|
         expect(item).to be_a(AGCOD::GiftCardActivity)
       end

data/spec/aws_agcod/request_spec.rb

@@ -1,23 +1,23 @@
-require "spec_helper"
-require "aws_agcod/request"
+require 'spec_helper'
+require 'aws_agcod/request'
 
 describe AGCOD::Request do
-  let(:action) { "Foo" }
+  let(:action) { 'Foo' }
   let(:params) { {} }
-  let(:signature) { double("Signature") }
-  let(:signed_headers) { double("signed_headers") }
-  let(:base_uri) { "https://example.com" }
-  let(:partner_id) { "BAR" }
+  let(:signature) { double('Signature') }
+  let(:signed_headers) { double('signed_headers') }
+  let(:base_uri) { 'https://example.com' }
+  let(:partner_id) { 'BAR' }
   let(:config) { double(uri: base_uri, partner_id: partner_id) }
   let(:httpable) { HTTP }
 
-  context "#new" do
+  context '#new' do
     before do
       allow(AGCOD).to receive(:config) { config }
       allow(AGCOD::Signature).to receive(:new).with(config) { signature }
     end
 
-    context "with creationRequestId as special testing value" do
+    context 'with creationRequestId as special testing value' do
       let(:params) { { creationRequestId: creationRequestId } }
 
       subject { AGCOD::Request.new(httpable, action, params) }
@@ -26,48 +26,48 @@ describe AGCOD::Request do
         allow(signature).to receive(:sign).and_return(signed_headers)
       end
 
-      context "with special creationRequestId F0000" do
-        let(:creationRequestId) { "F0000" }
+      context 'with special creationRequestId F0000' do
+        let(:creationRequestId) { 'F0000' }
 
-        it "does not add partnerId to creationRequestId" do
+        it 'does not add partnerId to creationRequestId' do
           expect(HTTP).to receive(:post) do |_, options|
-            expect(JSON.parse(options[:body])["creationRequestId"]).to eq("F0000")
+            expect(JSON.parse(options[:body])['creationRequestId']).to eq('F0000')
           end.and_return(double(body: params.to_json))
           subject
         end
       end
 
-      context "with special creationRequestId F2005" do
-        let(:creationRequestId) { "F2005" }
+      context 'with special creationRequestId F2005' do
+        let(:creationRequestId) { 'F2005' }
 
-        it "does not add partnerId to creationRequestId" do
+        it 'does not add partnerId to creationRequestId' do
           expect(HTTP).to receive(:post) do |_, options|
-            expect(JSON.parse(options[:body])["creationRequestId"]).to eq("F2005")
+            expect(JSON.parse(options[:body])['creationRequestId']).to eq('F2005')
           end.and_return(double(body: params.to_json))
           subject
         end
       end
     end
 
-    it "sends post request to endpoint uri" do
+    it 'sends post request to endpoint uri' do
       expect(signature).to receive(:sign) do |uri, headers, body|
         expect(uri).to eq(URI("#{base_uri}/#{action}"))
         expect(headers.keys).to match_array(%w(content-type x-amz-date accept host x-amz-target date))
-        expect(headers["content-type"]).to eq("application/json")
-        expect(headers["x-amz-target"]).to eq("com.amazonaws.agcod.AGCODService.#{action}")
-        expect(JSON.parse(body)["partnerId"]).to eq(partner_id)
+        expect(headers['content-type']).to eq('application/json')
+        expect(headers['x-amz-target']).to eq("com.amazonaws.agcod.AGCODService.#{action}")
+        expect(JSON.parse(body)['partnerId']).to eq(partner_id)
       end.and_return(signed_headers)
 
       expect(HTTP).to receive(:post) do |uri, options|
         expect(uri).to eq(URI("#{base_uri}/#{action}"))
-        expect(JSON.parse(options[:body])["partnerId"]).to eq(partner_id)
+        expect(JSON.parse(options[:body])['partnerId']).to eq(partner_id)
         expect(options[:headers]).to eq(signed_headers)
       end.and_return(double(body: params.to_json))
 
       AGCOD::Request.new(httpable, action, params)
     end
 
-    it "sets response" do
+    it 'sets response' do
       expect(signature).to receive(:sign) { signed_headers }
       expect(HTTP).to receive(:post) { (double(body: params.to_json)) }
 

data/spec/aws_agcod/response_spec.rb

@@ -1,65 +1,65 @@
-require "spec_helper"
-require "aws_agcod/response"
+require 'spec_helper'
+require 'aws_agcod/response'
 
 describe AGCOD::Response do
-  let(:payload) { { foo: "bar" }.to_json }
+  let(:payload) { { foo: 'bar' }.to_json }
 
-  context "#new" do
-    it "sets payload and status" do
+  context '#new' do
+    it 'sets payload and status' do
       response = AGCOD::Response.new(payload)
 
       expect(response.payload).not_to be_nil
       expect(response.status).not_to be_nil
     end
 
-    context "when no status in payload" do
-      it "sets status to failure" do
-        expect(AGCOD::Response.new(payload).status).to eq("FAILURE")
+    context 'when no status in payload' do
+      it 'sets status to failure' do
+        expect(AGCOD::Response.new(payload).status).to eq('FAILURE')
       end
     end
 
-    context "when has status in payload" do
-      let(:status) { "foo" }
+    context 'when has status in payload' do
+      let(:status) { 'foo' }
       let(:payload) { { status: status }.to_json }
 
-      it "sets status as payload's status" do
+      it 'sets status as payload\'s status' do
         expect(AGCOD::Response.new(payload).status).to eq(status)
       end
     end
 
-    context "when has agcodResponse in payload" do
-      let(:status) { "foo" }
+    context 'when has agcodResponse in payload' do
+      let(:status) { 'foo' }
       let(:payload) { { agcodResponse: { status: status } }.to_json }
 
-      it "sets status as agcodResponse's status" do
+      it 'sets status as agcodResponse\'s status' do
         expect(AGCOD::Response.new(payload).status).to eq(status)
       end
     end
   end
 
-  context "success?" do
-    context "when status is SUCCESS" do
-      let(:payload) { { status: "SUCCESS" }.to_json }
+  context 'success?' do
+    context 'when status is SUCCESS' do
+      let(:payload) { { status: 'SUCCESS' }.to_json }
 
-      it "returns true" do
+      it 'returns true' do
         expect(AGCOD::Response.new(payload).success?).to be_truthy
       end
     end
 
-    context "when status is not SUCCESS" do
-      it "returns false" do
+    context 'when status is not SUCCESS' do
+      it 'returns false' do
         expect(AGCOD::Response.new(payload).success?).to be_falsey
       end
     end
   end
 
-  context "error_message" do
-    let(:error_code) { "foo" }
-    let(:error_type) { "bar" }
-    let(:error_message) { "baz" }
+  context 'error_message' do
+    let(:error_code) { 'foo' }
+    let(:error_type) { 'bar' }
+    let(:error_message) { 'baz' }
     let(:payload) { { errorCode: error_code, errorType: error_type, message: error_message }.to_json }
 
-    it "composes error message by error code, type, and message from payload" do
+    it 'composes error message by error code, type, and message from payload' do
       expect(AGCOD::Response.new(payload).error_message).to eq("#{error_code} #{error_type} - #{error_message}")
     end
   end

data/spec/aws_agcod_spec.rb

@@ -1,9 +1,9 @@
-require "spec_helper"
-require "aws_agcod"
+require 'spec_helper'
+require 'aws_agcod'
 
 describe AGCOD do
-  context ".configure" do
-    it "yields config" do
+  context '.configure' do
+    it 'yields config' do
       AGCOD.configure do |config|
         expect(config).to be_an_instance_of(AGCOD::Config)
       end

data/spec/spec_helper.rb

@@ -1,15 +1,15 @@
-require "pathname"
-require "http"
+require 'pathname'
+require 'http'
 
-ROOT_PATH = Pathname.new(__FILE__).join("../..").expand_path
-$LOAD_PATH.unshift(ROOT_PATH.join("lib"))
+ROOT_PATH = Pathname.new(__FILE__).join('../..').expand_path
+$LOAD_PATH.unshift(ROOT_PATH.join('lib'))
 
 RSpec.configure do |config|
   # Run specs in random order to surface order dependencies. If you find an
   # order dependency and want to debug it, you can fix the order by providing
   # the seed, which is printed after each run.
   #     --seed 1234
-  config.order = "random"
+  config.order = 'random'
 
   # Disable the should syntax compeletely; we use the expect syntax only.
   config.expect_with :rspec do |c|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aws_agcod_2
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Xenor Chang
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-24 00:00:00.000000000 Z
+date: 2018-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http
@@ -67,11 +67,11 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3'
-description: |-
-  Amazon Gift Code On Demand (AGCOD) API v2 implementation for
-                            distribute Amazon gift cards (gift codes) instantly in any denomination
+description: Amazon Gift Code On Demand (AGCOD) API v2 implementation for distributing
+  Amazon gift cards (gift codes) instantly in any denomination
 email:
 - xenor@listia.com
+- compiledwrong+aws_agcod_2@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []