Solution to #8: Let parseRepoUrl accept a repository URL that ends in .git and a slash #21
18
tests/acceptance/8/CRITERIA.json
Normal file
18
tests/acceptance/8/CRITERIA.json
Normal file
@@ -0,0 +1,18 @@
|
||||
{
|
||||
"schema": "elffuss-t2t/criteria@2",
|
||||
"issue": 8,
|
||||
"criteria": [
|
||||
{
|
||||
"index": 1,
|
||||
"tests": [
|
||||
"tests/acceptance/8/criteria.test.js::criterion 1"
|
||||
]
|
||||
},
|
||||
{
|
||||
"index": 2,
|
||||
"tests": [
|
||||
"tests/acceptance/8/criteria.test.js::criterion 2"
|
||||
]
|
||||
}
|
||||
]
|
||||
}
|
||||
24
tests/acceptance/8/criteria.test.js
Normal file
24
tests/acceptance/8/criteria.test.js
Normal file
@@ -0,0 +1,24 @@
|
||||
import test from 'node:test';
|
||||
import assert from 'node:assert/strict';
|
||||
const mod = await import('../../../web/js/tools/clone.js');
|
||||
|
||||
test('criterion 1', () => {
|
||||
const result = mod.parseRepoUrl('https://' + 'github.com/o/r.git/');
|
||||
assert.equal(result.host, 'github');
|
||||
assert.equal(result.owner, 'o');
|
||||
assert.equal(result.repo, 'r');
|
||||
assert.equal(result.branch, null);
|
||||
|
||||
const treeResult = mod.parseRepoUrl('https://' + 'github.com/o/r/tree/dev');
|
||||
assert.equal(treeResult.branch, 'dev');
|
||||
});
|
||||
|
||||
test('criterion 2', () => {
|
||||
const result = mod.parseRepoUrl('https://' + 'bitbucket.org/o/r.git/');
|
||||
assert.equal(result.host, 'bitbucket');
|
||||
assert.equal(result.owner, 'o');
|
||||
assert.equal(result.repo, 'r');
|
||||
assert.equal(result.branch, null);
|
||||
|
||||
assert.throws(() => mod.parseRepoUrl('https://' + 'example.org/o/r'), /URL no reconocida/);
|
||||
});
|
||||
@@ -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(/\/+$/, '').replace(/\.git$/, '');
|
||||
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);
|
||||
|
||||
Reference in New Issue
Block a user