fix(plugins/plugin-client-common): when there're 3 splits, clicking table cells in the first split fail

Fixes #6937

Author: myan9

plugins/plugin-client-common/src/components/Views/Terminal/ScrollableTerminal.tsx

@@ -914,7 +914,7 @@ export default class ScrollableTerminal extends React.PureComponent<Props, State
           : !(await thisSplit.facade.REPL.qexec<boolean>(request.spec.options.ifnot).catch(() => true))
         if (!respIf || !respIfNot) {
           const { cmdline } = request.spec.options
-          const mainSplit = this.findMainSplit(thisSplitIdx)
+          const mainSplit = this.findMainSplit(thisSplitIdx) || thisSplit
           request.spec.options.cmdline = undefined // null this out, since we got it!
           mainSplit.facade.REPL.pexec(cmdline)
           return

plugins/plugin-core-support/src/test/core-support2/split-terminals.ts

@@ -325,3 +325,65 @@ describe('split close and reopen', function(this: Common.ISuite) {
     }
   })
 })
+
+describe('click and show in splits', function(this: Common.ISuite) {
+  before(Common.before(this))
+  after(Common.after(this))
+  Util.closeAllExceptFirstTab.bind(this)()
+
+  const splitTheTerminalViaButton = splitViaButton.bind(this)
+  const count = expectSplits.bind(this)
+
+  let selector: string
+
+  const doClickAndValidate = (splitIndex: number) => {
+    return this.app.client.waitUntil(async () => {
+      await this.app.client.$(selector).then(_ => _.click())
+
+      const text = await this.app.client.$(`${Selectors.OUTPUT_LAST_FOR_SPLIT(splitIndex)}`).then(_ => _.getText())
+
+      return text === 'hi'
+    })
+  }
+
+  const clickAndValidate = splitIndex => {
+    it(`should click in the first split and show in split: ${splitIndex}`, async () => {
+      try {
+        await doClickAndValidate(splitIndex)
+      } catch (err) {
+        await Common.oops(this, true)(err)
+      }
+    })
+  }
+
+  it('should click in the first split and show in the second split', async () => {
+    try {
+      await CLI.command('echo hi', this.app).then(ReplExpect.okWithPtyOutput('hi'))
+
+      selector = `${await Util.doList(this, 'history 1', 'echo hi')} [data-value="echo hi"].clickable`
+      await doClickAndValidate(2)
+    } catch (err) {
+      await Common.oops(this, true)(err)
+    }
+  })
+
+  splitTheTerminalViaButton(3)
+  count(3)
+  clickAndValidate(3)
+
+  splitTheTerminalViaButton(4)
+  count(4)
+  clickAndValidate(4)
+
+  splitTheTerminalViaButton(5)
+  count(5)
+  clickAndValidate(5)
+
+  splitTheTerminalViaButton(6)
+  count(6)
+  clickAndValidate(6)
+
+  splitTheTerminalViaButton(7)
+  count(7)
+  clickAndValidate(7)
+})