Switch test assertions to canonical commissionAndFees, drop deprecation warnings

The protobuf-side tests asserted on the legacy ``.commission``
attribute and ``commission=`` kwarg in ~15 places, each producing a
``DeprecationWarning`` on every test run. The legacy alias is
documented behaviour through v4.0 — the warnings are noise, not
signal — so the tests should use the canonical
``.commissionAndFees`` name everywhere except where the deprecation
alias itself is the contract under test.

Changes:
* Switch all assertions / constructor kwargs / handler-captured
  reads from ``.commission`` / ``commission=`` to
  ``.commissionAndFees`` / ``commissionAndFees=`` across
  ``test_proto_audit_regressions.py``,
  ``test_proto_decoder_dispatch.py``,
  ``test_proto_orders.py``,
  ``test_round6_cross_handler_invariants.py``.
* Two tests in ``test_proto_orders.py`` specifically exercise the
  rename / deprecation alias surface
  (``test_create_order_state_renames_commission_and_fees_to_commission``,
  ``test_create_commission_report_renames_fields``). Those keep the
  legacy ``.commission`` access and gain
  ``@pytest.mark.filterwarnings("ignore::DeprecationWarning")`` so
  the alias contract stays pinned without polluting the warning
  stream.

Net: 1001 owned tests still pass; the DeprecationWarning chatter is
gone from the test output.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: mattsta

tests/test_contract.py

@@ -1,8 +1,13 @@
 import pandas as pd
+import pytest
 
 import ib_async
 from ib_async import Stock, util
 
+# Live: requires a running TWS / IB Gateway on 127.0.0.1:4001.
+# Skipped by default. Opt in with ``pytest -m live``.
+pytestmark = pytest.mark.live
+
 
 def test_contract_format_data_pd():
     """Simple smoketest to verify everything still works minimally."""

tests/test_proto_audit_regressions.py

@@ -81,7 +81,7 @@ def test_commission_report_unset_realized_pnl_stays_none_not_nan():
     ib.client.decoder.processProtoBuf(59, proto.SerializeToString())
 
     # commission did flow.
-    assert fill.commissionReport.commission == Decimal("1.25")
+    assert fill.commissionReport.commissionAndFees == Decimal("1.25")
     # realizedPNL / yield_ stay None — NOT Decimal('NaN'), NOT 0.
     assert fill.commissionReport.realizedPNL is None
     assert fill.commissionReport.yield_ is None
@@ -124,14 +124,14 @@ def test_commission_report_explicit_none_survives_wrapper_dispatch():
 
     explicit = ibi.CommissionReport(
         execId="exec-explicit-none",
-        commission=Decimal("0.50"),
+        commissionAndFees=Decimal("0.50"),
         currency="USD",
         realizedPNL=None,
         yield_=None,
     )
     ib.wrapper.commissionReport(explicit)
 
-    assert fill.commissionReport.commission == Decimal("0.50")
+    assert fill.commissionReport.commissionAndFees == Decimal("0.50")
     assert fill.commissionReport.realizedPNL is None
     assert fill.commissionReport.yield_ is None
     # Direct NaN guard: nothing morphed silently into a NaN sentinel.
@@ -2582,7 +2582,7 @@ def test_commission_report_arriving_before_fill_is_buffered_and_drained():
 
     # 1) Commission report arrives FIRST. It must be parked, not dropped.
     early_report = ibi.CommissionReport(
-        execId="exec-early", commission=2.5, currency="USD"
+        execId="exec-early", commissionAndFees=2.5, currency="USD"
     )
     ib.wrapper.commissionReport(early_report)
     # Not yet emitted — fill doesn't exist.
@@ -2605,9 +2605,9 @@ def test_commission_report_arriving_before_fill_is_buffered_and_drained():
 
     assert "exec-early" not in ib.wrapper._pendingCommissionReports
     assert len(trade.fills) == 1
-    assert trade.fills[0].commissionReport.commission == 2.5
+    assert trade.fills[0].commissionReport.commissionAndFees == 2.5
     assert len(seenCommission) == 1
-    assert seenCommission[0][2].commission == 2.5
+    assert seenCommission[0][2].commissionAndFees == 2.5
 
 
 def test_completed_order_for_existing_trade_updates_status_to_terminal():

tests/test_proto_decoder_dispatch.py

@@ -177,7 +177,7 @@ def test_commission_report_proto_attaches_to_existing_fill():
     ib.client.decoder.processProtoBuf(59, proto.SerializeToString())
 
     assert fill.commissionReport.execId == "exec-7"
-    assert fill.commissionReport.commission == Decimal("1.25")
+    assert fill.commissionReport.commissionAndFees == Decimal("1.25")
     assert fill.commissionReport.realizedPNL == Decimal(10)
 
 

tests/test_proto_orders.py

@@ -21,6 +21,8 @@
 
 from decimal import Decimal
 
+import pytest
+
 from ib_async._pb import (
     AttachedOrders_pb2,
     CommissionAndFeesReport_pb2,
@@ -421,6 +423,7 @@ def test_order_status_total_property_sums_when_both_set():
 # ---------------------------------------------------------------------------
 
 
+@pytest.mark.filterwarnings("ignore::DeprecationWarning")
 def test_create_order_state_renames_commission_and_fees_to_commission():
     proto = OrderState_pb2.OrderState()
     proto.status = "PreSubmitted"
@@ -552,6 +555,7 @@ def test_create_execution_reads_submitter_and_opt_exercise_type():
 # ---------------------------------------------------------------------------
 
 
+@pytest.mark.filterwarnings("ignore::DeprecationWarning")
 def test_create_commission_report_renames_fields():
     proto = CommissionAndFeesReport_pb2.CommissionAndFeesReport()
     proto.execId = "exec-1"
@@ -578,13 +582,13 @@ def test_create_commission_report_unparseable_redemption_date_safe():
     proto.yieldRedemptionDate = "garbage"
     report = createCommissionReport(proto)
     assert report.yieldRedemptionDate == 0
-    assert report.commission == Decimal("1.25")
+    assert report.commissionAndFees == Decimal("1.25")
 
 
 def test_create_commission_report_handles_empty_proto():
     proto = CommissionAndFeesReport_pb2.CommissionAndFeesReport()
     report = createCommissionReport(proto)
-    assert report.commission is None
+    assert report.commissionAndFees is None
     assert report.realizedPNL is None
     assert report.yield_ is None
 

tests/test_round6_cross_handler_invariants.py

@@ -444,13 +444,13 @@ def test_execDetails_then_commissionReport_pairs_on_fill():
     )
     ib.wrapper.execDetails(reqId=99, contract=contract, execution=execution)
     assert len(trade.fills) == 1
-    assert trade.fills[0].commissionReport.commission is None
+    assert trade.fills[0].commissionReport.commissionAndFees is None
 
     report = ibi.CommissionReport(
-        execId=execId, commission=Decimal("1.0"), currency="USD"
+        execId=execId, commissionAndFees=Decimal("1.0"), currency="USD"
     )
     ib.wrapper.commissionReport(report)
-    assert trade.fills[0].commissionReport.commission == Decimal("1.0")
+    assert trade.fills[0].commissionReport.commissionAndFees == Decimal("1.0")
     assert len(seenComm) == 1
 
 
@@ -476,7 +476,7 @@ def test_execDetails_without_commissionReport_keeps_default_empty():
 
     assert len(trade.fills) == 1
     cr = trade.fills[0].commissionReport
-    assert cr.commission is None
+    assert cr.commissionAndFees is None
     assert cr.realizedPNL is None
     assert cr.execId == ""
 
@@ -637,7 +637,7 @@ def test_fills_aggregate_invariants_match_orderStatus_after_lifecycle():
         )
         ib.wrapper.commissionReport(
             ibi.CommissionReport(
-                execId=execId, commission=Decimal("0.5"), currency="USD"
+                execId=execId, commissionAndFees=Decimal("0.5"), currency="USD"
             )
         )
         cum += Decimal(sharesStr)
@@ -659,7 +659,9 @@ def test_fills_aggregate_invariants_match_orderStatus_after_lifecycle():
     assert trade.orderStatus.status == "Filled"
     assert trade.filled() == Decimal(50)
     assert trade.remaining() == Decimal(0)
-    assert all(f.commissionReport.commission == Decimal("0.5") for f in trade.fills)
+    assert all(
+        f.commissionReport.commissionAndFees == Decimal("0.5") for f in trade.fills
+    )
     # All fills carry the same permId — the wire's link from the
     # execution back to the order. Here we asserted on
     # ``trade.orderStatus.permId`` (set on the orderStatus callback)