Tests and CI #44

Merged
Vylpes merged 17 commits from feature/tests into develop 2021-08-21 16:00:28 +01:00
5 changed files with 2699 additions and 3 deletions
Showing only changes of commit 99e375633b - Show all commits

View file

@ -1,3 +1,5 @@
/** @type {import('@ts-jest/dist/types').InitialOptionsTsJest} */
module.exports = {
testEnvironment: 'node'
}
preset: 'ts-jest',
testEnvironment: 'node',
};

View file

@ -25,7 +25,10 @@
],
"repository": "github:vylpes/vylbot-core",
"devDependencies": {
"@types/jest": "^26.0.24",
"@types/node": "^16.3.2",
"jest": "^27.0.6",
"ts-jest": "^27.0.4",
"typescript": "^4.3.5"
}
}

135
tests/client/client.test.ts Normal file
View file

@ -0,0 +1,135 @@
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
import { CoreClient } from "../../src/client/client";
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
import { Client } from "discord.js";
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
import * as dotenv from "dotenv";
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
import { Events } from "../../src/client/events";
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
import { Util } from "../../src/client/util";
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
jest.mock("discord.js");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
jest.mock("dotenv");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
jest.mock("../../src/client/events");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
jest.mock("../../src/client/util");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Constructor_ExpectSuccessfulInitialisation', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(coreClient).toBeInstanceOf(Client);
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(dotenv.config).toBeCalledTimes(1);
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(Events).toBeCalledTimes(1);
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(Util).toBeCalledTimes(1);
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenEnvIsValid_ExpectSuccessfulStart', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).not.toThrow();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(coreClient.on).toBeCalledWith("message", expect.any(Function));
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(coreClient.on).toBeCalledWith("ready", expect.any(Function));
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenBotTokenIsNull_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("BOT_TOKEN is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenBotTokenIsEmpty_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: '',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("BOT_TOKEN is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenBotPrefixIsNull_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("BOT_PREFIX is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenBotPrefixIsEmpty_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("BOT_PREFIX is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenFoldersCommandsIsNull_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("FOLDERS_COMMANDS is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenFoldersCommandsIsEmpty_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: '',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: 'events',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("FOLDERS_COMMANDS is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenFoldersEventsIsNull_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("FOLDERS_EVENTS is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
test('Start_GivenFoldersCommandsIsEmpty_ExpectFailure', () => {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
process.env = {
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_TOKEN: 'TOKEN',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
BOT_PREFIX: '!',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_COMMANDS: 'commands',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
FOLDERS_EVENTS: '',
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
}
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
const coreClient = new CoreClient();
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
expect(() => coreClient.start()).toThrow("FOLDERS_EVENTS is not defined in .env");
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also
});
VylpesTester commented 2021-08-21 15:10:11 +01:00 (Migrated from github.com)
Review

Should these be put into a describe group?

Should these be put into a describe group?
VylpesTester commented 2021-08-21 15:10:21 +01:00 (Migrated from github.com)
Review

This would apply to all other tests also

This would apply to all other tests also

View file

@ -71,5 +71,8 @@
},
"include": [
"./src",
],
"exclude": [
"./tests"
]
}

2555
yarn.lock

File diff suppressed because it is too large Load diff