Replace test config with .env config.

Less risk of publishing api details.
chrisspiegl · Mar 22, 2022 · c21cdaf · c21cdaf
1 parent 54713ab
commit c21cdaf
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/.env.template b/.env.template
@@ -0,0 +1,2 @@
+PUSHNOTICE_CHAT_ID=
+PUSHNOTICE_CHAT_SECRET=
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
+.env
 *.sublime-project
 *.sublime-workspace
 node_modules

diff --git a/package.json b/package.json
@@ -43,6 +43,7 @@
   },
   "devDependencies": {
     "ava": "^4.1.0",
+    "dotenv": "^16.0.0",
     "npm-check-updates": "^12.5.4",
     "xo": "^0.48.0"
   }

diff --git a/test.js b/test.js
@@ -1,15 +1,16 @@
+import process from 'node:process'
+import 'dotenv/config.js'// eslint-disable-line import/no-unassigned-import
 import test from 'ava'
 import pnotice from './index.js'
 
-// TODO: find a way to load config.chat from file?
 const config = {
   chat: {
-    id: '',
-    secret: '',
+    id: process.env.PUSHNOTICE_CHAT_ID,
+    secret: process.env.PUSHNOTICE_CHAT_SECRET,
   },
 }
 
-if (config.chat.id !== '') {
+if (config.chat.id) {
   console.log('Test file currently being run:', test.meta.file)
 
   test('handle options.disabled = true', async (t) => {