fix: Should setup TestNG correctly (#1362)

Signed-off-by: Sheng Chen <[email protected]>
microsoft · Jan 24, 2022 · 41e71f6 · 41e71f6
1 parent b9563a2
commit 41e71f6
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/commands/testDependenciesCommands.ts b/src/commands/testDependenciesCommands.ts
@@ -159,6 +159,10 @@ function getJarIds(testKind: TestKind): IArtifactMetadata[] {
                 groupId: 'com.beust',
                 artifactId: 'jcommander',
                 defaultVersion: '1.81',
+            }, {
+                groupId: 'org.slf4j',
+                artifactId: 'slf4j-api',
+                defaultVersion: '1.7.33',
             }];
         default:
             return [];
@@ -255,9 +259,11 @@ async function updateProjectSettings(projectUri: Uri, libFolder: string): Promis
     let referencedLibraries: any = configuration.get('project.referencedLibraries');
     if (_.isArray(referencedLibraries)) {
         referencedLibraries.push(testDependencies);
+        referencedLibraries = Array.from(new Set(referencedLibraries));
     } else if (_.isObject(referencedLibraries)) {
         referencedLibraries = referencedLibraries as {include: string[]};
         referencedLibraries.include.push(testDependencies);
+        referencedLibraries.include = Array.from(new Set(referencedLibraries.include));
         if (!referencedLibraries.exclude && !referencedLibraries.sources) {
             referencedLibraries = referencedLibraries.include;
         }