Remove mock in unit tests

Author: randoum

.github/workflows/unittest.yml

@@ -11,7 +11,7 @@ jobs:
         node-version: [10.x, 12.x, 14.x, 15.x]
 
     env:
-      EMAIL_VALIDATION_API_KEY: ${{ secrets.API_KEY }}
+      IP_GEOLOCATION_API_KEY: ${{ secrets.API_KEY }}
 
     steps:
     - uses: actions/checkout@v2

README.md

@@ -156,5 +156,7 @@ To build the lib, you need to run `npm run build:lib`.
 
 ### Test
 
-To run the test suite, you need to run: `npm run test`.
+To run the test suite, you need the API key from the abstract website and you can run:
+
+    IP_GEOLOCATION_API_KEY=(your key here) npm run test
 

test/index.test.js

@@ -1,26 +1,7 @@
 import 'regenerator-runtime/runtime';
 import {AbstractIpGeolocation} from '../src'
-const mocks = require('./mocks.js')
 
-function mockFetch(status, data) {
-    const xhrMockObj = {
-        open: jest.fn(),
-        send: jest.fn(),
-        setRequestHeader: jest.fn(),
-        onreadystatechange: jest.fn(),
-        readyState: 4,
-        status,
-        responseText: JSON.stringify(data),
-    };
-
-    const xhrMockClass = () => xhrMockObj;
-
-    global.XMLHttpRequest = jest.fn().mockImplementation(xhrMockClass);
-
-    setTimeout(() => {
-        xhrMockObj.onreadystatechange();
-    }, 0);
-}
+const API_KEY = process.env.IP_GEOLOCATION_API_KEY;
 
 test('Should throw error when no key is configured', () => {
     let thrownError;
@@ -36,9 +17,7 @@ test('Should throw error when no key is configured', () => {
 });
 
 test('Request OK when everything is set up correctly', async () => {
-    mockFetch(200, mocks);
-
-    AbstractIpGeolocation.configure('API_KEY');
+    AbstractIpGeolocation.configure(API_KEY);
     let response = await AbstractIpGeolocation.look_up('166.171.248.255')
     expect(response.ip_address).toBe('166.171.248.255')
 });