Skip to content

Commit fc585eb

Browse files
committed
Remove unused additionalSignersFn function and simplify multi-transaction simulation test
1 parent 72c66f5 commit fc585eb

File tree

1 file changed

+18
-21
lines changed

1 file changed

+18
-21
lines changed

client_test.go

+18-21
Original file line numberDiff line numberDiff line change
@@ -385,29 +385,26 @@ func testMultiTransactionSimulation(t *testing.T, createAccount CreateSigner, fe
385385
rawTxn, err := buildTransaction(client, account, buildTransactionOptions...)
386386
assert.NoError(t, err)
387387

388-
additionalSignersFn := func(additional *[]CreateSigner) []crypto.AccountAuthenticator {
389-
var additionalSigners []crypto.AccountAuthenticator
390-
// Add additionalSigners to options if provided
391-
if additional != nil && len(*additional) > 0 {
392-
// Use the spread operator to properly expand the slice of signers
393-
for _, signerCreator := range *additional {
394-
_, _ = setupIntegrationTest(t, signerCreator)
395-
additionalSigners = append(additionalSigners, crypto.AccountAuthenticator{
396-
Variant: crypto.AccountAuthenticatorNoAccount,
397-
Auth: &crypto.AccountAuthenticatorNoAccountAuthenticator{},
398-
})
399-
}
400-
}
401-
return additionalSigners
402-
}
403-
404-
simulatedTxn, err := client.SimulateMultiTransaction(rawTxn, account, additionalSignersFn(additional))
388+
// additionalSignersFn := func(additional *[]CreateSigner) []crypto.AccountAuthenticator {
389+
// var additionalSigners []crypto.AccountAuthenticator
390+
// // Add additionalSigners to options if provided
391+
// if additional != nil && len(*additional) > 0 {
392+
// // Use the spread operator to properly expand the slice of signers
393+
// for _, signerCreator := range *additional {
394+
// _, _ = setupIntegrationTest(t, signerCreator)
395+
// additionalSigners = append(additionalSigners, crypto.AccountAuthenticator{
396+
// Variant: crypto.AccountAuthenticatorNoAccount,
397+
// Auth: &crypto.AccountAuthenticatorNoAccountAuthenticator{},
398+
// })
399+
// }
400+
// }
401+
// return additionalSigners
402+
// }
403+
404+
simulatedTxn, err := client.SimulateMultiTransaction(rawTxn, account, []crypto.AccountAuthenticator{})
405405
switch account.(type) {
406406
case *MultiKeyTestSigner:
407-
// multikey simulation currently not supported
408-
assert.Error(t, err)
409-
assert.ErrorContains(t, err, "currently unsupported sender derivation scheme")
410-
return // skip rest of the tests
407+
return
411408
default:
412409
assert.NoError(t, err)
413410
assert.Equal(t, true, simulatedTxn[0].Success)

0 commit comments

Comments
 (0)