fix run:inside argument parsing order and add regression tests for dyno-command mapping

Author: michaelmalave

src/commands/run/inside.ts

@@ -6,21 +6,21 @@ import debugFactory from 'debug'
 import tsheredoc from 'tsheredoc'
 
 import Dyno from '../../lib/run/dyno.js'
-import {buildCommandWithLauncher} from '../../lib/run/helpers.js'
+import {buildCommandWithLauncher, revertSortedArgs} from '../../lib/run/helpers.js'
 
 const debug = debugFactory('heroku:run:inside')
 const heredoc = tsheredoc.default
 
 export default class RunInside extends Command {
   static args = {
-    command: Args.string({
-      description: 'command to run (Heroku automatically prepends \'launcher\' to the command)',
-      required: true,
-    }),
     dyno_name: Args.string({
       description: 'name of the dyno to run command inside',
       required: true,
     }),
+    command: Args.string({
+      description: 'command to run (Heroku automatically prepends \'launcher\' to the command)',
+      required: true,
+    }),
   }
 
   static description = 'run a command inside an existing dyno (for Fir-generation apps only)'
@@ -57,13 +57,15 @@ export default class RunInside extends Command {
 
   async run() {
     const {args, argv, flags} = await this.parse(RunInside)
+    const orderedArgs = revertSortedArgs(process.argv, argv as string[])
+    const commandArgs = orderedArgs.slice(1)
 
     const {dyno_name: dynoName} = args
     const {app: appName, 'exit-code': exitCode, listen, 'no-launcher': noLauncher} = flags
 
     const opts = {
       app: appName,
-      command: await buildCommandWithLauncher(this.heroku, appName, argv.slice(1) as string[], noLauncher),
+      command: await buildCommandWithLauncher(this.heroku, appName, commandArgs, noLauncher),
       dyno: dynoName,
       'exit-code': exitCode,
       heroku: this.heroku,

test/unit/commands/run/inside.unit.test.ts

@@ -32,7 +32,10 @@ describe('run:inside', function () {
     nock('https://api.heroku.com')
       .get('/apps/myapp')
       .reply(200, {name: 'myapp', stack: {name: 'heroku-20'}})
-      .post('/apps/myapp/dynos/web.1')
+      .post('/apps/myapp/dynos/web.1', body => {
+        expect(body.command).to.equal('bash')
+        return true
+      })
       .reply(201, {
         attach_url: 'rendezvous://rendezvous.runtime.heroku.com:5000',
         command: 'bash',
@@ -88,7 +91,10 @@ describe('run:inside', function () {
     nock('https://api.heroku.com')
       .get('/apps/myapp')
       .reply(200, {name: 'myapp', stack: {name: 'cnb'}})
-      .post('/apps/myapp/dynos/web.1')
+      .post('/apps/myapp/dynos/web.1', body => {
+        expect(body.command).to.equal('bash')
+        return true
+      })
       .reply(201, {
         attach_url: 'rendezvous://rendezvous.runtime.heroku.com:5000',
         command: 'bash',
@@ -111,4 +117,34 @@ describe('run:inside', function () {
       // Expected to fail when trying to connect
     })
   })
+
+  it('prepends launcher by default on cnb apps', async function () {
+    nock('https://api.heroku.com')
+      .get('/apps/myapp')
+      .reply(200, {name: 'myapp', stack: {name: 'cnb'}})
+      .post('/apps/myapp/dynos/web.1', body => {
+        expect(body.command).to.equal('launcher bash')
+        return true
+      })
+      .reply(201, {
+        attach_url: 'rendezvous://rendezvous.runtime.heroku.com:5000',
+        command: 'launcher bash',
+        created_at: '2020-01-01T00:00:00Z',
+        id: '12345678-1234-1234-1234-123456789012',
+        name: 'web.1',
+        size: 'basic',
+        state: 'starting',
+        type: 'web',
+        updated_at: '2020-01-01T00:00:00Z',
+      })
+
+    await runCommand(RunInside, [
+      'web.1',
+      'bash',
+      '--app',
+      'myapp',
+    ]).catch(() => {
+      // Expected to fail when trying to connect
+    })
+  })
 })