dear_inventory 1.4.0 → 1.5.0

data/lib/dear_inventory/resources/sale.rb

@@ -1,19 +1,12 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   class Sale < Resource
     class << self
-      extend T::Sig
-
       # Sales
       #
       # @param params [Hash] URL query string parameters that conform to
       #   DearInventory::Parameters::Sale::Index
-      sig do
-        params(params: T::Hash[Symbol, T.untyped]).
-          returns(DearInventory::Response)
-      end
       def index(params = {})
         new.request(
           :get,
@@ -29,10 +22,6 @@ module DearInventory
       #
       # @param params [Hash] URL query string parameters that conform to
       #   DearInventory::Parameters::Sale::Show
-      sig do
-        params(params: T::Hash[Symbol, T.untyped]).
-          returns(DearInventory::Response)
-      end
       def show(params = {})
         new.request(
           :get,
@@ -46,10 +35,6 @@ module DearInventory
       #
       # @param params [Hash] URL query string parameters that conform to
       #   DearInventory::Parameters::Sale::Delete
-      sig do
-        params(params: T::Hash[Symbol, T.untyped]).
-          returns(DearInventory::Response)
-      end
       def delete(params = {})
         new.request(
           :delete,
@@ -62,14 +47,10 @@ module DearInventory
 
     private
 
-    sig { params(endpoint: T.nilable(String)).returns(String) }
     def resource_uri(endpoint)
-      case endpoint
-      when "index"
-        "#{self.class.const_get(:URI_BASE)}/salelist"
-      when "show", "delete"
-        "#{self.class.const_get(:URI_BASE)}/sale"
-      end
+      return super unless endpoint == "index"
+
+      "#{self.class.const_get(:URI_BASE)}/salelist"
     end
   end
 end

data/lib/dear_inventory/response.rb

@@ -1,46 +1,30 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   class Response
-    extend T::Sig
     extend DearInventory::IsASubclass
 
-    sig { returns(DearInventory::Models::Request) }
-    attr_reader :request
-    sig { returns(HTTP::Response) }
-    attr_reader :response
+    attr_reader :request, :response
 
-    # rubocop:disable Metrics/AbcSize
     # rubocop:disable Metrics/MethodLength
-    sig do
-      params(
-        request: DearInventory::Models::Request,
-        response: HTTP::Response,
-        num_previous_records: Integer
-      ).void
-    end
     def initialize(request:, response:, num_previous_records: 0)
-      @request = T.let(request, DearInventory::Models::Request)
-      @response = T.let(response, HTTP::Response)
-      @num_previous_records = T.let(num_previous_records, Integer)
+      @request = request
+      @response = response
+      @num_previous_records = num_previous_records
 
-      @fields = T.let(nil, T.nilable(T::Array[Symbol]))
-      @http_status = T.let(nil, T.nilable(Integer))
-      @load_full_record = T.let(nil, T.nilable(T::Boolean))
-      @num_records_paged = T.let(nil, T.nilable(Integer))
-      @uri = T.let(nil, T.nilable(String))
+      @fields = nil
+      @http_status = nil
+      @load_full_record = nil
+      @num_records_paged = nil
+      @uri = nil
 
       raise_error unless success?
 
-      body_copy = T.cast(body, T.nilable(T::Hash[String, T.untyped]))
-      @model = T.let(@request.model.new(body_copy), DearInventory::Model)
+      @model = @request.model.new(body)
       assign_values
     end
-    # rubocop:enable Metrics/AbcSize
     # rubocop:enable Metrics/MethodLength
 
-    sig { returns(T::Array[Symbol]) }
     def fields
       @fields ||= begin
         values = @request.model.const_get(:FIELDS).values.map do |field|
@@ -51,32 +35,27 @@ module DearInventory
       end
     end
 
-    sig { returns(T.nilable(String)) }
     def error
       if body.respond_to?(:fetch)
-        body_copy = T.cast(body, T::Hash[String, T.untyped])
+        body_copy = body
         return body_copy.fetch("Exception", nil)
       end
 
-      T.cast(body, T.nilable(String))
+      body
     end
 
-    sig { returns(T::Hash[Symbol, String]) }
     def headers
       @response.headers
     end
 
-    sig { returns(Integer) }
     def http_status
       @http_status ||= @response.status.code
     end
 
-    sig { returns(T::Boolean) }
     def paginated?
       @model.respond_to?(:page)
     end
 
-    sig { params(block: T.proc.params(arg0: DearInventory::Model).void).void }
     def each(&block)
       raise_not_paginated unless paginated?
 
@@ -89,51 +68,44 @@ module DearInventory
       end
     end
 
-    sig { returns(DearInventory::Response) }
     def next_page
       raise_not_paginated unless paginated?
       raise DearInventory::NoMorePagesError unless next_page?
 
       request = @request.dup
-      T.unsafe(request.params).page = T.unsafe(@model).page + 1
+      request.params.page = @model.page + 1
 
       DearInventory::Request.(request, num_previous_records: num_records_paged)
     end
 
-    sig { returns(T::Boolean) }
     def next_page?
       raise_not_paginated unless paginated?
 
-      num_records_paged < T.unsafe(self).total
+      num_records_paged < total
     end
 
-    sig { returns(Integer) }
     def num_records_paged
       @num_records_paged ||= begin
         raise_not_paginated unless paginated?
 
-        @num_previous_records + T.unsafe(self).records.count
+        @num_previous_records + records.count
       end
     end
 
-    sig { void }
     def raise_not_paginated
-      raise DearInventory::NotPaginatedError.new(uri: uri)
+      raise DearInventory::NotPaginatedError.new(uri)
     end
 
-    sig { returns(T::Boolean) }
     def success?
       http_status == 200
     end
 
-    sig { returns(String) }
     def uri
       @uri ||= @response.uri.to_s
     end
 
-    protected
+    private
 
-    sig { returns(T.any(T::Hash[String, T.untyped], String)) }
     def body
       string_body = @response.body.to_s
       parsed_body = JSON.parse(string_body)
@@ -147,59 +119,39 @@ module DearInventory
       string_body
     end
 
-    private
-
-    sig { void }
     def assign_values
       fields.each { |field| define_alias_method(field) }
     end
 
-    sig { params(method_name: Symbol).void }
     def define_alias_method(method_name)
       define_singleton_method(method_name) do
         @model.public_send(method_name)
       end
     end
 
-    API_LIMIT_ERROR = T.let(
-      "You have reached 60 calls per 60 seconds API limit.", String
-    )
-
-    # rubocop:disable Metrics/AbcSize
-    sig { returns(DearInventory::Error) }
+    API_LIMIT_ERROR = "You have reached 60 calls per 60 seconds API limit."
+    private_constant :API_LIMIT_ERROR
     def raise_error
-      if http_status == 400
-        raise T.unsafe(DearInventory::BadRequestError).new(error, self)
-      end
+      raise DearInventory::BadRequestError.new(error, self) if http_status == 400
 
-      if http_status == 503 && error == API_LIMIT_ERROR
-        raise T.unsafe(DearInventory::APILimitError)
-      end
+      raise DearInventory::APILimitError if http_status == 503 && error == API_LIMIT_ERROR
 
-      raise T.unsafe(DearInventory::Error).
+      raise DearInventory::Error.
         new("Unknown error (#{http_status}): #{error}")
     end
-    # rubocop:enable Metrics/AbcSize
 
-    sig do
-      params(
-        response: DearInventory::Response,
-        block: T.proc.params(arg0: DearInventory::Model).void
-      ).void
-    end
     def iterate_over_records(response, block)
-      return if T.unsafe(response).records.nil?
+      return if response.records.nil?
 
-      T.unsafe(response).records.each do |record|
+      response.records.each do |record|
         record = record.full_record if load_full_record?
         block.call(record)
       end
     end
 
-    sig { returns(T::Boolean) }
     def load_full_record?
       @load_full_record ||=
-        T.unsafe(self).records.first.respond_to?(:full_record)
+        records.first.respond_to?(:full_record)
     end
   end
 end

data/lib/dear_inventory/test_response.rb

@@ -1,42 +1,27 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   class TestResponse < Response
-    extend T::Sig
     extend DearInventory::IsASubclass
 
-    sig { returns(DearInventory::Models::Request) }
-    attr_reader :request
-    sig { returns(HTTP::Response) }
-    attr_reader :response
+    attr_reader :request, :response
 
-    # rubocop:disable Metrics/AbcSize
-    sig do
-      params(
-        request: DearInventory::Models::TestRequest,
-        body: T.nilable(T::Hash[String, T.untyped]),
-        num_previous_records: Integer
-      ).void
-    end
     def initialize(request:, body:, num_previous_records: 0)
-      @request = T.let(request, DearInventory::Models::Request)
-      @num_previous_records = T.let(num_previous_records, Integer)
+      @request = request
+      @num_previous_records = num_previous_records
 
-      @fields = T.let(nil, T.nilable(T::Array[Symbol]))
-      @http_status = T.let(nil, T.nilable(Integer))
-      @load_full_record = T.let(nil, T.nilable(T::Boolean))
-      @num_records_paged = T.let(nil, T.nilable(Integer))
-      @uri = T.let(nil, T.nilable(String))
+      @fields = nil
+      @http_status = nil
+      @load_full_record = nil
+      @num_records_paged = nil
+      @uri = nil
 
       raise_error unless success?
 
-      @model = T.let(@request.model.new(body), DearInventory::Model)
+      @model = @request.model.new(body)
       assign_values
     end
-    # rubocop:enable Metrics/AbcSize
 
-    sig { returns(Integer) }
     def http_status
       200
     end

data/lib/dear_inventory/validator.rb

@@ -1,59 +1,31 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   class Validator
-    extend T::Sig
-
-    extend T::Helpers
-    abstract!
-
-    DEFAULT_OPTIONS = T.let(
-      {
-        limit: nil,
-        max_length: nil,
-        values: nil,
-      }.freeze,
-      T::Hash[Symbol, T.nilable(T.any(Integer, T::Array[String]))]
-    )
+    DEFAULT_OPTIONS = {
+      limit: nil,
+      max_length: nil,
+      values: nil,
+    }.freeze
     private_constant :DEFAULT_OPTIONS
 
-    sig do
-      params(
-        field_name: Symbol,
-        value: T.nilable(T.any(Date, Numeric, String, T::Boolean, Time)),
-        options: T::Hash[Symbol, T.untyped]
-      ).void
-    end
     def self.call(field_name, value, options: DEFAULT_OPTIONS)
       new(field_name, value, options: options).call
     end
 
-    sig do
-      params(
-        field_name: Symbol,
-        value: T.nilable(T.any(Date, Numeric, String, T::Boolean, Time)),
-        options: T::Hash[Symbol, T.untyped]
-      ).void
-    end
     def initialize(field_name, value, options: DEFAULT_OPTIONS)
-      @field_name = T.let(field_name, Symbol)
-      @value =
-        T.let(value, T.nilable(T.any(Date, Numeric, String, T::Boolean, Time)))
+      @field_name = field_name
+      @value = value
 
-      @limit = T.let(options[:limit], T.nilable(Integer))
-      @max_length = T.let(options[:limit], T.nilable(Integer))
-      @values = T.let(options[:values], T.nilable(T::Array[String]))
+      @limit = options[:limit]
+      @max_length = options[:limit]
+      @values = options[:values]
     end
 
     private
 
-    sig { abstract.void }
     def call; end
 
-    protected
-
-    sig { params(message: String).void }
     def raise_error(message)
       raise ValidationError, "#{@field_name} is invalid, #{message}"
     end

data/lib/dear_inventory/validators/array.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module DearInventory
+  module Validators
+    class Array < DearInventory::Validator
+      def call
+        value = instance_variable_get(:@value)
+        return if value.nil?
+        return if value.is_a?(::Array)
+
+        raise_error("expected an Array but received #{value.inspect}")
+      end
+    end
+  end
+end

data/lib/dear_inventory/validators/boolean.rb

@@ -1,12 +1,8 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class Boolean < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if [true, false, nil].include?(value)

data/lib/dear_inventory/validators/enum.rb

@@ -1,20 +1,16 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class Enum < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if value.nil?
-        return if T.must(@values).include?(value)
+        return if @values.include?(value)
 
         raise_error(
           "expected one of " \
-          "#{T.must(@values).map(&:inspect).join(", ")} " \
+          "#{@values.map(&:inspect).join(", ")} " \
           "but received #{value.inspect}"
         )
       end

data/lib/dear_inventory/validators/guid.rb

@@ -1,20 +1,13 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class Guid < DearInventory::Validator
-      extend T::Sig
+      REGEX = [
+        /\A[\da-f]{32}\z/i,
+        /\A(urn:uuid:)?[\da-f]{8}-([\da-f]{4}-){3}[\da-f]{12}\z/i,
+      ].freeze
 
-      REGEX = T.let(
-        [
-          /\A[\da-f]{32}\z/i,
-          /\A(urn:uuid:)?[\da-f]{8}-([\da-f]{4}-){3}[\da-f]{12}\z/i,
-        ].freeze,
-        T::Array[Regexp]
-      )
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if value.nil?

data/lib/dear_inventory/validators/integer.rb

@@ -1,12 +1,8 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class Integer < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if value.nil?

data/lib/dear_inventory/validators/required.rb

@@ -1,12 +1,8 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class Required < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         return if present?(instance_variable_get(:@value))
 

data/lib/dear_inventory/validators/string.rb

@@ -1,12 +1,8 @@
-# typed: true
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class String < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if value.nil?

data/lib/dear_inventory/validators/time.rb

@@ -1,12 +1,8 @@
-# typed: strict
 # frozen_string_literal: true
 
 module DearInventory
   module Validators
     class DateTime < DearInventory::Validator
-      extend T::Sig
-
-      sig { override.void }
       def call
         value = instance_variable_get(:@value)
         return if value.nil?

data/lib/dear_inventory/version.rb

@@ -1,6 +1,5 @@
-# typed: strong
 # frozen_string_literal: true
 
 module DearInventory
-  VERSION = "1.4.0"
+  VERSION = "1.5.0"
 end

data/lib/dear_inventory.rb

@@ -1,8 +1,6 @@
-# typed: strong
 # frozen_string_literal: true
 
 require "http"
-require "sorbet-runtime"
 
 require "dear_inventory/config"
 require "dear_inventory/environment"
@@ -91,6 +89,7 @@ require "dear_inventory/models/request"
 require "dear_inventory/models/test_request"
 
 require "dear_inventory/validator"
+require "dear_inventory/validators/array"
 require "dear_inventory/validators/boolean"
 require "dear_inventory/validators/enum"
 require "dear_inventory/validators/guid"
@@ -101,7 +100,10 @@ require "dear_inventory/validators/time"
 
 require "dear_inventory/parameters"
 require "dear_inventory/parameters/advanced_purchase/show"
+require "dear_inventory/parameters/customer/create"
 require "dear_inventory/parameters/customer/index"
+require "dear_inventory/parameters/customer/show"
+require "dear_inventory/parameters/customer/update"
 require "dear_inventory/parameters/product/index"
 require "dear_inventory/parameters/product_availability/index"
 require "dear_inventory/parameters/purchase/index"
@@ -124,21 +126,13 @@ require "dear_inventory/test_response"
 require "dear_inventory/version"
 
 module DearInventory
-  extend T::Sig
-
-  @config = T.let(Config.new, Config)
+  @config = Config.new
 
   class << self
-    extend T::Sig
-
-    sig { returns(Config) }
     attr_reader :config
 
-    # rubocop:disable Lint/UnusedMethodArgument
-    sig { params(block: T.proc.params(config: Config).void).void }
-    def configure(&block)
+    def configure(&)
       yield config
     end
-    # rubocop:enable Lint/UnusedMethodArgument
   end
 end