chore: streamline logging

Author: savetheclocktower

lib/job-manager.js

@@ -42,8 +42,8 @@ class JobManager {
   }
 
   createWorker () {
-    console.log('JobManager creating worker at:', nodeBin, this.workerPath);
     let nodeBin = Config.get('nodeBin');
+    console.debug('JobManager creating worker at:', nodeBin, this.workerPath);
     this.killWorker();
 
     // We should not try to start the worker without testing the value we have
@@ -101,7 +101,7 @@ class JobManager {
 
   receiveMessage (data) {
     if (data.log) {
-      console.log('WORKER LOG:', data.log);
+      console.debug('WORKER LOG:', data.log);
       return;
     }
     let key = data.key;
@@ -138,7 +138,7 @@ class JobManager {
   async send (bundle) {
     let key = generateKey();
     bundle.key = key;
-    console.log('JobManager#send:', bundle);
+    console.debug('JobManager#send:', bundle);
     try {
       this.ensureWorker();
     } catch (err) {

spec/linter-eslint-node-spec.js

@@ -44,8 +44,6 @@ function getNotification(expectedMessage) {
     * @param {Promise<import("atom").Notification>} notification
     */
     const newNotification = (notification) => {
-      console.warn(`Got a notification with text:`, notification.getMessage());
-      console.warn(`We want it to have text:`, expectedMessage);
       if (notification.getMessage() !== expectedMessage) {
         // As the specs execute asynchronously, it's possible a notification
         // from a different spec was grabbed, if the message doesn't match what
@@ -95,7 +93,6 @@ async function makeFixes(textEditor, expectedFixCount) {
   await atom.commands.dispatch(atom.views.getView(textEditor), 'linter-eslint-node:fix-file');
 
   const notification = await notificationPromise;
-  console.log('THE MESSAGE IS NOW:', notification.getMessage());
   expect(notification.getMessage()).toBe(expectedMessage);
   expect(notification.getType()).toBe('success');