Clean up ConfigTester after decaffeinate

References #460

lib/ConfigTester.js

@@ -1,17 +1,12 @@
-/*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
-let ConfigTester;
-const child_process = require('child_process');
+'use strict';
 
 module.exports =
 
-//#*
-// Tests the user's PHP setup to see if it is properly usable.
-//#
-(ConfigTester = class ConfigTester {
+/**
+ * Tests the user's PHP setup to see if it is properly usable.
+ */
+class ConfigTester
+{
     /**
      * Constructor.
      *
@@ -25,15 +20,17 @@ module.exports =
      * @return {Promise}
     */
     test() {
-        return new Promise((resolve, reject) => {
+        return new Promise((resolve) => {
             const process = this.phpInvoker.invoke(['-v']);
-            return process.on('close', code => {
+
+            process.on('close', (code) => {
                 if (code === 0) {
                     resolve(true);
+                    return;
                 }
 
-                return resolve(false);
+                resolve(false);
             });
         });
     }
-});
+};