chore: testing on CI

Author: JounQin

.github/workflows/ci.yml

@@ -14,8 +14,8 @@ jobs:
     strategy:
       matrix:
         os: 
-          - ubuntu-latest
-          - macos-latest
+          # - ubuntu-latest
+          # - macos-latest
           - windows-latest
     runs-on: ${{ matrix.os }}
     steps:
@@ -35,4 +35,4 @@ jobs:
       run: cargo build
 
     - name: Run tests
-      run: cargo test
+      run: cargo test -- --nocapture

src/lib.rs

@@ -292,12 +292,26 @@ pub fn init_pnp_manifest<P: AsRef<Path>>(manifest: &mut Manifest, p: P) {
 
     for (name, ranges) in manifest.package_registry_data.iter_mut() {
         for (reference, info) in ranges.iter_mut() {
+            println!(
+                "manifest.manifest_dir, {}",
+                manifest.manifest_dir.to_string_lossy()
+            );
+
             let package_location = manifest.manifest_dir.join(info.package_location.clone());
 
+            println!("package_location: {}", package_location.to_string_lossy());
+
             let normalized_location = util::normalize_path(&package_location.to_string_lossy());
 
+            println!("normalized_location: {}", normalized_location);
+
             info.package_location = PathBuf::from(normalized_location);
 
+            println!(
+                "info.package_location: {}",
+                info.package_location.to_string_lossy()
+            );
+
             if !info.discard_from_lookup {
                 manifest.location_trie.insert(
                     &info.package_location,
@@ -349,6 +363,12 @@ pub fn find_locator<'a, P: AsRef<Path>>(
         }
     }
 
+    println!("path {}", path.as_ref().to_string_lossy());
+
+    let path = util::normalize_path(path.as_ref().to_string_lossy());
+
+    println!("path {}", path);
+
     manifest.location_trie.get_ancestor_value(&path)
 }
 

src/lib_tests.rs

@@ -23,7 +23,7 @@ mod tests {
     use super::*;
     use crate::{
         init_pnp_manifest, load_pnp_manifest, parse_bare_identifier, resolve_to_unqualified,
-        resolve_to_unqualified_via_manifest, ResolutionHost,
+        resolve_to_unqualified_via_manifest, util, ResolutionHost,
     };
 
     #[test]
@@ -191,11 +191,12 @@ mod tests {
         )
         .unwrap();
 
-        let global_cache = dirs::home_dir()
-            .unwrap()
-            .join(".yarn")
-            .join("berry")
-            .join("cache");
+        let home_dir = dirs::home_dir().unwrap();
+
+        #[cfg(windows)]
+        let global_cache = home_dir.join("AppData\\Local\\Yarn\\Berry");
+        #[cfg(not(windows))]
+        let global_cache = home_dir.join(".yarn/berry/cache");
 
         let result = resolve_to_unqualified_via_manifest(
             &manifest,
@@ -210,12 +211,15 @@ mod tests {
         match result {
             Ok(Resolution::Resolved(path, subpath)) => {
                 assert_eq!(
-                    path,
-                    global_cache
-                        .join("source-map-npm-0.6.1-1a3621db16-10c0.zip")
-                        .join("node_modules")
-                        .join("source-map")
-                        .join("")
+                    path.to_string_lossy(),
+                    util::normalize_path(
+                        global_cache
+                            .join("source-map-npm-0.6.1-1a3621db16-10c0.zip")
+                            .join("node_modules")
+                            .join("source-map")
+                            .join("")
+                            .to_string_lossy()
+                    )
                 );
                 assert_eq!(subpath, None);
             }

src/util.rs

@@ -3,7 +3,7 @@ use serde::{de::Error, Deserialize, Deserializer};
 use std::borrow::Cow;
 
 use path_slash::PathBufExt;
-use std::path::{Path, PathBuf};
+use std::path::{Path, PathBuf, MAIN_SEPARATOR_STR};
 
 #[derive(Debug, Default, Clone)]
 pub struct Trie<T> {
@@ -39,7 +39,9 @@ pub fn normalize_path<P: AsRef<str>>(original: P) -> String {
     let p = PathBuf::from(original_str);
     let mut str = clean_path::clean(p).to_slash_lossy().to_string();
 
-    if original_str.ends_with('/') && !str.ends_with('/') {
+    if (original_str.ends_with('/') || original_str.ends_with(MAIN_SEPARATOR_STR))
+        && !str.ends_with('/')
+    {
         str.push('/');
     }