fix: enable package manifest file verification during PackageManifestFile::new

forc-pkg/src/manifest.rs

@@ -238,7 +238,9 @@ impl PackageManifestFile {
     pub fn from_file(path: PathBuf) -> Result<Self> {
         let path = path.canonicalize()?;
         let manifest = PackageManifest::from_file(&path)?;
-        Ok(Self { manifest, path })
+        let manifest_file = Self { manifest, path };
+        manifest_file.validate()?;
+        Ok(manifest_file)
     }
 
     /// Read the manifest from the `Forc.toml` in the directory specified by the given `path` or
@@ -253,13 +255,14 @@ impl PackageManifestFile {
         Self::from_file(path)
     }
 
-    /// Validate the `PackageManifest`.
+    /// Validate the `PackageManifestFile`.
     ///
-    /// This checks the project and organization names against a set of reserved/restricted
-    /// keywords and patterns, and if a given entry point exists.
-    pub fn validate(&self, path: &Path) -> Result<()> {
+    /// This checks:
+    /// 1. Validity of the underlying `PackageManifet`.
+    /// 2. Existence of the entry file.
+    pub fn validate(&self) -> Result<()> {
         self.manifest.validate()?;
-        let mut entry_path = path.to_path_buf();
+        let mut entry_path = self.path.clone();
         entry_path.pop();
         let entry_path = entry_path
             .join(constants::SRC_DIR)

forc-plugins/forc-client/src/op/deploy.rs

@@ -267,8 +267,8 @@ mod test {
 
         let manifests_dir = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
             .join("test")
-            .join("data")
-            .join("manifests");
+            .join("data");
+
         for entry in manifests_dir.read_dir().unwrap() {
             let manifest =
                 PackageManifestFile::from_file(entry.unwrap().path().join("Forc.toml")).unwrap();

forc-plugins/forc-client/test/data/contract_with_dep/.gitignore

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

forc-plugins/forc-client/test/data/contract_with_dep/src/main.sw

@@ -0,0 +1,11 @@
+contract;
+
+abi MyContract {
+    fn test_function() -> bool;
+}
+
+impl MyContract for Contract {
+    fn test_function() -> bool {
+        true
+    }
+}

forc-plugins/forc-client/test/data/contract_with_dep_with_salt_conflict/.gitignore

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

forc-plugins/forc-client/test/data/contract_with_dep_with_salt_conflict/src/main.sw

@@ -0,0 +1,11 @@
+contract;
+
+abi MyContract {
+    fn test_function() -> bool;
+}
+
+impl MyContract for Contract {
+    fn test_function() -> bool {
+        true
+    }
+}

forc-plugins/forc-client/test/data/standalone_contract/.gitignore

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

forc-plugins/forc-client/test/data/standalone_contract/src/main.sw

@@ -0,0 +1,11 @@
+contract;
+
+abi MyContract {
+    fn test_function() -> bool;
+}
+
+impl MyContract for Contract {
+    fn test_function() -> bool {
+        true
+    }
+}

forc-plugins/forc-client/test/data/standalone_contract_b/.gitignore

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

forc-plugins/forc-client/test/data/standalone_contract_b/src/main.sw

@@ -0,0 +1,11 @@
+contract;
+
+abi MyContract {
+    fn test_function() -> bool;
+}
+
+impl MyContract for Contract {
+    fn test_function() -> bool {
+        true
+    }
+}