Skip to content

Commit b2a98a9

Browse files
committed
multi: make use of require.Len where possible
This commit changes instances of `require.Equal(t, <int>, len(<var>))` to `require.Len(t, <var>, <int>)` where possible.
1 parent f124681 commit b2a98a9

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

itest/assertions.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1567,10 +1567,10 @@ func assertGroups(t *testing.T, client taprpc.TaprootAssetsClient,
15671567
require.NoError(t, err)
15681568

15691569
groupKeys := maps.Keys(assetGroups.Groups)
1570-
require.Equal(t, 2, len(groupKeys))
1570+
require.Len(t, groupKeys, 2)
15711571

15721572
groupedAssets := assetGroups.Groups[groupKeys[0]].Assets
1573-
require.Equal(t, 1, len(groupedAssets))
1573+
require.Len(t, groupedAssets, 1)
15741574
require.Equal(t, 1, len(assetGroups.Groups[groupKeys[1]].Assets))
15751575

15761576
groupedAssets = append(

tapdb/assets_store_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1261,7 +1261,7 @@ func TestAssetExportLog(t *testing.T) {
12611261
// looking for all unconfirmed transfers.
12621262
assetTransfers, err := db.QueryAssetTransfers(ctx, TransferQuery{})
12631263
require.NoError(t, err)
1264-
require.Equal(t, 1, len(assetTransfers))
1264+
require.Len(t, assetTransfers, 1)
12651265

12661266
// We should also be able to find it based on its outpoint.
12671267
firstOutput := spendDelta.Outputs[0]
@@ -1270,7 +1270,7 @@ func TestAssetExportLog(t *testing.T) {
12701270
AnchorTxHash: firstOutputAnchor.OutPoint.Hash[:],
12711271
})
12721272
require.NoError(t, err)
1273-
require.Equal(t, 1, len(assetTransfers))
1273+
require.Len(t, assetTransfers, 1)
12741274

12751275
// Check that the new UTXO is found among our managed UTXOs.
12761276
utxos, err = assetsStore.FetchManagedUTXOs(ctx)
@@ -1302,13 +1302,13 @@ func TestAssetExportLog(t *testing.T) {
13021302
UnconfOnly: true,
13031303
})
13041304
require.NoError(t, err)
1305-
require.Equal(t, 1, len(assetTransfers))
1305+
require.Len(t, assetTransfers, 1)
13061306

13071307
// This should also show up in the set of pending parcels. It should
13081308
// also match exactly the inbound parcel we used to make the delta.
13091309
parcels, err := assetsStore.PendingParcels(ctx)
13101310
require.NoError(t, err)
1311-
require.Equal(t, 1, len(parcels))
1311+
require.Len(t, parcels, 1)
13121312
require.Equal(t, spendDelta, parcels[0])
13131313

13141314
// We should also be able to query for the parcel when filtering on its
@@ -1428,7 +1428,7 @@ func TestAssetExportLog(t *testing.T) {
14281428
// At this point, there should be no more pending parcels.
14291429
parcels, err = assetsStore.PendingParcels(ctx)
14301430
require.NoError(t, err)
1431-
require.Equal(t, 0, len(parcels))
1431+
require.Len(t, parcels, 0)
14321432
}
14331433

14341434
// TestAssetGroupWitnessUpsert tests that if you try to insert another asset

0 commit comments

Comments
 (0)