fix: Revert "fix: failing release due to renamed repository (#878)" (#…

…887) This reverts commit 76fdaa3.
semantic-release · Jul 26, 2024 · 325952e · 325952e
1 parent 76fdaa3
commit 325952e
Show file tree

Hide file tree

Showing 4 changed files with 92 additions and 270 deletions.
diff --git a/lib/definitions/errors.js b/lib/definitions/errors.js
@@ -139,17 +139,6 @@ By default the \`repositoryUrl\` option is retrieved from the \`repository\` pro
   };
 }
 
-export function EMISMATCHGITHUBURL() {
-  return {
-    message: "The git repository URL mismatches the GitHub URL.",
-    details: `The **semantic-release** \`repositoryUrl\` option must match your GitHub URL with the format \`<GitHub_or_GHE_URL>/<owner>/<repo>.git\`.
-
-By default the \`repositoryUrl\` option is retrieved from the \`repository\` property of your \`package.json\` or the [git origin url](https://git-scm.com/book/en/v2/Git-Basics-Working-with-Remotes) of the repository cloned by your CI environment.
-
-Note: If you have recently changed your GitHub repository name or owner, update the value in **semantic-release** \`repositoryUrl\` option and the \`repository\` property of your \`package.json\` respectively to match the new GitHub URL.`,
-  };
-}
-
 export function EINVALIDPROXY({ proxy }) {
   return {
     message: "Invalid `proxy` option.",

diff --git a/lib/verify.js b/lib/verify.js
@@ -144,33 +144,6 @@ export default async function verify(pluginConfig, context, { Octokit }) {
     }
   }
 
-  // Verify if Repository Name wasn't changed
-  if (
-    owner &&
-    repo &&
-    githubToken &&
-    !errors.find(({ code }) => code === "EINVALIDPROXY") &&
-    !errors.find(({ code }) => code === "EMISSINGREPO")
-  ) {
-    const octokit = new Octokit(
-      toOctokitOptions({
-        githubToken,
-        githubUrl,
-        githubApiPathPrefix,
-        githubApiUrl,
-        proxy,
-      }),
-    );
-
-    const {
-      status,
-      data: { clone_url },
-    } = await octokit.request("GET /repos/{owner}/{repo}", { owner, repo });
-    if (status !== 200 || repositoryUrl !== clone_url) {
-      errors.push(getError("EMISMATCHGITHUBURL"));
-    }
-  }
-
   if (!githubToken) {
     errors.push(getError("ENOGHTOKEN", { owner, repo }));
   }

diff --git a/test/integration.test.js b/test/integration.test.js
@@ -24,16 +24,11 @@ test("Verify GitHub auth", async (t) => {
     repositoryUrl: `git+https://othertesturl.com/${owner}/${repo}.git`,
   };
 
-  const fetch = fetchMock.sandbox().get(
-    `https://api.github.local/repos/${owner}/${repo}`,
-    {
+  const fetch = fetchMock
+    .sandbox()
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
       permissions: { push: true },
-      clone_url: `git+https://othertesturl.com/${owner}/${repo}.git`,
-    },
-    {
-      repeat: 2,
-    },
-  );
+    });
 
   await t.notThrowsAsync(
     t.context.m.verifyConditions(
@@ -59,16 +54,11 @@ test("Verify GitHub auth with publish options", async (t) => {
     publish: { path: "@semantic-release/github" },
     repositoryUrl: `git+https://othertesturl.com/${owner}/${repo}.git`,
   };
-  const fetch = fetchMock.sandbox().get(
-    `https://api.github.local/repos/${owner}/${repo}`,
-    {
+  const fetch = fetchMock
+    .sandbox()
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
       permissions: { push: true },
-      clone_url: `git+https://othertesturl.com/${owner}/${repo}.git`,
-    },
-    {
-      repeat: 2,
-    },
-  );
+    });
 
   await t.notThrowsAsync(
     t.context.m.verifyConditions(
@@ -101,16 +91,11 @@ test("Verify GitHub auth and assets config", async (t) => {
     publish: [{ path: "@semantic-release/npm" }],
     repositoryUrl: `git+https://othertesturl.com/${owner}/${repo}.git`,
   };
-  const fetch = fetchMock.sandbox().get(
-    `https://api.github.local/repos/${owner}/${repo}`,
-    {
+  const fetch = fetchMock
+    .sandbox()
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
       permissions: { push: true },
-      clone_url: `git+https://othertesturl.com/${owner}/${repo}.git`,
-    },
-    {
-      repeat: 2,
-    },
-  );
+    });
 
   await t.notThrowsAsync(
     t.context.m.verifyConditions(
@@ -211,16 +196,9 @@ test("Publish a release with an array of assets", async (t) => {
 
   const fetch = fetchMock
     .sandbox()
-    .get(
-      `https://api.github.local/repos/${owner}/${repo}`,
-      {
-        permissions: { push: true },
-        clone_url: `https://github.com/${owner}/${repo}.git`,
-      },
-      {
-        repeat: 2,
-      },
-    )
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
+      permissions: { push: true },
+    })
     .postOnce(
       `https://api.github.local/repos/${owner}/${repo}/releases`,
       { upload_url: uploadUrl, html_url: releaseUrl, id: releaseId },
@@ -310,16 +288,9 @@ test("Publish a release with release information in assets", async (t) => {
 
   const fetch = fetchMock
     .sandbox()
-    .get(
-      `https://api.github.local/repos/${owner}/${repo}`,
-      {
-        permissions: { push: true },
-        clone_url: `https://github.com/${owner}/${repo}.git`,
-      },
-      {
-        repeat: 2,
-      },
-    )
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
+      permissions: { push: true },
+    })
     .postOnce(
       `https://api.github.local/repos/${owner}/${repo}/releases`,
       { upload_url: uploadUrl, html_url: releaseUrl, id: releaseId },
@@ -387,16 +358,9 @@ test("Update a release", async (t) => {
 
   const fetch = fetchMock
     .sandbox()
-    .get(
-      `https://api.github.local/repos/${owner}/${repo}`,
-      {
-        permissions: { push: true },
-        clone_url: `https://github.com/${owner}/${repo}.git`,
-      },
-      {
-        repeat: 2,
-      },
-    )
+    .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
+      permissions: { push: true },
+    })
     .getOnce(
       `https://api.github.local/repos/${owner}/${repo}/releases/tags/${nextRelease.gitTag}`,
       { id: releaseId },
@@ -462,10 +426,10 @@ test("Comment and add labels on PR included in the releases", async (t) => {
       {
         permissions: { push: true },
         full_name: `${owner}/${repo}`,
-        clone_url: `https://github.com/${owner}/${repo}.git`,
       },
       {
-        repeat: 3,
+        // TODO: why do we call the same endpoint twice?
+        repeat: 2,
       },
     )
     .postOnce("https://api.github.local/graphql", {
@@ -565,10 +529,9 @@ test("Open a new issue with the list of errors", async (t) => {
       {
         permissions: { push: true },
         full_name: `${owner}/${repo}`,
-        clone_url: `https://github.com/${owner}/${repo}.git`,
       },
       {
-        repeat: 3,
+        repeat: 2,
       },
     )
     .getOnce(
@@ -662,10 +625,9 @@ test("Verify, release and notify success", async (t) => {
       {
         permissions: { push: true },
         full_name: `${owner}/${repo}`,
-        clone_url: `https://github.com/${owner}/${repo}.git`,
       },
       {
-        repeat: 3,
+        repeat: 2,
       },
     )
     .postOnce(
@@ -823,10 +785,9 @@ test("Verify, update release and notify success", async (t) => {
       {
         permissions: { push: true },
         full_name: `${owner}/${repo}`,
-        clone_url: `https://github.com/${owner}/${repo}.git`,
       },
       {
-        repeat: 3,
+        repeat: 2,
       },
     )
     .getOnce(
@@ -956,10 +917,9 @@ test("Verify and notify failure", async (t) => {
       {
         permissions: { push: true },
         full_name: `${owner}/${repo}`,
-        clone_url: `https://github.com/${owner}/${repo}.git`,
       },
       {
-        repeat: 3,
+        repeat: 2,
       },
     )
     .getOnce(