test(nextcloud,dynamite_end_to_end_test): migrate to dart test configuration

Signed-off-by: Nikolas Rimikis <leptopoda@users.noreply.github.com>

.github/workflows/dart.yml

@@ -46,4 +46,5 @@ jobs:
           if [ -z "$packages" ]; then exit 0; fi
           packages_glob="$(printf ",%s" "${packages[@]}")"
           packages_glob="${packages_glob:1}"
-          melos exec --scope="$packages_glob" --concurrency=1 --fail-fast --dir-exists=test -- "flutter test --concurrency=$(nproc --all)"
+          melos exec --scope="$packages_glob" --concurrency=1 --fail-fast --dir-exists=test -- "flutter test --concurrency=$(nproc --all) -x source_verification"
+          melos exec --scope="$packages_glob" --concurrency=1 --fail-fast --file-exists=test/ensure_validity_test.dart -- "flutter test --concurrency=1 -t source_verification"

melos.yaml

@@ -22,6 +22,7 @@ scripts:
   format: dart format --fix --line-length 120 .
   format:check: dart format --output=none --set-exit-if-changed --line-length 120 .
   analyze: dart analyze --fatal-infos .
-  test: melos exec --concurrency=1 --fail-fast --dir-exists=test -- "flutter test --concurrency=$(nproc --all)"
+  test: melos exec --concurrency=1 --fail-fast --dir-exists=test -- "flutter test --concurrency=$(nproc --all) -x source_verification"
+  test:source_verification: melos exec --concurrency=1 --fail-fast --file-exists=test/ensure_validity_test.dart -- "flutter test --concurrency=1 -t source_verification"
   generate:neon:build_runner: melos exec --scope="neon*" --file-exists="build.yaml" -- dart run build_runner build --delete-conflicting-outputs && melos run format
   generate:neon:l10n: melos exec --flutter --dir-exists="lib/l10n" flutter gen-l10n && melos run format

packages/dynamite/dynamite_end_to_end_test/dart_test.yaml

@@ -0,0 +1 @@
+include: ../../../tool/dart_test_base.yaml

packages/dynamite/dynamite_end_to_end_test/test/ensure_validity_test.dart

@@ -8,6 +8,6 @@ void main() {
       packageRelativeDirectory: 'packages/dynamite/dynamite_end_to_end_test',
       customCommand: ['bash', 'generate.sh'],
     ),
-    timeout: const Timeout.factor(20),
+    tags: 'source_verification',
   );
 }

packages/nextcloud/dart_test.yaml

@@ -0,0 +1 @@
+include: ../../tool/dart_test_base.yaml

packages/nextcloud/test/core_test.dart

@@ -177,7 +177,6 @@ void main() {
         );
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/dashboard_test.dart

@@ -39,7 +39,6 @@ void main() {
         });
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/helper.dart

@@ -8,9 +8,6 @@ import 'package:process_run/cmd_run.dart';
 import 'package:test/test.dart';
 import 'package:universal_io/io.dart';
 
-const retryCount = 3;
-const timeout = Timeout(Duration(seconds: 30));
-
 class DockerContainer {
   DockerContainer({
     required this.id,

packages/nextcloud/test/news_test.dart

@@ -426,8 +426,7 @@ void main() {
         expect(response.body.items, hasLength(0));
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }
 

packages/nextcloud/test/notes_test.dart

@@ -178,7 +178,6 @@ void main() {
         expect(response.body.noteMode, notes.Settings_NoteMode.preview);
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/notifications_test.dart

@@ -140,7 +140,6 @@ void main() {
         await client.notifications.push.removeDevice();
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/provisioning_api_test.dart

@@ -61,7 +61,6 @@ void main() {
         });
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/settings_test.dart

@@ -6,11 +6,10 @@ import 'package:test/test.dart';
 import 'helper.dart';
 
 Future<void> main() async {
-  await run(await getDockerImage());
-}
-
-Future<void> run(final DockerImage image) async {
   group('settings', () {
+    late DockerImage image;
+    setUpAll(() async => image = await getDockerImage());
+
     late DockerContainer container;
     late TestNextcloudClient client;
     setUp(() async {

packages/nextcloud/test/uppush_test.dart

@@ -85,7 +85,6 @@ void main() {
 
       // Deleting an app, sending a notification (also via matrix gateway) or listening for notifications is not possible because redis is not set up
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/user_status_test.dart

@@ -197,7 +197,6 @@ void main() {
         expect(response.body.ocs.data.statusIsUserDefined, false);
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

packages/nextcloud/test/webdav_test.dart

@@ -530,7 +530,6 @@ void main() {
         // props: Most of them are either not applicable or hard/impossible to implement because we don't allow just writing any props
       });
     },
-    retry: retryCount,
-    timeout: timeout,
+    tags: 'integration',
   );
 }

tool/dart_test_base.yaml

@@ -0,0 +1,5 @@
+tags:
+  integration:
+    retry: 3
+  source_verification:
+    timeout: 20x