fix: detect ai review bot commits via api

This commit is contained in:
2026-05-15 14:17:55 +00:00
parent f3319b5ec4
commit 82ecbd3463
6 changed files with 84 additions and 6 deletions
+40 -1
View File
@@ -1,7 +1,7 @@
import { describe, it, afterEach, mock } from 'node:test';
import assert from 'node:assert/strict';
import axios from 'axios';
import { getPRDiff, filterDiff, postComment } from './gitea.js';
import { getPRDiff, filterDiff, postComment, getCommitMessageBySha, getBranchHeadCommitMessage, shouldSkipBotCommit } from './gitea.js';
afterEach(() => mock.restoreAll());
@@ -56,6 +56,45 @@ describe('gitea', () => {
mock.method(axios, 'post', async () => { throw new Error('api error'); });
await assert.rejects(() => postComment('test'), /api error/);
});
it('getCommitMessageBySha reads commit message from Gitea API', async () => {
let capturedUrl;
mock.method(axios, 'get', async (url) => {
capturedUrl = url;
return { data: { message: 'chore: update ai-review findings [ai-review-bot]' } };
});
const message = await getCommitMessageBySha('abc123');
assert.ok(capturedUrl.includes('/git/commits/abc123'));
assert.ok(message.includes('[ai-review-bot]'));
});
it('getBranchHeadCommitMessage reads branch head commit message from Gitea API', async () => {
const urls = [];
mock.method(axios, 'get', async (url) => {
urls.push(url);
if (url.includes('/branches/feat%2Ftest')) {
return { data: { commit: { id: 'abc123' } } };
}
return { data: { message: 'chore: update ai-review findings [ai-review-bot]' } };
});
const message = await getBranchHeadCommitMessage('feat/test');
assert.ok(urls.some(url => url.includes('/branches/feat%2Ftest')));
assert.ok(urls.some(url => url.includes('/git/commits/abc123')));
assert.ok(message.includes('[ai-review-bot]'));
});
it('shouldSkipBotCommit returns true when either sha or branch head is bot commit', async () => {
mock.method(axios, 'get', async (url) => {
if (url.includes('/git/commits/sha-bot')) {
return { data: { message: 'chore: update ai-review findings [ai-review-bot]' } };
}
if (url.includes('/branches/feat%2Ftest')) {
return { data: { commit: { id: 'sha-bot' } } };
}
return { data: { message: 'regular commit' } };
});
await assert.equal(await shouldSkipBotCommit({ sha: 'sha-bot', branch: 'feat/test' }), true);
});
});
describe('filterDiff', () => {