fix: ignore case when checking for repo rename (#903)

jedwards1211 · web-flow · commit 2b1b9b65dcc1 · 2024-08-20T12:22:11.000-07:00
fix #901
diff --git a/lib/verify.js b/lib/verify.js
@@ -109,7 +109,10 @@ export default async function verify(pluginConfig, context, { Octokit }) {
       } = await octokit.request("GET /repos/{owner}/{repo}", { repo, owner });
       // Verify if Repository Name wasn't changed
       const parsedCloneUrl = parseGithubUrl(clone_url);
-      if (owner !== parsedCloneUrl.owner || repo !== parsedCloneUrl.repo) {
+      if (
+        `${owner}/${repo}`.toLowerCase() !==
+        `${parsedCloneUrl.owner}/${parsedCloneUrl.repo}`.toLowerCase()
+      ) {
         errors.push(
           getError("EMISMATCHGITHUBURL", { repositoryUrl, clone_url }),
         );
diff --git a/test/verify.test.js b/test/verify.test.js
@@ -1269,6 +1269,40 @@ test("Throw SemanticReleaseError if the repository doesn't exist", async (t) =>
   t.true(fetch.done());
 });
 
+test(`Don't throw an error if owner/repo only differs in case`, async (t) => {
+  const env = { GH_TOKEN: "github_token" };
+
+  const fetch = fetchMock.sandbox().getOnce(
+    `https://api.github.local/repos/org/foo`,
+    {
+      permissions: { push: true },
+      clone_url: `https://github.com/ORG/FOO.git`,
+    },
+    { repeat: 2 },
+  );
+
+  await t.notThrowsAsync(
+    verify(
+      {},
+      {
+        env,
+        options: {
+          repositoryUrl: `https://github.com/org/foo.git`,
+        },
+        logger: t.context.logger,
+      },
+      {
+        Octokit: TestOctokit.defaults((options) => ({
+          ...options,
+          request: { ...options.request, fetch },
+        })),
+      },
+    ),
+  );
+
+  t.true(fetch.done());
+});
+
 const urlFormats = [
   (owner, repo) => `https://github.com/${owner}/${repo}.git`,
   (owner, repo) => `git+https://github.com/${owner}/${repo}.git`,
