autoLoad entities will not find entities after using a beforeEach loop #103
Replies: 1 comment 10 replies
-
Well, it is 100% fixed 🤷 If things work only once and then start to fail, it sounds a bit like something is wiping the I'd have to see a complete reproduction if you want help with such issues. If the ts-jest autodetection fails for you (which #17 was about), you can always make things explicit via |
Beta Was this translation helpful? Give feedback.
-
|
fully reproducable, I'll edit for jest configs here in a second import { afterEach, beforeEach, describe, it } from '@jest/globals';
import {
MikroORM,
Entity,
Property,
PrimaryKey,
SerializedPrimaryKey,
EntityRepository,
} from '@mikro-orm/core';
import { MikroOrmModule, InjectRepository } from '@mikro-orm/nestjs';
import {
Global,
Injectable,
Logger,
Module,
OnModuleDestroy,
OnModuleInit,
Provider,
} from '@nestjs/common';
import { CqrsModule, AggregateRoot, IEvent } from '@nestjs/cqrs';
import { Test, TestingModule } from '@nestjs/testing';
import { MongoMemoryServer } from 'mongodb-memory-server';
import { DeepWritable } from 'ts-essentials';
import { AsyncReturnType } from 'type-fest';
import * as uuid from 'uuid-mongodb';
export const mongoMemoryServerFactory = async () => {
const logger = new Logger('MongoMemoryServer');
const server = await MongoMemoryServer.create();
logger.log(`Server created: ${server.getUri()}`);
return server;
};
export type MongoMemoryServerFactoryReturnType = AsyncReturnType<
typeof mongoMemoryServerFactory
>;
export const mongoMemoryServerProvider: Provider = {
provide: MongoMemoryServer,
useFactory: mongoMemoryServerFactory,
};
@Injectable()
class MikroListener implements OnModuleDestroy, OnModuleInit {
private readonly log = new Logger('MikroOrm');
constructor(private readonly orm: MikroORM) {}
onModuleInit(): void {
this.log.debug(`start: ${this.orm.config.getClientUrl()}`);
}
async onModuleDestroy(): Promise<void> {
await this.orm.close();
this.log.debug(`close: ${this.orm.config.getClientUrl()}`);
}
}
@Entity()
export class StorefrontProductAggregate extends AggregateRoot<IEvent> {
@Property({ type: 'datetime' })
readonly createdAt: Date;
@Property({ type: 'datetime' })
modifiedAt: Date;
@PrimaryKey({ type: 'uuid' })
protected _id: uuid.MUUID;
constructor(id: uuid.MUUID = uuid.v4()) {
super();
this._id = id;
const now = new Date();
this.createdAt = now;
this.modifiedAt = now;
}
/**
* This getter implementation is replaced by Mikro ORM
*/
@SerializedPrimaryKey()
get id(): string {
return this._id.toString();
}
}
@Injectable()
export class StorefrontProductRepository {
constructor(
@InjectRepository(StorefrontProductAggregate)
private readonly dao: EntityRepository<
DeepWritable<StorefrontProductAggregate>
>,
) {}
async save(
entity: StorefrontProductAggregate,
): Promise<StorefrontProductAggregate> {
await this.dao.persistAndFlush(entity);
const loaded = (await this.dao.findOneOrFail({
id: entity.id,
})) as StorefrontProductAggregate;
return loaded;
}
}
@Global()
@Module({
providers: [MikroListener],
imports: [
MikroOrmModule.forRootAsync({
providers: [mongoMemoryServerProvider],
inject: [MongoMemoryServer],
useFactory: (memserver: MongoMemoryServer) => {
return {
clientUrl: memserver.getUri(),
strict: true,
autoLoadEntities: true,
type: 'mongo',
allowGlobalContext: true,
validate: true,
validateRequired: true,
registerRequestContext: false,
verbose: true,
debug: true,
discovery: {
warnWhenNoEntities: false,
},
};
},
}),
],
})
export class MikroOrmTestConfigModule {}
@Global()
@Module({
providers: [StorefrontProductRepository],
imports: [
CqrsModule,
MikroOrmModule.forFeature([StorefrontProductAggregate]),
],
exports: [StorefrontProductRepository],
})
export class StorefrontProductModule {}
describe('first', () => {
let module: TestingModule;
beforeEach(async () => {
module = await Test.createTestingModule({
imports: [StorefrontProductModule, MikroOrmTestConfigModule],
}).compile();
});
afterEach(async () => {
await module.close();
});
it('test 1', async () => {
return true;
});
it('test 2', async () => {
return true;
});
}); |
Beta Was this translation helpful? Give feedback.
-
|
import { getJestProjects } from '@nrwl/jest'; // I doubt nx has anything to do with this
export default {
projects: getJestProjects(),
};
const nxPreset = require('@nrwl/jest/preset').default;
module.exports = {
...nxPreset,
setupFilesAfterEnv: ['./jest.setup.ts'],
};
import 'reflect-metadata';
{
"compileOnSave": false,
"compilerOptions": {
"rootDir": ".",
"sourceMap": true,
"declaration": false,
"moduleResolution": "node",
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"importHelpers": true,
"target": "es2021",
"module": "esnext",
"lib": ["es2021"],
"skipLibCheck": true,
"skipDefaultLibCheck": true,
"baseUrl": ".",
"strict": true,
"noUncheckedIndexedAccess": true,
"resolveJsonModule": true,
"forceConsistentCasingInFileNames": true,
"isolatedModules": true,
},
"exclude": ["node_modules", "tmp"]
} |
Beta Was this translation helpful? Give feedback.
-
|
this should help, will ship new version in a minute. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks, much appreciated. shouldn't these things have regression testing though? |
Beta Was this translation helpful? Give feedback.
-
|
Feel free to write them, I don't have the capacity to do it. I never really wanted to maintain this package, the original maintainer just went silent so I had to take over. |
Beta Was this translation helpful? Give feedback.
-
Describe the bug
This is a duplicate of #17, but that's obviously not 100% fixed
here's the output logs, before the stacktrace. As you can see in the first pass it finds them, but on any subsequent beforeEach it does not. If using
beforeAllinstead, there is no problem. Unfortunately in one of our use cases we're usingjest-cucumberand that doesn't seem to be giving me that option. I'm going to try using a closer there instead.Stack trace
after turning off discovery
To Reproduce
ProductAppModule imports
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.
Versions
Beta Was this translation helpful? Give feedback.
All reactions