🐛 Fix passing environment variables to Qodana container

JetBrains · Jan 4, 2022 · edd36a9 · edd36a9
1 parent 78235b1
commit edd36a9
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 9 deletions.
diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts
@@ -64,14 +64,14 @@ function inputsDefaultFixture(): Inputs {
     resultsDir: '${{ runner.temp }}/qodana-results',
     cacheDir: '${{ runner.temp }}/qodana-caches',
     additionalCacheHash: '',
-    additionalVolumes: [],
-    additionalEnvVars: [],
+    additionalVolumes: ['/tmp/project:/data/project'],
+    additionalEnvVars: ['TESTS=1', 'RANDOM_SEED=42'],
     inspectedDir: '',
     ideaConfigDir: '',
     baselinePath: '',
     baselineIncludeAbsent: false,
-    failThreshold: '',
-    profileName: '',
+    failThreshold: '10',
+    profileName: 'qodana.recommended',
     profilePath: '',
     gradleSettingsPath: '',
     changes: false,
@@ -97,9 +97,19 @@ function defaultDockerRunCommandFixture(): string[] {
     '-v',
     '${{ runner.temp }}/qodana-results:/data/results',
     '-u',
-    `${process.getuid()}:${process.getgid()}`,
+    '501:20',
+    '-v',
+    '/tmp/project:/data/project',
+    '-e',
+    'TESTS=1',
+    '-e',
+    'RANDOM_SEED=42',
     'jetbrains/qodana-jvm-community',
-    '--save-report'
+    '--save-report',
+    '--fail-threshold',
+    '10',
+    '-n',
+    'qodana.recommended'
   ]
 }
 

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/docker.ts b/src/docker.ts
@@ -40,7 +40,7 @@ export function getQodanaRunArgs(inputs: Inputs): string[] {
   }
   if (inputs.additionalEnvVars.length > 0) {
     for (const envVar of inputs.additionalEnvVars) {
-      args.push('e', envVar)
+      args.push('-e', envVar)
     }
   }
   if (inputs.ideaConfigDir !== '') {