fix: fetchPartialPackument fallback is not done right

raineorshine · Aug 7, 2024 · 6a7de59 · 6a7de59
1 parent ed0767b
commit 6a7de59
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/package-managers/npm.ts b/src/package-managers/npm.ts
@@ -117,7 +117,7 @@ const fetchPartialPackument = async (
     }
 
     // possible that corgis are not supported by this registry
-    return fetchPartialPackument(name, fields, tag, { opts, fullMetadata: true })
+    return fetchPartialPackument(name, fields, tag, { ...opts, fullMetadata: true }, version)
   }
 }
 

diff --git a/test/package-managers/npm/index.test.ts b/test/package-managers/npm/index.test.ts
@@ -34,6 +34,7 @@ describe('npm', function () {
 
   it('getEngines', async () => {
     await npm.getEngines('del', '2.0.0').should.eventually.deep.equal({ node: '>=0.10.0' })
-    await npm.getEngines('ncu-test-return-version', '1.0').should.eventually.deep.equal({})
+    await npm.getEngines('ncu-test-return-version', '1.0.0').should.eventually.deep.equal({})
+    await npm.getEngines('ncu-test-return-version', '1.0').should.eventually.be.rejectedWith('404 Not Found - GET https://registry.npmjs.org/ncu-test-return-version/1.0')
   })
 })
-Original file line number
+Diff line change
@@ Expand Up / @@ -117,7 +117,7 @@ const fetchPartialPackument = async ( @@
         }
         // possible that corgis are not supported by this registry
-        return fetchPartialPackument(name, fields, tag, { opts, fullMetadata: true })
+        return fetchPartialPackument(name, fields, tag, { ...opts, fullMetadata: true }, version)
       }
     }
@@ Expand Down @@