DFI-795 Remove round-robin mechanism from issuing new currencies func…

…tional (#206)

app/unit_currencies_test.go

@@ -162,13 +162,13 @@ func TestCurrenciesApp_Issue(t *testing.T) {
 		CheckResultError(t, currencies.ErrWrongIssueID, res, err)
 	}
 
-	// fail: currency issue with already existing uniqueMsgID
+	// fail: currency issue with already approved uniqueMsgID
 	{
 		msgId, issueId := "1", "non-existing-issue"
 		coin := sdk.NewCoin(denom, amount)
 
 		res, err := IssueCurrency(t, app, coin, msgId, issueId, recipientIdx, genAccs, genPrivKeys, false)
-		CheckResultError(t, multisig.ErrWrongCallUniqueId, res, err)
+		CheckResultError(t, multisig.ErrVoteAlreadyApproved, res, err)
 	}
 }
 

x/multisig/internal/keeper/call.go

@@ -13,9 +13,13 @@ import (
 func (k Keeper) SubmitCall(ctx sdk.Context, msg msmodule.MsMsg, uniqueID string, sender sdk.AccAddress) error {
 	k.modulePerms.AutoCheck(types.PermWrite)
 
-	// check call exists
+	// confirm call if exists
 	if k.HasCallByUniqueID(ctx, uniqueID) {
-		return sdkErrors.Wrapf(types.ErrWrongCallUniqueId, "%q exists", uniqueID)
+		id, err := k.GetCallIDByUniqueID(ctx, uniqueID)
+		if err != nil {
+			return err
+		}
+		return k.ConfirmCall(ctx, id, sender)
 	}
 
 	// create a new call and check its validity

x/multisig/internal/keeper/call_test.go

@@ -150,7 +150,7 @@ func TestMSKeeper_SubmitCall_InvalidInputs(t *testing.T) {
 	// invalid: call
 	{
 		msg := NewMockMsMsg(true)
-		require.Error(t, keeper.SubmitCall(ctx, msg, uniqueID1, sdk.AccAddress{}))
+		require.NoError(t, keeper.SubmitCall(ctx, msg, uniqueID1, sdk.AccAddress{}))
 	}
 
 	// invalid: non-existing msg route