fortnox-api 0.8.0 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 02c5fbd6c659d3c6cc4e9ed8dfef6e739ded62fc33107623f4dddc13d33f6672
-  data.tar.gz: 14f0815a2fd32e98055d71ef231da2b6fac4b95ecfd9dfcda59b88d1c1445c63
+  metadata.gz: 345228c827e624083ca2557504b5c426f3e4719a9252e35bb31d87e44fe9f7f7
+  data.tar.gz: 3b9fc903f0f5dc8b7e98ae5610af503dd09c6cc218af6c7339e197de23c39e08
 SHA512:
-  metadata.gz: 293665ff9e0b860e88c7c6c78fbcfdd0b8a8047444f23cd0560d504442e1a0ac3439c52349413cac63e0c5973d1db74dabcf7ebdd9c10bc3f1b7de922db58cbc
-  data.tar.gz: 011b4785ce46af9c4cd57939431f6b09dc6ab44d650023616fe7b0784dcde890727999eb918ac3c3db311afaaff8785d240c54e5c30f198a630b9a1bf41483b7
+  metadata.gz: 8f74bbe0e82253b3addac22a96fc2cde1892ff8b2d0a48bab032b3b7306bb6e215646b2e365622f9671eeecc90167531b7fa9dc92227b83e4dfd6aaefe04f202
+  data.tar.gz: 0f1f3b7a1702950973bdd23d4bda90fdfb03ac2532031067f15f84d4abf6d2f629a7bda54cc2a405e868629627785a722c4f32dd5d719f3f6f36d5593cbc39da

data/.codeclimate.yml

@@ -12,6 +12,7 @@ engines:
     enabled: false
   rubocop:
     enabled: true
+    channel: rubocop-1-39-0
 ratings:
   paths:
   - "**.rb"

data/.rubocop.yml

@@ -1,7 +1,7 @@
 require: rubocop-rspec
 
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 2.7
   DisplayCopNames: true
 
 RSpec/FilePath:

data/.tool-versions

@@ -0,0 +1 @@
+ruby 3.2.0

data/.travis.yml

@@ -3,9 +3,11 @@ env:
   - CC_TEST_REPORTER_ID=43ea0acb059b164537ce31bf67e03e1cb552d63df1ad3e9bd95dc3d844697bb4
 language: ruby
 rvm:
-- 2.5.8
-- 2.6.6
-- 2.7.1
+  - 2.7.7
+  - 3.0.5
+  - 3.1.3
+  - 3.2.0
+
 notifications:
   slack:
     secure: UEYDtwLaQgAoMRY7ct1y0Uu5HbInMwrLo+TiYRc3rjDrzqDfDyInEr3bpckyPJ79lnzccrFezMdY4jWzRjmsvQzvYMu+DWJ1mAWOa/9Ws4YZ8nHd5KqXxVQ0EqPWuWwMOyOdyy3DK/MjUOobGSc0//Pv5DeaAAEBKLJzN+e1BQM=

data/CHANGELOG.md

@@ -1,46 +1,87 @@
 # Changelog
+
 All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
-and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
+and this project adheres to
+[Semantic Versioning](http://semver.org/spec/v2.0.0.html).
+
+## [0.8.2]
+
+### Fixed
+- Restored undocumented support for string keys in model attributes,
+  which was inadvertently removed in `v0.8.1`.
+
+## [0.8.1]
+
+### Fixed
+
+- Missing runtime dependencies dry-configurable and dry-container is added.
+  Those were indirect dependencies before via other dry libs, but we use them
+  explicitly in the gem, so they should be included as real dependencies. Also,
+  newer versions of these gems did actually break the gem.
+
+### Removed
+
+- **Breaking** Drops support for Ruby 2.6 since it's reached end of life
+- Token rotation is removed since Fortnox counts rate limit on tenant and client
+  id. Before, you could get around this by using multiple access tokens. That's
+  not possible anymore, so token rotation is simply removed.
 
 ## [0.8.0]
+
 ### Changed
+
 - `Fortnox::API::CURRENT_HOUSEWORK_TYPES` is now renamed to `HOUSEWORK_TYPES`
-  and is instead a Hash with keys for the different categories of types.
-  It also includes legacy types, which means `Fortnox::API::LEGACY_HOUSEWORK_TYPES` is removed.
+  and is instead a Hash with keys for the different categories of types. It also
+  includes legacy types, which means `Fortnox::API::LEGACY_HOUSEWORK_TYPES` is
+  removed.
 
 ### Removed
+
 - Drops support for Ruby < `2.5.0` since they are deprecated
 
 ## [0.7.2]
+
 ### Fixed
+
 - Invalid validation for Customer's account number attribute
 
 ## [0.7.1]
+
 ### Fixed
+
 - Invalid validation for Customer's country attributes
 
 ## [0.7.0]
+
 ### Added
+
 - Adds build test for Ruby `2.6.0`, `2.6.3` and `2.7.0-preview`.
 
 ### Removed
+
 - Drops support for Ruby < `2.4.0` since they are deprecated
 
 ### Fixed
+
 - Country attribute for `Invoice` and `Order` is now validated correctly.
 - Fixes deprecation warnings
 - Unlocks pinned `HTTParty` version
 - Invalid email validation
 
 ## [0.6.3]
+
 ### Changed
+
 - Pins `dry-types` to `< 0.13.0` due to breaking changes
 
 ### Fixed
+
 - Model attribute `url` is no longer null
 
+[0.8.2]: https://github.com/accodeing/fortnox-api/compare/v0.8.1...v0.8.2
+[0.8.1]: https://github.com/accodeing/fortnox-api/compare/v0.8.0...v0.8.1
 [0.8.0]: https://github.com/accodeing/fortnox-api/compare/v0.7.2...v0.8.0
 [0.7.2]: https://github.com/accodeing/fortnox-api/compare/v0.7.1...v0.7.2
 [0.7.1]: https://github.com/accodeing/fortnox-api/compare/v0.7.0...v0.7.1

data/fortnox-api.gemspec

@@ -3,12 +3,12 @@
 
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'fortnox/api/version'
+require_relative 'lib/fortnox/api/version'
 
-Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
+Gem::Specification.new do |spec|
   spec.name          = 'fortnox-api'
   spec.version       = Fortnox::API::VERSION
-  spec.authors       = ['Jonas Schubert Erlandsson', 'Hannes Elvemyr', 'Felix Holmgren']
+  spec.authors       = ['Jonas Schubert Erlandsson', 'Hannes Elvemyr', 'Felix Holmgren', 'Mike Eirih']
   spec.email         = ['info@accodeing.com']
   spec.summary       = 'Gem to use Fortnox REST API in Ruby.'
   spec.description   = 'This gem uses the HTTParty library to abstract away the REST calls. It gives you access to a '\
@@ -23,22 +23,27 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
   spec.test_files    = spec.files.grep(%r{^(spec)/})
   spec.require_paths = ['lib']
 
-  spec.required_ruby_version = '>= 2.5'
-  spec.add_dependency 'countries', '~> 3.0'
-  spec.add_dependency 'dry-struct', '~> 0.1'
-  spec.add_dependency 'dry-types', '~> 0.8', '< 0.13.0'
+  spec.required_ruby_version = '>= 2.7'
+  spec.add_dependency 'countries', '~> 5.0'
+  spec.add_dependency 'dry-configurable', '~> 1.0'
+  spec.add_dependency 'dry-container', '~> 0.10'
+  spec.add_dependency 'dry-struct', '~> 1.6'
+  spec.add_dependency 'dry-types', '~> 1.7'
   spec.add_dependency 'httparty', '~> 0.17'
+  spec.add_dependency 'jwt', '~> 2.3'
 
-  spec.add_development_dependency 'bundler', '~> 2.0'
-  spec.add_development_dependency 'guard', '~> 2.12'
-  spec.add_development_dependency 'guard-rspec', '~> 4.5'
+  spec.add_development_dependency 'bundler', '~> 2.4'
+  spec.add_development_dependency 'dotenv', '~> 2.8'
+  spec.add_development_dependency 'guard', '~> 2.18'
+  spec.add_development_dependency 'guard-rspec', '~> 4.7'
   spec.add_development_dependency 'pry', '~> 0'
-  spec.add_development_dependency 'rake', '>= 12.3.3'
-  spec.add_development_dependency 'rspec', '~> 3.2'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rspec', '~> 3.12'
   spec.add_development_dependency 'rspec-collection_matchers', '~> 0'
-  spec.add_development_dependency 'rubocop', '~> 0.52.0'
-  spec.add_development_dependency 'rubocop-rspec', '~> 1.22.2'
-  spec.add_development_dependency 'simplecov', '~> 0.15'
-  spec.add_development_dependency 'vcr', '~> 4.0'
-  spec.add_development_dependency 'webmock', '~> 3.5'
+  spec.add_development_dependency 'rubocop', '~> 1.39.0'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.16.0'
+  spec.add_development_dependency 'simplecov', '~> 0.22'
+  spec.add_development_dependency 'timecop', '~> 0.9.6'
+  spec.add_development_dependency 'vcr', '~> 6.1'
+  spec.add_development_dependency 'webmock', '~> 3.18'
 end

data/lib/fortnox/api/mappers/article.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/base/from_json.rb

@@ -50,7 +50,7 @@ module Fortnox
 
             raise MissingModelOrMapperException, message if ENV['RUBY_ENV']
             Fortnox::API.logger.warn("Missing Model or Mapper implementation for #{key} with attributes: #{collection}")
-            return convert_hash_keys_from_json_format(collection, {})
+            convert_hash_keys_from_json_format(collection, {})
           end
         end
 

data/lib/fortnox/api/mappers/base/to_json.rb

@@ -25,8 +25,8 @@ module Fortnox
           # PRIVATE
 
           def convert_hash_keys_to_json_format(hash)
-            hash.each_with_object({}) do |(key, value), json_hash|
-              json_hash[convert_key_to_json(key)] = value
+            hash.transform_keys do |key|
+              convert_key_to_json(key)
             end
           end
 

data/lib/fortnox/api/mappers/base.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base/canonical_name_sym'
-require 'fortnox/api/mappers/base/from_json'
-require 'fortnox/api/mappers/base/to_json'
+require_relative 'base/canonical_name_sym'
+require_relative 'base/from_json'
+require_relative 'base/to_json'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/customer.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/default_delivery_types.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/default_templates.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/edi_information.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/email_information.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/invoice.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
-require 'fortnox/api/mappers/edi_information'
-require 'fortnox/api/mappers/email_information'
-require 'fortnox/api/mappers/invoice_row'
+require_relative 'base'
+require_relative 'edi_information'
+require_relative 'email_information'
+require_relative 'invoice_row'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/invoice_row.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/order.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
-require 'fortnox/api/mappers/edi_information'
-require 'fortnox/api/mappers/email_information'
-require 'fortnox/api/mappers/order_row'
+require_relative 'base'
+require_relative 'edi_information'
+require_relative 'email_information'
+require_relative 'order_row'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/order_row.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/project.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/terms_of_payment.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/unit.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base'
+require_relative 'base'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers/value/country_string.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/mappers/base/canonical_name_sym'
+require_relative '../base/canonical_name_sym'
 
 module Fortnox
   module API

data/lib/fortnox/api/mappers.rb

@@ -1,21 +1,21 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
+require_relative 'types'
 
-require 'fortnox/api/mappers/value/array'
-require 'fortnox/api/mappers/value/country_string'
-require 'fortnox/api/mappers/value/date'
-require 'fortnox/api/mappers/value/hash'
-require 'fortnox/api/mappers/value/identity'
-require 'fortnox/api/mappers/customer'
-require 'fortnox/api/mappers/default_delivery_types'
-require 'fortnox/api/mappers/default_templates'
-require 'fortnox/api/mappers/edi_information'
-require 'fortnox/api/mappers/email_information'
-require 'fortnox/api/mappers/invoice_row'
-require 'fortnox/api/mappers/invoice'
-require 'fortnox/api/mappers/order_row'
-require 'fortnox/api/mappers/order'
-require 'fortnox/api/mappers/project'
-require 'fortnox/api/mappers/unit'
-require 'fortnox/api/mappers/terms_of_payment'
+require_relative 'mappers/value/array'
+require_relative 'mappers/value/country_string'
+require_relative 'mappers/value/date'
+require_relative 'mappers/value/hash'
+require_relative 'mappers/value/identity'
+require_relative 'mappers/customer'
+require_relative 'mappers/default_delivery_types'
+require_relative 'mappers/default_templates'
+require_relative 'mappers/edi_information'
+require_relative 'mappers/email_information'
+require_relative 'mappers/invoice_row'
+require_relative 'mappers/invoice'
+require_relative 'mappers/order_row'
+require_relative 'mappers/order'
+require_relative 'mappers/project'
+require_relative 'mappers/unit'
+require_relative 'mappers/terms_of_payment'

data/lib/fortnox/api/models/article.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/base'
+require_relative 'base'
+require_relative '../types'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/base.rb

@@ -1,8 +1,9 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
 require 'ice_nine'
 
+require_relative '../types'
+
 module Fortnox
   module API
     module Model
@@ -36,6 +37,10 @@ module Fortnox
           new(self::STUB.dup)
         end
 
+        def unique_id
+          send(self.class::UNIQUE_ID)
+        end
+
         # This filtering logic could be improved since it is currently O(N*M).
         def attributes(*options)
           return self.class.schema if options.nil?
@@ -47,8 +52,13 @@ module Fortnox
           end
         end
 
-        def unique_id
-          send(self.class::UNIQUE_ID)
+        def to_hash(recursive = false)
+          return super() if recursive
+
+          self.class.schema.each_with_object({}) do |key, result|
+              # Only output attributes that have a value set
+              result[key.name] = self[key.name] if self.send("#{key.name}?")
+          end
         end
 
         def update(hash)
@@ -85,19 +95,9 @@ module Fortnox
           @parent || self.class.new(self.class::STUB.dup)
         end
 
-        def to_hash(recursive = false)
-          return super() if recursive
-
-          self.class.schema.keys.each_with_object({}) do |key, result|
-            result[key] = self[key]
-          end
-        end
-
-        private_class_method
-
         # dry-types filter anything that isn't specified as an attribute on the
         # class that is being instantiated. This wrapper preserves the meta
-        # properties we need to track object state during that initilisation and
+        # properties we need to track object state during that initialisation and
         # sets them on the object after dry-types is done with it.
         def self.preserve_meta_properties(hash)
           is_unsaved = hash.delete(:unsaved) { true }

data/lib/fortnox/api/models/customer.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/base'
+require_relative 'base'
+require_relative '../types'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/document.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/label'
+require_relative '../types'
+require_relative 'label'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/invoice.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/models/base'
-require 'fortnox/api/models/document'
+require_relative 'base'
+require_relative 'document'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/label.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
+require_relative '../types'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/order.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/models/base'
-require 'fortnox/api/models/document'
+require_relative 'base'
+require_relative 'document'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/project.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/base'
+require_relative 'base'
+require_relative '../types'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/terms_of_payment.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/base'
+require_relative '../types'
+require_relative '../models/base'
 
 module Fortnox
   module API

data/lib/fortnox/api/models/unit.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/types'
-require 'fortnox/api/models/base'
+require_relative 'base'
+require_relative '../types'
 
 module Fortnox
   module API

data/lib/fortnox/api/models.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/models/article'
-require 'fortnox/api/models/customer'
-require 'fortnox/api/models/invoice'
-require 'fortnox/api/models/order'
-require 'fortnox/api/models/project'
-require 'fortnox/api/models/unit'
-require 'fortnox/api/models/terms_of_payment'
+require_relative 'models/article'
+require_relative 'models/customer'
+require_relative 'models/invoice'
+require_relative 'models/order'
+require_relative 'models/project'
+require_relative 'models/unit'
+require_relative 'models/terms_of_payment'

data/lib/fortnox/api/repositories/article.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/repositories/base'
-require 'fortnox/api/models/article'
-require 'fortnox/api/mappers/article'
+require_relative 'base'
+require_relative '../models/article'
+require_relative '../mappers/article'
 
 module Fortnox
   module API

data/lib/fortnox/api/repositories/base.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
 require 'httparty'
-require 'fortnox/api/request_handling'
-require 'fortnox/api/repositories/base/loaders'
-require 'fortnox/api/repositories/base/savers'
+
+require_relative 'base/loaders'
+require_relative 'base/savers'
+require_relative '../request_handling'
 
 module Fortnox
   module API

data/lib/fortnox/api/repositories/customer.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/repositories/base'
-require 'fortnox/api/models/customer'
-require 'fortnox/api/mappers/customer'
+require_relative 'base'
+require_relative '../models/customer'
+require_relative '../mappers/customer'
 
 module Fortnox
   module API

data/lib/fortnox/api/repositories/invoice.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'fortnox/api/repositories/base'
-require 'fortnox/api/models/invoice'
-require 'fortnox/api/mappers/invoice'
+require_relative 'base'
+require_relative '../models/invoice'
+require_relative '../mappers/invoice'
 
 module Fortnox
   module API