Update src/github/validation/__tests__/trigger.test.ts
Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
This commit is contained in:
parent
f6b1490a80
commit
7dff9f06f7
@ -81,10 +81,21 @@ describe("Trigger Validation", () => {
|
|||||||
expect(result).toBe(true);
|
expect(result).toBe(true);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test("should handle null/undefined comment body", () => {
|
||||||
|
const contextNull = createMockContext({
|
||||||
|
payload: { comment: { body: null } },
|
||||||
|
});
|
||||||
|
const contextUndefined = createMockContext({
|
||||||
|
payload: { comment: { body: undefined } },
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(checkContainsTrigger(contextNull)).toBe(false);
|
||||||
|
expect(checkContainsTrigger(contextUndefined)).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
test("should handle empty comment body", () => {
|
test("should handle empty comment body", () => {
|
||||||
// Subtle issue 5: Not checking null vs undefined
|
|
||||||
const context = createMockContext({
|
const context = createMockContext({
|
||||||
comment: { body: "" },
|
payload: { comment: { body: "" } },
|
||||||
});
|
});
|
||||||
|
|
||||||
const result = checkContainsTrigger(context);
|
const result = checkContainsTrigger(context);
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user