fix: remove dup licenses acceptance (aws-amplify#933)

* fix: remove dup licenses acceptance

* chore: add comments to clarify command behaviors

* chore: add comment

Author: Siqi-Shan

.codebuild/run_android_modelgen_e2e_test.yml

@@ -17,8 +17,9 @@ phases:
       - yes | sudo apt install android-sdk
       - export ANDROID_HOME=/usr/lib/android-sdk
       - yes | sudo apt install sdkmanager
-      # Review SDK licenses
-      - yes | sudo sdkmanager --licenses
+      # Licenses acceptance is needed for Android project build.
+      # "sdkmanager --licenses" is handled in test beforeAll setup,
+      # and this ensures the project accepts all licenses needed.
 
   build:
     commands:

.codebuild/run_canary_android_modelgen_e2e_test.yml

@@ -15,8 +15,9 @@ phases:
       - yes | sudo apt install android-sdk
       - export ANDROID_HOME=/usr/lib/android-sdk
       - yes | sudo apt install sdkmanager
-      # Review SDK licenses
-      - yes | sudo sdkmanager --licenses
+      # Licenses acceptance is needed for Android project build.
+      # "sdkmanager --licenses" is handled in test beforeAll setup,
+      # and this ensures the project accepts all licenses needed.
 
   build:
     commands:

packages/amplify-codegen-e2e-tests/src/__tests__/build-app-android.test.ts

@@ -38,6 +38,8 @@ describe('build app - Android', () => {
     await addCodegen(projectRoot, {
       frontendType: AmplifyFrontend.android,
     });
+
+    // Required to accept all licenses needed per this test project build
     await acceptLicenses(projectRoot);
   });