fix: integrate the logic with excludedPeerDependencies with the new testInitialization service

Author: Fatme

config/test-dependencies.json

@@ -5,6 +5,10 @@
     {
         "name": "karma-nativescript-launcher"
     },
+    {
+        "name": "karma-webpack",
+        "excludedPeerDependencies": ["webpack"]
+    },
     {
         "name": "mocha",
         "framework": "mocha"

lib/commands/test-init.ts

@@ -36,13 +36,11 @@ class TestInitCommand implements ICommand {
 			const dependencies = this.$testInitializationService.getDependencies(frameworkToInstall);
 			const dependenciesVersions = this.$testInitializationService.getDependenciesVersions();
 			modulesToInstall = dependencies.map(dependency => {
-				const dependencyVersion = dependenciesVersions[dependency];
-				if (!dependencyVersion) {
+				dependency.version = dependenciesVersions[dependency.name];
+				if (!dependency.version) {
 					this.$errors.failWithoutHelp(`'${dependency}' is not a registered dependency.`);
 				}
-
-
-				return { name: dependency, version: dependencyVersion };
+				return dependency;
 			});
 		} catch (err) {
 			this.$errors.failWithoutHelp(`Unable to install the unit testing dependencies. Error: '${err.message}'`);

lib/definitions/project.d.ts

@@ -482,7 +482,7 @@ interface ITestExecutionService {
 }
 
 interface ITestInitializationService {
-	getDependencies(framework: string): string[];
+	getDependencies(framework: string): IDependencyInformation[];
 	getDependenciesVersions(): IDictionary<string>;
 }
 

lib/services/test-initialization-service.ts

@@ -7,12 +7,11 @@ export class TestInitializationService implements ITestInitializationService {
 	constructor(private $fs: IFileSystem) { }
 
 	@cache()
-	public getDependencies(selectedFramework: string): string[] {
+	public getDependencies(selectedFramework: string): IDependencyInformation[] {
 		const dependenciesPath = path.join(this.configsPath, "test-dependencies.json");
 		const allDependencies: { name: string, framework: string }[] = this.$fs.readJson(dependenciesPath);
-		const targetFrameworkDependencies: string[] = allDependencies
-			.filter(dependency => !dependency.framework || dependency.framework === selectedFramework)
-			.map(dependency => dependency.name);
+		const targetFrameworkDependencies: IDependencyInformation[] = allDependencies
+			.filter(dependency => !dependency.framework || dependency.framework === selectedFramework);
 
 		return targetFrameworkDependencies;
 	}