easypost 3.1.4 → 3.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f2b1945f7283b88880be5fa9700332289d7007137056113d6223a61e59be79a
-  data.tar.gz: a168444a606c919ca7a5f1a940040b2822d25ee88d555ca7a11fb289a3e06962
+  metadata.gz: b9e4ea3a1f185f475aac49ab7985092a8ba095e928a812ad77c68eb7b450afdc
+  data.tar.gz: b1291c1f7325559896f109257231fba6f0e781e7f9778c619ec115a86abf9731
 SHA512:
-  metadata.gz: e53924a901d7dc9f25272991d589d7c08b1f28ccba941792925d9fd6ad8b65a57568caf5a15062e227086a2c9d997d85d72d655b21c1be08addc47093a956bff
-  data.tar.gz: 6bf95b960ec71c8b79effae947a38fd11a902fba63d98db28b6f7d448a05b4914f7f8a56ffc33452d8660e5898b60a0f953de10f7be0764b9b2b40058034ec4c
+  metadata.gz: f3c88ce92a498fa4a0bb45e4d0ca06aeee55158af398632ddce04c0513b0e3dae99f27d1d752f3fb6e69daf3cc536d3872c4c18ba141e20eab9e60fdc58872ae
+  data.tar.gz: ec5ec3ad97341782372ba9c7b3e836b451a8c23dd5b9bf27e51dfcfeb561eb1140751c9529b13bbe8b384151ac5023628dff6803fe70a57a4b34070b1d83d64a

data/CHANGELOG

@@ -1,3 +1,9 @@
+=== 3.1.5 2020-12-16
+
+* Fix attribute lookup when manually constructing objects (#105; thanks @drewtempelmeyer)
+* Flatten class names and clean up some other style issues
+* Fix `EasyPost::Address.create_and_verify`, broken since 3.1.0 (#108; thanks @rajbirverma)
+
 === 3.1.4 2020-09-29
 
 * Don't modify params passed into Address#create (#78; thanks @TheRusskiy)

data/README.md

@@ -1,6 +1,7 @@
 # EasyPost Ruby Client Library
 
 [![Build Status](https://travis-ci.com/EasyPost/easypost-ruby.svg?branch=master)](https://travis-ci.com/EasyPost/easypost-ruby)
+[![Gem Version](https://badge.fury.io/rb/easypost.svg)](https://badge.fury.io/rb/easypost)
 
 
 EasyPost is a simple shipping API. You can sign up for an account at https://easypost.com

data/VERSION

@@ -1 +1 @@
-3.1.4
+3.1.5

data/lib/easypost.rb

@@ -118,7 +118,7 @@ module EasyPost
 
     request = Net::HTTP.const_get(method.capitalize).new(path)
     if body
-      request.body = JSON.dump(Util.objects_to_ids(body))
+      request.body = JSON.dump(EasyPost::Util.objects_to_ids(body))
     end
 
     request["Content-Type"] = "application/json"
@@ -131,7 +131,7 @@ module EasyPost
 
     if (400..599).include? response.code.to_i
       error = JSON.parse(response.body)["error"]
-      raise Error.new(error["message"], response.code.to_i, error["code"], error["errors"], response.body)
+      raise EasyPost::Error.new(error["message"], response.code.to_i, error["code"], error["errors"], response.body)
     end
 
     if response["Content-Type"].include? "application/json"

data/lib/easypost/address.rb

@@ -1,60 +1,58 @@
-module EasyPost
-  class Address < Resource
-    attr_accessor :message # Backwards compatibility
-
-    def self.create(params={}, api_key=nil)
-      url = self.url
-
-      address = params.reject { |k,_| k == :verify || k == :verify_strict }
-      
-      if params[:verify] || params[:verify_strict]
-        verify = params[:verify] || []
-        verify_strict = params[:verify_strict] || []
-
-        url += "?"
-        verify.each do |verification|
-          url += "verify[]=#{verification}&"
-        end
-        verify_strict.each do |verification|
-          url += "verify_strict[]=#{verification}&"
-        end
-      end
+class EasyPost::Address < EasyPost::Resource
+  attr_accessor :message # Backwards compatibility
 
-      response = EasyPost.make_request(:post, url, api_key, {address: address})
-      return Util.convert_to_easypost_object(response, api_key)
-    end
+  def self.create(params={}, api_key=nil)
+    url = self.url
+
+    address = params.reject { |k,_| k == :verify || k == :verify_strict }
+
+    if params[:verify] || params[:verify_strict]
+      verify = params[:verify] || []
+      verify_strict = params[:verify_strict] || []
 
-    def self.create_and_verify(params={}, carrier=nil, api_key=nil)
-      wrapped_params = {}
-      wrapped_params[self.class_name().to_sym] = params
-      wrapped_params[:carrier] = carrier
-      response = EasyPost.make_request(:post, url + '/create_and_verify', api_key, wrapped_params)
-
-      if response.has_key?(:address)
-        if response.has_key?(:message)
-          response[:address][:message] = response[:message]
-        end
-        verified_address = EasyPost::Util::convert_to_easypost_object(response[:address], api_key)
-        return verified_address
-      else
-        raise Error.new("Unable to verify address.")
+      url += "?"
+      verify.each do |verification|
+        url += "verify[]=#{verification}&"
+      end
+      verify_strict.each do |verification|
+        url += "verify_strict[]=#{verification}&"
       end
     end
 
-    def verify(params={}, carrier=nil)
-      begin
-        response = EasyPost.make_request(:get, url + '/verify?carrier=' + String(carrier), @api_key, params)
-      rescue
-        raise Error.new("Unable to verify address.")
-      end
+    response = EasyPost.make_request(:post, url, api_key, {address: address})
+    return EasyPost::Util.convert_to_easypost_object(response, api_key)
+  end
 
-      if response.has_key?("address")
-        return EasyPost::Util::convert_to_easypost_object(response["address"], api_key)
-      else
-        raise Error.new("Unable to verify address.")
+  def self.create_and_verify(params={}, carrier=nil, api_key=nil)
+    wrapped_params = {}
+    wrapped_params[self.class_name().to_sym] = params
+    wrapped_params[:carrier] = carrier
+    response = EasyPost.make_request(:post, url + '/create_and_verify', api_key, wrapped_params)
+
+    if response.has_key?("address")
+      if response.has_key?("message")
+        response["address"]["message"] = response["message"]
       end
+      verified_address = EasyPost::Util::convert_to_easypost_object(response["address"], api_key)
+      return verified_address
+    else
+      raise EasyPost::Error.new("Unable to verify address.")
+    end
+  end
 
-      return self
+  def verify(params={}, carrier=nil)
+    begin
+      response = EasyPost.make_request(:get, url + '/verify?carrier=' + String(carrier), @api_key, params)
+    rescue
+      raise EasyPost::Error.new("Unable to verify address.")
     end
+
+    if response.has_key?("address")
+      return EasyPost::Util::convert_to_easypost_object(response["address"], api_key)
+    else
+      raise EasyPost::Error.new("Unable to verify address.")
+    end
+
+    return self
   end
 end

data/lib/easypost/batch.rb

@@ -1,53 +1,49 @@
-module EasyPost
-  class Batch < Resource
+class EasyPost::Batch < EasyPost::Resource
+  def self.create_and_buy(params={}, api_key=nil)
+    wrapped_params = {}
+    wrapped_params[self.class_name().to_sym] = params
+    response = EasyPost.make_request(:post, url + '/create_and_buy', api_key, wrapped_params)
 
-    def self.create_and_buy(params={}, api_key=nil)
-      wrapped_params = {}
-      wrapped_params[self.class_name().to_sym] = params
-      response = EasyPost.make_request(:post, url + '/create_and_buy', api_key, wrapped_params)
-
-      return Util.convert_to_easypost_object(response, api_key)
-    end
-
-    def buy(params={})
-      response = EasyPost.make_request(:post, url + '/buy', @api_key, params)
-      self.refresh_from(response, @api_key, true)
+    return EasyPost::Util.convert_to_easypost_object(response, api_key)
+  end
 
-      return self
-    end
+  def buy(params={})
+    response = EasyPost.make_request(:post, url + '/buy', @api_key, params)
+    self.refresh_from(response, @api_key, true)
 
-    def label(params={})
-      response = EasyPost.make_request(:post, url + '/label', @api_key, params)
-      self.refresh_from(response, @api_key, true)
+    return self
+  end
 
-      return self
-    end
+  def label(params={})
+    response = EasyPost.make_request(:post, url + '/label', @api_key, params)
+    self.refresh_from(response, @api_key, true)
 
-    def remove_shipments(params={})
-      response = EasyPost.make_request(:post, url + '/remove_shipments', @api_key, params)
-      self.refresh_from(response, @api_key, true)
+    return self
+  end
 
-      return self
-    end
+  def remove_shipments(params={})
+    response = EasyPost.make_request(:post, url + '/remove_shipments', @api_key, params)
+    self.refresh_from(response, @api_key, true)
 
-    def add_shipments(params={})
-      response = EasyPost.make_request(:post, url + '/add_shipments', @api_key, params)
-      self.refresh_from(response, @api_key, true)
+    return self
+  end
 
-      return self
-    end
+  def add_shipments(params={})
+    response = EasyPost.make_request(:post, url + '/add_shipments', @api_key, params)
+    self.refresh_from(response, @api_key, true)
 
-    def stamp_and_barcode_by_reference(params={})
-      response = EasyPost.make_request(:get, url + '/stamp_and_barcode_by_reference', @api_key, params)
+    return self
+  end
 
-      return response
-    end
+  def stamp_and_barcode_by_reference(params={})
+    response = EasyPost.make_request(:get, url + '/stamp_and_barcode_by_reference', @api_key, params)
 
-    def create_scan_form(params={})
-      response = EasyPost.make_request(:post, url + '/scan_form', @api_key, params)
+    return response
+  end
 
-      return response
-    end
+  def create_scan_form(params={})
+    response = EasyPost.make_request(:post, url + '/scan_form', @api_key, params)
 
+    return response
   end
 end

data/lib/easypost/carrier_account.rb

@@ -1,8 +1,6 @@
-module EasyPost
-  class CarrierAccount < Resource
-    def self.types
-      response = EasyPost.make_request(:get, "/carrier_types", @api_key)
-      return Util.convert_to_easypost_object(response, api_key)
-    end
+class EasyPost::CarrierAccount < EasyPost::Resource
+  def self.types
+    response = EasyPost.make_request(:get, "/carrier_types", @api_key)
+    return EasyPost::Util.convert_to_easypost_object(response, api_key)
   end
 end

data/lib/easypost/customs_info.rb

@@ -1,9 +1,5 @@
-module EasyPost
-  class CustomsInfo < Resource
-
-    def self.all(filters={}, api_key=nil)
-      raise NotImplementedError.new('CustomsInfo.all not implemented.')
-    end
-
+class EasyPost::CustomsInfo < EasyPost::Resource
+  def self.all(filters={}, api_key=nil)
+    raise NotImplementedError.new('CustomsInfo.all not implemented.')
   end
 end

data/lib/easypost/customs_item.rb

@@ -1,9 +1,5 @@
-module EasyPost
-  class CustomsItem < Resource
-
-    def self.all(filters={}, api_key=nil)
-      raise NotImplementedError.new('CustomsItem.all not implemented.')
-    end
-
+class EasyPost::CustomsItem < EasyPost::Resource
+  def self.all(filters={}, api_key=nil)
+    raise NotImplementedError.new('CustomsItem.all not implemented.')
   end
 end

data/lib/easypost/error.rb

@@ -1,33 +1,31 @@
-module EasyPost
-  class Error < StandardError
-    attr_reader :message
-    attr_reader :status
-    attr_reader :http_status # deprecated
-    attr_reader :http_body
-    attr_reader :code
-    attr_reader :errors
+class EasyPost::Error < StandardError
+  attr_reader :message
+  attr_reader :status
+  attr_reader :http_status # deprecated
+  attr_reader :http_body
+  attr_reader :code
+  attr_reader :errors
 
-    def initialize(message = nil, status = nil, code = nil, errors = nil, http_body = nil)
-      @message = message
-      @status = status
-      @http_status = status # deprecated
-      @code = code
-      @errors = errors
-      @http_body = http_body
+  def initialize(message = nil, status = nil, code = nil, errors = nil, http_body = nil)
+    @message = message
+    @status = status
+    @http_status = status # deprecated
+    @code = code
+    @errors = errors
+    @http_body = http_body
 
-      super(message)
-    end
+    super(message)
+  end
 
-    def to_s
-      "#{code} (#{status}): #{message} #{errors}".strip
-    end
+  def to_s
+    "#{code} (#{status}): #{message} #{errors}".strip
+  end
 
-    def ==(other)
-      other.is_a?(Error) &&
-        message == other.message &&
-        status == other.status &&
-        code == other.code &&
-        errors == other.errors
-    end
+  def ==(other)
+    other.is_a?(EasyPost::Error) &&
+      message == other.message &&
+      status == other.status &&
+      code == other.code &&
+      errors == other.errors
   end
 end

data/lib/easypost/event.rb

@@ -1,10 +1,7 @@
 require 'json'
 
-module EasyPost
-  class Event < Resource
-      
-    def self.receive(values)
-        return EasyPost::Util::convert_to_easypost_object(JSON.parse(values), nil)
-    end
+class EasyPost::Event < EasyPost::Resource
+  def self.receive(values)
+      return EasyPost::Util::convert_to_easypost_object(JSON.parse(values), nil)
   end
 end

data/lib/easypost/insurance.rb

@@ -1,4 +1,2 @@
-module EasyPost
-  class Insurance < Resource
-  end
+class EasyPost::Insurance < EasyPost::Resource
 end

data/lib/easypost/item.rb

@@ -1,10 +1,6 @@
-module EasyPost
-  class Item < Resource
-
-    def self.retrieve_reference(params={}, api_key=nil)
-      response = EasyPost.make_request(:get, url + '/retrieve_reference', api_key, params)
-      return EasyPost::Util::convert_to_easypost_object(response, api_key) if response
-    end
-
+class EasyPost::Item < EasyPost::Resource
+  def self.retrieve_reference(params={}, api_key=nil)
+    response = EasyPost.make_request(:get, url + '/retrieve_reference', api_key, params)
+    return EasyPost::Util::convert_to_easypost_object(response, api_key) if response
   end
 end

data/lib/easypost/object.rb

@@ -1,146 +1,144 @@
 require "set"
 
-module EasyPost
-  class EasyPostObject
-    include Enumerable
-
-    attr_accessor :parent, :name, :api_key, :unsaved_values
-    @@immutable_values = Set.new([:api_key, :id])
-
-    def initialize(id=nil, api_key=nil, parent=nil, name=nil)
-      @api_key = api_key
-      @values = {}
-      @unsaved_values = Set.new
-      @transient_values = Set.new
-      @parent = parent
-      @name = name
-      self.id = id if id
-    end
-
-    def self.construct_from(values, api_key=nil, parent=nil, name=nil)
-      obj = self.new(values[:id], api_key, parent, name)
-      obj.refresh_from(values, api_key)
-      obj
-    end
+class EasyPost::EasyPostObject
+  include Enumerable
+
+  attr_accessor :parent, :name, :api_key, :unsaved_values
+  @@immutable_values = Set.new([:api_key, :id])
+
+  def initialize(id=nil, api_key=nil, parent=nil, name=nil)
+    @api_key = api_key
+    @values = {}
+    @unsaved_values = Set.new
+    @transient_values = Set.new
+    @parent = parent
+    @name = name
+    self.id = id if id
+  end
 
-    def to_s(*args)
-      JSON.dump(@values)
-    end
+  def self.construct_from(values, api_key=nil, parent=nil, name=nil)
+    obj = self.new(values[:id], api_key, parent, name)
+    obj.refresh_from(values, api_key)
+    obj
+  end
 
-    def inspect
-      id_string = (self.respond_to?(:id) && !self.id.nil?) ? " id=#{self.id}" : ""
-      "#<#{self.class}:#{id_string}> JSON: " + to_json
-    end
+  def to_s(*args)
+    JSON.dump(@values)
+  end
 
-    def refresh_from(values, api_key, partial=false)
-      @api_key = api_key
+  def inspect
+    id_string = (self.respond_to?(:id) && !self.id.nil?) ? " id=#{self.id}" : ""
+    "#<#{self.class}:#{id_string}> JSON: " + to_json
+  end
 
-      added = Set.new(values.keys - @values.keys)
+  def refresh_from(values, api_key, partial=false)
+    @api_key = api_key
 
-      instance_eval do
-        add_accessors(added)
-      end
+    added = Set.new(values.keys - @values.keys)
 
-      values.each do |k, v|
-        @values[k.to_s] = Util.convert_to_easypost_object(v, api_key, self, k)
-        @transient_values.delete(k)
-        @unsaved_values.delete(k)
-      end
+    instance_eval do
+      add_accessors(added)
     end
 
-    def [](k)
-      @values[k.to_s]
+    values.each do |k, v|
+      @values[k.to_s] = EasyPost::Util.convert_to_easypost_object(v, api_key, self, k)
+      @transient_values.delete(k)
+      @unsaved_values.delete(k)
     end
+  end
 
-    def []=(k, v)
-      send(:"#{k}=", v)
-    end
+  def [](k)
+    @values[k.to_s]
+  end
 
-    def keys
-      @values.keys
-    end
+  def []=(k, v)
+    send(:"#{k}=", v)
+  end
 
-    def values
-      @values.values
-    end
+  def keys
+    @values.keys
+  end
 
-    def to_json(options = {})
-      JSON.dump(@values)
-    end
+  def values
+    @values.values
+  end
 
-    def as_json(options = {})
-      @values.as_json
-    end
+  def to_json(options = {})
+    JSON.dump(@values)
+  end
 
-    def to_hash
-      @values
-    end
+  def as_json(options = {})
+    @values.as_json
+  end
 
-    def each(&blk)
-      @values.each(&blk)
-    end
+  def to_hash
+    @values
+  end
 
-    def id=(id)
-      @values[:id] = id
-    end
+  def each(&blk)
+    @values.each(&blk)
+  end
 
-    def id
-      @values[:id]
-    end
+  def id=(id)
+    @values[:id] = id
+  end
+
+  def id
+    @values[:id]
+  end
 
-    protected
+  protected
 
-    def flatten_unsaved
-      values = {}
-      for key in @unsaved_values
-        value = @values[key]
+  def flatten_unsaved
+    values = {}
+    for key in @unsaved_values
+      value = @values[key]
 
-        values[key] = value
+      values[key] = value
 
-        if value.is_a?(EasyPost::EasyPostObject)
-          values[key] = flatten_unsaved(value)
-        end
+      if value.is_a?(EasyPost::EasyPostObject)
+        values[key] = flatten_unsaved(value)
       end
-
-      return values
     end
 
-    def metaclass
-      class << self; self; end
-    end
+    return values
+  end
 
-    def remove_accessors(keys)
-      metaclass.instance_eval do
-        keys.each do |k|
-          next if @@immutable_values.include?(k)
-          k_eq = :"#{k}="
-          remove_method(k) if method_defined?(k)
-          remove_method(k_eq) if method_defined?(k_eq)
-        end
+  def metaclass
+    class << self; self; end
+  end
+
+  def remove_accessors(keys)
+    metaclass.instance_eval do
+      keys.each do |k|
+        next if @@immutable_values.include?(k)
+        k_eq = :"#{k}="
+        remove_method(k) if method_defined?(k)
+        remove_method(k_eq) if method_defined?(k_eq)
       end
     end
+  end
 
-    def add_accessors(keys)
-      metaclass.instance_eval do
-        keys.each do |k|
-          next if @@immutable_values.include?(k)
-          k_eq = :"#{k}="
-          define_method(k) { @values[k] }
-          define_method(k_eq) do |v|
-            @values[k] = v
-            @unsaved_values.add(k)
-
-            cur = self
-            cur_parent = self.parent
-            param = {}
-            while cur_parent
-              if cur.name
-                cur_parent.unsaved_values.add(cur.name)
-              end
-
-              cur = cur_parent
-              cur_parent = cur.parent
+  def add_accessors(keys)
+    metaclass.instance_eval do
+      keys.each do |k|
+        next if @@immutable_values.include?(k)
+        k = k.to_s
+        k_eq = :"#{k}="
+        define_method(k) { @values[k] }
+        define_method(k_eq) do |v|
+          @values[k] = v
+          @unsaved_values.add(k)
+
+          cur = self
+          cur_parent = self.parent
+          while cur_parent
+            if cur.name
+              cur_parent.unsaved_values.add(cur.name)
             end
+
+            cur = cur_parent
+            cur_parent = cur.parent
           end
         end
       end