honeymaker 0.12.2 → 0.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6862c70498fbf141e4808db14c19d5bd63a5dc59d7877797048756252b1d2836
-  data.tar.gz: e593c15692fcf4fb4677349b269076314cfac14fbe09b65b1c05b7aada91894d
+  metadata.gz: 2306f41b9132fa6da84591b6ec38ca160a3fd052ef74a0893d3c27f01f64e8f3
+  data.tar.gz: 40bb6abacd74cfbd7fded2bb0d560700be9be0bf29c4fbf2aa7ab725d816a294
 SHA512:
-  metadata.gz: f1547ddeaa92b66ba371d21f5452d88dc5aec87895649d28dec3ea9d54b36cb76582084b923381aadcbc004a95ba17291951b717ad4a7601a4401966f72af760
-  data.tar.gz: 96ad193d8d2ca903ac26e261c7a493ce77f25f2e644e4e5e152d8d4a3f717b5ae5cb6bfa6d1f3f33d49656b3a5decd5ee98402ce85ea5d3c72f28819858a93c2
+  metadata.gz: 326a5af73ef2f8564dda06d0d6f82a3f873d5b05f725d579666ccbb1df42a74603221904190972ac8b7b7fbe74d673898f1bc5e08da799655fd11d3959dfc1a6
+  data.tar.gz: 1c24efe089db21b89d7c24bd563da3841f5cfd3899c339d5524a3c55970245632bcdf8c36b2f060bbae6bc50e8a2f7e918c87684ba9e7e2e91002cdabf14c31e

data/lib/honeymaker/clients/kraken.rb

@@ -7,6 +7,7 @@ module Honeymaker
   module Clients
     class Kraken < Client
       URL = "https://api.kraken.com"
+      UNREADABLE = "Kraken: unreadable response"
 
       RATE_LIMITS = { default: 1000, orders: 1000 }.freeze
 
@@ -23,6 +24,7 @@ module Honeymaker
           txid: txid, consolidate_taker: consolidate_taker
         })
         return result if result.failure?
+        return unreadable unless result.data.is_a?(Hash)
 
         errors = result.data["error"]
         return Result::Failure.new(*errors) if errors.is_a?(Array) && errors.any?
@@ -53,6 +55,7 @@ module Honeymaker
           "deadline" => deadline, "validate" => validate
         })
         return result if result.failure?
+        return unreadable unless result.data.is_a?(Hash)
 
         errors = result.data["error"]
         return Result::Failure.new(*errors) if errors.is_a?(Array) && errors.any?
@@ -96,6 +99,7 @@ module Honeymaker
       def get_balances
         result = get_extended_balance
         return result if result.failure?
+        return unreadable unless result.data.is_a?(Hash)
 
         errors = result.data["error"]
         return Result::Failure.new(*errors) if errors.is_a?(Array) && errors.any?
@@ -149,6 +153,7 @@ module Honeymaker
         loop do
           result = get_trades_history(start: start, ofs: offset)
           return result if result.failure?
+          return unreadable unless result.data.is_a?(Hash)
 
           errors = result.data["error"]
           return Result::Failure.new(*errors) if errors.is_a?(Array) && errors.any?
@@ -196,6 +201,12 @@ module Honeymaker
 
       private
 
+      # A 2xx body that isn't a JSON object (a maintenance page, "null"): the venue's answer is
+      # unknown, which is not the same as empty. Callers treat data[:unreadable] as ambiguous.
+      def unreadable
+        Result::Failure.new(UNREADABLE, data: { unreadable: true })
+      end
+
       def aggregate_trades(trades)
         first = trades.first
         vol  = trades.sum { |t| BigDecimal(t["vol"].to_s) }

data/lib/honeymaker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Honeymaker
-  VERSION = "0.12.2"
+  VERSION = "0.12.3"
 end

data/test/honeymaker/clients/kraken_client_test.rb

@@ -15,6 +15,18 @@ class Honeymaker::Clients::KrakenTest < Minitest::Test
     assert_equal "https://api.kraken.com", Honeymaker::Clients::Kraken::URL
   end
 
+  def test_non_object_2xx_is_unreadable_not_empty
+    [["<html>maintenance</html>", ->(c) { c.get_balances }],
+     [nil, ->(c) { c.query_orders_info(txid: "O1") }],
+     ["oops", ->(c) { c.add_order(ordertype: "market", type: "buy", volume: "1", pair: "XBTUSD") }],
+     ["oops", ->(c) { c.closed_orders_from_trades(order_ids: ["O1"]) }]].each do |body, call|
+      stub_connection(:post, body)
+      r = call.call(@client)
+      assert_equal ["Kraken: unreadable response"], r.errors
+      assert_equal({ unreadable: true }, r.data)
+    end
+  end
+
   def test_get_tradable_asset_pairs
     stub_connection(:get, { "error" => [], "result" => { "XBTUSDT" => { "altname" => "XBTUSDT" } } })
     result = @client.get_tradable_asset_pairs

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: honeymaker
 version: !ruby/object:Gem::Version
-  version: 0.12.2
+  version: 0.12.3
 platform: ruby
 authors:
 - Deltabadger