chore: add happy path tests

noble-assets · Jul 8, 2024 · 9a77cc4 · 9a77cc4
1 parent 2910fdb
commit 9a77cc4
Showing 1 changed file with 52 additions and 16 deletions.
diff --git a/e2e/fiat_tf_test.go b/e2e/fiat_tf_test.go
@@ -22,11 +22,9 @@ func TestFiatTFUpdatePauser(t *testing.T) {
 	noble := nw.chain
 	val := noble.Validators[0]
 
-	// ACTION: Update Pauser while TF is paused
+	// ACTION: Happy path: Update Pauser
 	// EXPECTED: Success; pauser updated
 
-	pauseFiatTF(t, ctx, val, nw.fiatTfRoles.Pauser)
-
 	w := interchaintest.GetAndFundTestUsers(t, ctx, "new-pauser-1", math.OneInt(), noble)
 	newPauser1 := w[0]
 
@@ -42,18 +40,40 @@ func TestFiatTFUpdatePauser(t *testing.T) {
 	}
 	require.Equal(t, expectedGetPauserResponse.Pauser, showPauserRes.Pauser)
 
-	unpauseFiatTF(t, ctx, val, newPauser1)
+	// ACTION: Update Pauser while TF is paused
+	// EXPECTED: Success; pauser updated
+	// Status:
+	// 	Pauser: newPauser1
+
+	pauseFiatTF(t, ctx, val, newPauser1)
+
+	w = interchaintest.GetAndFundTestUsers(t, ctx, "new-pauser-2", math.OneInt(), noble)
+	newPauser2 := w[0]
+
+	_, err = val.ExecTx(ctx, nw.fiatTfRoles.Owner.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser2.FormattedAddress())
+	require.NoError(t, err, "failed to broadcast update-pauser message")
+
+	showPauserRes, err = showPauser(ctx, val)
+	require.NoError(t, err, "failed to query show-pauser")
+	expectedGetPauserResponse = fiattokenfactorytypes.QueryGetPauserResponse{
+		Pauser: fiattokenfactorytypes.Pauser{
+			Address: string(newPauser2.FormattedAddress()),
+		},
+	}
+	require.Equal(t, expectedGetPauserResponse.Pauser, showPauserRes.Pauser)
+
+	unpauseFiatTF(t, ctx, val, newPauser2)
 
 	// ACTION: Update Pauser from non owner account
 	// EXPECTED: Request fails; pauser not updated
 	// Status:
-	// 	Pauser: newPauser1
+	// 	Pauser: newPauser2
 
 	w = interchaintest.GetAndFundTestUsers(t, ctx, "default", math.OneInt(), noble, noble)
-	newPauser2 := w[0]
+	newPauser3 := w[0]
 	alice := w[1]
 
-	_, err = val.ExecTx(ctx, alice.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser2.FormattedAddress())
+	_, err = val.ExecTx(ctx, alice.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser3.FormattedAddress())
 	require.ErrorContains(t, err, "you are not the owner: unauthorized")
 
 	showPauserRes, err = showPauser(ctx, val)
@@ -63,18 +83,18 @@ func TestFiatTFUpdatePauser(t *testing.T) {
 	// ACTION: Update Pauser from blacklisted owner account
 	// EXPECTED: Success; pauser updated
 	// Status:
-	// 	Pauser: newPauser1
+	// 	Pauser: newPauser2
 
 	blacklistAccount(t, ctx, val, nw.fiatTfRoles.Blacklister, nw.fiatTfRoles.Owner)
 
-	_, err = val.ExecTx(ctx, nw.fiatTfRoles.Owner.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser2.FormattedAddress())
+	_, err = val.ExecTx(ctx, nw.fiatTfRoles.Owner.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser3.FormattedAddress())
 	require.NoError(t, err, "failed to broadcast update-pauser message")
 
 	showPauserRes, err = showPauser(ctx, val)
 	require.NoError(t, err, "failed to query show-pauser")
 	expectedGetPauserResponse = fiattokenfactorytypes.QueryGetPauserResponse{
 		Pauser: fiattokenfactorytypes.Pauser{
-			Address: string(newPauser2.FormattedAddress()),
+			Address: string(newPauser3.FormattedAddress()),
 		},
 	}
 	require.Equal(t, expectedGetPauserResponse.Pauser, showPauserRes.Pauser)
@@ -84,21 +104,21 @@ func TestFiatTFUpdatePauser(t *testing.T) {
 	// ACTION: Update Pauser to blacklisted Pauser account
 	// EXPECTED: Success; pauser updated
 	// Status:
-	// 	Pauser: newPauser2
+	// 	Pauser: newPauser3
 
-	w = interchaintest.GetAndFundTestUsers(t, ctx, "new-pauser-3", math.OneInt(), noble)
-	newPauser3 := w[0]
+	w = interchaintest.GetAndFundTestUsers(t, ctx, "new-pauser-4", math.OneInt(), noble)
+	newPauser4 := w[0]
 
-	blacklistAccount(t, ctx, val, nw.fiatTfRoles.Blacklister, newPauser3)
+	blacklistAccount(t, ctx, val, nw.fiatTfRoles.Blacklister, newPauser4)
 
-	_, err = val.ExecTx(ctx, nw.fiatTfRoles.Owner.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser3.FormattedAddress())
+	_, err = val.ExecTx(ctx, nw.fiatTfRoles.Owner.KeyName(), "fiat-tokenfactory", "update-pauser", newPauser4.FormattedAddress())
 	require.NoError(t, err, "failed to broadcast update-pauser message")
 
 	showPauserRes, err = showPauser(ctx, val)
 	require.NoError(t, err, "failed to query show-pauser")
 	expectedGetPauserResponse = fiattokenfactorytypes.QueryGetPauserResponse{
 		Pauser: fiattokenfactorytypes.Pauser{
-			Address: string(newPauser3.FormattedAddress()),
+			Address: string(newPauser4.FormattedAddress()),
 		},
 	}
 	require.Equal(t, expectedGetPauserResponse.Pauser, showPauserRes.Pauser)
@@ -116,8 +136,17 @@ func TestFiatTFPause(t *testing.T) {
 	noble := nw.chain
 	val := noble.Validators[0]
 
+	// ACTION: Happy path: Pause TF
+	// EXPECTED: Success; TF is paused
+
+	pauseFiatTF(t, ctx, val, nw.fiatTfRoles.Pauser)
+
 	// ACTION: Pause TF from an account that is not the Pauser
 	// EXPECTED: Request fails; TF not paused
+	// Status:
+	// 	Paused: true
+
+	unpauseFiatTF(t, ctx, val, nw.fiatTfRoles.Pauser)
 
 	w := interchaintest.GetAndFundTestUsers(t, ctx, "alice", math.OneInt(), noble)
 	alice := w[0]
@@ -165,6 +194,13 @@ func TestFiatTFUnpause(t *testing.T) {
 	noble := nw.chain
 	val := noble.Validators[0]
 
+	// ACTION: Happy path: Unpause TF
+	// EXPECTED: Success; TF is unpaused
+
+	pauseFiatTF(t, ctx, val, nw.fiatTfRoles.Pauser)
+
+	unpauseFiatTF(t, ctx, val, nw.fiatTfRoles.Pauser)
+
 	// ACTION: Unpause TF from an account that is not a Pauser
 	// EXPECTED: Request fails; TF remains paused