Issue nestjs#39 naming refactor

Author: kamil.mysliwiec

src/core/injector/injector.ts

@@ -1,6 +1,6 @@
 import 'reflect-metadata';
 import { InstanceWrapper } from './container';
-import { UnkownDependenciesException } from '../../errors/exceptions/unkown-dependencies.exception';
+import { UnknownDependenciesException } from '../../errors/exceptions/unknown-dependencies.exception';
 import { RuntimeException } from '../../errors/exceptions/runtime.exception';
 import { Module } from './module';
 import { Metatype } from '../../common/interfaces/metatype.interface';
@@ -122,7 +122,7 @@ export class Injector {
 
         const instanceWrapper = this.scanForComponentInRelatedModules(module, name);
         if (isNil(instanceWrapper)) {
-            throw new UnkownDependenciesException(metatype.name);
+            throw new UnknownDependenciesException(metatype.name);
         }
         return instanceWrapper;
     }

src/core/injector/module.ts

@@ -1,6 +1,6 @@
 import { InstanceWrapper } from './container';
 import { Injectable, Controller, NestModule } from '../../common/interfaces';
-import { UnkownExportException } from '../../errors/exceptions/unkown-export.exception';
+import { UnknownExportException } from '../../errors/exceptions/unknown-export.exception';
 import { NestModuleMetatype } from '../../common/interfaces/module-metatype.interface';
 import { Metatype } from '../../common/interfaces/metatype.interface';
 import { ModuleRef } from './module-ref';
@@ -152,7 +152,7 @@ export class Module {
 
     public addExportedComponent(exportedComponent: Metatype<Injectable>) {
         if (!this._components.get(exportedComponent.name)) {
-            throw new UnkownExportException(exportedComponent.name);
+            throw new UnknownExportException(exportedComponent.name);
         }
         this._exports.add(exportedComponent.name);
     }

src/core/middlewares/routes-mapper.ts

@@ -1,6 +1,6 @@
 import 'reflect-metadata';
 import { ExpressRouterExplorer } from '../router/router-explorer';
-import { UnkownRequestMappingException } from '../../errors/exceptions/unkown-request-mapping.exception';
+import { UnknownRequestMappingException } from '../../errors/exceptions/unknown-request-mapping.exception';
 import { RequestMethod } from '../../common/enums/request-method.enum';
 import { isUndefined, validatePath } from '../../common/utils/shared.utils';
 import { PATH_METADATA } from '../../common/constants';
@@ -24,7 +24,7 @@ export class RoutesMapper {
     private mapObjectToRouteProps(route) {
         const { path, method } = route;
         if (isUndefined(path)) {
-            throw new UnkownRequestMappingException();
+            throw new UnknownRequestMappingException();
         }
         return {
             path: this.validateRoutePath(path),

src/core/router/router-explorer.ts

@@ -2,7 +2,7 @@ import 'reflect-metadata';
 import { Controller } from '../../common/interfaces/controller.interface';
 import { RequestMethod } from '../../common/enums/request-method.enum';
 import { RouterProxy, RouterProxyCallback } from './router-proxy';
-import { UnkownRequestMappingException } from '../../errors/exceptions/unkown-request-mapping.exception';
+import { UnknownRequestMappingException } from '../../errors/exceptions/unknown-request-mapping.exception';
 import { ExpressAdapter } from '../adapters/express-adapter';
 import { Metatype } from '../../common/interfaces/metatype.interface';
 import { isUndefined, validatePath } from '../../common/utils/shared.utils';
@@ -90,7 +90,7 @@ export class ExpressRouterExplorer implements RouterExplorer {
 
     private validateRoutePath(path: string): string {
         if (isUndefined(path)) {
-            throw new UnkownRequestMappingException();
+            throw new UnknownRequestMappingException();
         }
         return validatePath(path);
     }

src/core/test/injector/injector.spec.ts

@@ -5,7 +5,7 @@ import { Injector } from '../../injector/injector';
 import { Component } from '../../../common/utils/decorators/component.decorator';
 import { RuntimeException } from '../../../errors/exceptions/runtime.exception';
 import { Module } from '../../injector/module';
-import { UnkownDependenciesException } from '../../../errors/exceptions/unkown-dependencies.exception';
+import { UnknownDependenciesException } from '../../../errors/exceptions/unknown-dependencies.exception';
 
 describe('Injector', () => {
     let injector: Injector;
@@ -178,25 +178,25 @@ describe('Injector', () => {
             expect(scanForComponentInRelatedModules.called).to.be.true;
         });
 
-        it('should throw "UnkownDependenciesException" instanceWrapper is null', () => {
+        it('should throw "UnknownDependenciesException" instanceWrapper is null', () => {
             scanForComponentInRelatedModules.returns(null);
             const collection = {
                 has: () => false,
             };
 
             expect(
                 () => injector.scanForComponent(collection as any, metatype.name, null, metatype)
-            ).throws(UnkownDependenciesException);
+            ).throws(UnknownDependenciesException);
         });
 
-        it('should not throw "UnkownDependenciesException" instanceWrapper is not null', () => {
+        it('should not throw "UnknownDependenciesException" instanceWrapper is not null', () => {
             scanForComponentInRelatedModules.returns({});
             const collection = {
                 has: () => false,
             };
             expect(
                 () => injector.scanForComponent(collection as any, metatype.name, null, metatype)
-            ).not.throws(UnkownDependenciesException);
+            ).not.throws(UnknownDependenciesException);
         });
 
     });

src/core/test/injector/module.spec.ts

@@ -1,7 +1,7 @@
 import { expect } from 'chai';
 import * as sinon from 'sinon';
 import { Module as ModuleDecorator } from '../../../common/utils/decorators/module.decorator';
-import { UnkownExportException } from '../../../errors/exceptions/unkown-export.exception';
+import { UnknownExportException } from '../../../errors/exceptions/unknown-export.exception';
 import { Module } from '../../injector/module';
 import { Component } from '../../../common/utils/decorators/component.decorator';
 import { RuntimeException } from '../../../errors/exceptions/runtime.exception';
@@ -16,10 +16,10 @@ describe('Module', () => {
         module = new Module(TestModule);
     });
 
-    it('should throw "UnkownExportException" when given exported component is not a part of components array', () => {
+    it('should throw "UnknownExportException" when given exported component is not a part of components array', () => {
         expect(
             () => module.addExportedComponent(TestComponent),
-        ).throws(UnkownExportException);
+        ).throws(UnknownExportException);
     });
 
     it('should add route', () => {

src/core/test/middlewares/routes-mapper.spec.ts

@@ -3,7 +3,7 @@ import { RoutesMapper } from '../../middlewares/routes-mapper';
 import { Controller } from '../../../common/utils/decorators/controller.decorator';
 import { RequestMapping } from '../../../common/utils/decorators/request-mapping.decorator';
 import { RequestMethod } from '../../../common/enums/request-method.enum';
-import { UnkownRequestMappingException } from '../../../errors/exceptions/unkown-request-mapping.exception';
+import { UnknownRequestMappingException } from '../../../errors/exceptions/unknown-request-mapping.exception';
 
 describe('RoutesMapper', () => {
     @Controller({ path: 'test' })
@@ -50,7 +50,7 @@ describe('RoutesMapper', () => {
 
         expect(
             mapper.mapRouteToRouteProps.bind(mapper, config.forRoutes[0]),
-        ).throws(UnkownRequestMappingException);
+        ).throws(UnknownRequestMappingException);
     });
 
 });

src/errors/exceptions/unknown-dependencies.exception.ts

@@ -0,0 +1,8 @@
+import { RuntimeException } from './runtime.exception';
+import { UnknownDependenciesMessage } from '../messages';
+
+export class UnknownDependenciesException extends RuntimeException {
+    constructor(type: string) {
+        super(UnknownDependenciesMessage(type));
+    }
+}

src/errors/exceptions/unknown-export.exception.ts

@@ -0,0 +1,8 @@
+import { RuntimeException } from './runtime.exception';
+import { UnknownExportMessage } from '../messages';
+
+export class UnknownExportException extends RuntimeException {
+    constructor(name: string) {
+        super(UnknownExportMessage(name));
+    }
+}

src/errors/exceptions/unknown-request-mapping.exception.ts

@@ -0,0 +1,8 @@
+import { RuntimeException } from './runtime.exception';
+import { UNKNOWN_REQUEST_MAPPING } from '../messages';
+
+export class UnknownRequestMappingException extends RuntimeException {
+    constructor() {
+        super(UNKNOWN_REQUEST_MAPPING);
+    }
+}