fix(api): Remove optional cleanUp() method from AWSAPIPlugin.reset() (#2585)

thisisabhash · web-flow · commit 48b60e959861 · 2023-03-06T09:41:55.000-08:00
* fix(api): Use async APIs in GraphQL IAM integration tests and remove cleanUp() from AWSAPIPlugin.reset()

* Remove unused code
diff --git a/AmplifyPlugins/API/Sources/AWSAPIPlugin/AWSAPIPlugin+Resettable.swift b/AmplifyPlugins/API/Sources/AWSAPIPlugin/AWSAPIPlugin+Resettable.swift
@@ -7,7 +7,6 @@
 
 import Amplify
 import Foundation
-// import AwsCommonRuntimeKit
 
 extension AWSAPIPlugin: Resettable {
 
@@ -27,9 +26,6 @@ extension AWSAPIPlugin: Resettable {
         }
 
         subscriptionConnectionFactory = nil
-
-        // Issue: https://github.com/aws-amplify/amplify-ios/issues/2120
-        // AwsCommonRuntimeKit.cleanUp()        
     }
 
 }
diff --git a/AmplifyPlugins/API/Tests/APIHostApp/AWSAPIPluginGraphQLIAMTests/README.md b/AmplifyPlugins/API/Tests/APIHostApp/AWSAPIPluginGraphQLIAMTests/README.md
@@ -44,7 +44,7 @@ Do you want to use the default authentication and security configuration?
  Please enter a name for your identity pool. 
     <amplifyintegtestCIDP>
  Allow unauthenticated logins? (Provides scoped down permissions that you can control via AWS IAM) 
-    No
+    Yes
  Do you want to enable 3rd party authentication providers in your identity pool? 
     No
  Please provide a name for your user pool: 

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,6 @@`
`7`	`7`
`8`	`8`	`import Amplify`
`9`	`9`	`import Foundation`
`10`		`-// import AwsCommonRuntimeKit`
`11`	`10`
`12`	`11`	`extension AWSAPIPlugin: Resettable {`
`13`	`12`
`@@ -27,9 +26,6 @@ extension AWSAPIPlugin: Resettable {`
`27`	`26`	`}`
`28`	`27`
`29`	`28`	`subscriptionConnectionFactory = nil`
`30`		`-`
`31`		`- // Issue: https://github.com/aws-amplify/amplify-ios/issues/2120`
`32`		`- // AwsCommonRuntimeKit.cleanUp()`
`33`	`29`	`}`
`34`	`30`
`35`	`31`	`}`