alula-ruby 1.2.4 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c33d8891eaf1c48cac11a52b1afb25ac8e0b0085337035ed62cce0f149a39ca
-  data.tar.gz: a744baa78d697b48c49a1adb8a1037ecf42fc57345eb93193ee62448349aeb4b
+  metadata.gz: 8e7c4adad8dc9335e13282c8c0ea43d962312e5e4e34113d52877099a072b56c
+  data.tar.gz: 800ed5ba6ddcb94d33742c3b0ecf46ac55d846b9d24d708ea080f55bc6013952
 SHA512:
-  metadata.gz: 24325ab8af9e6e69168d7eac1f09ffd8c6b9fdabac9058d076fbf742d7049764f11c555568f3618f81950af779e3dd55a5cba6263f203a3a7852928b3f6b8a12
-  data.tar.gz: 90e797fdfe125ea6f593c5bcf425dfe17b2eb2100adb602a0277032bf9e83c93baa12052ba67a07ed3b024806f8bc34fe3cdd1aa74fe646e03557c7cc4894b0b
+  metadata.gz: 3c5f72d61ad7f378b1eb412ec3d824d08e33a73f90dcc1a9ec91dec51e552b935af89f7fea74a553243938c3991bc5c3c1504d5a37e2740155d85a9f07f656da
+  data.tar.gz: 54d92e69be82bf5d9a2a2321f02631e2a73e0d26fc887e4541fe30872e0e507c14a2dabbd9224a8eda5fdeffc7cbcbe0bce2412b726c92d4c4c965cc45286b32

data/README.md

@@ -380,7 +380,7 @@ Success responses are custom per RPC method. Some provide response data, some do
 
 After checking out the repo, run `bin/setup` to install dependencies. You can also run `bundle exec bin/console` for an interactive prompt that will allow you to experiment.
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, and push git commits and tags.
+To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, commit and push your changes. GitHub Actions takes care of the gem release.
 
 ### Using Docker
 

data/VERSION.md

@@ -2,6 +2,7 @@
 
 | Version | Date      | Description                                                                 |
 | ------- | --------- | --------------------------------------------------------------------------- |
+| v1.3.0  | 2023-11-20  | Update NotImplemented error's message to include a JSONized hash, add more known errors that use a different format         |
 | v1.2.4  | 2023-09-15  | Update UI Settings to object         |
 | v1.2.3  | 2023-08-31  | Update typo device patchable field `uiSettings`         |
 | v1.2.2  | 2023-08-30  | Add new device patchable field `uiSettings`         |

data/alula.gemspec

@@ -31,7 +31,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "dotenv"
   spec.add_development_dependency "guard-rspec"
-  spec.add_development_dependency "nokogiri", "~> 1.10.3"
+  spec.add_development_dependency "nokogiri", "~> 1.11.7"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency "simplecov", "~> 0.21.2"

data/lib/alula/errors.rb

@@ -71,9 +71,20 @@ module Alula
         # RPC errors are identified by jsonrpc in the body.
         return ProcedureError.new(response) if response.data['jsonrpc']
 
-
-        Alula.logger.error response
-        raise NotImplementedError.new("Unable to derive error for #{response.data['error']}")
+        # Error messages coming from the API are inconsistent.
+        # Sometimes they are in the msg key, sometimes in the message key
+        # Example: https://github.com/alula-net/alula-connect/pull/3417#issue-1985813211
+        # Here we have some errors that are actually known but at least one of them is not properly formatted by the API
+        message = response.data.dig('error', 'data', 'msg') || response.data.dig('error', 'data', 'message')
+        case message
+        when 'dealer does not exist'
+          NotFoundError.new(message)
+        when 'Insufficient User Scope'
+          InsufficientScopeError.new(message)
+        else
+          Alula.logger.error response
+          raise NotImplementedError.new('Unable to derive error for', response.data['error'].to_json)
+        end
       end
     end
 
@@ -100,8 +111,17 @@ module Alula
         # RPC errors are identified by jsonrpc in the body.
         return ProcedureError.new(response) if response.data['jsonrpc']
 
-        Alula.logger.error response
-        raise NotImplementedError.new("Unable to derive error for #{response.data['errors']}")
+        # See comment in the error_for_response method
+        message = error.dig('data', 'msg') || error.dig('data', 'message')
+        case message
+        when 'dealer does not exist'
+          NotFoundError.new(message)
+        when 'Insufficient User Scope'
+          InsufficientScopeError.new(message)
+        else
+          Alula.logger.error response
+          raise NotImplementedError.new('Unable to derive error for', response.data['errors'].to_json)
+        end
       end
     end
   end

data/lib/alula/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Alula
-  VERSION = '1.2.4'
+  VERSION = '1.3.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: alula-ruby
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.3.0
 platform: ruby
 authors:
 - Titus Johnson
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-09-15 00:00:00.000000000 Z
+date: 2023-11-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -100,14 +100,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: 1.11.7
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: 1.11.7
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement