refactor(urlSlug): remove urlSlug as a dependency and update packages…

… to remove security alert (#120)

package.json

@@ -62,7 +62,6 @@
   },
   "dependencies": {
     "lodash-es": "^4.17.15",
-    "url-slug": "^2.2.0",
     "winston": "^3.2.1"
   },
   "peerDependencies": {

src/transformer/mockDefiner/factoryUniqueName.ts

@@ -1,7 +1,5 @@
 import * as ts from 'typescript';
 import { KeyCounter } from './keyCounter';
-// tslint:disable-next-line:no-any
-const urlSlug: any = require('url-slug');
 
 export type PossibleDeclaration = ts.InterfaceDeclaration | ts.ClassDeclaration | ts.TypeAliasDeclaration;
 
@@ -14,7 +12,7 @@ export class FactoryUniqueName {
 
     public createForDeclaration(declaration: PossibleDeclaration): string {
         const declarationNameIdentifier: ts.Identifier = declaration.name;
-        const declarationNameSanitized: string = urlSlug((declarationNameIdentifier && declarationNameIdentifier.text) || 'Anonymous', '_');
+        const declarationNameSanitized: string = (declarationNameIdentifier && declarationNameIdentifier.text) || 'Anonymous';
         const baseFactoryName: string = `create__${declarationNameSanitized}__mock`;
         const count: number = this._keyCounter.getFor(baseFactoryName);
 
@@ -34,7 +32,7 @@ export class FactoryUniqueName {
             return acc;
         }, '');
 
-        const declarationNameSanitized: string = urlSlug(nameOfDeclarations || 'Anonymous', '_');
+        const declarationNameSanitized: string = nameOfDeclarations || 'Anonymous';
         const baseFactoryName: string = `create__${declarationNameSanitized}__mock`;
         const count: number = this._keyCounter.getFor(baseFactoryName);
 

src/transformer/mockDefiner/mockDefiner.ts

@@ -14,9 +14,6 @@ import { ModuleName } from './modules/moduleName';
 import { ModuleNameIdentifier } from './modules/moduleNameIdentifier';
 import { ModulesImportUrl } from './modules/modulesImportUrl';
 
-// tslint:disable-next-line:no-any
-const urlSlug: any = require('url-slug');
-
 interface FactoryRegistrationPerFile {
     [key: string]: Array<{
         key: ts.Declaration;
@@ -69,9 +66,9 @@ export class MockDefiner {
         }
 
         this._internalModuleImportIdentifierPerFile[this._fileName] = {
-            [ModuleName.Extension]: this._createUniqueFileName(ModuleName.Extension),
-            [ModuleName.Merge]: this._createUniqueFileName(ModuleName.Merge),
-            [ModuleName.Repository]: this._createUniqueFileName(ModuleName.Repository),
+            [ModuleName.Extension]: ts.createFileLevelUniqueName(ModuleName.Extension),
+            [ModuleName.Merge]: ts.createFileLevelUniqueName(ModuleName.Merge),
+            [ModuleName.Repository]: ts.createFileLevelUniqueName(ModuleName.Repository),
         };
 
         this._neededImportIdentifierPerFile[this._fileName] = this._neededImportIdentifierPerFile[this._fileName] || [];
@@ -140,10 +137,6 @@ export class MockDefiner {
         return this._declarationCache.get(declaration);
     }
 
-    private _createUniqueFileName(name: string): ts.Identifier {
-        return ts.createFileLevelUniqueName(`${urlSlug(this._fileName, '_')}_${name}`);
-    }
-
     private _mockRepositoryAccess(filename: string): ts.Expression {
         const repository: ts.Identifier = this._getModuleIdentifier(filename, ModuleName.Repository);