feat: check user for tester container and env file (#57)

* feat: check user for tester container and env file
* fix: missing space
* fix: lint

bin/cmds/test.js

@@ -199,6 +199,11 @@ exports.builder = (yargs) => (
       type: 'boolean',
       default: false,
     })
+    .option('env-file', {
+      alias: 'env_file',
+      describe: '.env file for docker-compose',
+      type: 'string',
+    })
     .help()
 );
 exports.handler = () => {};

bin/cmds/test_cmds/run.js

@@ -72,6 +72,10 @@ exports.handler = async (argv) => {
     }
   }
 
+  if (argv.envFile) {
+    composeArgs.unshift('--env-file', argv.envFile);
+  }
+
   // start containers
   if ((await echoAndExec(compose, [...composeArgs, 'up', '-d'])).exitCode !== 0) {
     echo('failed to start docker containers. Exit 128');
@@ -199,6 +203,40 @@ exports.handler = async (argv) => {
     };
   }
 
+  async function checkUser(name) {
+    echo(`checking container user ${name}`);
+
+    try {
+      await dockerExec('getent', ['passwd', name], { user: 'root' });
+      return true;
+    } catch (e) {
+      return false;
+    }
+  }
+
+  async function createUser(name, uid) {
+    echo(`create user "${name}"`);
+
+    const userExtraArgs = uid ? ['-u', uid, '-g', uid] : [];
+    await dockerExec('adduser', ['-D', ...userExtraArgs, name], { user: 'root' });
+  }
+
+  if (!argv.isRootless) {
+    const uid = process.getuid();
+
+    if (!(await checkUser(uid))) {
+      createUser('tester', uid);
+    }
+
+    if (argv.ruser && !(await checkUser(argv.ruser))) {
+      createUser(argv.ruser);
+    }
+
+    if (argv.tuser && !(await checkUser(argv.tuser))) {
+      createUser(argv.tuser);
+    }
+  }
+
   // easy way to wait for containers, can do improved detection, but it's not generic
   if (argv.rebuild.length > 0) {
     echo('rebuilding modules');