Add skipped tab in test report (microsoft#963)

Author: jdneo

resources/templates/report.pug

@@ -35,18 +35,22 @@ html(lang="en")
     body.container-fluid
         ul.nav.nav-pills.mt-3(role='tablist')
             li.nav-item
-                button.btn.btn-sm.btn-info.active(id='all-tab', data-toggle='pill', href='#all', role='tab', aria-controls='all') All
+                button.btn.btn-sm.btn-info.active.mr-2.mb-2(id='all-tab', data-toggle='pill', href='#all', role='tab', aria-controls='all') All
                     span.badge.badge-light.ml-2 #{allCount}
             if failedCount
                 li.nav-item
-                    button.btn.btn-sm.btn-danger.ml-3(id='fail-tab', data-toggle='pill', href='#fail', role='tab', aria-controls='fail') Failed
+                    button.btn.btn-sm.btn-danger.mr-2.mb-2(id='fail-tab', data-toggle='pill', href='#fail', role='tab', aria-controls='fail') Failed
                         span.badge.badge-light.ml-2 #{failedCount}
             if passedCount
                 li.nav-item
-                    button.btn.btn-sm.btn-success.ml-3(id='pass-tab', data-toggle='pill', href='#pass', role='tab', aria-controls='pass') Passed
+                    button.btn.btn-sm.btn-success.mr-2.mb-2(id='pass-tab', data-toggle='pill', href='#pass', role='tab', aria-controls='pass') Passed
                         span.badge.badge-light.ml-2 #{passedCount}
+            if skippedCount
+                li.nav-item
+                    button.btn.btn-sm.btn-secondary.mb-2(id='skip-tab', data-toggle='pill', href='#skip', role='tab', aria-controls='skip') Skipped
+                        span.badge.badge-light.ml-2 #{skippedCount}
 
-        div.tab-content.mt-3
+        div.tab-content
             div.tab-pane.fade.show.active(id='all', role='tabpanel', aria-labelledby='all-tab')
                 +collapseMethodTable(tests, 'detail-all')
             if failedCount
@@ -55,3 +59,6 @@ html(lang="en")
             if passedCount
                 div.tab-pane.fade(id='pass', role='tabpanel', aria-labelledby='pass-tab')
                     +collapseMethodTable(passedTests, 'detail-pass')
+            if skippedCount
+                div.tab-pane.fade(id='skip', role='tabpanel', aria-labelledby='skip-tab')
+                    +collapseMethodTable(skippedTests, 'detail-skip')

resources/templates/report_method_table.pug

@@ -11,10 +11,10 @@ mixin collapseMethodTable(children, type)
                 - methodIdx++
                 li.list-group-item.list-group-item-action
                     div.row.accordion-toggle
-                        div.col-8.text-truncate.collapsed(data-toggle="collapse", data-target=`#${type}-${classIdx}-${methodIdx}`, style="cursor:pointer", title="View details")
+                        div.col-7.text-truncate.px-1.collapsed(data-toggle="collapse", data-target=`#${type}-${classIdx}-${methodIdx}`, style="cursor:pointer", title="View details")
                             include /images/chevron-right.svg
                             span.ml-1 #{method.displayName}
-                        div.col-1.text-right.p-0
+                        div.col-2.text-right.p-0
                             if !method.status
                                 span.badge.badge-warning Not run
                             else if method.status === 'Pass'
@@ -23,8 +23,8 @@ mixin collapseMethodTable(children, type)
                                 span.badge.badge-danger Failed
                             else
                                 span.badge.badge-secondary Skipped
-                        div.col-2.text-right #{method.duration >= 0 ? parseFloat((method.duration/1000).toFixed(2)) + "s" : "N/A"}
-                        div.col-1.text-right.pl-0
+                        div.col-2.text-right.p-0 #{method.duration >= 0 ? parseFloat((method.duration/1000).toFixed(2)) + "s" : "N/A"}
+                        div.col-1.text-right.px-1
                             a(href="#", title="Navigate to Source", uri=`${method.location && method.location.uri ? method.location.uri : ''}`, range=`${method.location && method.location.range ? JSON.stringify(method.location.range) : ''}`, fullname=`${method.fullName}`)
                                 include /images/go-to-file.svg
                     div.mt-2.pl-2.accordion-body(id=`${type}-${classIdx}-${methodIdx}`, class="collapse")

src/testReportProvider.ts

@@ -89,6 +89,7 @@ class TestReportProvider implements Disposable {
         const allResultsMap: Map<string, ITestReportItem[]> = new Map();
         const passedResultMap: Map<string, ITestReportItem[]> = new Map();
         const failedResultMap: Map<string, ITestReportItem[]> = new Map();
+        const skippedResultMap: Map<string, ITestReportItem[]> = new Map();
         let passedCount: number = 0;
         let failedCount: number = 0;
         let skippedCount: number = 0;
@@ -115,6 +116,7 @@ class TestReportProvider implements Disposable {
                         failedCount++;
                         break;
                     case TestStatus.Skip:
+                        this.putMethodResultIntoMap(skippedResultMap, reportItem, classFullName);
                         skippedCount++;
                         break;
                 }
@@ -125,6 +127,7 @@ class TestReportProvider implements Disposable {
             tests: allResultsMap,
             passedTests: passedResultMap,
             failedTests: failedResultMap,
+            skippedTests: skippedResultMap,
             allCount: testResults.length,
             passedCount,
             failedCount,