@k03mad/ip2geo 2.0.0 → 2.1.0

package/README.md

@@ -8,7 +8,7 @@
 
 ```bash
 npm i @k03mad/ip2geo --save-exact
-echo "/geoip" >> .gitignore
+echo .geoip >> .gitignore
 ```
 
 ```js
@@ -23,7 +23,7 @@ const {
     isp,
 } = await ip2geo('1.1.1.1', {
     // defaults
-    cacheDir: 'geoip',
+    cacheDir: '.geoip',
     cacheFileName: 'ips.log',
     cacheFileSeparator: ';;',
     cacheFileNewline: '\n',

package/app/lib/ip2geo.js

@@ -21,7 +21,7 @@ const debug = _debug('mad:geoip');
 
 const API = 'https://ipwho.is/';
 
-const DEFAULT_CACHE_FILE_DIR = 'geoip';
+const DEFAULT_CACHE_FILE_DIR = '.geoip';
 const DEFAULT_CACHE_FILE_NAME = 'ips.log';
 const DEFAULT_CACHE_FILE_SEPARATOR = ';;';
 const DEFAULT_CACHE_FILE_NEWLINE = '\n';
@@ -61,7 +61,7 @@ const collectOutputData = dataArr => {
  * @returns {string}
  */
 const getCacheFileFullPath = (ip, cacheDir, cacheFileName) => {
-    const [firstOctet] = ip.split('.');
+    const [firstOctet] = ip.split(/\.|:/);
     return path.join(cacheDir, `${firstOctet}_${cacheFileName}`);
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@k03mad/ip2geo",
-  "version": "2.0.0",
+  "version": "2.1.0",
   "description": "GeoIP library",
   "maintainers": [
     "Kirill Molchanov <k03.mad@gmail.com"

package/tests/helpers/path.js

@@ -0,0 +1,7 @@
+import path from 'node:path';
+
+/**
+ * @param {string} file import.meta.url
+ * @returns {string}
+ */
+export const getCurrentFilename = file => path.basename(file, '.js');

package/tests/opts-assigned-subfolder.js

@@ -1,47 +0,0 @@
-import assert from 'node:assert/strict';
-import {describe, it} from 'node:test';
-
-import {ip2geo} from '../app/index.js';
-
-import {checkCacheFile, removeCacheFolder} from './shared/fs.js';
-
-describe('opts-assigned', () => {
-    const CACHE_FILE_DIR = 'geoip-subfolder/hello_there';
-    const CACHE_FILE_NAME = 'ips.log';
-    const CACHE_FILE_SEPARATOR = ';;';
-    const CACHE_FILE_NEWLINE = '\n';
-
-    const REQUEST_IP = '9.9.9.9';
-
-    const cacheFile = `${REQUEST_IP.split('.')[0]}_${CACHE_FILE_NAME}`;
-
-    const response = {
-        ip: REQUEST_IP,
-        emoji: '🇨🇭',
-        country: 'Switzerland',
-        countryA2: 'CH',
-        region: 'Zurich',
-        city: 'Zürich',
-        org: 'Quad9',
-        isp: 'Quad9',
-        ispDomain: 'quad9.net',
-    };
-
-    removeCacheFolder(CACHE_FILE_DIR);
-
-    it(`should return correct response for IP: "${REQUEST_IP}"`, async () => {
-        const data = await ip2geo(REQUEST_IP, {
-            cacheDir: CACHE_FILE_DIR,
-        });
-
-        assert.deepEqual(data, response);
-    });
-
-    checkCacheFile(
-        CACHE_FILE_DIR,
-        cacheFile,
-        CACHE_FILE_SEPARATOR,
-        CACHE_FILE_NEWLINE,
-        response,
-    );
-});

package/tests/opts-assigned.js

@@ -1,50 +0,0 @@
-import assert from 'node:assert/strict';
-import {describe, it} from 'node:test';
-
-import {ip2geo} from '../app/index.js';
-
-import {checkCacheFile, removeCacheFolder} from './shared/fs.js';
-
-describe('opts-assigned', () => {
-    const CACHE_FILE_DIR = 'geoip-cache-opts';
-    const CACHE_FILE_NAME = 'ips.md';
-    const CACHE_FILE_SEPARATOR = '-_-';
-    const CACHE_FILE_NEWLINE = '%%%';
-
-    const REQUEST_IP = '8.8.8.8';
-
-    const cacheFile = `${REQUEST_IP.split('.')[0]}_${CACHE_FILE_NAME}`;
-
-    const response = {
-        ip: REQUEST_IP,
-        emoji: '🇺🇸',
-        country: 'United States',
-        countryA2: 'US',
-        region: 'California',
-        city: 'Mountain View',
-        org: 'Google LLC',
-        isp: 'Google LLC',
-        ispDomain: 'google.com',
-    };
-
-    removeCacheFolder(CACHE_FILE_DIR);
-
-    it(`should return correct response for IP: "${REQUEST_IP}"`, async () => {
-        const data = await ip2geo(REQUEST_IP, {
-            cacheDir: CACHE_FILE_DIR,
-            cacheFileName: CACHE_FILE_NAME,
-            cacheFileSeparator: CACHE_FILE_SEPARATOR,
-            cacheFileNewline: CACHE_FILE_NEWLINE,
-        });
-
-        assert.deepEqual(data, response);
-    });
-
-    checkCacheFile(
-        CACHE_FILE_DIR,
-        cacheFile,
-        CACHE_FILE_SEPARATOR,
-        CACHE_FILE_NEWLINE,
-        response,
-    );
-});

package/tests/opts-default.js

@@ -1,77 +0,0 @@
-import assert from 'node:assert/strict';
-import {describe, it} from 'node:test';
-
-import {ip2geo} from '../app/index.js';
-
-import {checkCacheFile, removeCacheFolder} from './shared/fs.js';
-
-describe('opts-default', () => {
-    const CACHE_FILE_DIR = 'geoip';
-    const CACHE_FILE_NAME = 'ips.log';
-    const CACHE_FILE_SEPARATOR = ';;';
-    const CACHE_FILE_NEWLINE = '\n';
-
-    const REQUEST_IP = '1.1.1.1';
-
-    const cacheFile = `${REQUEST_IP.split('.')[0]}_${CACHE_FILE_NAME}`;
-
-    const response = {
-        ip: REQUEST_IP,
-        emoji: '🇺🇸',
-        country: 'United States',
-        countryA2: 'US',
-        region: 'District of Columbia',
-        city: 'Washington',
-        org: 'APNIC and Cloudflare DNS Resolver project',
-        isp: 'Cloudflare, Inc.',
-        ispDomain: 'cloudflare.com',
-    };
-
-    const outputKeys = [
-        'ip',
-        'emoji',
-        'country',
-        'countryA2',
-        'region',
-        'city',
-        'org',
-        'isp',
-        'ispDomain',
-    ];
-
-    removeCacheFolder(CACHE_FILE_DIR);
-
-    describe('with ip arg', () => {
-        it(`should return correct response for IP: "${REQUEST_IP}"`, async () => {
-            const data = await ip2geo(REQUEST_IP);
-
-            assert.deepEqual(data, response);
-        });
-
-        checkCacheFile(
-            CACHE_FILE_DIR,
-            cacheFile,
-            CACHE_FILE_SEPARATOR,
-            CACHE_FILE_NEWLINE,
-            response,
-        );
-    });
-
-    describe('without ip arg', () => {
-        let data;
-
-        it('should request geoip without ip arg', async () => {
-            data = await ip2geo();
-        });
-
-        outputKeys.forEach(key => {
-            it(`should have "${key}" in request response`, () => {
-                assert.ok(data[key]);
-            });
-        });
-
-        it('should not have extra keys in request response', () => {
-            assert.deepEqual(Object.keys(data), outputKeys);
-        });
-    });
-});