forked from heavy-duty/stoke
Fixes found during a full audit of the CLI: - auth logout: remote token revocation always failed with 401 because Forgejo only accepts Basic auth on the token endpoints. Logout now asks for (or accepts) the account password, supports --password, --password-file and --local-only, and clearly reports when the token is left active. - Silent password prompt actually echoed the password on a TTY: overriding rl.write does not suppress readline echo. Switched to the callback readline module and mute _writeToOutput instead (the readline/promises interface does not honor that hook). - Global --config flag was silently ignored: config paths were resolved at require time, before the preAction hook set STOKE_CONFIG_FILE. Paths are now resolved lazily on every access. - XDG_CONFIG_HOME handling put the config in $XDG_CONFIG_HOME/.config/stoke; per the XDG spec it now resolves to $XDG_CONFIG_HOME/stoke. - repo create: --auto-init defaulted to true with no way to disable it; added --no-auto-init. - repo import/import-batch: a GitHub token was required even for non-GitHub services (e.g. --service git), making those imports fail without gh/GITHUB_TOKEN. Tokens are now only auto-resolved for the github service; batch imports resolve per entry and memoize. - Branding leftovers: 'Run: forgejo auth login' hint and forgejo-cli/1.0.0 User-Agent now say stoke (UA tracks pkg.version). - Added request timeouts (30s default, 10m for migrations). - --limit and --team-id are validated as integers instead of silently misbehaving on garbage (NaN made -l show all results). New commands (per the repo's every-operation-becomes-a-command design): - stoke issue create (title/body/body-file/assignees) - stoke pr create (head/base/title/body/body-file) Tests and metadata: - New test suite on the built-in node:test runner (25 tests) covering config resolution/persistence, the API client with a mocked fetch, and end-to-end CLI behavior. npm test previously matched no files. - package.json: engines >=22.12.0 (required by commander@15 — the README claimed Node 18), repository, keywords, author; version 1.1.0. - README: corrected Node requirement, documented repo rename (was missing), issue create, pr create, logout options and revocation caveat, --no-auto-init, XDG behavior, import token rules, testing. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
134 lines
5.4 KiB
JavaScript
134 lines
5.4 KiB
JavaScript
const { test, afterEach } = require('node:test');
|
|
const assert = require('node:assert/strict');
|
|
|
|
const { ForgejoClient } = require('../src/api');
|
|
const pkg = require('../package.json');
|
|
|
|
const realFetch = global.fetch;
|
|
|
|
afterEach(() => {
|
|
global.fetch = realFetch;
|
|
});
|
|
|
|
function mockFetch(handler) {
|
|
const calls = [];
|
|
global.fetch = async (url, opts) => {
|
|
calls.push({ url, opts });
|
|
return handler(url, opts, calls.length);
|
|
};
|
|
return calls;
|
|
}
|
|
|
|
function jsonResponse(body, status = 200) {
|
|
return {
|
|
ok: status >= 200 && status < 300,
|
|
status,
|
|
text: async () => JSON.stringify(body),
|
|
};
|
|
}
|
|
|
|
test('fromConfig rejects missing credentials with a stoke-branded hint', () => {
|
|
assert.throws(() => ForgejoClient.fromConfig(null), /stoke auth login/);
|
|
assert.throws(() => ForgejoClient.fromConfig({ url: 'https://x' }), /stoke auth login/);
|
|
});
|
|
|
|
test('trailing slash in base URL is normalized', async () => {
|
|
const calls = mockFetch(() => jsonResponse({ ok: true }));
|
|
const client = new ForgejoClient('https://forge.test/', 'tok');
|
|
await client.get('/user');
|
|
assert.equal(calls[0].url, 'https://forge.test/api/v1/user');
|
|
});
|
|
|
|
test('token auth wins and User-Agent matches the package', async () => {
|
|
const calls = mockFetch(() => jsonResponse({}));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await client.get('/user');
|
|
const headers = calls[0].opts.headers;
|
|
assert.equal(headers.Authorization, 'token tok');
|
|
assert.equal(headers['User-Agent'], `stoke/${pkg.version}`);
|
|
});
|
|
|
|
test('withBasicAuth sends Basic credentials and drops the token', async () => {
|
|
const calls = mockFetch(() => jsonResponse({}));
|
|
const client = new ForgejoClient('https://forge.test', 'tok').withBasicAuth('user', 'pass');
|
|
await client.get('/user');
|
|
const expected = `Basic ${Buffer.from('user:pass').toString('base64')}`;
|
|
assert.equal(calls[0].opts.headers.Authorization, expected);
|
|
});
|
|
|
|
test('deleteToken uses Basic auth (Forgejo rejects token auth on token endpoints)', async () => {
|
|
const calls = mockFetch(() => jsonResponse(null, 204));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await client.deleteToken('user@example.test', 'pass', 'user', 42);
|
|
const { url, opts } = calls[0];
|
|
assert.equal(url, 'https://forge.test/api/v1/users/user/tokens/42');
|
|
assert.equal(opts.method, 'DELETE');
|
|
assert.match(opts.headers.Authorization, /^Basic /);
|
|
});
|
|
|
|
test('API errors carry message, status and body', async () => {
|
|
mockFetch(() => jsonResponse({ message: 'user does not exist', url: 'https://forge.test/api/swagger' }, 404));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await assert.rejects(() => client.get('/users/ghost'), (err) => {
|
|
assert.equal(err.message, 'user does not exist');
|
|
assert.equal(err.status, 404);
|
|
assert.equal(err.body.url, 'https://forge.test/api/swagger');
|
|
return true;
|
|
});
|
|
});
|
|
|
|
test('non-JSON error bodies are surfaced raw', async () => {
|
|
mockFetch(() => ({ ok: false, status: 502, text: async () => 'Bad Gateway' }));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await assert.rejects(() => client.get('/user'), /Bad Gateway/);
|
|
});
|
|
|
|
test('network failures are wrapped with the base URL', async () => {
|
|
global.fetch = async () => { throw new Error('ECONNREFUSED'); };
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await assert.rejects(() => client.get('/user'), /Network error reaching https:\/\/forge\.test/);
|
|
});
|
|
|
|
test('getAll paginates until a short page', async () => {
|
|
const pageOf = (n, count) => Array.from({ length: count }, (_, i) => ({ id: (n - 1) * 50 + i }));
|
|
mockFetch((url) => {
|
|
const page = Number(new URL(url).searchParams.get('page'));
|
|
if (page === 1) return jsonResponse(pageOf(1, 50));
|
|
if (page === 2) return jsonResponse(pageOf(2, 3));
|
|
throw new Error('should not fetch beyond a short page');
|
|
});
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
const all = await client.getAll('/user/repos');
|
|
assert.equal(all.length, 53);
|
|
});
|
|
|
|
test('getAll stops on an empty first page', async () => {
|
|
const calls = mockFetch(() => jsonResponse([]));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
const all = await client.getAll('/user/repos');
|
|
assert.deepEqual(all, []);
|
|
assert.equal(calls.length, 1);
|
|
});
|
|
|
|
test('searchUsers unwraps the {data: []} envelope and paginates', async () => {
|
|
mockFetch((url) => {
|
|
const page = Number(new URL(url).searchParams.get('page'));
|
|
if (page === 1) return jsonResponse({ data: Array.from({ length: 50 }, (_, i) => ({ login: `u${i}` })) });
|
|
return jsonResponse({ data: [{ login: 'last' }] });
|
|
});
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
const users = await client.searchUsers('u');
|
|
assert.equal(users.length, 51);
|
|
assert.equal(users.at(-1).login, 'last');
|
|
});
|
|
|
|
test('createIssue and createPullRequest hit the expected endpoints', async () => {
|
|
const calls = mockFetch(() => jsonResponse({ number: 1 }));
|
|
const client = new ForgejoClient('https://forge.test', 'tok');
|
|
await client.createIssue('own/er', 'repo', { title: 't' });
|
|
await client.createPullRequest('owner', 're po', { title: 't', head: 'h', base: 'b' });
|
|
assert.equal(calls[0].url, 'https://forge.test/api/v1/repos/own%2Fer/repo/issues');
|
|
assert.equal(calls[1].url, 'https://forge.test/api/v1/repos/owner/re%20po/pulls');
|
|
assert.equal(calls[0].opts.method, 'POST');
|
|
assert.equal(JSON.parse(calls[1].opts.body).head, 'h');
|
|
});
|