chore: Improve testing

.github/workflows/native_test.yml

@@ -24,7 +24,7 @@ jobs:
         uses: actions/cache@4504faf7e9bcf8f3ed0bc863c4e1d21499ab8ef8
         with:
           path: "~/.pub-cache/hosted"
-          key: "os:ubuntu-latest;pub-cache-hosted;sdk:dev;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen;commands:test_5"
+          key: "os:ubuntu-latest;pub-cache-hosted;sdk:dev;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen;commands:test_3"
           restore-keys: |
             os:ubuntu-latest;pub-cache-hosted;sdk:dev;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen
             os:ubuntu-latest;pub-cache-hosted;sdk:dev
@@ -71,7 +71,7 @@ jobs:
         uses: actions/cache@4504faf7e9bcf8f3ed0bc863c4e1d21499ab8ef8
         with:
           path: "~/.pub-cache/hosted"
-          key: "os:ubuntu-latest;pub-cache-hosted;sdk:stable;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen;commands:test_5"
+          key: "os:ubuntu-latest;pub-cache-hosted;sdk:stable;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen;commands:test_3"
           restore-keys: |
             os:ubuntu-latest;pub-cache-hosted;sdk:stable;packages:packages/smithy/smithy-packages/smithy/smithy_aws-packages/smithy/smithy_codegen
             os:ubuntu-latest;pub-cache-hosted;sdk:stable

packages/aft/pubspec_overrides.yaml

@@ -1,4 +1,7 @@
-dependency_overrides:
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
+  amplify_lints:
+    path: ../amplify_lints
   aws_common:
     path: ../aws_common
   aws_signature_v4:

packages/amplify_authenticator/lib/amplify_authenticator.dart

@@ -44,7 +44,6 @@ import 'package:amplify_authenticator/src/widgets/form.dart';
 import 'package:amplify_flutter/amplify_flutter.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
-import 'package:meta/meta.dart';
 
 export 'package:amplify_auth_cognito/amplify_auth_cognito.dart'
     show AuthProvider;

packages/amplify_core/build.yaml

@@ -0,0 +1,8 @@
+targets:
+  $default:
+    builders:
+      build_web_compilers:entrypoint:
+        release_options:
+          dart2js_args:
+            - --define=dart.vm.product=true
+            - --enable-asserts

packages/amplify_core/mono_pkg.yaml

@@ -8,9 +8,11 @@ stages:
           - format
           - analyze: --fatal-infos .
   - unit_test:
+    - command: dart --version
     - test:
       os:
         - windows
         - macos
         - linux
-    - test: -p chrome,firefox
+    - command: dart run build_runner test --delete-conflicting-outputs -- -p chrome,firefox
+    - command: dart run build_runner test --release --delete-conflicting-outputs -- -p chrome,firefox

packages/amplify_core/pubspec.yaml

@@ -24,6 +24,8 @@ dev_dependencies:
   amplify_lints:
     path: ../amplify_lints
   build_runner: ^2.0.0
+  build_test: ^2.1.5
+  build_web_compilers: ^3.2.4
   json_serializable: ^6.3.0
   path: any
   test: ^1.17.0

packages/amplify_core/pubspec_overrides.yaml

@@ -1,9 +1,7 @@
-# This file is checked in until it can be easily generated locally
-# and in CI pipelines, e.g. with melos.
-#
-# It may cause issues with publishing until https://github.com/dart-lang/pub/pull/3419
-# is released in stable.
-dependency_overrides:
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
+  amplify_lints:
+    path: ../amplify_lints
   aws_common:
     path: ../aws_common
   aws_signature_v4:

packages/auth/amplify_auth_cognito/example/pubspec_overrides.yaml

@@ -1,39 +1,39 @@
-# This file is checked in until it can be easily generated locally
-# and in CI pipelines, e.g. with melos.
-#
-# It may cause issues with publishing until https://github.com/dart-lang/pub/pull/3419
-# is released in stable.
-dependency_overrides:
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
   amplify_api:
     path: ../../../api/amplify_api
   amplify_api_android:
     path: ../../../api/amplify_api_android
   amplify_api_ios:
     path: ../../../api/amplify_api_ios
-  amplify_authenticator:
-    path: ../../../amplify_authenticator
   amplify_auth_cognito:
-    path: ../
+    path: ..
   amplify_auth_cognito_android:
     path: ../../amplify_auth_cognito_android
   amplify_auth_cognito_dart:
     path: ../../amplify_auth_cognito_dart
   amplify_auth_cognito_ios:
     path: ../../amplify_auth_cognito_ios
+  amplify_authenticator:
+    path: ../../../amplify_authenticator
   amplify_core:
     path: ../../../amplify_core
   amplify_datastore_plugin_interface:
     path: ../../../amplify_datastore_plugin_interface
   amplify_flutter:
     path: ../../../amplify/amplify_flutter
-  amplify_flutter_ios:
-    path: ../../../amplify/amplify_flutter_ios
   amplify_flutter_android:
     path: ../../../amplify/amplify_flutter_android
+  amplify_flutter_ios:
+    path: ../../../amplify/amplify_flutter_ios
+  amplify_lints:
+    path: ../../../amplify_lints
   amplify_secure_storage:
     path: ../../../secure_storage/amplify_secure_storage
   amplify_secure_storage_dart:
     path: ../../../secure_storage/amplify_secure_storage_dart
+  amplify_test:
+    path: ../../../amplify_test
   aws_common:
     path: ../../../aws_common
   aws_signature_v4:

packages/auth/amplify_auth_cognito/pubspec_overrides.yaml

@@ -1,9 +1,5 @@
-# This file is checked in until it can be easily generated locally
-# and in CI pipelines, e.g. with melos.
-#
-# It may cause issues with publishing until https://github.com/dart-lang/pub/pull/3419
-# is released in stable.
-dependency_overrides:
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
   amplify_auth_cognito_android:
     path: ../amplify_auth_cognito_android
   amplify_auth_cognito_dart:
@@ -14,12 +10,14 @@ dependency_overrides:
     path: ../../amplify_core
   amplify_datastore_plugin_interface:
     path: ../../amplify_datastore_plugin_interface
-  amplify_flutter_android:
-    path: ../../amplify/amplify_flutter_android
   amplify_flutter:
     path: ../../amplify/amplify_flutter
+  amplify_flutter_android:
+    path: ../../amplify/amplify_flutter_android
   amplify_flutter_ios:
     path: ../../amplify/amplify_flutter_ios
+  amplify_lints:
+    path: ../../amplify_lints
   amplify_secure_storage:
     path: ../../secure_storage/amplify_secure_storage
   amplify_secure_storage_dart:

packages/auth/amplify_auth_cognito_dart/build.yaml

@@ -34,6 +34,7 @@ targets:
           compiler: dart2js
           dart2js_args:
             - -O2
+            - --define=dart.vm.product=true
   workers:
     auto_apply_builders: false
     builders:

packages/auth/amplify_auth_cognito_dart/example/build.yaml

@@ -0,0 +1,7 @@
+targets:
+  $default:
+    builders:
+      build_web_compilers:entrypoint:
+        release_options:
+          dart2js_args:
+            - --define=dart.vm.product=true

packages/auth/amplify_auth_cognito_dart/example/pubspec_overrides.yaml

@@ -0,0 +1,22 @@
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
+  amplify_auth_cognito_dart:
+    path: ..
+  amplify_core:
+    path: ../../../amplify_core
+  amplify_lints:
+    path: ../../../amplify_lints
+  amplify_secure_storage_dart:
+    path: ../../../secure_storage/amplify_secure_storage_dart
+  aws_common:
+    path: ../../../aws_common
+  aws_signature_v4:
+    path: ../../../aws_signature_v4
+  example_common:
+    path: ../../../example_common
+  smithy:
+    path: ../../../smithy/smithy
+  smithy_aws:
+    path: ../../../smithy/smithy_aws
+  worker_bee:
+    path: ../../../worker_bee/worker_bee

packages/auth/amplify_auth_cognito_dart/lib/src/auth_plugin_impl.dart

@@ -101,7 +101,7 @@ class AmplifyAuthCognitoDart extends AuthPluginInterface
 
   @visibleForTesting
   set stateMachine(CognitoAuthStateMachine stateMachine) {
-    if (!zDebugMode) throw StateError('Can only be called in tests');
+    if (!zAssertsEnabled) throw StateError('Can only be called in tests');
     _stateMachine = stateMachine;
   }
 

packages/auth/amplify_auth_cognito_dart/mono_pkg.yaml

@@ -6,12 +6,7 @@ stages:
   - analyze_and_format:
       - group:
           - format
-          - analyze: --fatal-infos lib test
-  - unit_test:
-      - group:
-          - command: dart --version
-          - test: --exclude-tags=build
-          - test: -p chrome
+          - analyze: --fatal-infos lib
   - build_test:
       - test: --tags=build
         sdk:

packages/auth/amplify_auth_cognito_dart/pubspec_overrides.yaml

@@ -1,11 +1,9 @@
-# This file is checked in until it can be easily generated locally
-# and in CI pipelines, e.g. with melos.
-#
-# It may cause issues with publishing until https://github.com/dart-lang/pub/pull/3419
-# is released in stable.
-dependency_overrides:
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
   amplify_core:
     path: ../../amplify_core
+  amplify_lints:
+    path: ../../amplify_lints
   amplify_secure_storage_dart:
     path: ../../secure_storage/amplify_secure_storage_dart
   aws_common:
@@ -18,3 +16,5 @@ dependency_overrides:
     path: ../../smithy/smithy_aws
   worker_bee:
     path: ../../worker_bee/worker_bee
+  worker_bee_builder:
+    path: ../../worker_bee/worker_bee_builder

packages/auth/amplify_auth_cognito_test/.gitignore

@@ -0,0 +1,6 @@
+# Files and directories created by pub.
+.dart_tool/
+.packages
+
+# Conventional directory for build output.
+build/

packages/auth/amplify_auth_cognito_test/README.md

@@ -0,0 +1,11 @@
+Tests for the [`amplify_auth_cognito_dart`](../amplify_auth_cognito_dart) package separated out so that they can be easily tested using `build_runner`.
+
+## Testing
+
+To run tests:
+
+```sh
+$ dart test
+$ dart run build_runner test --delete-conflicting-outputs -- -p chrome,firefox
+$ dart run build_runner test --release --delete-conflicting-outputs -- -p chrome,firefox
+```

packages/auth/amplify_auth_cognito_test/analysis_options.yaml

@@ -0,0 +1,7 @@
+include: package:amplify_lints/library.yaml
+
+analyzer:
+  errors:
+    invalid_use_of_internal_member: ignore
+  exclude:
+    - '**/*.g.dart'

packages/auth/amplify_auth_cognito_test/build.yaml

@@ -0,0 +1,8 @@
+targets:
+  $default:
+    builders:
+      build_web_compilers:entrypoint:
+        release_options:
+          dart2js_args:
+            - --define=dart.vm.product=true
+            - --enable-asserts

packages/auth/amplify_auth_cognito_test/dart_test.yaml

@@ -0,0 +1,8 @@
+override_platforms:
+  firefox:
+    settings:
+      arguments: -headless
+on_platform:
+  # Tests with workers seems to be really slow on browsers
+  browser:
+    timeout: 2x

packages/auth/amplify_auth_cognito_test/lib/amplify_auth_cognito_test.dart

@@ -0,0 +1,2 @@
+/// Tests for the `amplify_auth_cognito_dart` package.
+library amplify_auth_cognito_test;

packages/auth/amplify_auth_cognito_test/mono_pkg.yaml

@@ -0,0 +1,22 @@
+sdk:
+  - stable
+  - dev
+
+stages:
+  - analyze_and_format:
+      - group:
+          - format
+          - analyze: --fatal-infos .
+  - unit_test:
+      - command: dart --version
+      - test:
+        os:
+          - windows
+          - macos
+          - linux
+      - command: dart run build_runner test --delete-conflicting-outputs -- -p chrome,firefox
+        sdk:
+          - stable
+          # TODO(dnys1): https://github.com/dart-lang/sdk/issues/49474
+          # - dev
+      - command: dart run build_runner test --release --delete-conflicting-outputs -- -p chrome,firefox

packages/auth/amplify_auth_cognito_test/pubspec.yaml

@@ -0,0 +1,46 @@
+name: amplify_auth_cognito_test
+description: Tests for the amplify_auth_cognito_dart package.
+publish_to: none
+
+environment:
+  sdk: '>=2.17.0 <3.0.0'
+
+dependency_overrides:
+  amplify_core:
+    path: ../../amplify_core
+  amplify_secure_storage_dart:
+    path: ../../secure_storage/amplify_secure_storage_dart
+  aws_common:
+    path: ../../aws_common
+  aws_signature_v4:
+    path: ../../aws_signature_v4
+  worker_bee:
+    path: ../../worker_bee/worker_bee
+
+dev_dependencies:
+  amplify_auth_cognito_dart:
+    path: ../amplify_auth_cognito_dart
+  amplify_core:
+    path: ../../amplify_core
+  amplify_lints:
+    path: ../../amplify_lints
+  amplify_secure_storage_dart:
+    path: ../../secure_storage/amplify_secure_storage_dart
+  build_runner: ^2.0.0
+  build_test: ^2.0.0
+  build_web_compilers: ^3.2.2
+  built_collection: ^5.0.0
+  built_value: ^8.0.0
+  convert: ^3.0.0
+  http: ^0.13.0
+  mockito: ^5.0.0
+  smithy:
+    hosted: https://pub.dillonnys.com
+    version: ^0.5.0
+  smithy_aws:
+    hosted: https://pub.dillonnys.com
+    version: ^0.5.0
+  stream_transform: ^2.0.0
+  test: ^1.16.0
+  worker_bee:
+    path: ../../worker_bee/worker_bee

packages/auth/amplify_auth_cognito_test/pubspec_overrides.yaml

@@ -0,0 +1,20 @@
+# Generated with `aft`. Do not modify by hand or check into source control.
+dependency_overrides: 
+  amplify_auth_cognito_dart:
+    path: ../amplify_auth_cognito_dart
+  amplify_core:
+    path: ../../amplify_core
+  amplify_lints:
+    path: ../../amplify_lints
+  amplify_secure_storage_dart:
+    path: ../../secure_storage/amplify_secure_storage_dart
+  aws_common:
+    path: ../../aws_common
+  aws_signature_v4:
+    path: ../../aws_signature_v4
+  smithy:
+    path: ../../smithy/smithy
+  smithy_aws:
+    path: ../../smithy/smithy_aws
+  worker_bee:
+    path: ../../worker_bee/worker_bee

packages/auth/amplify_auth_cognito_dart/test/flows/srp/srp_password_verifier_worker_test.dart → packages/auth/amplify_auth_cognito_test/test/flows/srp/srp_password_verifier_worker_test.dart

@@ -12,6 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+@OnPlatform({
+  'chrome': Skip('TODO(dnys1): Failing in CI'),
+})
+
 import 'dart:async';
 
 import 'package:amplify_auth_cognito_dart/src/flows/constants.dart';