Skip to content

Commit

Permalink
test: fix after boxes and prefetcher remerge (#4716)
Browse files Browse the repository at this point in the history
  • Loading branch information
algorandskiy authored Nov 1, 2022
1 parent e6f396b commit 97c232d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion ledger/internal/eval.go
Original file line number Diff line number Diff line change
Expand Up @@ -1558,7 +1558,7 @@ transactionGroupLoop:
if !ok {
break transactionGroupLoop
} else if txgroup.Err != nil {
logging.Base().Errorf("eval prefetcher error: %w", txgroup.Err)
logging.Base().Errorf("eval prefetcher error: %v", txgroup.Err)
}

if txgroup.Err == nil {
Expand Down
18 changes: 9 additions & 9 deletions test/e2e-go/features/transactions/accountv2_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -343,31 +343,31 @@ func TestAccountInformationWithBadAssetIdx(t *testing.T) {
defer fixtures.ShutdownSynchronizedTest(t)

t.Parallel()
AccountInformationCheckWithOffendingFields(t, []basics.AssetIndex{12181853637140359511}, nil, nil)
accountInformationCheckWithOffendingFields(t, []basics.AssetIndex{12181853637140359511}, nil, nil)
}

// Add missing asset index
func TestAccountInformationWithMissingAssetIdx(t *testing.T) {
partitiontest.PartitionTest(t)
defer fixtures.ShutdownSynchronizedTest(t)

AccountInformationCheckWithOffendingFields(t, []basics.AssetIndex{121818}, nil, nil)
accountInformationCheckWithOffendingFields(t, []basics.AssetIndex{121818}, nil, nil)
}

// Add offending app index greater than uint64
func TestAccountInformationWithBadAppIdx(t *testing.T) {
partitiontest.PartitionTest(t)
defer fixtures.ShutdownSynchronizedTest(t)

AccountInformationCheckWithOffendingFields(t, nil, []basics.AppIndex{12181853637140359511}, nil)
accountInformationCheckWithOffendingFields(t, nil, []basics.AppIndex{12181853637140359511}, nil)
}

// Add missing app index
func TestAccountInformationWithMissingApp(t *testing.T) {
partitiontest.PartitionTest(t)
defer fixtures.ShutdownSynchronizedTest(t)

AccountInformationCheckWithOffendingFields(t, nil, []basics.AppIndex{121818}, nil)
accountInformationCheckWithOffendingFields(t, nil, []basics.AppIndex{121818}, nil)
}

// Add missing account address
Expand All @@ -377,10 +377,10 @@ func TestAccountInformationWithMissingAddress(t *testing.T) {

randAddr := basics.Address{}
crypto.RandBytes(randAddr[:])
AccountInformationCheckWithOffendingFields(t, nil, nil, []basics.Address{randAddr})
accountInformationCheckWithOffendingFields(t, nil, nil, []basics.Address{randAddr})
}

func AccountInformationCheckWithOffendingFields(t *testing.T,
func accountInformationCheckWithOffendingFields(t *testing.T,
foreignAssets []basics.AssetIndex,
foreignApps []basics.AppIndex,
accounts []basics.Address) {
Expand Down Expand Up @@ -464,7 +464,7 @@ int 1

// create the app
tx, err := client.MakeUnsignedAppCreateTx(
transactions.OptInOC, approvalOps.Program, clearstateOps.Program, schema, schema, nil, nil, nil, nil, 0)
transactions.OptInOC, approvalOps.Program, clearstateOps.Program, schema, schema, nil, nil, nil, nil, nil, 0)
a.NoError(err)
tx, err = client.FillUnsignedTxTemplate(creator, 0, 0, fee, tx)
a.NoError(err)
Expand Down Expand Up @@ -519,7 +519,7 @@ int 1
checkEvalDelta(t, &client, txnRound, txnRound+1, 1, 1)

// call the app
tx, err = client.MakeUnsignedAppOptInTx(uint64(appIdx), nil, nil, nil, nil)
tx, err = client.MakeUnsignedAppOptInTx(uint64(appIdx), nil, nil, nil, nil, nil)
a.NoError(err)
if foreignAssets != nil {
tx.ForeignAssets = foreignAssets
Expand Down Expand Up @@ -608,7 +608,7 @@ int 1
a.Equal(creator, app.Params.Creator)

// call the app
tx, err = client.MakeUnsignedAppNoOpTx(uint64(appIdx), nil, nil, nil, nil)
tx, err = client.MakeUnsignedAppNoOpTx(uint64(appIdx), nil, nil, nil, nil, nil)
a.NoError(err)
tx, err = client.FillUnsignedTxTemplate(user, 0, 0, fee, tx)
a.NoError(err)
Expand Down

0 comments on commit 97c232d

Please sign in to comment.