resolve conflicts

Author: kamilmysliwiec

lerna.json

@@ -1,7 +1,5 @@
 {
   "lerna": "2.4.0",
-  "packages": [
-    "packages/*"
-  ],
+  "packages": ["packages/*"],
   "version": "6.0.0-alpha.3"
 }

packages/core/errors/exceptions/unknown-dependencies.exception.ts

@@ -5,7 +5,7 @@ import { Module } from '../../injector/module';
 
 export class UnknownDependenciesException extends RuntimeException {
   constructor(
-    type: string,
+    type: string | symbol,
     unknownDependencyContext: InjectorDependencyContext,
     module?: Module,
   ) {

packages/core/errors/messages.ts

@@ -1,6 +1,9 @@
 import { Type } from '@nestjs/common';
-import { isNil } from '@nestjs/common/utils/shared.utils';
-import { InjectorDependency, InjectorDependencyContext } from '../injector/injector';
+import { isNil, isSymbol } from '@nestjs/common/utils/shared.utils';
+import {
+  InjectorDependency,
+  InjectorDependencyContext,
+} from '../injector/injector';
 import { Module } from '../injector/module';
 
 // TODO: Replace `any` with `unknown` type when TS 3.0.0 is supported
@@ -18,7 +21,15 @@ const getInstanceName = (instance: unknown) =>
  * @param dependency The dependency whichs name should get displayed
  */
 const getDependencyName = (dependency: InjectorDependency) =>
-  getInstanceName(dependency) || dependency || '+';
+  // use class name
+  getInstanceName(dependency) ||
+  // use injection token (symbol)
+  (isSymbol(dependency) && dependency.toString()) ||
+  // use string directly
+  dependency ||
+  // otherwise
+  '+';
+
 /**
  * Returns the name of the module
  * Tries to get the class name. As fallback it returns 'current'.
@@ -28,15 +39,15 @@ const getModuleName = (module: Module) =>
   (module && getInstanceName(module.metatype)) || 'current';
 
 export const UNKNOWN_DEPENDENCIES_MESSAGE = (
-  type: string,
+  type: string | symbol,
   unknownDependencyContext: InjectorDependencyContext,
   module: Module,
 ) => {
   const { index, dependencies, key } = unknownDependencyContext;
-  let message = `Nest can't resolve dependencies of the ${type}`;
+  let message = `Nest can't resolve dependencies of the ${type.toString()}`;
 
   if (isNil(index)) {
-    message += `. Please make sure that the "${key}" property is available in the current context.`;
+    message += `. Please make sure that the "${key.toString()}" property is available in the current context.`;
     return message;
   }
   const dependenciesName = (dependencies || []).map(getDependencyName);

packages/core/injector/injector.ts

@@ -49,7 +49,7 @@ export interface InjectorDependencyContext {
   /**
    * The name of the property key (property-based injection)
    */
-  key?: string;
+  key?: string | symbol;
   /**
    * The name of the function or injection token
    */

packages/core/test/errors/test/messages.spec.ts

@@ -46,4 +46,14 @@ describe('UnknownDependenciesMessage', () => {
     myModule.metatype = myMetaType;
     expect(new UnknownDependenciesException('CatService', { index, dependencies: ['', 'MY_TOKEN'] }, myModule as Module).message).to.equal(expectedResult);
   });
+  it('should display the symbol name of the provider', () => {
+    const expectedResult = 'Nest can\'t resolve dependencies of the Symbol(CatProvider) (?). ' +
+      'Please make sure that the argument at index [0] is available in the current context.';
+    expect(new UnknownDependenciesException(Symbol('CatProvider'), { index, dependencies: [''] }).message).to.equal(expectedResult);
+  });
+  it('should display the symbol dependency of the provider', () => {
+    const expectedResult = 'Nest can\'t resolve dependencies of the CatProvider (?, Symbol(DogProvider)). ' +
+      'Please make sure that the argument at index [0] is available in the current context.';
+    expect(new UnknownDependenciesException('CatProvider', { index, dependencies: ['', Symbol('DogProvider')] }).message).to.equal(expectedResult);
+  });
 });