Merge pull request #3 from arshadkazmi42/process-name-lib

Updated to use process-name library for getting process name

Author: arshadkazmi42

index.js

@@ -1,16 +1,18 @@
 const fkill = require('fkill');
 
-const { ProcessName } = require('./lib');
+const { ProcessName, ProcessNameConstants } = require('process-name');
 
 
 const chrome = async (force) => {
-  const procName = ProcessName.chrome();
+  const { BROWSERS: { CHROME } } = ProcessNameConstants;
+  const procName = ProcessName.BROWSERS[CHROME][process.platform];
   return await fkill(procName, { force: force || true });
 };
 
 
 const firefox = async (force) => {
-  const procName = ProcessName.firefox();
+  const { BROWSERS: { FIREFOX } } = ProcessNameConstants;
+  const procName = ProcessName.BROWSERS[FIREFOX][process.platform];
   return await fkill(procName, { force: force || true });
 };
 

lib/constants.js

@@ -9,8 +9,7 @@
     "test": "node_modules/mocha/bin/mocha tests/*"
   },
   "files": [
-    "index.js",
-    "lib/"
+    "index.js"
   ],
   "repository": {
     "type": "git",
@@ -38,6 +37,7 @@
   "author": "Arshad Kazmi",
   "license": "MIT",
   "dependencies": {
-    "fkill": "^6.2.0"
+    "fkill": "^6.2.0",
+    "process-name": "^1.0.1"
   }
 }

lib/index.js

@@ -2,7 +2,10 @@ const { expect } = require('chai');
 const processExists = require('process-exists');
 
 const BrKill = require('../index');
-const { ProcessName } = require('../lib');
+const { ProcessName, ProcessNameConstants } = require('process-name');
+const { BROWSERS: { CHROME, FIREFOX } } = ProcessNameConstants;
+const procNameChrome = ProcessName.BROWSERS[CHROME][process.platform];
+const procNameFirefox = ProcessName.BROWSERS[FIREFOX][process.platform];
 
 const ERROR_MESSAGE_CHROME = 'killing process chrome failed: process doesn\'t exist';
 const ERROR_MESSAGE_FIREFOX = 'killing process firefox failed: process doesn\'t exist';
@@ -15,7 +18,7 @@ describe('test kill browser process', () => {
   it('should kill chrome process', async () => {
     try  {
       await BrKill.chrome();
-      const exists = await processExists(ProcessName.chrome());
+      const exists = await processExists(procNameChrome);
       expect(exists).to.equal(false);
     } catch (err) {
       expect(err.message.toLowerCase().includes(ERROR_MESSAGE_CHROME)).to.equal(true);
@@ -24,7 +27,7 @@ describe('test kill browser process', () => {
   it('should kill firefox process', async () => {
     try  {
       await BrKill.firefox();
-      const exists = await processExists(ProcessName.firefox());      
+      const exists = await processExists(procNameFirefox);
       expect(exists).to.equal(false);
     } catch (err) {
       expect(err.message.toLowerCase().includes(ERROR_MESSAGE_FIREFOX)).to.equal(true);