refactor: modify new update_asset_holding method so asset is the first parameter

Author: daniel-makerx

src/_algopy_testing/context_helpers/ledger_context.py

@@ -93,8 +93,8 @@ def update_account(
 
     def update_asset_holdings(
         self,
-        account: algopy.Account | str,
         asset: algopy.Asset | algopy.UInt64 | int,
+        account: algopy.Account | str,
         *,
         balance: algopy.UInt64 | int | None = None,
         frozen: bool | None = None,

src/_algopy_testing/value_generators/avm.py

@@ -98,8 +98,8 @@ def account(
         account_data.fields.update(account_fields)
         for asset_id, balance in (opted_asset_balances or {}).items():
             ledger.update_asset_holdings(
-                new_account_address,
                 asset_id,
+                new_account_address,
                 balance=balance,
             )
         account_data.opted_apps = {_get_app_id(app): ledger.get_app(app) for app in opted_apps}

tests/models/test_asset.py

@@ -37,7 +37,7 @@ def test_asset_from_int() -> None:
 def test_asset_balance(context: AlgopyTestContext) -> None:
     account = context.any.account()
     asset = context.any.asset()
-    context.ledger.update_asset_holdings(account, asset, balance=1000)
+    context.ledger.update_asset_holdings(asset, account, balance=1000)
 
     assert asset.balance(account) == UInt64(1000)