fix(plugins/plugin-kubectl): kubectl api-resources table shows 'unknown' in title

Fixes #4145

Author: myan9

plugins/plugin-kubectl/src/controller/kubectl/api-resources.ts

@@ -34,7 +34,7 @@ export const doGet = (command: string) => async (args: Arguments<KubeOptions>):
       content: { stderr, stdout }
     } = response
 
-    const table = stringToTable(stdout, stderr, args, command, 'explain')
+    const table = stringToTable(stdout, stderr, args, command, 'explain', 'api-resources')
 
     if (isTable(table)) {
       table.body.forEach(_ => {

plugins/plugin-kubectl/src/lib/view/formatTable.ts

@@ -145,7 +145,7 @@ export const preprocessTable = (raw: string[]): Pair[][][] => {
 
 /** normalize the status badge by capitalization */
 const capitalize = (str: string): string => {
-  return !str ? 'Unknown' : str[0].toUpperCase() + str.slice(1).toLowerCase()
+  return !str ? '' : str[0].toUpperCase() + str.slice(1).toLowerCase()
 }
 
 /**
@@ -294,7 +294,7 @@ export const formatTable = <O extends KubeOptions>(
                 ((idx > 0 && cssForKey[key]) || '') +
                 ' ' +
                 (cssForValue[column] || (key === 'READY' && cssForReadyCount(column)) || maybeRed(column)),
-              value: key === 'STATUS' && idx > 0 ? capitalize(column) : column
+              value: key === 'STATUS' && idx > 0 ? capitalize(column || 'unknown') : column
             })
           )
           .concat(fillTo(rows.length, maxColumns))

plugins/plugin-kubectl/src/test/k8s2/api-resources.ts

@@ -17,6 +17,8 @@
 import { Common, CLI, Selectors, ReplExpect } from '@kui-shell/test'
 import { createNS, allocateNS, doHelp } from '../../../tests/lib/k8s/utils'
 
+import * as assert from 'assert'
+
 const commonModes = ['Introduction', 'Options']
 const kubectlApiResourcesModes = commonModes.concat(['api-resources'])
 
@@ -29,7 +31,11 @@ describe('kubectl api-resources', function(this: Common.ISuite) {
 
   it('should get a list of api resources', () =>
     CLI.command('kubectl api-resources', this.app)
-      .then(ReplExpect.okWithCustom({ selector: Selectors.BY_NAME('bindings') }))
+      .then(async res => {
+        await ReplExpect.okWithCustom({ selector: Selectors.BY_NAME('bindings') })(res)
+        const actualTitle = await this.app.client.getText(Selectors.TABLE_TITLE(res.count))
+        assert.strictEqual(actualTitle, 'Api-resources')
+      })
       .catch(Common.oops(this, true)))
 
   it('should get a list of api resources', () =>