fix: remove getDependenciesVersions method from test-initialization-service

Author: Fatme

lib/commands/test-init.ts

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

lib/definitions/project.d.ts

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

lib/services/test-initialization-service.ts

@@ -4,24 +4,28 @@ import { cache } from "../common/decorators";
 export class TestInitializationService implements ITestInitializationService {
 	private configsPath = path.join(__dirname, "..", "..", "config");
 
-	constructor(private $fs: IFileSystem) { }
+	constructor(private $errors: IErrors,
+		private $fs: IFileSystem) { }
 
 	@cache()
 	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: IDependencyInformation[] = allDependencies
-			.filter(dependency => !dependency.framework || dependency.framework === selectedFramework);
-
-		return targetFrameworkDependencies;
-	}
+		const allDependencies: { name: string, framework?: string, excludedPeerDependencies?: string[] }[] = this.$fs.readJson(dependenciesPath);
 
-	@cache()
-	public getDependenciesVersions(): IDictionary<string> {
 		const dependenciesVersionsPath = path.join(this.configsPath, "test-deps-versions-generated.json");
 		const dependenciesVersions = this.$fs.readJson(dependenciesVersionsPath);
 
-		return dependenciesVersions;
+		const targetFrameworkDependencies: IDependencyInformation[] = allDependencies
+			.filter(dependency => !dependency.framework || dependency.framework === selectedFramework)
+			.map(dependency => {
+				const dependencyVersion = dependenciesVersions[dependency.name];
+				if (!dependencyVersion) {
+					this.$errors.failWithoutHelp(`'${dependency}' is not a registered dependency.`);
+				}
+				return { ...dependency, version: dependencyVersion };
+			});
+
+		return targetFrameworkDependencies;
 	}
 }