fix(plugin-npm): normalize registry (#5582)

**What's the problem this PR addresses?**

While working on #5581 I noticed
that we're not normalizing the registry passed to the `npmHttpUtils`
methods.
There are tests that are supposed to catch this but the mock wasn't
reset before each test so they didn't.

**How did you fix it?**

Normalize the registry and update the tests to use `wrapNetworkRequest`.

**Checklist**
- [x] I have read the [Contributing
Guide](https://yarnpkg.com/advanced/contributing).
- [x] I have set the packages that need to be released for my changes to
be effective.
- [x] I will check that all automated PR checks pass before the PR gets
reviewed.

.yarn/versions/4a8968a1.yml

@@ -0,0 +1,24 @@
+releases:
+  "@yarnpkg/cli": patch
+  "@yarnpkg/plugin-npm": patch
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-constraints"
+  - "@yarnpkg/plugin-dlx"
+  - "@yarnpkg/plugin-essentials"
+  - "@yarnpkg/plugin-init"
+  - "@yarnpkg/plugin-interactive-tools"
+  - "@yarnpkg/plugin-nm"
+  - "@yarnpkg/plugin-npm-cli"
+  - "@yarnpkg/plugin-pack"
+  - "@yarnpkg/plugin-patch"
+  - "@yarnpkg/plugin-pnp"
+  - "@yarnpkg/plugin-pnpm"
+  - "@yarnpkg/plugin-stage"
+  - "@yarnpkg/plugin-typescript"
+  - "@yarnpkg/plugin-version"
+  - "@yarnpkg/plugin-workspace-tools"
+  - "@yarnpkg/builder"
+  - "@yarnpkg/core"
+  - "@yarnpkg/doctor"

packages/plugin-npm/sources/npmHttpUtils.ts

@@ -353,7 +353,7 @@ function normalizeRegistry(configuration: Configuration, {ident, registry}: Part
   if (typeof registry !== `string`)
     throw new Error(`Assertion failed: The registry should be a string`);
 
-  return registry;
+  return npmConfigUtils.normalizeRegistry(registry);
 }
 
 async function getAuthenticationHeader(registry: string, {authType = AuthType.CONFIGURATION, configuration, ident}: {authType?: AuthType, configuration: Configuration, ident: RegistryOptions['ident']}) {

packages/plugin-npm/tests/npmHttpUtils.test.js → packages/plugin-npm/tests/npmHttpUtils.test.ts

@@ -1,16 +1,7 @@
-import {httpUtils}         from '@yarnpkg/core';
 import {npmHttpUtils}      from '@yarnpkg/plugin-npm';
 
 import {makeConfiguration} from './_makeConfiguration';
 
-jest.mock(`@yarnpkg/core`, () => ({
-  ...jest.requireActual(`@yarnpkg/core`),
-  httpUtils: {
-    ...jest.requireActual(`@yarnpkg/core`).httpUtils,
-    get: jest.fn(() => Promise.resolve()),
-  },
-}));
-
 describe(`npmHttpUtils.get`, () => {
   for (const registry of [`https://example.org`, `https://example.org/`, `https://example.org/foo`, `https://example.org/foo/`]) {
     for (const path of [`/bar`]) {
@@ -19,12 +10,18 @@ describe(`npmHttpUtils.get`, () => {
       it(`should craft the final path correctly (${registry} + ${path} = ${expected})`, async () => {
         const configuration = await makeConfiguration();
 
+        let actualTarget: string | undefined;
         await npmHttpUtils.get(path, {
           configuration,
           registry,
+          async wrapNetworkRequest(executor, extra) {
+            actualTarget = extra.target.toString();
+
+            return () => Promise.resolve({body: {}, headers: {}, statusCode: 200});
+          },
         });
 
-        expect(httpUtils.get).toHaveBeenCalledWith(expected, expect.anything());
+        expect(actualTarget).toEqual(expected);
       });
     }
   }