refactor: allow creating multiple migrations as part of test setup.

SandPod · Nov 17, 2023 · 73d3199 · 73d3199
1 parent 730c7b3
commit 73d3199
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 20 deletions.
diff --git a/tests/serverpod_test_server/lib/test_util/migration_test_utils.dart b/tests/serverpod_test_server/lib/test_util/migration_test_utils.dart
@@ -5,16 +5,20 @@ import 'package:path/path.dart' as path;
 import 'package:serverpod_cli/src/migrations/migration_registry.dart';
 import 'package:serverpod_service_client/serverpod_service_client.dart';
 import 'package:serverpod/protocol.dart' as serverProtocol;
+import 'package:uuid/uuid.dart';
 
 abstract class MigrationTestUtils {
   static Future<void> createInitialState({
-    required Map<String, String> protocols,
+    required List<Map<String, String>> migrationProtocols,
     String tag = 'test',
   }) async {
-    assert(
-      await createMigrationFromProtocols(protocols: protocols, tag: tag) == 0,
-      'Failed to create migration.',
-    );
+    for (var protocols in migrationProtocols) {
+      assert(
+        await createMigrationFromProtocols(protocols: protocols, tag: tag) == 0,
+        'Failed to create migration.',
+      );
+    }
+
     assert(
       await runApplyMigrations() == 0,
       'Failed to create migration.',
@@ -38,12 +42,13 @@ abstract class MigrationTestUtils {
       protocolFile.writeAsStringSync(contents);
     });
 
+    var suffixedTag = '$tag-${Uuid().v4()}';
     return await _runProcess(
       'serverpod',
       arguments: [
         'create-migration',
         '--tag',
-        tag,
+        suffixedTag,
         if (force) '--force',
       ],
     );

diff --git a/tests/serverpod_test_server/test_e2e_migrations/migrations_roundtrip_test.dart b/tests/serverpod_test_server/test_e2e_migrations/migrations_roundtrip_test.dart
@@ -152,7 +152,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = <String, String>{};
       var createMigrationExitCode =
@@ -191,7 +191,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = <String, String>{};
       var createMigrationExitCode =
@@ -246,7 +246,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var addedColumn = 'addedColumn';
       var targetStateProtocols = {
@@ -322,7 +322,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -395,7 +395,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -438,7 +438,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var addedColumn = 'addedColumn';
       var targetStateProtocols = {
@@ -485,7 +485,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var addedColumn = 'addedColumn';
       var targetStateProtocols = {
@@ -562,7 +562,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -646,7 +646,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -729,7 +729,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -774,7 +774,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var addedIndex = 'migrated_table_index';
       var targetStateProtocols = {
@@ -857,7 +857,7 @@ indexes:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -930,7 +930,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''
@@ -1002,7 +1002,7 @@ fields:
 '''
       };
       await MigrationTestUtils.createInitialState(
-          protocols: initialStateProtocols, tag: tag);
+          migrationProtocols: [initialStateProtocols], tag: tag);
 
       var targetStateProtocols = {
         'migrated_table': '''