Add more tests.

crates/migrate-turborepo/src/migrate_turborepo.rs

@@ -101,27 +101,22 @@ impl TurboMigrator {
             for dep in global_deps {
                 implicit_inputs.push(InputPath::from_str(&dep)?);
             }
-
-            self.global_config_modified = true;
         }
 
         if let Some(global_dot_env) = turbo_json.global_dot_env.take() {
             for env_file in global_dot_env {
                 implicit_inputs.push(InputPath::from_str(&env_file)?);
             }
-
-            self.global_config_modified = true;
         }
 
         if let Some(global_env) = turbo_json.global_env.take() {
             for env in global_env {
                 implicit_inputs.push(InputPath::EnvVar(env.to_owned()));
             }
-
-            self.global_config_modified = true;
         }
 
         if !implicit_inputs.is_empty() {
+            self.global_config_modified = true;
             self.global_config
                 .implicit_inputs
                 .get_or_insert(vec![])
@@ -237,7 +232,7 @@ impl TurboMigrator {
                     continue;
                 }
 
-                // task
+                // ~:task
                 deps.push(Target::parse(dep).map_err(map_miette_error)?);
             }
 

crates/migrate-turborepo/tests/__fixtures__/error-missing-project-deps/.moon/workspace.yml

@@ -0,0 +1,2 @@
+projects:
+  - "packages/*"

crates/migrate-turborepo/tests/__fixtures__/error-missing-project-deps/turbo.json

@@ -0,0 +1,8 @@
+{
+  "$schema": "https://turbo.build/schema.json",
+  "pipeline": {
+    "build": {
+      "dependsOn": ["client#build"]
+    }
+  }
+}

crates/migrate-turborepo/tests/__fixtures__/error-missing-project/.moon/workspace.yml

@@ -0,0 +1,2 @@
+projects:
+  - "packages/*"

crates/migrate-turborepo/tests/__fixtures__/error-missing-project/turbo.json

@@ -0,0 +1,8 @@
+{
+  "$schema": "https://turbo.build/schema.json",
+  "pipeline": {
+    "client#build": {
+      "dependsOn": ["^build"]
+    }
+  }
+}

crates/migrate-turborepo/tests/__fixtures__/root-merge-existing/.moon/tasks/node.yml

@@ -0,0 +1,3 @@
+tasks:
+  lint:
+    command: "eslint"

crates/migrate-turborepo/tests/__fixtures__/root-merge-existing/.moon/workspace.yml

@@ -0,0 +1,2 @@
+projects:
+  - "packages/*"

crates/migrate-turborepo/tests/__fixtures__/root-merge-existing/turbo.json

@@ -0,0 +1,9 @@
+{
+  "$schema": "https://turbo.build/schema.json",
+  "pipeline": {
+    "build": {
+      "dependsOn": ["^build"],
+      "outputs": ["dist/**"]
+    }
+  }
+}

crates/migrate-turborepo/tests/__fixtures__/root-project/.moon/workspace.yml

@@ -0,0 +1,2 @@
+projects:
+  - "packages/*"

crates/migrate-turborepo/tests/__fixtures__/root-project/turbo.json

@@ -0,0 +1,9 @@
+{
+  "$schema": "https://turbo.build/schema.json",
+  "pipeline": {
+    "//#build": {
+      "cache": true,
+      "persistent": false
+    }
+  }
+}

crates/migrate-turborepo/tests/migrate_turborepo_test.rs

@@ -42,4 +42,58 @@ mod migrate_turborepo {
         assert_snapshot!(fs::read_to_string(sandbox.path().join("client/moon.yml")).unwrap());
         assert_snapshot!(fs::read_to_string(sandbox.path().join("server/moon.yml")).unwrap());
     }
+
+    #[test]
+    fn converts_to_a_root_project() {
+        let sandbox = create_sandbox("root-project");
+        let plugin = create_extension("test", sandbox.path());
+
+        plugin.execute_extension(ExecuteExtensionInput {
+            args: vec![],
+            context: plugin.create_context(sandbox.path()),
+        });
+
+        assert!(!sandbox.path().join("turbo.json").exists());
+        assert!(!sandbox.path().join(".moon/tasks/node.yml").exists());
+        assert!(sandbox.path().join("moon.yml").exists());
+
+        assert_snapshot!(fs::read_to_string(sandbox.path().join("moon.yml")).unwrap());
+    }
+
+    #[test]
+    fn merges_with_existing_root_tasks() {
+        let sandbox = create_sandbox("root-merge-existing");
+        let plugin = create_extension("test", sandbox.path());
+
+        plugin.execute_extension(ExecuteExtensionInput {
+            args: vec![],
+            context: plugin.create_context(sandbox.path()),
+        });
+
+        assert_snapshot!(fs::read_to_string(sandbox.path().join(".moon/tasks/node.yml")).unwrap());
+    }
+
+    #[test]
+    #[should_panic(expected = "Unable to migrate task for package client.")]
+    fn errors_if_a_task_points_to_an_unknown_project() {
+        let sandbox = create_sandbox("error-missing-project");
+        let plugin = create_extension("test", sandbox.path());
+
+        plugin.execute_extension(ExecuteExtensionInput {
+            args: vec![],
+            context: plugin.create_context(sandbox.path()),
+        });
+    }
+
+    #[test]
+    #[should_panic(expected = "Unable to migrate task for package client.")]
+    fn errors_if_a_dependson_points_to_an_unknown_project() {
+        let sandbox = create_sandbox("error-missing-project-deps");
+        let plugin = create_extension("test", sandbox.path());
+
+        plugin.execute_extension(ExecuteExtensionInput {
+            args: vec![],
+            context: plugin.create_context(sandbox.path()),
+        });
+    }
 }

crates/migrate-turborepo/tests/snapshots/migrate_turborepo_test__migrate_turborepo__converts_to_a_root_project.snap

@@ -0,0 +1,10 @@
+---
+source: crates/migrate-turborepo/tests/migrate_turborepo_test.rs
+expression: "fs::read_to_string(sandbox.path().join(\"moon.yml\")).unwrap()"
+---
+language: javascript
+platform: node
+tasks:
+  build:
+    command: npm run build
+

crates/migrate-turborepo/tests/snapshots/migrate_turborepo_test__migrate_turborepo__merges_with_existing_root_tasks.snap

@@ -0,0 +1,14 @@
+---
+source: crates/migrate-turborepo/tests/migrate_turborepo_test.rs
+expression: "fs::read_to_string(sandbox.path().join(\".moon/tasks/node.yml\")).unwrap()"
+---
+tasks:
+  build:
+    command: pnpm run build
+    deps:
+    - ^:build
+    outputs:
+    - dist/**/*
+  lint:
+    command: eslint
+