From d8fc401229367a9d46f3282cb39ff593e51969dd Mon Sep 17 00:00:00 2001
From: Filipa Lacerda <filipa@gitlab.com>
Date: Tue, 7 Aug 2018 11:09:56 +0100
Subject: Show resolved failures

---
 .../components/grouped_test_reports_app_spec.js    | 36 +++++++++++++++++++++
 .../reports/mock_data/resolved_failures.json       | 37 ++++++++++++++++++++++
 2 files changed, 73 insertions(+)
 create mode 100644 spec/javascripts/reports/mock_data/resolved_failures.json

(limited to 'spec')

diff --git a/spec/javascripts/reports/components/grouped_test_reports_app_spec.js b/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
index d86e565036c..c40227b74df 100644
--- a/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
+++ b/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
@@ -7,6 +7,7 @@ import mountComponent from '../../helpers/vue_mount_component_helper';
 import newFailedTestReports from '../mock_data/new_failures_report.json';
 import successTestReports from '../mock_data/no_failures_report.json';
 import mixedResultsTestReports from '../mock_data/new_and_fixed_failures_report.json';
+import resolvedFailures from '../mock_data/resolved_failures.json';
 
 describe('Grouped Test Reports App', () => {
   let vm;
@@ -123,6 +124,41 @@ describe('Grouped Test Reports App', () => {
     });
   });
 
+  describe('with resolved failures', () => {
+    beforeEach(() => {
+      mock.onGet('test_results.json').reply(200, resolvedFailures, {});
+      vm = mountComponent(Component, {
+        endpoint: 'test_results.json',
+      });
+    });
+
+    it('renders summary text', done => {
+      setTimeout(() => {
+        expect(vm.$el.querySelector('.fa-spinner')).toBeNull();
+        expect(vm.$el.querySelector('.js-code-text').textContent.trim()).toEqual(
+          'Test summary contained 2 fixed test results out of 11 total tests',
+        );
+
+        expect(vm.$el.textContent).toContain(
+          'rspec:pg found 2 fixed test results out of 8 total tests',
+        );
+        done();
+      }, 0);
+    });
+
+    it('renders resolved failures', done => {
+      setTimeout(() => {
+        expect(vm.$el.querySelector('.js-mr-code-resolved-issues').textContent).toContain(
+          resolvedFailures.suites[0].resolved_failures[0].name,
+        );
+        expect(vm.$el.querySelector('.js-mr-code-resolved-issues').textContent).toContain(
+          resolvedFailures.suites[0].resolved_failures[1].name,
+        );
+        done()
+      }, 0);
+    });
+  });
+
   describe('with error', () => {
     beforeEach(() => {
       mock.onGet('test_results.json').reply(500, {}, {});
diff --git a/spec/javascripts/reports/mock_data/resolved_failures.json b/spec/javascripts/reports/mock_data/resolved_failures.json
new file mode 100644
index 00000000000..d1f347ce5e6
--- /dev/null
+++ b/spec/javascripts/reports/mock_data/resolved_failures.json
@@ -0,0 +1,37 @@
+{
+  "status": "success",
+  "summary": { "total": 11, "resolved": 2, "failed": 0 },
+  "suites": [
+    {
+      "name": "rspec:pg",
+      "status": "success",
+      "summary": { "total": 8, "resolved": 2, "failed": 0 },
+      "new_failures": [],
+      "resolved_failures": [
+        {
+          "status": "success",
+          "name": "Test#sum when a is 1 and b is 2 returns summary",
+          "execution_time": 0.000411,
+          "system_output": null,
+          "stack_trace": null
+        },
+        {
+          "status": "success",
+          "name": "Test#sum when a is 100 and b is 200 returns summary",
+          "execution_time": 7.6e-5,
+          "system_output": null,
+          "stack_trace": null
+        }
+      ],
+      "existing_failures": []
+    },
+    {
+      "name": "java ant",
+      "status": "success",
+      "summary": { "total": 3, "resolved": 0, "failed": 0 },
+      "new_failures": [],
+      "resolved_failures": [],
+      "existing_failures": []
+    }
+  ]
+}
-- 
cgit v1.2.1


From 0f0e70bc13a680efb438c0e6b6007c3f5fc98eca Mon Sep 17 00:00:00 2001
From: Filipa Lacerda <filipa@gitlab.com>
Date: Tue, 7 Aug 2018 13:53:39 +0100
Subject: Reset error state in success mutation to handle error-success state

---
 spec/javascripts/reports/components/grouped_test_reports_app_spec.js | 2 +-
 spec/javascripts/reports/store/mutations_spec.js                     | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

(limited to 'spec')

diff --git a/spec/javascripts/reports/components/grouped_test_reports_app_spec.js b/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
index c40227b74df..333cefe5f8a 100644
--- a/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
+++ b/spec/javascripts/reports/components/grouped_test_reports_app_spec.js
@@ -154,7 +154,7 @@ describe('Grouped Test Reports App', () => {
         expect(vm.$el.querySelector('.js-mr-code-resolved-issues').textContent).toContain(
           resolvedFailures.suites[0].resolved_failures[1].name,
         );
-        done()
+        done();
       }, 0);
     });
   });
diff --git a/spec/javascripts/reports/store/mutations_spec.js b/spec/javascripts/reports/store/mutations_spec.js
index 8f99d2675a5..7d19b16efb9 100644
--- a/spec/javascripts/reports/store/mutations_spec.js
+++ b/spec/javascripts/reports/store/mutations_spec.js
@@ -72,6 +72,10 @@ describe('Reports Store Mutations', () => {
       expect(stateCopy.isLoading).toEqual(false);
     });
 
+    it('should reset hasError', () => {
+      expect(stateCopy.hasError).toEqual(false);
+    });
+
     it('should set summary counts', () => {
       expect(stateCopy.summary.total).toEqual(mockedResponse.summary.total);
       expect(stateCopy.summary.resolved).toEqual(mockedResponse.summary.resolved);
-- 
cgit v1.2.1