fix: tests (#526)

Author: AmirSa12

packages/app/e2e.test.ts

@@ -75,7 +75,7 @@ beforeAll(async () => {
       shell: true,
     },
   );
-}, 70_000);
+}, 180_000);
 
 afterAll(async () => {
   await server.ensureClose();
@@ -156,7 +156,7 @@ describe.sequential.each([
     expect(process.stderr).toContain(
       "private-playground/ because the package is private",
     );
-  }, 20_000);
+  }, 60_000);
 
   it(`serves and installs playground-a for ${mode}`, async () => {
     const [owner, repo] = payload.repository.full_name.split("/");

packages/utils/index.test.ts

@@ -47,6 +47,24 @@ describe("utils", () => {
         utils.extractOwnerAndRepo("git+http://github.com/org/repo.git"),
       ).toEqual(["org", "repo"]);
     });
+
+    it("returns org and repo for GitHub shorthand", () => {
+      expect(utils.extractOwnerAndRepo("org/repo")).toEqual(["org", "repo"]);
+    });
+
+    it("returns org and repo for github: shorthand", () => {
+      expect(utils.extractOwnerAndRepo("github:org/repo")).toEqual([
+        "org",
+        "repo",
+      ]);
+    });
+
+    it("returns org and repo for URLs without .git suffix", () => {
+      expect(utils.extractOwnerAndRepo("https://github.com/org/repo")).toEqual([
+        "org",
+        "repo",
+      ]);
+    });
   });
 
   describe("extractRepository", () => {

packages/utils/index.ts

@@ -1,15 +1,25 @@
 import type { PackageManifest } from "query-registry";
 
 const githubUrlRegex =
-  /^(?:git\+)?https?:\/\/github\.com\/([^/]+)\/([^/]+)\.git$/;
+  /^(?:git\+)?https?:\/\/github\.com\/([^/]+)\/([^/]+)\/?$/;
+const githubShorthandRegex = /^(?:github:)?([^/]+)\/([^/]+)$/;
 
 export function extractOwnerAndRepo(
   repositoryUrl: string,
 ): [string, string] | null {
-  const match = repositoryUrl.match(githubUrlRegex);
+  const match =
+    repositoryUrl.match(githubUrlRegex) ??
+    repositoryUrl.match(githubShorthandRegex);
 
   if (match) {
-    return [match[1], match[2]];
+    let repo = match[2];
+    if (repo.endsWith(".git")) {
+      repo = repo.slice(0, -4);
+    } else if (repo.includes(".git")) {
+      return null;
+    }
+
+    return [match[1], repo];
   }
 
   return null;