fix(commerce): fix formatTime handling for get-command-execution implementation. fixes #441 (#442)

Author: thoughtassassin

src/commands/cloudmanager/commerce/get-command-execution.js

@@ -44,11 +44,11 @@ class GetCommandExecutionCommand extends BaseCommand {
       },
       startedAt: {
         header: 'Started At',
-        get: formatTime(startedAt),
+        get: formatTime('startedAt'),
       },
       completedAt: {
         header: 'Completed At',
-        get: formatTime(completedAt),
+        get: formatTime('completedAt'),
       },
       status: {},
     })

test/commands/commerce/get-command-execution.test.js

@@ -19,7 +19,7 @@ beforeEach(() => {
   resetCurrentOrgId()
 })
 
-test('get-command (commerce) - missing arg', async () => {
+test('get-command-execution (commerce) - missing arg', async () => {
   expect.assertions(4)
 
   const runResultOne = GetCommandExecutionCommand.run([])
@@ -34,15 +34,15 @@ test('get-command (commerce) - missing arg', async () => {
   )
 })
 
-test('get-command (commerce) - missing config', async () => {
+test('get-command-execution (commerce) - missing config', async () => {
   expect.assertions(2)
 
   const runResult = GetCommandExecutionCommand.run(['--programId', '5', '10', '20'])
   await expect(runResult instanceof Promise).toBeTruthy()
   await expect(runResult).rejects.toSatisfy(err => err.message === '[CloudManagerCLI:NO_IMS_CONTEXT] Unable to find IMS context aio-cli-plugin-cloudmanager.')
 })
 
-test('get-command (commerce)', async () => {
+test('get-command-execution (commerce)', async () => {
   setCurrentOrgId('good')
   mockSdk.getCommerceCommandExecution = jest.fn(() => {
     return Promise.resolve({
@@ -59,7 +59,7 @@ test('get-command (commerce)', async () => {
     })
   })
 
-  // expect.assertions(9)
+  expect.assertions(9)
 
   const runResult = GetCommandExecutionCommand.run(['--programId', '5', '10', '100'])
   await expect(runResult instanceof Promise).toBeTruthy()
@@ -79,7 +79,7 @@ test('get-command (commerce)', async () => {
   await expect(cli.action.stop).toHaveBeenCalled()
 })
 
-test('get-command (commerce) - api error', async () => {
+test('get-command-execution (commerce) - api error', async () => {
   setCurrentOrgId('good')
   mockSdk.getCommerceCommandExecution = jest.fn(() =>
     Promise.reject(new Error('Command failed.')),