fix: fix failing tests

Author: thomasconner

test/ipdata.test.ts

@@ -4,6 +4,7 @@ const API_KEY = 'test';
 
 describe('constructor()', () => {
   it('should throw an error if an apiKey is not provided', () => {
+    // eslint-disable-next-line @typescript-eslint/ban-ts-ignore
     // @ts-ignore
     expect(() => new IPData()).toThrowError('An API key is required.');
   });
@@ -12,28 +13,22 @@ describe('constructor()', () => {
     const ipdata = new IPData(API_KEY);
     expect(ipdata.apiKey).toEqual(API_KEY);
   });
-
-  it('should turn on the cache by default', () => {
-    const ipdata = new IPData(API_KEY);
-    expect(ipdata.useCache).toEqual(true);
-  });
-
-  it('should turn off the cache', () => {
-    const ipdata = new IPData(API_KEY, false);
-    expect(ipdata.useCache).toEqual(false);
-  });
 })
 
 describe('lookup()', () => {
-  const ipdata = new IPData(API_KEY, false);
+  const ipdata = new IPData(API_KEY);
   const IP = '1.1.1.1';
 
-  it('should throw an error if the ip is invalid', async () => {
+  afterEach(() => {
+    return ipdata.cache.reset();
+  });
+
+  it('should throw an error if the ip address is invalid', async () => {
     const badIP = '1.1.11';
     await expect(ipdata.lookup(badIP)).rejects.toThrowError(`${badIP} is an invalid IP address.`);
   });
 
-  it('should return default information when no ip is provided', async () => {
+  it('should return default information when no ip address is provided', async () => {
     const info = await ipdata.lookup();
     expect(info).toHaveProperty('ip');
     expect(info).toHaveProperty('status');
@@ -51,12 +46,26 @@ describe('lookup()', () => {
     expect(info).toHaveProperty('status');
   });
 
-  it('should set an ip in the uri', async () => {
+  it('should cache the ip address info for the default ip address', async () => {
+    await ipdata.lookup();
+    const info = ipdata.cache.get('DEFAULT_IP');
+    expect(info).toHaveProperty('ip');
+    expect(info).toHaveProperty('status');
+  });
+
+  it('should accept an ip address', async () => {
     const info = await ipdata.lookup(IP);
     expect(info).toHaveProperty('ip', IP);
     expect(info).toHaveProperty('status');
   });
 
+  it('should cache the ip address info', async () => {
+    await ipdata.lookup(IP);
+    const info = ipdata.cache.get(IP);
+    expect(info).toHaveProperty('ip', IP);
+    expect(info).toHaveProperty('status');
+  });
+
   it('should throw an error if a selectField and fields is provided', async () => {
     await expect(ipdata.lookup(null, 'field', ['field'])).rejects.toThrowError(
       'The selectField and fields parameters cannot be used at the same time.',
@@ -97,24 +106,28 @@ describe('lookup()', () => {
 });
 
 describe('bulkLookup()', () => {
-  const ipdata = new IPData(API_KEY, false);
+  const ipdata = new IPData(API_KEY);
   const IP1 = '1.1.1.1';
   const IP2 = '8.8.8.8';
 
-  it('should throw an error if less than 2 ips are provided', async () => {
+  afterEach(() => {
+    return ipdata.cache.reset();
+  });
+
+  it('should throw an error if less than 2 ip addresses are provided', async () => {
     const ips = [IP1];
     await expect(ipdata.bulkLookup(ips)).rejects.toThrowError(
       'Bulk Lookup requires more than 1 IP Address in the payload.',
     );
   });
 
-  it('should throw an error if an ip is invalid', async () => {
+  it('should throw an error if an ip address is invalid', async () => {
     const badIP = '1.1.11';
     const ips = [badIP, IP2];
     await expect(ipdata.bulkLookup(ips)).rejects.toThrowError(`${badIP} is an invalid IP address.`);
   });
 
-  it('should return info for the ips', async () => {
+  it('should return info for the ip addresses', async () => {
     const info = await ipdata.bulkLookup([IP1, IP2]);
     expect(info).toHaveProperty('responses');
     expect(info).toHaveProperty('status');

tsconfig.json

@@ -12,6 +12,7 @@
     "target": "es5"
   },
   "include": [
-    "src/**/*"
+    "src/**/*",
+    "test/**/*"
   ]
 }