forc-pkg: throw error for nested workspaces (FuelLabs#4321)

forc-pkg/src/manifest.rs

@@ -835,6 +835,10 @@ impl WorkspaceManifest {
                     member_path
                 );
             }
+            if Self::from_file(&member_path).is_ok() {
+                bail!("Unexpected nested workspace '{}'. Workspaces are currently only allowed in the project root.", member.display());
+            };
+
             let member_manifest_file = PackageManifestFile::from_file(member_path.clone())?;
             let pkg_name = member_manifest_file.manifest.project.name;
             pkg_name_to_paths

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/.gitignore

@@ -0,0 +1,2 @@
+out
+target

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/Forc.toml

@@ -0,0 +1,2 @@
+[workspace]
+members = ["test_workspace"] 

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test.toml

@@ -0,0 +1,3 @@
+category = "fail"
+
+# check: $()Unexpected nested workspace 'test_workspace'. Workspaces are currently only allowed in the project root.

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/Forc.lock

@@ -0,0 +1,14 @@
+[[package]]
+name = 'test_contract'
+source = 'member'
+dependencies = ['test_lib']
+
+[[package]]
+name = 'test_lib'
+source = 'member'
+
+[[package]]
+name = 'test_script'
+source = 'member'
+dependencies = ['test_lib']
+contract-dependencies = ['test_contract']

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/Forc.toml

@@ -0,0 +1,2 @@
+[workspace]
+members = ["test_contract", "test_script", "test_lib"] 

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_contract/.gitignore

@@ -0,0 +1,2 @@
+out
+target

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_contract/Forc.toml

@@ -0,0 +1,9 @@
+[project]
+authors = ["Fuel Labs <[email protected]>"]
+entry = "main.sw"
+license = "Apache-2.0"
+name = "test_contract"
+implicit-std = false
+
+[dependencies]
+test_lib = { path = "../test_lib/" }

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_contract/src/main.sw

@@ -0,0 +1 @@
+contract;

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_lib/.gitignore

@@ -0,0 +1,2 @@
+out
+target

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_lib/Forc.toml

@@ -0,0 +1,6 @@
+[project]
+authors = ["Fuel Labs <[email protected]>"]
+entry = "lib.sw"
+license = "Apache-2.0"
+name = "test_lib"
+implicit-std = false

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_lib/src/lib.sw

@@ -0,0 +1 @@
+library;

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_script/.gitignore

@@ -0,0 +1,2 @@
+out
+target

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_script/Forc.toml

@@ -0,0 +1,12 @@
+[project]
+authors = ["Fuel Labs <[email protected]>"]
+entry = "main.sw"
+license = "Apache-2.0"
+name = "test_script"
+implicit-std = false
+
+[dependencies]
+test_lib = { path = "../test_lib/" }
+
+[contract-dependencies]
+test_contract = { path = "../test_contract" }

test/src/e2e_vm_tests/test_programs/should_fail/forc/unexpected_nested_workspace/test_workspace/test_script/src/main.sw

@@ -0,0 +1,5 @@
+script;
+
+fn main() {
+
+}