fix: revert http as default communication mode

Author: AVVS

bin/cmds/test.js

@@ -165,7 +165,7 @@ exports.builder = (yargs) => (
     .option('http', {
       describe: 'uses http exec instead of docker exec to run tests',
       type: 'boolean',
-      default: true,
+      default: false,
     })
     .option('in-one', {
       alias: 'in_one',

bin/cmds/test_cmds/auto-compose.js

@@ -107,7 +107,7 @@ function tester(compose, argv) {
     }
   }
 
-  if (argv.services.includes('redisCluster')) {
+  if (argv.services && argv.services.includes('redisCluster')) {
     const cmdWait = process.env.DEPLOY_CLUSTER_SCRIPT || '/deploy-scripts/wait-for-cluster.sh';
     volumes.push(
       `${resolve(__dirname, '../../../scripts/wait-for-cluster.sh')}:${cmdWait}`

bin/cmds/test_cmds/run.js

@@ -82,7 +82,7 @@ exports.handler = async (argv) => {
    */
   let client;
   let dockerExec;
-  if (argv.http) {
+  if (argv.auto_compose && argv.http) {
     const getSocketId = async (attempt = 0) => {
       const { stdout } = await execAsync('docker', ['logs', container]);
       const lines = stdout.split('\n');