Allow PIDUSAGE_USE_PS to have string type of "true" (#141)

soyuka · Oct 25, 2022 · 2dbd8c8 · 2dbd8c8
1 parent 766b657
commit 2dbd8c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -22,7 +22,7 @@ function pidusage (pids, options, callback) {
   }
 
   options = Object.assign({
-    usePs: process.env.PIDUSAGE_USE_PS,
+    usePs: /^true$/i.test(process.env.PIDUSAGE_USE_PS),
     maxage: process.env.PIDUSAGE_MAXAGE
   }, options)
 

diff --git a/test/ps.js b/test/ps.js
@@ -158,11 +158,11 @@ test('should be able to set usePs from env var', async t => {
   let usePsFromStats
 
   mockery.registerMock('./lib/stats', (_, options) => {
-    usePsFromStats = !!options.usePs
+    usePsFromStats = options.usePs
   })
 
   const beforeValue = process.env.PIDUSAGE_USE_PS
-  process.env.PIDUSAGE_USE_PS = '1'
+  process.env.PIDUSAGE_USE_PS = 'true'
 
   const pidusage = require('../')
   pidusage(1, () => {})