chore: better napi test.mjs usage (#19)

JounQin · JounQin · commit 2a6741252b64 · 2025-03-18T18:55:11.000+08:00
diff --git a/napi/test.mjs b/napi/test.mjs
@@ -4,18 +4,18 @@ import resolve, { ResolverFactory } from './index.js';
 
 console.log(`Testing on ${process.platform}-${process.arch}`);
 
-const cwd = process.cwd();
+const dir = import.meta.dirname;
 
 // `resolve`
-assert.deepStrictEqual(resolve.sync(cwd, './index.js').path, path.join(cwd, 'index.js'));
+assert.deepStrictEqual(resolve.sync(dir, './index.js').path, path.join(dir, 'index.js'));
 
 // `ResolverFactory`
 const resolver = new ResolverFactory();
-assert.deepStrictEqual(resolver.sync(cwd, './index.js').path, path.join(cwd, 'index.js'));
+assert.deepStrictEqual(resolver.sync(dir, './index.js').path, path.join(dir, 'index.js'));
 
-assert.strict(resolver.sync(cwd, './ts').error.length > 0);
+assert.strict(resolver.sync(dir, './ts').error.length > 0);
 
-resolver.async(cwd, './ts')
+resolver.async(dir, './ts')
   .then((result) => assert.strict(result.error.length > 0));
 
 const newResolver = resolver.cloneWithOptions({});
@@ -29,4 +29,4 @@ const resolver2 = new ResolverFactory(
 );
 
 // After add `.ts` extension, resolver can resolve `ts` as `ts.ts` now
-assert.deepStrictEqual(resolver2.sync(cwd, './test.mjs').path, path.join(cwd, 'test.mjs'));
+assert.deepStrictEqual(resolver2.sync(dir, './test.mjs').path, path.join(dir, 'test.mjs'));
diff --git a/src/file_system.rs b/src/file_system.rs
@@ -295,7 +295,7 @@ impl FileSystem for FileSystemOs {
                         VPath::Virtual(info) => dunce::canonicalize(info.physical_base_path()),
                         VPath::Native(path) => dunce::canonicalize(path),
                     }
-            }
+                }
             }
         }
 
diff --git a/src/lib.rs b/src/lib.rs
@@ -148,7 +148,6 @@ impl<C: Cache + Default> Default for ResolverGeneric<C> {
 impl<C: Cache + Default> ResolverGeneric<C> {
     #[must_use]
     pub fn new(options: ResolveOptions) -> Self {
-        println!("enable_pnp: {:?}", options.enable_pnp);
         Self {
             options: options.sanitize(),
             cache: Arc::new(C::default()),

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ impl FileSystem for FileSystemOs {`
`295`	`295`	`VPath::Virtual(info) => dunce::canonicalize(info.physical_base_path()),`
`296`	`296`	`VPath::Native(path) => dunce::canonicalize(path),`
`297`	`297`	`}`
`298`		`- }`
	`298`	`+ }`
`299`	`299`	`}`
`300`	`300`	`}`
`301`	`301`