refactor: move preCommands out of docker options

lib/manager/bundler/artifacts.ts

@@ -183,8 +183,8 @@ export async function updateArtifacts(
         image: 'ruby',
         tagScheme: 'ruby',
         tagConstraint: await getRubyConstraint(updateArtifact),
-        preCommands,
       },
+      preCommands,
     };
     await exec(cmd, execOptions);
 

lib/manager/gradle/deep/index.ts

@@ -74,8 +74,8 @@ export async function executeGradle(
       tagConstraint:
         config.constraints?.java ?? (await getDockerConstraint(gradleRoot)),
       tagScheme: getJavaVersioning(),
-      preCommands: await getDockerPreCommands(gradleRoot),
     },
+    preCommands: await getDockerPreCommands(gradleRoot),
     extraEnv,
   };
   try {

lib/manager/mix/artifacts.ts

@@ -75,8 +75,8 @@ export async function updateArtifacts({
     cwdFile: packageFileName,
     docker: {
       image: 'elixir',
-      preCommands,
     },
+    preCommands,
   };
   const command = [
     'mix',

lib/manager/npm/post-update/lerna.ts

@@ -83,8 +83,8 @@ export async function generateLockFiles(
         image: 'node',
         tagScheme: 'node',
         tagConstraint,
-        preCommands,
       },
+      preCommands,
     };
     // istanbul ignore if
     if (GlobalConfig.get('exposeAllEnv')) {

lib/manager/npm/post-update/npm.ts

@@ -67,8 +67,8 @@ export async function generateLockFile(
         image: 'node',
         tagScheme: 'node',
         tagConstraint,
-        preCommands,
       },
+      preCommands,
     };
     // istanbul ignore if
     if (GlobalConfig.get('exposeAllEnv')) {

lib/manager/npm/post-update/pnpm.ts

@@ -46,8 +46,8 @@ export async function generateLockFile(
         image: 'node',
         tagScheme: 'node',
         tagConstraint,
-        preCommands,
       },
+      preCommands,
     };
     // istanbul ignore if
     if (GlobalConfig.get('exposeAllEnv')) {

lib/manager/npm/post-update/yarn.ts

@@ -148,8 +148,8 @@ export async function generateLockFile(
         image: 'node',
         tagScheme: 'node',
         tagConstraint,
-        preCommands,
       },
+      preCommands,
     };
     // istanbul ignore if
     if (GlobalConfig.get('exposeAllEnv')) {

lib/manager/pip-compile/artifacts.ts

@@ -67,10 +67,10 @@ export async function updateArtifacts({
         image: 'python',
         tagConstraint,
         tagScheme: 'pep440',
-        preCommands: [
-          `pip install --user ${quote(`pip-tools${pipToolsConstraint}`)}`,
-        ],
       },
+      preCommands: [
+        `pip install --user ${quote(`pip-tools${pipToolsConstraint}`)}`,
+      ],
     };
     logger.debug({ cmd }, 'pip-compile command');
     await exec(cmd, execOptions);

lib/manager/pip_requirements/artifacts.ts

@@ -40,8 +40,8 @@ export async function updateArtifacts({
       docker: {
         image: 'python',
         tagScheme: 'pip_requirements',
-        preCommands: ['pip install hashin'],
       },
+      preCommands: ['pip install hashin'],
     };
     await exec(cmd, execOptions);
     const newContent = await readLocalFile(packageFileName, 'utf8');

lib/manager/pipenv/artifacts.ts

@@ -103,10 +103,8 @@ export async function updateArtifacts({
         image: 'python',
         tagConstraint,
         tagScheme: 'pep440',
-        preCommands: [
-          `pip install --user ${quote(`pipenv${pipenvConstraint}`)}`,
-        ],
       },
+      preCommands: [`pip install --user ${quote(`pipenv${pipenvConstraint}`)}`],
     };
     logger.debug({ cmd }, 'pipenv lock command');
     await exec(cmd, execOptions);

lib/manager/poetry/artifacts.ts

@@ -139,8 +139,8 @@ export async function updateArtifacts({
         image: 'python',
         tagConstraint,
         tagScheme: 'poetry',
-        preCommands: [poetryInstall],
       },
+      preCommands: [poetryInstall],
     };
     await exec(cmd, execOptions);
     const newPoetryLockContent = await readLocalFile(lockFileName, 'utf8');

lib/util/exec/common.ts

@@ -20,8 +20,6 @@ export interface DockerOptions {
   volumes?: Opt<VolumeOption[]>;
   envVars?: Opt<Opt<string>[]>;
   cwd?: Opt<string>;
-  preCommands?: DockerExtraCommands;
-  postCommands?: DockerExtraCommands;
 }
 
 export interface RawExecOptions extends ChildProcessExecOptions {

lib/util/exec/docker/index.spec.ts

@@ -213,8 +213,6 @@ describe('util/exec/docker/index', () => {
     const envVars = ['FOO', 'BAR'];
     const image = 'sample_image';
     const dockerOptions = {
-      preCommands,
-      postCommands,
       image,
       cwd: '/tmp/foobar',
       envVars,
@@ -236,7 +234,12 @@ describe('util/exec/docker/index', () => {
 
     it('returns executable command', async () => {
       mockExecAll(exec);
-      const res = await generateDockerCommand(commands, dockerOptions);
+      const res = await generateDockerCommand(
+        commands,
+        preCommands,
+        postCommands,
+        dockerOptions
+      );
       expect(res).toBe(command(image));
     });
 
@@ -247,10 +250,15 @@ describe('util/exec/docker/index', () => {
         ['/tmp/bar', `/tmp/bar`],
         ['/tmp/baz', `/home/baz`],
       ];
-      const res = await generateDockerCommand(commands, {
-        ...dockerOptions,
-        volumes: [...volumes, ...volumes],
-      });
+      const res = await generateDockerCommand(
+        commands,
+        preCommands,
+        postCommands,
+        {
+          ...dockerOptions,
+          volumes: [...volumes, ...volumes],
+        }
+      );
       expect(res).toBe(
         command(
           image,
@@ -261,10 +269,15 @@ describe('util/exec/docker/index', () => {
 
     it('handles tag parameter', async () => {
       mockExecAll(exec);
-      const res = await generateDockerCommand(commands, {
-        ...dockerOptions,
-        tag: '1.2.3',
-      });
+      const res = await generateDockerCommand(
+        commands,
+        preCommands,
+        postCommands,
+        {
+          ...dockerOptions,
+          tag: '1.2.3',
+        }
+      );
       expect(res).toBe(command(`${image}:1.2.3`));
     });
 
@@ -277,11 +290,16 @@ describe('util/exec/docker/index', () => {
           { version: '2.0.0' },
         ],
       } as never);
-      const res = await generateDockerCommand(commands, {
-        ...dockerOptions,
-        tagScheme: 'npm',
-        tagConstraint: '^1.2.3',
-      });
+      const res = await generateDockerCommand(
+        commands,
+        preCommands,
+        postCommands,
+        {
+          ...dockerOptions,
+          tagScheme: 'npm',
+          tagConstraint: '^1.2.3',
+        }
+      );
       expect(res).toBe(command(`${image}:1.2.4`));
     });
   });

lib/util/exec/docker/index.ts

@@ -199,13 +199,13 @@ export async function removeDanglingContainers(): Promise<void> {
 
 export async function generateDockerCommand(
   commands: string[],
+  preCommands: string[] = [],
+  postCommands: string[] = [],
   options: DockerOptions
 ): Promise<string> {
   const { envVars, cwd, tagScheme, tagConstraint } = options;
   let image = options.image;
   const volumes = options.volumes || [];
-  const preCommands = options.preCommands || [];
-  const postCommands = options.postCommands || [];
   const {
     localDir,
     cacheDir,

lib/util/exec/index.spec.ts

@@ -464,9 +464,9 @@ describe('util/exec/index', () => {
         inOpts: {
           docker: {
             image,
-            preCommands: ['preCommand1', 'preCommand2', null],
-            postCommands: ['postCommand1', undefined, 'postCommand2'],
           },
+          preCommands: ['preCommand1', 'preCommand2', null],
+          postCommands: ['postCommand1', undefined, 'postCommand2'],
         },
         outCmd: [
           dockerPullCmd,
@@ -496,8 +496,6 @@ describe('util/exec/index', () => {
         inOpts: {
           docker: {
             image,
-            preCommands: null,
-            postCommands: undefined,
           },
         },
         outCmd: [

lib/util/exec/index.ts

@@ -5,6 +5,7 @@ import { TEMPORARY_ERROR } from '../../constants/error-messages';
 import { logger } from '../../logger';
 import { generateInstallCommands } from './buildpack';
 import {
+  DockerExtraCommands,
   DockerOptions,
   ExecResult,
   Opt,
@@ -22,6 +23,8 @@ export interface ExecOptions extends ChildProcessExecOptions {
   extraEnv?: Opt<ExtraEnv>;
   docker?: Opt<DockerOptions>;
   toolConstraints?: Opt<ToolConstraint[]>;
+  preCommands?: DockerExtraCommands;
+  postCommands?: DockerExtraCommands;
 }
 
 function getChildEnv({
@@ -74,6 +77,8 @@ function getRawExecOptions(opts: ExecOptions): RawExecOptions {
   delete execOptions.docker;
   delete execOptions.cwdFile;
   delete execOptions.toolConstraints;
+  delete execOptions.preCommands;
+  delete execOptions.postCommands;
 
   const childEnv = getChildEnv(opts);
   const cwd = getCwd(opts);
@@ -125,12 +130,14 @@ async function prepareRawExec(
     const envVars = dockerEnvVars(extraEnv, childEnv);
     const cwd = getCwd(opts);
     const dockerOptions: DockerOptions = { ...docker, cwd, envVars };
-    dockerOptions.preCommands = [
+    const preCommands = [
       ...(await generateInstallCommands(opts.toolConstraints)),
-      ...(dockerOptions.preCommands || []),
+      ...(opts.preCommands || []),
     ];
     const dockerCommand = await generateDockerCommand(
       rawCommands,
+      preCommands,
+      opts.postCommands,
       dockerOptions
     );
     rawCommands = [dockerCommand];