generated from nhcarrigan/template
Naomi Carrigan
296a50fedd
### Explanation This should set up everything we need for our initial launch. Test coverage is at 100% to ensure nothing breaks. ### Issue _No response_ ### Attestations - [x] I have read and agree to the [Code of Conduct](https://docs.nhcarrigan.com/community/coc/) - [x] I have read and agree to the [Community Guidelines](https://docs.nhcarrigan.com/community/guide/). - [x] My contribution complies with the [Contributor Covenant](https://docs.nhcarrigan.com/dev/covenant/). ### Dependencies - [x] I have pinned the dependencies to a specific patch version. ### Style - [x] I have run the linter and resolved any errors. - [x] My pull request uses an appropriate title, matching the conventional commit standards. - [x] My scope of feat/fix/chore/etc. correctly matches the nature of changes in my pull request. ### Tests - [x] My contribution adds new code, and I have added tests to cover it. - [ ] My contribution modifies existing code, and I have updated the tests to reflect these changes. - [x] All new and existing tests pass locally with my changes. - [x] Code coverage remains at or above the configured threshold. ### Documentation Coming soon - I'm working on the infra for docs next ### Versioning Major - My pull request introduces a breaking change. Reviewed-on: https://codeberg.org/nhcarrigan/rig-task-bot/pulls/2 Co-authored-by: Naomi Carrigan <commits@nhcarrigan.com> Co-committed-by: Naomi Carrigan <commits@nhcarrigan.com>
167 lines
4.6 KiB
TypeScript
167 lines
4.6 KiB
TypeScript
/**
|
|
* @copyright nhcarrigan
|
|
* @license Naomi's Public License
|
|
* @author Naomi Carrigan
|
|
*/
|
|
|
|
import {
|
|
ApplicationCommandOptionType,
|
|
InteractionContextType,
|
|
} from "discord.js";
|
|
import { describe, it, expect, vi } from "vitest";
|
|
import { complete } from "../../src/commands/complete.js";
|
|
import { errorHandler } from "../../src/utils/errorHandler.js";
|
|
|
|
vi.mock("../../src/utils/errorHandler.ts", () => {
|
|
return {
|
|
errorHandler: vi.fn(),
|
|
};
|
|
});
|
|
|
|
describe("complete command", () => {
|
|
it("should have the correct data", () => {
|
|
expect.assertions(11);
|
|
expect(complete.data.name, "did not have the correct name").toBe(
|
|
"complete",
|
|
);
|
|
expect(complete.data.name.length, "name is too long").toBeLessThanOrEqual(
|
|
32,
|
|
);
|
|
expect(complete.data.name, "name has invalid characters").toMatch(
|
|
/^[-_\p{L}\p{N}\p{sc=Deva}\p{sc=Thai}]{1,32}$/u,
|
|
);
|
|
expect(
|
|
complete.data.description,
|
|
"did not have the correct description",
|
|
).toBe("Mark a task as completed.");
|
|
expect(
|
|
complete.data.description.length,
|
|
"description is too long",
|
|
).toBeLessThanOrEqual(100);
|
|
expect(
|
|
complete.data.contexts,
|
|
"did not have the correct context",
|
|
).toStrictEqual([ InteractionContextType.Guild ]);
|
|
expect(complete.data.options, "should have 1 option").toHaveLength(1);
|
|
expect(
|
|
complete.data.options[0].toJSON().name,
|
|
"did not have the correct option name",
|
|
).toBe("id");
|
|
expect(
|
|
complete.data.options[0].toJSON().description,
|
|
"did not have the correct option description",
|
|
).toBe("The ID of the task to complete.");
|
|
expect(
|
|
complete.data.options[0].toJSON().required,
|
|
"did not have the correct option required value",
|
|
).toBeTruthy();
|
|
expect(
|
|
complete.data.options[0].toJSON().type,
|
|
"did not have the correct option type",
|
|
).toBe(ApplicationCommandOptionType.Integer);
|
|
});
|
|
|
|
it("should execute correctly", async() => {
|
|
expect.assertions(3);
|
|
const mockBot = {
|
|
database: {
|
|
tasks: {
|
|
update: vi.fn().mockReturnValue({
|
|
catch: vi.fn().mockReturnValue([ { assignees: [ "123" ] } ]),
|
|
}),
|
|
},
|
|
},
|
|
} as never;
|
|
const mockInteraction = {
|
|
deferReply: vi.fn(),
|
|
editReply: vi.fn(),
|
|
options: { getInteger: vi.fn().mockReturnValue(1) },
|
|
} as never;
|
|
await complete.run(mockBot, mockInteraction);
|
|
expect(
|
|
mockInteraction.deferReply,
|
|
"should defer the reply",
|
|
).toHaveBeenCalledWith({ ephemeral: true });
|
|
expect(
|
|
mockBot.database.tasks.update,
|
|
"should update database",
|
|
).toHaveBeenCalledWith({
|
|
data: {
|
|
completed: true,
|
|
},
|
|
where: {
|
|
numericalId: 1,
|
|
},
|
|
});
|
|
expect(
|
|
mockInteraction.editReply,
|
|
"should call editReply",
|
|
).toHaveBeenCalledWith({
|
|
content: "Task 1 has been marked as complete.",
|
|
});
|
|
});
|
|
|
|
it("should execute correctly when task not found", async() => {
|
|
expect.assertions(3);
|
|
const mockBot = {
|
|
database: {
|
|
tasks: {
|
|
update: vi.fn().mockReturnValue({
|
|
catch: vi.fn().mockImplementation((callback) => {
|
|
return callback();
|
|
}),
|
|
}),
|
|
},
|
|
},
|
|
} as never;
|
|
const mockInteraction = {
|
|
deferReply: vi.fn(),
|
|
editReply: vi.fn(),
|
|
options: {
|
|
getInteger: vi.fn().mockReturnValue(1),
|
|
},
|
|
} as never;
|
|
await complete.run(mockBot, mockInteraction);
|
|
expect(
|
|
mockInteraction.deferReply,
|
|
"should defer the reply",
|
|
).toHaveBeenCalledWith({ ephemeral: true });
|
|
expect(
|
|
mockBot.database.tasks.update,
|
|
"should update database",
|
|
).toHaveBeenCalledWith({
|
|
data: {
|
|
completed: true,
|
|
},
|
|
where: {
|
|
numericalId: 1,
|
|
},
|
|
});
|
|
expect(
|
|
mockInteraction.editReply,
|
|
"should call editReply",
|
|
).toHaveBeenCalledWith({
|
|
content: "Task 1 does not exist.",
|
|
});
|
|
});
|
|
|
|
it("should handle errors correctly", async() => {
|
|
expect.assertions(1);
|
|
await complete.run(
|
|
{} as never,
|
|
{ editReply: vi.fn(), replied: false, reply: vi.fn() } as never,
|
|
);
|
|
expect(errorHandler, "should call error handler").toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("should handle errors with interaction.reply correctly", async() => {
|
|
expect.assertions(1);
|
|
vi.resetAllMocks();
|
|
await complete.run(
|
|
{} as never,
|
|
{ editReply: vi.fn(), replied: true, reply: vi.fn() } as never,
|
|
);
|
|
expect(errorHandler, "should call error handler").toHaveBeenCalledTimes(1);
|
|
});
|
|
});
|