From fc9adc95a4cf2ed591d6477bd86935e9a31cf95f Mon Sep 17 00:00:00 2001
From: Vladimir Volek <vladimir.volek@satoshilabs.com>
Date: Thu, 13 Sep 2018 13:34:31 +0200
Subject: [PATCH] Added more tests

---
 .../__snapshots__/networkUtils.test.js.snap   | 12 +++++--
 src/utils/__tests__/networkUtils.test.js      | 36 +++++++++++++++++--
 2 files changed, 43 insertions(+), 5 deletions(-)

diff --git a/src/utils/__tests__/__snapshots__/networkUtils.test.js.snap b/src/utils/__tests__/__snapshots__/networkUtils.test.js.snap
index 514a0f3d..b3339859 100644
--- a/src/utils/__tests__/__snapshots__/networkUtils.test.js.snap
+++ b/src/utils/__tests__/__snapshots__/networkUtils.test.js.snap
@@ -1,13 +1,21 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`network utils JSONRequest ok 1`] = `
+exports[`network utils JSONRequest 1`] = `
 Object {
   "test_json": "01234",
 }
 `;
 
-exports[`network utils httpRequest json ok response 1`] = `
+exports[`network utils JSONRequest error 1`] = `[Error: jsonRequest error: [object Object]]`;
+
+exports[`network utils httpRequest error (500) 1`] = `[Error: /test/ Internal Server Error]`;
+
+exports[`network utils httpRequest json ok response binary 1`] = `Promise {}`;
+
+exports[`network utils httpRequest json ok response json 1`] = `
 Object {
   "test_json": "12345",
 }
 `;
+
+exports[`network utils httpRequest json ok response text 1`] = `Promise {}`;
diff --git a/src/utils/__tests__/networkUtils.test.js b/src/utils/__tests__/networkUtils.test.js
index b5b00116..8b306c4d 100644
--- a/src/utils/__tests__/networkUtils.test.js
+++ b/src/utils/__tests__/networkUtils.test.js
@@ -5,15 +5,45 @@ describe('network utils', () => {
         fetch.resetMocks();
     });
 
-    it('httpRequest json ok response', async () => {
+    it('httpRequest json ok response json', async () => {
         fetch.mockResponse(JSON.stringify({ test_json: '12345' }));
-        const result = await networkUtils.httpRequest('/test/', 'json');
+        const result = await networkUtils.httpRequest('/http-request-test-response-json/', 'json');
         expect(result).toMatchSnapshot();
     });
 
-    it('JSONRequest ok', async () => {
+    it('httpRequest json ok response text', async () => {
+        fetch.mockResponse(JSON.stringify({ test_json: '12345' }));
+        const result = networkUtils.httpRequest('/http-request-test-response-text/', 'text');
+        expect(result).toMatchSnapshot();
+    });
+
+    it('httpRequest json ok response binary', async () => {
+        fetch.mockResponse(1);
+        const result = networkUtils.httpRequest('/http-request-test-response-binary/', 'binary');
+        expect(result).toMatchSnapshot();
+    });
+
+    it('httpRequest error (500)', async () => {
+        fetch.mockResponse(JSON.stringify({ test_json: '12345' }), { status: 500 });
+        try {
+            await networkUtils.httpRequest('/test/', 'json');
+        } catch (err) {
+            expect(err).toMatchSnapshot();
+        }
+    });
+
+    it('JSONRequest', async () => {
         fetch.mockResponse(JSON.stringify({ test_json: '01234' }));
         const result = await networkUtils.JSONRequest('/test/');
         expect(result).toMatchSnapshot();
     });
+
+    it('JSONRequest error', async () => {
+        fetch.mockResponse(JSON.stringify({ test_json: '12345' }), { status: 500 });
+        try {
+            await networkUtils.JSONRequest('/test/', 'json');
+        } catch (err) {
+            expect(err).toMatchSnapshot();
+        }
+    });
 });