Merge pull request #1318 from mikepenz/fix/1316

Include time in summary tables

Author: mikepenz

README.md

@@ -100,6 +100,7 @@ jobs:
 | `verbose_summary`          | Optional. Detail table will note if there were no test annotations for a test suite (Also applies to comment). Defaults to `true`.                                                                  |
 | `skip_success_summary`     | Optional. Skips the summary table if only successful tests were detected (Also applies to comment). Defaults to `false`.                                                                            |
 | `include_empty_in_summary` | Optional. Include entries in summaries that have 0 count. Defaults to `true`.                                                                                                                       |
+| `include_time_in_summary`  | Optional. Include spent time in summaries. Defaults to `false`.                                                                                                                                     |
 | `simplified_summary`       | Optional. Use icons instead of text to indicate status in summary. Defaults to `false`.                                                                                                             |
 | `group_suite`              | Optional. If enabled, will group the testcases by test suite in the `detailed_summary`. Defaults to `false`.                                                                                        |
 | `comment`                  | Optional. Enables a comment being added to the PR with the summary tables (Respects the summary configuration flags). Defaults to `false`.                                                          |

__tests__/table.test.ts

@@ -27,9 +27,13 @@ const NORMAL_TABLE = [
     {
       data: 'Failed',
       header: true
+    },
+    {
+      data: 'Time ⏱',
+      header: true
     }
   ],
-  ['checkName', '3 ran', '3 passed', '0 skipped', '0 failed']
+  ['checkName', '3 ran', '3 passed', '0 skipped', '0 failed', '100ms']
 ]
 const FLAKY_TABLE = [
   [
@@ -40,6 +44,10 @@ const FLAKY_TABLE = [
     {
       data: 'Retries',
       header: true
+    },
+    {
+      data: 'Time ⏱',
+      header: true
     }
   ]
 ]
@@ -71,17 +79,25 @@ describe('buildSummaryTables', () => {
         {
           data: 'Result',
           header: true
+        },
+        {
+          data: 'Time ⏱',
+          header: true
         }
       ],
       [
         {
           data: '<strong>checkName</strong>',
-          colspan: '2'
+          colspan: '3'
         }
       ],
-      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingAsync (Normal)', '✅ passed'],
-      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingServer (Normal)', '✅ passed'],
-      ['ABC-0045: Multi-User Chat/MultiUserIntegrationTest.mucRoleTestForReceivingModerator (Normal)', '✅ passed']
+      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingAsync (Normal)', '✅ passed', '54ms'],
+      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingServer (Normal)', '✅ passed', ''],
+      [
+        'ABC-0045: Multi-User Chat/MultiUserIntegrationTest.mucRoleTestForReceivingModerator (Normal)',
+        '✅ passed',
+        '46ms'
+      ]
     ])
     expect(flakyTable).toStrictEqual(FLAKY_TABLE)
   })
@@ -132,29 +148,37 @@ describe('buildSummaryTables', () => {
         {
           data: 'Result',
           header: true
+        },
+        {
+          data: 'Time ⏱',
+          header: true
         }
       ],
       [
         {
           data: '<strong>checkName</strong>',
-          colspan: '2'
+          colspan: '3'
         }
       ],
       [
         {
           data: '<em>ABC-0199: XMPP Ping</em>',
-          colspan: '2'
+          colspan: '3'
         }
       ],
-      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingAsync (Normal)', '✅ passed'],
-      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingServer (Normal)', '✅ passed'],
+      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingAsync (Normal)', '✅ passed', '54ms'],
+      ['ABC-0199: XMPP Ping/PingIntegrationTest.pingServer (Normal)', '✅ passed', ''],
       [
         {
           data: '<em>ABC-0045: Multi-User Chat</em>',
-          colspan: '2'
+          colspan: '3'
         }
       ],
-      ['ABC-0045: Multi-User Chat/MultiUserIntegrationTest.mucRoleTestForReceivingModerator (Normal)', '✅ passed']
+      [
+        'ABC-0045: Multi-User Chat/MultiUserIntegrationTest.mucRoleTestForReceivingModerator (Normal)',
+        '✅ passed',
+        '46ms'
+      ]
     ])
     expect(flakyTable).toStrictEqual(FLAKY_TABLE)
   })