fix: better check for 'is_source_file'

Author: DaniPopes

crates/compilers/src/cache.rs

@@ -670,7 +670,7 @@ impl<T: ArtifactOutput<CompilerContract = C::CompilerContract>, C: Compiler>
 {
     /// Whether given file is a source file or a test/script file.
     fn is_source_file(&self, file: &Path) -> bool {
-        !self.project.paths.is_test_or_script(file)
+        self.project.paths.is_source_file(file)
     }
 
     /// Creates a new cache entry for the file

crates/compilers/src/config.rs

@@ -251,9 +251,8 @@ impl<L> ProjectPathsConfig<L> {
     }
 
     pub(crate) fn is_test_or_script(&self, path: &Path) -> bool {
-        let test_dir = self.tests.strip_prefix(&self.root).unwrap_or(&self.tests);
-        let script_dir = self.scripts.strip_prefix(&self.root).unwrap_or(&self.scripts);
-        path.starts_with(test_dir) || path.starts_with(script_dir)
+        path_starts_with_rooted(path, &self.tests, &self.root)
+            || path_starts_with_rooted(path, &self.scripts, &self.root)
     }
 
     pub(crate) fn is_source_file(&self, path: &Path) -> bool {
@@ -984,6 +983,17 @@ impl SolcConfigBuilder {
     }
 }
 
+/// Return true if `a` starts with `b` or `b - root`.
+fn path_starts_with_rooted(a: &Path, b: &Path, root: &Path) -> bool {
+    if a.starts_with(b) {
+        return true;
+    }
+    if let Ok(b) = b.strip_prefix(root) {
+        return a.starts_with(b);
+    }
+    false
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;