Catch-All signature for object initializations. (#24)

* adding **kwargs catch-all param

* making unit test timezone independent

Author: KmiQ

coinbaseadvanced/models/accounts.py

@@ -19,7 +19,7 @@ class AvailableBalance:
     value: str
     currency: str
 
-    def __init__(self, value: str, currency: str) -> None:
+    def __init__(self, value: str, currency: str, **kwargs) -> None:
         self.value = value
         self.currency = currency
 
@@ -45,7 +45,7 @@ class Account:
     def __init__(
         self, uuid: UUID, name: str, currency: str, available_balance: dict, default: bool,
             active: bool, created_at: datetime, updated_at: datetime, deleted_at: datetime,
-            type: str, ready: bool, hold: dict) -> None:
+            type: str, ready: bool, hold: dict, **kwargs) -> None:
         self.uuid = uuid
         self.name = name
         self.currency = currency
@@ -89,6 +89,7 @@ def __init__(self,
                  has_next: bool,
                  cursor: str,
                  size: int,
+                 **kwargs
                  ) -> None:
 
         self.accounts = list(map(lambda x: Account(**x), accounts))\

coinbaseadvanced/models/fees.py

@@ -24,7 +24,7 @@ def __init__(self,
                  usd_from: int,
                  usd_to: str,
                  taker_fee_rate: str,
-                 maker_fee_rate: str) -> None:
+                 maker_fee_rate: str, **kwargs) -> None:
         self.pricing_tier = pricing_tier
         self.usd_from = usd_from
         self.usd_to = usd_to
@@ -40,7 +40,7 @@ class GoodsAndServicesTax:
     rate: str
     type: str
 
-    def __init__(self, rate: str, type: str) -> None:
+    def __init__(self, rate: str, type: str, **kwargs) -> None:
         self.rate = rate
         self.type = type
 
@@ -52,7 +52,7 @@ class MarginRate:
 
     value: str
 
-    def __init__(self, value: str) -> None:
+    def __init__(self, value: str, **kwargs) -> None:
         self.value = value
 
 
@@ -73,7 +73,7 @@ class TransactionsSummary:
 
     def __init__(self, total_volume: int, total_fees: int, fee_tier: dict, margin_rate: dict,
                  goods_and_services_tax: dict, advanced_trade_only_volume: int, advanced_trade_only_fees: int,
-                 coinbase_pro_volume: int, coinbase_pro_fees: int) -> None:
+                 coinbase_pro_volume: int, coinbase_pro_fees: int, **kwargs) -> None:
         self.total_volume = total_volume
         self.total_fees = total_fees
         self.fee_tier = FeeTier(**fee_tier) if fee_tier is not None else None

coinbaseadvanced/models/orders.py

@@ -63,7 +63,7 @@ class OrderError:
     new_order_failure_reason: str
 
     def __init__(self, error: str = '', message: str = '', error_details: str = '',
-                 preview_failure_reason: str = '', new_order_failure_reason: str = '') -> None:
+                 preview_failure_reason: str = '', new_order_failure_reason: str = '', **kwargs) -> None:
         self.error = error
         self.message = message
         self.error_details = error_details
@@ -80,7 +80,7 @@ class LimitGTC:
     limit_price: str
     post_only: bool
 
-    def __init__(self, base_size: str, limit_price: str, post_only: bool) -> None:
+    def __init__(self, base_size: str, limit_price: str, post_only: bool, **kwargs) -> None:
         self.base_size = base_size
         self.limit_price = limit_price
         self.post_only = post_only
@@ -96,7 +96,7 @@ class LimitGTD:
     post_only: bool
     end_time: datetime
 
-    def __init__(self, base_size: str, limit_price: str, post_only: bool, end_time: str) -> None:
+    def __init__(self, base_size: str, limit_price: str, post_only: bool, end_time: str, **kwargs) -> None:
         self.base_size = base_size
         self.limit_price = limit_price
         self.post_only = post_only
@@ -112,7 +112,7 @@ class MarketIOC:
     quote_size: str
     base_size: str
 
-    def __init__(self, quote_size: str = None, base_size: str = None) -> None:
+    def __init__(self, quote_size: str = None, base_size: str = None, **kwargs) -> None:
         self.quote_size = quote_size
         self.base_size = base_size
 
@@ -131,7 +131,7 @@ def __init__(self,
                  base_size: str,
                  limit_price: str,
                  stop_price: str,
-                 stop_direction: str) -> None:
+                 stop_direction: str, **kwargs) -> None:
         self.base_size = base_size
         self.limit_price = limit_price
         self.stop_price = stop_price
@@ -154,7 +154,7 @@ def __init__(self,
                  limit_price: str,
                  stop_price: str,
                  end_time: str,
-                 stop_direction: str) -> None:
+                 stop_direction: str, **kwargs) -> None:
         self.base_size = base_size
         self.limit_price = limit_price
         self.stop_price = stop_price
@@ -176,7 +176,7 @@ class OrderConfiguration:
 
     def __init__(self, market_market_ioc: dict = None, limit_limit_gtc: dict = None,
                  limit_limit_gtd: dict = None, stop_limit_stop_limit_gtc: dict = None,
-                 stop_limit_stop_limit_gtd: dict = None) -> None:
+                 stop_limit_stop_limit_gtd: dict = None, **kwargs) -> None:
         self.market_market_ioc = MarketIOC(
             **market_market_ioc) if market_market_ioc is not None else None
         self.limit_limit_gtc = LimitGTC(
@@ -258,7 +258,7 @@ def __init__(self, order_id: str, product_id: str, side: str, client_order_id: s
                  order_placement_source: str = None,
                  outstanding_hold_amount: str = None,
 
-                 order_error: dict = None) -> None:
+                 order_error: dict = None, **kwargs) -> None:
         self.order_id = order_id
         self.product_id = product_id
         self.side = side
@@ -348,7 +348,7 @@ def __init__(self,
                  orders: List[dict],
                  has_next: bool,
                  cursor: str,
-                 sequence: int,
+                 sequence: int, **kwargs
                  ) -> None:
 
         self.orders = list(map(lambda x: Order(**x), orders)) if orders is not None else None
@@ -379,7 +379,7 @@ class OrderCancellation:
     failure_reason: str
     order_id: str
 
-    def __init__(self, success: bool, failure_reason: str, order_id: str) -> None:
+    def __init__(self, success: bool, failure_reason: str, order_id: str, **kwargs) -> None:
         self.success = success
         self.failure_reason = failure_reason
         self.order_id = order_id
@@ -392,7 +392,7 @@ class OrderBatchCancellation:
 
     results: List[OrderCancellation]
 
-    def __init__(self, results: List[OrderCancellation]) -> None:
+    def __init__(self, results: List[OrderCancellation], **kwargs) -> None:
         self.results = results
 
     @classmethod
@@ -444,7 +444,7 @@ def __init__(
             liquidity_indicator: str,
             size_in_quote: bool,
             user_id: str,
-            side: str) -> None:
+            side: str, **kwargs) -> None:
         self.entry_id = entry_id
         self.trade_id = trade_id
         self.order_id = order_id
@@ -475,7 +475,7 @@ class FillsPage:
 
     def __init__(self,
                  fills: List[dict],
-                 cursor: str,
+                 cursor: str, **kwargs
                  ) -> None:
 
         self.fills = list(map(lambda x: Fill(**x), fills)) if fills is not None else None

coinbaseadvanced/models/products.py

@@ -105,7 +105,7 @@ def __init__(self,
                  alias: str,
                  alias_to: list,
                  base_display_symbol: str,
-                 quote_display_symbol: str,
+                 quote_display_symbol: str, **kwargs
                  ) -> None:
         self.product_id = product_id
         self.price = price
@@ -161,7 +161,7 @@ class ProductsPage:
     products: List[Product]
     num_products: int
 
-    def __init__(self, products: List[Product], num_products: int) -> None:
+    def __init__(self, products: List[Product], num_products: int, **kwargs) -> None:
         self.products = list(map(lambda x: Product(**x), products)) \
             if products is not None else None
 
@@ -192,7 +192,7 @@ class Candle:
     close: str
     volume: int
 
-    def __init__(self, start: int, low: str, high: str, open: str, close: str, volume: int) -> None:
+    def __init__(self, start: int, low: str, high: str, open: str, close: str, volume: int, **kwargs) -> None:
         self.start = start
         self.low = low
         self.high = high
@@ -208,7 +208,7 @@ class CandlesPage:
 
     candles: List[Candle]
 
-    def __init__(self, candles: List[Candle]) -> None:
+    def __init__(self, candles: List[Candle], **kwargs) -> None:
         self.candles = list(map(lambda x: Candle(**x), candles)) if candles is not None else None
 
     @classmethod
@@ -246,7 +246,7 @@ def __init__(self,
                  time: datetime,
                  side: str,
                  bid: str,
-                 ask: str) -> None:
+                 ask: str, **kwargs) -> None:
         self.trade_id = trade_id
         self.product_id = product_id
         self.price = price
@@ -269,7 +269,7 @@ class TradesPage:
     def __init__(self,
                  trades: List[Trade],
                  best_bid: str,
-                 best_ask: str,
+                 best_ask: str, **kwargs
                  ) -> None:
         self.trades = list(map(lambda x: Trade(**x), trades)) if trades is not None else None
         self.best_bid = best_bid

tests/fixtures/fixtures.py

@@ -98,6 +98,14 @@ def fixture_list_orders_success_response() -> str:
             text=content)
 
 
+def fixture_list_orders_with_extra_unnamed_success_response() -> str:
+    with open('tests/fixtures/list_orders_with_extra_unnamed_success_response.json', 'r', encoding="utf-8") as file:
+        content = file.read()
+        return fixtured_mock_response(
+            ok=True,
+            text=content)
+
+
 def fixture_list_fills_success_response() -> str:
     with open('tests/fixtures/list_fills_success_response.json', 'r', encoding="utf-8") as file:
         content = file.read()