fix test

Author: brianolson

netdeploy/remote/deployedNetwork_test.go

@@ -23,11 +23,12 @@ import (
 
 	"github.com/algorand/go-algorand/config"
 	"github.com/algorand/go-algorand/data/basics"
+	"github.com/algorand/go-algorand/logging"
 	"github.com/algorand/go-algorand/protocol"
 	"github.com/algorand/go-algorand/test/partitiontest"
 )
 
-func TestCreateSignedTx(t *testing.T) {
+func TestCreateSignedTxBasic(t *testing.T) {
 	partitiontest.PartitionTest(t)
 
 	var networkState netState
@@ -36,6 +37,7 @@ func TestCreateSignedTx(t *testing.T) {
 	networkState.nAccounts = 10
 	networkState.roundTxnCnt = 4
 	networkState.txnState = protocol.PaymentTx
+	networkState.log = logging.TestingLog(t)
 
 	params := config.Consensus[protocol.ConsensusCurrentVersion]
 
@@ -50,7 +52,8 @@ func TestCreateSignedTx(t *testing.T) {
 		require.Equal(t, protocol.PaymentTx, sntx.Txn.Type)
 	}
 
-	initialAccounts := networkState.accounts
+	initialAccounts := make([]basics.Address, len(networkState.accounts))
+	copy(initialAccounts, networkState.accounts)
 
 	//	should be creating assets next
 	sgtxns, _ = createSignedTx(src, basics.Round(1), params, &networkState)
@@ -59,7 +62,7 @@ func TestCreateSignedTx(t *testing.T) {
 	require.Equal(t, protocol.ApplicationCallTx, networkState.txnState)
 	require.Equal(t, uint64(0), networkState.nAssets)
 	//	same accounts should be used
-	require.Equal(t, initialAccounts[0], accounts[0])
+	require.Equal(t, initialAccounts, accounts)
 	for _, sntx := range sgtxns {
 		require.Equal(t, protocol.AssetConfigTx, sntx.Txn.Type)
 	}
@@ -69,7 +72,7 @@ func TestCreateSignedTx(t *testing.T) {
 	require.Equal(t, 2, len(sgtxns))
 	require.Equal(t, protocol.PaymentTx, networkState.txnState)
 	require.Equal(t, uint64(0), networkState.nApplications)
-	require.Equal(t, initialAccounts[0], accounts[0])
+	require.Equal(t, initialAccounts, accounts)
 	for _, sntx := range sgtxns {
 		require.Equal(t, protocol.ApplicationCallTx, sntx.Txn.Type)
 	}
@@ -78,14 +81,22 @@ func TestCreateSignedTx(t *testing.T) {
 	sgtxns, _ = createSignedTx(src, basics.Round(1), params, &networkState)
 	require.Equal(t, 4, len(sgtxns))
 	require.Equal(t, protocol.PaymentTx, networkState.txnState)
-	//new accounts should be created
-	accounts = networkState.accounts
-	require.NotEqual(t, initialAccounts[0], accounts[0])
+	require.Equal(t, initialAccounts, accounts)
 	for _, sntx := range sgtxns {
 		require.Equal(t, protocol.PaymentTx, sntx.Txn.Type)
 	}
+}
 
+func TestCreateSignedTxAssets(t *testing.T) {
 	//	assets per account should not exceed limit
+	partitiontest.PartitionTest(t)
+
+	params := config.Consensus[protocol.ConsensusCurrentVersion]
+	secretDst := keypair()
+	src := basics.Address(secretDst.SignatureVerifier)
+
+	var networkState netState
+	networkState.log = logging.TestingLog(t)
 	networkState.txnState = protocol.PaymentTx
 	networkState.nAssets = 10
 	networkState.nApplications = 10