Fix import_graph depth/cycle tests and windows locking

Author: gtfierro

lib/src/api.rs

@@ -1141,7 +1141,14 @@ impl OntoEnv {
             }
             // if location is a file, add it to the list
             if resolved.is_file() && self.config.is_included(&resolved) {
-                files.insert(OntologyLocation::File(resolved.clone()));
+                if let Err(err) = std::fs::File::open(&resolved) {
+                    if self.config.strict {
+                        return Err(err.into());
+                    }
+                    warn!("Skipping {:?} due to access error: {}", resolved, err);
+                } else {
+                    files.insert(OntologyLocation::File(resolved.clone()));
+                }
                 continue;
             }
             for entry in walkdir::WalkDir::new(&resolved) {
@@ -1160,6 +1167,18 @@ impl OntoEnv {
                     }
                 };
                 if entry.file_type().is_file() && self.config.is_included(entry.path()) {
+                    // Skip unreadable files when not strict
+                    if let Err(err) = std::fs::File::open(entry.path()) {
+                        if self.config.strict {
+                            return Err(err.into());
+                        }
+                        warn!(
+                            "Skipping {:?} due to access error while opening: {}",
+                            entry.path(),
+                            err
+                        );
+                        continue;
+                    }
                     files.insert(OntologyLocation::File(entry.path().to_path_buf()));
                 }
             }

lib/tests/test_ontoenv.rs

@@ -219,8 +219,8 @@ fn import_graph_handles_cycles() -> Result<()> {
 
     let a_path = dir.path().join("A.ttl");
     let b_path = dir.path().join("B.ttl");
-    let a_iri = format!("file://{}", a_path.display());
-    let b_iri = format!("file://{}", b_path.display());
+    let a_iri = url::Url::from_file_path(&a_path).unwrap().to_string();
+    let b_iri = url::Url::from_file_path(&b_path).unwrap().to_string();
 
     fs::write(
         &a_path,
@@ -307,9 +307,9 @@ fn import_graph_respects_recursion_depth() -> Result<()> {
     let b_path = dir.path().join("B.ttl");
     let c_path = dir.path().join("C.ttl");
 
-    let a_iri = format!("file://{}", a_path.display());
-    let b_iri = format!("file://{}", b_path.display());
-    let c_iri = format!("file://{}", c_path.display());
+    let a_iri = url::Url::from_file_path(&a_path).unwrap().to_string();
+    let b_iri = url::Url::from_file_path(&b_path).unwrap().to_string();
+    let c_iri = url::Url::from_file_path(&c_path).unwrap().to_string();
 
     fs::write(
         &a_path,