Can't configure BullModule with useClass about bull

[fwoelffel]

Thanks for reporting this @maksimkurb 🥇
It's nice to see that you prepared the failing test that can confirm your bug report. Could you create the pull request before we continue to work on this?

[maksimkurb]

Closing in favor of #89

[fwoelffel]

Don't close an issue until it's resolved.

[fwoelffel]

I believe this is rather an error in the BullModuleAsyncOptions interface:

export interface BullModuleAsyncOptions extends Pick<ModuleMetadata, 'imports'> {
  name?: string;
  useExisting?: Type<BullOptionsFactory>;
  useClass?: Type<BullOptionsFactory>;
  useFactory?: (
    ...args: any[]
  ) => Promise<BullModuleOptions> | BullModuleOptions;
  inject?: any[];
}

should be

export interface BullModuleAsyncOptions extends Pick<ModuleMetadata, 'imports'> {
  name?: string;
  useExisting?: Type<BullModuleOptions>;
  useClass?: Type<BullModuleOptions>;
  useFactory?: (
    ...args: any[]
  ) => Promise<BullModuleOptions> | BullModuleOptions;
  inject?: any[];
}

I'll try to have a look at this by the end of the week.

In the meantine, I think you could refactor your async configuration and use useFactory instead of useClass.

[fwoelffel]

I think I'll have a look at your PR and try to fix the current implementation of the useClass alternative without breaking the interface. But keep #94 in mind. I might change this behavior pretty soon.

[fwoelffel]

Since this is broken, I've made my mind to directly "fix" this through #94. I'm therefore closing this in favor of #94.