add better console info when no log options are availble. fixes #51 (#52)

* add better console info when no log options are availble. fixes #51

* coverage

Author: justinedelson

src/commands/cloudmanager/list-available-log-options.js

@@ -38,16 +38,20 @@ class ListAvailableLogOptionsCommand extends Command {
             this.error(error.message)
         }
 
-        cli.table(result, {
-            id: {
-                header: "Environment Id",
-                get: () => args.environmentId
-            },
-            service: {},
-            name: {}
-        }, {
-                printLine: this.log
-            })
+        if (result.length > 0) {
+            cli.table(result, {
+                id: {
+                    header: "Environment Id",
+                    get: () => args.environmentId
+                },
+                service: {},
+                name: {}
+            }, {
+                    printLine: this.log
+                })
+        } else {
+            cli.info(`No log options are available for environmentId ${args.environmentId}`)
+        }
 
         return result
     }

test/__mocks__/cli-ux.js

@@ -16,6 +16,7 @@ module.exports = {
         action: {
             start: jest.fn(),
             stop: jest.fn()
-        }
+        },
+        info: jest.fn()
     }
 };

test/commands/list-available-log-options.test.js

@@ -10,6 +10,7 @@ OF ANY KIND, either express or implied. See the License for the specific languag
 governing permissions and limitations under the License.
 */
 
+const { cli } = require('cli-ux')
 const { setStore } = require('@adobe/aio-cna-core-config')
 const ListAvailableLogOptionsCommand = require('../../src/commands/cloudmanager/list-available-log-options')
 
@@ -70,11 +71,12 @@ test('list-available-logs - success undefined', async () => {
         'cloudmanager_programid': "4"
     })
 
-    expect.assertions(2)
+    expect.assertions(3)
 
     let runResult = ListAvailableLogOptionsCommand.run(["3"])
     await expect(runResult instanceof Promise).toBeTruthy()
     await expect(runResult).resolves.toEqual([])
+    await expect(cli.info.mock.calls[0][0]).toBe("No log options are available for environmentId 3")
 })
 
 test('list-available-logs - success empty', async () => {
@@ -88,11 +90,12 @@ test('list-available-logs - success empty', async () => {
         'cloudmanager_programid': "4"
     })
 
-    expect.assertions(2)
+    expect.assertions(3)
 
     let runResult = ListAvailableLogOptionsCommand.run(["2"])
     await expect(runResult instanceof Promise).toBeTruthy()
     await expect(runResult).resolves.toEqual([])
+    await expect(cli.info.mock.calls[0][0]).toBe("No log options are available for environmentId 2")
 })
 
 test('list-available-logs - success', async () => {
@@ -106,7 +109,7 @@ test('list-available-logs - success', async () => {
         'cloudmanager_programid': "4"
     })
 
-    expect.assertions(2)
+    expect.assertions(3)
 
     let runResult = ListAvailableLogOptionsCommand.run(["1"])
     await expect(runResult instanceof Promise).toBeTruthy()
@@ -146,6 +149,8 @@ test('list-available-logs - success', async () => {
         "service" : "dispatcher",
         "name" : "aemdispatcher"
     }])
+
+    await expect(cli.table.mock.calls[0][1].id.get({environmentId: 1})).toBe("1")
 })