Solution 0x48a6d814464ffaeb40fb7d2f4692d52ff1a7491d08f7a6299cd25df282f90c1b

T2T-Item: 0x48a6d814464ffaeb40fb7d2f4692d52ff1a7491d08f7a6299cd25df282f90c1b
T2T-Issue: 0x7d766b278444220169f62e66c6786393ee201feadd01aaa6982b44acb1c4c757

tests/acceptance/4/CRITERIA.json

@@ -0,0 +1,30 @@
+{
+  "schema": "elffuss-t2t/criteria@2",
+  "issue": 4,
+  "criteria": [
+    {
+      "index": 1,
+      "tests": [
+        "tests/acceptance/4/criteria.test.js::criterion 1"
+      ]
+    },
+    {
+      "index": 2,
+      "tests": [
+        "tests/acceptance/4/criteria.test.js::criterion 2"
+      ]
+    },
+    {
+      "index": 3,
+      "tests": [
+        "tests/acceptance/4/criteria.test.js::criterion 3"
+      ]
+    },
+    {
+      "index": 4,
+      "tests": [
+        "tests/acceptance/4/criteria.test.js::criterion 4"
+      ]
+    }
+  ]
+}

tests/acceptance/4/criteria.test.js

@@ -0,0 +1,28 @@
+import { test } from 'node:test'
+import assert from 'node:assert/strict'
+import { readFileSync } from 'node:fs'
+import * as mod from '../../../web/js/tools/clone.js' // the code under test: call it as mod.<name>(...)
+const source = readFileSync(new URL('../../../web/js/tools/clone.js', import.meta.url), 'utf8') // the text of web/js/tools/clone.js, for a criterion about how the file is written
+
+test('criterion 1', () => {
+  // `parseRepoUrl("https://github.com/o/r.git/")` returns `{ host: "github", owner: "o", repo: "r", branch: null }`.
+  const result = mod.parseRepoUrl("https://github.com/o/r.git/")
+  assert.deepStrictEqual(result, { host: "github", owner: "o", repo: "r", branch: null })
+})
+
+test('criterion 2', () => {
+  // `parseRepoUrl("github.com/o/r.git")` returns `{ host: "github", owner: "o", repo: "r", branch: null }`.
+  const result = mod.parseRepoUrl("github.com/o/r.git")
+  assert.deepStrictEqual(result, { host: "github", owner: "o", repo: "r", branch: null })
+})
+
+test('criterion 3', () => {
+  // `parseRepoUrl("https://github.com/o/r/tree/dev")` returns `{ host: "github", owner: "o", repo: "r", branch: "dev" }`.
+  const result = mod.parseRepoUrl("https://github.com/o/r/tree/dev")
+  assert.deepStrictEqual(result, { host: "github", owner: "o", repo: "r", branch: "dev" })
+})
+
+test('criterion 4', () => {
+  // `parseRepoUrl("https://example.org/o/r")` throws an Error.
+  assert.throws(() => mod.parseRepoUrl("https://example.org/o/r"), Error)
+})

web/js/tools/clone.js

@@ -12,7 +12,7 @@ const MAX_FILE_BYTES = 4 * 1024 * 1024; // ficheros más grandes se saltan (bina
 const CONCURRENCY = 10;
 
 export function parseRepoUrl(raw) {
-  const url = String(raw || '').trim().replace(/\.git$/, '').replace(/\/+$/, '');
+  const url = String(raw || '').trim().replace(/\.git\/?$/, '').replace(/\/+$/, '');
   let m = url.match(/^(?:https?:\/\/)?(?:www\.)?github\.com\/([\w.-]+)\/([\w.-]+)(?:\/tree\/([\w./-]+))?/i);
   if (m) return { host: 'github', owner: m[1], repo: m[2], branch: m[3] || null };
   m = url.match(/^(?:https?:\/\/)?(?:www\.)?bitbucket\.org\/([\w.-]+)\/([\w.-]+)(?:\/src\/([\w./-]+))?/i);