worked on review comments

aws-amplify · Nov 29, 2023 · 94786f1 · 94786f1
1 parent 25043e5
commit 94786f1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/...ources/AWSCognitoAuthPlugin/Models/Options/AWSAuthConfirmSignInWithMagicLinkOptions.swift b/...ources/AWSCognitoAuthPlugin/Models/Options/AWSAuthConfirmSignInWithMagicLinkOptions.swift
@@ -6,6 +6,7 @@
 //
 import Amplify
 
+//[HS]: TODO: Make sure this is approved API for options
 public struct AWSAuthConfirmSignInWithMagicLinkOptions {
 
     /// A map of custom key-value pairs that you can provide as input for any custom workflows that this action triggers.

diff --git a/...Auth/Sources/AWSCognitoAuthPlugin/Models/Options/AWSAuthConfirmSignInWithOTPOptions.swift b/...Auth/Sources/AWSCognitoAuthPlugin/Models/Options/AWSAuthConfirmSignInWithOTPOptions.swift
@@ -6,6 +6,7 @@
 //
 import Amplify
 
+//[HS]: TODO: Make sure this is approved API for options
 public struct AWSAuthConfirmSignInWithOTPOptions {
 
     /// A map of custom key-value pairs that you can provide as input for any custom workflows that this action triggers.

diff --git a/...hPluginUnitTests/TaskTests/ClientBehaviorTests/SignIn/AWSAuthSignInWithOTPTaskTests.swift b/...hPluginUnitTests/TaskTests/ClientBehaviorTests/SignIn/AWSAuthSignInWithOTPTaskTests.swift
@@ -25,7 +25,7 @@ class AWSAuthSignInWithOTPTaskTests: BasePluginTest {
     /// - When:
     ///    - I invoke signInWithOTP
     /// - Then:
-    ///    - I should get a the info in next step
+    ///    - I should get `confirmSignInWithOTP` as the next step. 
     ///
     func testSignInWithOTP() async {