test(server): add MockJobQueue (#11032)
This commit is contained in:
@@ -5,6 +5,7 @@ import type { TestFn } from 'ava';
|
|||||||
import ava from 'ava';
|
import ava from 'ava';
|
||||||
import Sinon from 'sinon';
|
import Sinon from 'sinon';
|
||||||
|
|
||||||
|
import { JobQueue } from '../base';
|
||||||
import { ConfigModule } from '../base/config';
|
import { ConfigModule } from '../base/config';
|
||||||
import { AuthService } from '../core/auth';
|
import { AuthService } from '../core/auth';
|
||||||
import { WorkspaceModule } from '../core/workspaces';
|
import { WorkspaceModule } from '../core/workspaces';
|
||||||
@@ -88,6 +89,10 @@ test.before(async t => {
|
|||||||
WorkspaceModule,
|
WorkspaceModule,
|
||||||
CopilotModule,
|
CopilotModule,
|
||||||
],
|
],
|
||||||
|
tapModule: m => {
|
||||||
|
// use real JobQueue for testing
|
||||||
|
m.overrideProvider(JobQueue).useClass(JobQueue);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
const auth = app.get(AuthService);
|
const auth = app.get(AuthService);
|
||||||
|
|||||||
@@ -4,6 +4,7 @@ export * from './user.mock';
|
|||||||
export * from './workspace.mock';
|
export * from './workspace.mock';
|
||||||
|
|
||||||
import { MockMailer } from './mailer.mock';
|
import { MockMailer } from './mailer.mock';
|
||||||
|
import { MockJobQueue } from './queue.mock';
|
||||||
import { MockTeamWorkspace } from './team-workspace.mock';
|
import { MockTeamWorkspace } from './team-workspace.mock';
|
||||||
import { MockUser } from './user.mock';
|
import { MockUser } from './user.mock';
|
||||||
import { MockWorkspace } from './workspace.mock';
|
import { MockWorkspace } from './workspace.mock';
|
||||||
@@ -14,4 +15,4 @@ export const Mockers = {
|
|||||||
TeamWorkspace: MockTeamWorkspace,
|
TeamWorkspace: MockTeamWorkspace,
|
||||||
};
|
};
|
||||||
|
|
||||||
export { MockMailer };
|
export { MockJobQueue, MockMailer };
|
||||||
|
|||||||
26
packages/backend/server/src/__tests__/mocks/queue.mock.ts
Normal file
26
packages/backend/server/src/__tests__/mocks/queue.mock.ts
Normal file
@@ -0,0 +1,26 @@
|
|||||||
|
import Sinon from 'sinon';
|
||||||
|
|
||||||
|
import { JobQueue } from '../../base';
|
||||||
|
|
||||||
|
export class MockJobQueue {
|
||||||
|
add = Sinon.createStubInstance(JobQueue).add.resolves();
|
||||||
|
|
||||||
|
last<Job extends JobName>(name: Job): { name: Job; payload: Jobs[Job] } {
|
||||||
|
const addJobName = this.add.lastCall?.args[0];
|
||||||
|
const payload = this.add.lastCall?.args[1];
|
||||||
|
|
||||||
|
if (!payload) {
|
||||||
|
throw new Error('No job ever added');
|
||||||
|
}
|
||||||
|
|
||||||
|
if (addJobName !== name) {
|
||||||
|
throw new Error(`Job name mismatch: ${addJobName} !== ${name}`);
|
||||||
|
}
|
||||||
|
|
||||||
|
return { name, payload };
|
||||||
|
}
|
||||||
|
|
||||||
|
count(name: JobName) {
|
||||||
|
return this.add.getCalls().filter(call => call.args[0] === name).length;
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -8,11 +8,22 @@ import cookieParser from 'cookie-parser';
|
|||||||
import graphqlUploadExpress from 'graphql-upload/graphqlUploadExpress.mjs';
|
import graphqlUploadExpress from 'graphql-upload/graphqlUploadExpress.mjs';
|
||||||
import supertest from 'supertest';
|
import supertest from 'supertest';
|
||||||
|
|
||||||
import { AFFiNELogger, ApplyType, GlobalExceptionFilter } from '../../base';
|
import {
|
||||||
|
AFFiNELogger,
|
||||||
|
ApplyType,
|
||||||
|
GlobalExceptionFilter,
|
||||||
|
JobQueue,
|
||||||
|
} from '../../base';
|
||||||
import { AuthService } from '../../core/auth';
|
import { AuthService } from '../../core/auth';
|
||||||
import { Mailer } from '../../core/mail';
|
import { Mailer } from '../../core/mail';
|
||||||
import { UserModel } from '../../models';
|
import { UserModel } from '../../models';
|
||||||
import { createFactory, MockedUser, MockUser, MockUserInput } from '../mocks';
|
import {
|
||||||
|
createFactory,
|
||||||
|
MockedUser,
|
||||||
|
MockJobQueue,
|
||||||
|
MockUser,
|
||||||
|
MockUserInput,
|
||||||
|
} from '../mocks';
|
||||||
import { MockMailer } from '../mocks/mailer.mock';
|
import { MockMailer } from '../mocks/mailer.mock';
|
||||||
import { createTestingModule } from './testing-module';
|
import { createTestingModule } from './testing-module';
|
||||||
import { initTestingDB, TEST_LOG_LEVEL } from './utils';
|
import { initTestingDB, TEST_LOG_LEVEL } from './utils';
|
||||||
@@ -85,6 +96,7 @@ export class TestingApp extends ApplyType<INestApplication>() {
|
|||||||
|
|
||||||
readonly create!: ReturnType<typeof createFactory>;
|
readonly create!: ReturnType<typeof createFactory>;
|
||||||
readonly mails!: MockMailer;
|
readonly mails!: MockMailer;
|
||||||
|
readonly queue!: MockJobQueue;
|
||||||
|
|
||||||
[Symbol.asyncDispose](): Promise<void> {
|
[Symbol.asyncDispose](): Promise<void> {
|
||||||
return this.close();
|
return this.close();
|
||||||
@@ -285,6 +297,8 @@ function makeTestingApp(app: INestApplication): TestingApp {
|
|||||||
testingApp.create = createFactory(app.get(PrismaClient, { strict: false }));
|
testingApp.create = createFactory(app.get(PrismaClient, { strict: false }));
|
||||||
// @ts-expect-error allow
|
// @ts-expect-error allow
|
||||||
testingApp.mails = app.get(Mailer, { strict: false }) as MockMailer;
|
testingApp.mails = app.get(Mailer, { strict: false }) as MockMailer;
|
||||||
|
// @ts-expect-error allow
|
||||||
|
testingApp.queue = app.get(JobQueue, { strict: false }) as MockJobQueue;
|
||||||
|
|
||||||
return new Proxy(testingApp, {
|
return new Proxy(testingApp, {
|
||||||
get(target, prop) {
|
get(target, prop) {
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ import {
|
|||||||
import { PrismaClient } from '@prisma/client';
|
import { PrismaClient } from '@prisma/client';
|
||||||
|
|
||||||
import { AppModule, FunctionalityModules } from '../../app.module';
|
import { AppModule, FunctionalityModules } from '../../app.module';
|
||||||
import { AFFiNELogger, Runtime } from '../../base';
|
import { AFFiNELogger, JobQueue, Runtime } from '../../base';
|
||||||
import { GqlModule } from '../../base/graphql';
|
import { GqlModule } from '../../base/graphql';
|
||||||
import { AuthGuard, AuthModule } from '../../core/auth';
|
import { AuthGuard, AuthModule } from '../../core/auth';
|
||||||
import { Mailer, MailModule } from '../../core/mail';
|
import { Mailer, MailModule } from '../../core/mail';
|
||||||
@@ -17,11 +17,11 @@ import { ModelsModule } from '../../models';
|
|||||||
// for jsdoc inference
|
// for jsdoc inference
|
||||||
// oxlint-disable-next-line no-unused-vars
|
// oxlint-disable-next-line no-unused-vars
|
||||||
import type { createModule } from '../create-module';
|
import type { createModule } from '../create-module';
|
||||||
import { createFactory } from '../mocks';
|
import { createFactory, MockJobQueue } from '../mocks';
|
||||||
import { MockMailer } from '../mocks/mailer.mock';
|
import { MockMailer } from '../mocks/mailer.mock';
|
||||||
import { initTestingDB, TEST_LOG_LEVEL } from './utils';
|
import { initTestingDB, TEST_LOG_LEVEL } from './utils';
|
||||||
|
|
||||||
interface TestingModuleMeatdata extends ModuleMetadata {
|
interface TestingModuleMetadata extends ModuleMetadata {
|
||||||
tapModule?(m: TestingModuleBuilder): void;
|
tapModule?(m: TestingModuleBuilder): void;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -29,6 +29,7 @@ export interface TestingModule extends BaseTestingModule {
|
|||||||
initTestingDB(): Promise<void>;
|
initTestingDB(): Promise<void>;
|
||||||
create: ReturnType<typeof createFactory>;
|
create: ReturnType<typeof createFactory>;
|
||||||
mails: MockMailer;
|
mails: MockMailer;
|
||||||
|
queue: MockJobQueue;
|
||||||
[Symbol.asyncDispose](): Promise<void>;
|
[Symbol.asyncDispose](): Promise<void>;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -58,7 +59,7 @@ class MockResolver {
|
|||||||
* @deprecated use {@link createModule} instead
|
* @deprecated use {@link createModule} instead
|
||||||
*/
|
*/
|
||||||
export async function createTestingModule(
|
export async function createTestingModule(
|
||||||
moduleDef: TestingModuleMeatdata = {},
|
moduleDef: TestingModuleMetadata = {},
|
||||||
autoInitialize = true
|
autoInitialize = true
|
||||||
): Promise<TestingModule> {
|
): Promise<TestingModule> {
|
||||||
// setting up
|
// setting up
|
||||||
@@ -88,10 +89,11 @@ export async function createTestingModule(
|
|||||||
controllers: moduleDef.controllers,
|
controllers: moduleDef.controllers,
|
||||||
});
|
});
|
||||||
|
|
||||||
|
builder.overrideProvider(Mailer).useClass(MockMailer);
|
||||||
|
builder.overrideProvider(JobQueue).useClass(MockJobQueue);
|
||||||
if (moduleDef.tapModule) {
|
if (moduleDef.tapModule) {
|
||||||
moduleDef.tapModule(builder);
|
moduleDef.tapModule(builder);
|
||||||
}
|
}
|
||||||
builder.overrideProvider(Mailer).useClass(MockMailer);
|
|
||||||
|
|
||||||
const module = await builder.compile();
|
const module = await builder.compile();
|
||||||
|
|
||||||
@@ -114,6 +116,7 @@ export async function createTestingModule(
|
|||||||
};
|
};
|
||||||
|
|
||||||
testingModule.mails = module.get(Mailer, { strict: false }) as MockMailer;
|
testingModule.mails = module.get(Mailer, { strict: false }) as MockMailer;
|
||||||
|
testingModule.queue = module.get(JobQueue, { strict: false }) as MockJobQueue;
|
||||||
|
|
||||||
const logger = new AFFiNELogger();
|
const logger = new AFFiNELogger();
|
||||||
// we got a lot smoking tests try to break nestjs
|
// we got a lot smoking tests try to break nestjs
|
||||||
|
|||||||
@@ -68,6 +68,10 @@ test.before(async () => {
|
|||||||
JobModule.forRoot(),
|
JobModule.forRoot(),
|
||||||
],
|
],
|
||||||
providers: [JobHandlers],
|
providers: [JobHandlers],
|
||||||
|
tapModule: builder => {
|
||||||
|
// use real JobQueue for testing
|
||||||
|
builder.overrideProvider(JobQueue).useClass(JobQueue);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
queue = module.get(JobQueue);
|
queue = module.get(JobQueue);
|
||||||
|
|||||||
Reference in New Issue
Block a user