Added remove federated account intent and corresponding test (#292)

* Added remove federated account intent and corresponding test

* Fix intellisence summary

Author: BellringerQuinn

Assets/SequenceSDK/WaaS/Tests/MockWaaSWallet.cs

@@ -105,5 +105,11 @@ public string GetEmail()
         {
             throw new NotImplementedException();
         }
+
+        public event Action<string, bool> OnFederatedAccountRemovedComplete;
+        public Task<bool> RemoveFederatedAccount(Account account)
+        {
+            throw new NotImplementedException();
+        }
     }
 }

Assets/SequenceSDK/WaaS/Tests/WaaSFederatedAuthTests.cs

@@ -15,6 +15,8 @@ public class WaaSFederatedAuthTests
         private TaskCompletionSource<bool> _tcs;
         private string _walletAddress;
         SequenceLogin _login;
+        private IWallet _wallet;
+        private Account _federatedAccount;
 
         [Test]
         public async Task TestAccountAssociation()
@@ -152,5 +154,73 @@ private void OnAccountFederated(Account account)
             _tcs.SetResult(true);
             SequenceWallet.OnAccountFederated -= OnAccountFederated;
         }
+
+        [Test]
+        public async Task TestRemoveFederatedAccount()
+        {
+            _login = SequenceLogin.GetInstance();
+            SequenceWallet.OnAccountFederated += OnAccountFederatedBeforeDeletion;
+            _tcs = new TaskCompletionSource<bool>();
+            EndToEndTestHarness testHarness = new EndToEndTestHarness(_login);
+
+            await testHarness.Login(async wallet =>
+            {
+                _wallet = wallet;
+                try
+                {
+                    _walletAddress = wallet.GetWalletAddress();
+                    _email = WaaSEndToEndTestConfig.GetConfig().PlayFabEmail;
+                    string titleId = WaaSEndToEndTestConfig.GetConfig().PlayFabTitleId;
+
+                    if (string.IsNullOrEmpty(PlayFabSettings.staticSettings.TitleId))
+                    {
+                        PlayFabSettings.staticSettings.TitleId = titleId;
+                    }
+
+                    var request = new LoginWithEmailAddressRequest
+                    {
+                        Email = _email,
+                        Password = WaaSEndToEndTestConfig.GetConfig().PlayFabPassword
+                    };
+                    PlayFabClientAPI.LoginWithEmailAddress(request, PlayFabLoginResult, PlayFabLoginFailed);
+                }
+                catch (Exception e)
+                {
+                    _tcs.TrySetException(e);
+                }
+            }, (error, method, email, methods) =>
+            {
+                _tcs.TrySetException(new Exception(error));
+            });
+
+            await _tcs.Task;
+        }
+
+        private void OnAccountFederatedBeforeDeletion(Account account)
+        {
+            if (account.email != _email.ToLower())
+            {
+                _tcs.TrySetException(new Exception($"Emails do not match {_email} != {account.email}"));
+            }
+            SequenceWallet.OnAccountFederated -= OnAccountFederatedBeforeDeletion;
+
+            _wallet.OnFederatedAccountRemovedComplete += OnFederatedAccountRemoved;
+            _federatedAccount = account;
+            _wallet.RemoveFederatedAccount(account);
+        }
+
+        private void OnFederatedAccountRemoved(string accountId, bool success)
+        {
+            if (accountId == _federatedAccount.id)
+            {
+                _tcs.SetResult(true);
+            }
+            else
+            {
+                _tcs.TrySetException(new Exception($"Account ids do not match {_federatedAccount.id} != {accountId}"));
+            }
+            
+            _wallet.OnFederatedAccountRemovedComplete -= OnFederatedAccountRemoved;
+        }
     }
 }

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ParameterTypes/IntentDataRemoveAccount.cs

@@ -0,0 +1,19 @@
+using System;
+using UnityEngine.Scripting;
+
+namespace Sequence.EmbeddedWallet
+{
+    [Serializable]
+    internal class IntentDataRemoveAccount
+    {
+        public string accountId;
+        public string wallet;
+
+        [Preserve]
+        public IntentDataRemoveAccount(string wallet, string accountId)
+        {
+            this.wallet = wallet;
+            this.accountId = accountId;
+        }
+    }
+}

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ParameterTypes/IntentDataRemoveAccount.cs.meta

@@ -71,6 +71,7 @@ public IntentPayload(string version, IntentType type, JObject data, Signature[]
             {IntentType.FederateAccount, "federateAccount"},
             {IntentType.ListAccounts, "listAccounts"},
             {IntentType.GetIdToken, "getIdToken"},
+            {IntentType.RemoveAccount, "removeAccount"},
         };
     }
 
@@ -91,6 +92,7 @@ public enum IntentType
         FederateAccount,
         ListAccounts,
         GetIdToken,
+        RemoveAccount,
         None
     }
 }

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ParameterTypes/IntentPayload.cs

@@ -0,0 +1,10 @@
+using System;
+
+namespace Sequence.EmbeddedWallet
+{
+    [Serializable]
+    public class IntentResponseAccountRemoved
+    {
+        
+    }
+}

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ReturnTypes/IntentResponseAccountRemoved.cs

@@ -324,6 +324,12 @@ public string GetEmail()
         {
             throw new NotSupportedException();
         }
+
+        public event Action<string, bool> OnFederatedAccountRemovedComplete;
+        public Task<bool> RemoveFederatedAccount(Account account)
+        {
+            throw new NotSupportedException();
+        }
     }
     #endregion
 

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ReturnTypes/IntentResponseAccountRemoved.cs.meta

@@ -172,5 +172,16 @@ public Task<TransactionReturn> SendTransactionWithFeeOptions(Chain network, Tran
         /// </summary>
         /// <returns></returns>
         public string GetEmail();
+
+        public event Action<string, bool> OnFederatedAccountRemovedComplete;
+        
+        /// <summary>
+        /// Remove a federated account association
+        ///
+        /// Can be awaited directly and/or you can subscribe to the OnFederatedAccountRemovedComplete event
+        /// </summary>
+        /// <param name="dropSessionId"></param>
+        /// <returns></returns>
+        public Task<bool> RemoveFederatedAccount(Account account);
     }
 }

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/EOAWalletToSequenceWalletAdapter.cs

@@ -463,5 +463,24 @@ public string GetEmail()
         {
             return _email;
         }
+
+        public event Action<string, bool> OnFederatedAccountRemovedComplete;
+        public async Task<bool> RemoveFederatedAccount(Account account)
+        {
+            IntentDataRemoveAccount args = new IntentDataRemoveAccount(account.wallet, account.id);
+            try
+            {
+                var response =
+                    await _intentSender.SendIntent<IntentResponseAccountRemoved, IntentDataRemoveAccount>(args,
+                        IntentType.RemoveAccount);
+                OnFederatedAccountRemovedComplete?.Invoke(account.id, true);
+                return true;
+            }
+            catch (Exception e)
+            {
+                OnFederatedAccountRemovedComplete?.Invoke($"Failed to remove federated account with id {account.id}, reason: {e.Message}", false);
+                return false;
+            }
+        }
     }
 }