make npm run test script actually work - fix all eslint issues

.eslintignore

@@ -0,0 +1 @@
+src/functions-templates

.eslintrc

@@ -1,6 +1,14 @@
 {
   "extends": ["oclif", "plugin:prettier/recommended"],
   "rules": {
-    "prettier/prettier": "error"
+    "no-warning-comments": "off",
+    "prettier/prettier": "error",
+    "no-process-exit": "off",
+    "unicorn/no-process-exit": "off",
+    "camelcase": "off",
+    "guard-for-in": "off",
+    "valid-jsdoc": "off",
+    "no-inner-declarations": "off",
+    "no-missing-require": "off"
   }
 }

package.json

@@ -16,6 +16,11 @@
     "chalk": "^2.4.2",
     "chokidar": "^2.1.5",
     "copy-template-dir": "^1.4.0",
+    "debug": "^4.1.1",
+    "npm-packlist": "^1.4.1",
+    "precinct": "^6.1.2",
+    "read-pkg-up": "^5.0.0",
+    "require-package-name": "^2.0.1",
     "execa": "^1.0.0",
     "express": "^4.16.4",
     "express-logging": "^1.1.1",
@@ -30,7 +35,7 @@
     "netlify-cli-logo": "^1.0.0",
     "node-fetch": "^2.3.0",
     "ora": "^3.4.0",
-    "querystring": "^0.2.0",
+    "resolve": "^1.10.0",
     "safe-join": "^0.1.2",
     "static-server": "^2.2.1",
     "wait-port": "^0.2.2",
@@ -50,6 +55,7 @@
     "gh-release": "^3.5.0",
     "globby": "^8",
     "mocha": "^5",
+    "nock": "^10.0.6",
     "npm-run-all": "^4.1.5",
     "nyc": "^13",
     "prettier": "^1.16.4"

src/commands/dev/exec.js

@@ -2,9 +2,9 @@ const execa = require("execa");
 const Command = require("@netlify/cli-utils");
 const { track } = require("@netlify/cli-utils/src/utils/telemetry");
 const {
-  NETLIFYDEV,
+  // NETLIFYDEV,
   NETLIFYDEVLOG,
-  NETLIFYDEVWARN,
+  // NETLIFYDEVWARN,
   NETLIFYDEVERR
 } = require("netlify-cli-logo");
 

src/commands/dev/index.js

@@ -5,18 +5,17 @@ const httpProxy = require("http-proxy");
 const waitPort = require("wait-port");
 const getPort = require("get-port");
 const chokidar = require("chokidar");
-const { serveFunctions } = require("../../utils/serveFunctions");
+const { serveFunctions } = require("../../utils/serve-functions");
 const { serverSettings } = require("../../detect-server");
 const { detectFunctionsBuilder } = require("../../detect-functions-builder");
 const Command = require("@netlify/cli-utils");
-const { getAddons } = require("netlify/src/addons");
 const { track } = require("@netlify/cli-utils/src/utils/telemetry");
 const chalk = require("chalk");
 const {
   NETLIFYDEV,
   NETLIFYDEVLOG,
-  NETLIFYDEVWARN,
-  NETLIFYDEVERR
+  NETLIFYDEVWARN
+  // NETLIFYDEVERR
 } = require("netlify-cli-logo");
 const boxen = require("boxen");
 const { createTunnel, connectTunnel } = require("../../live-tunnel");
@@ -26,7 +25,7 @@ function isFunction(settings, req) {
 }
 
 function addonUrl(addonUrls, req) {
-  const m = req.url.match(/^\/.netlify\/([^\/]+)(\/.*)/);
+  const m = req.url.match(/^\/.netlify\/([^\/]+)(\/.*)/); // eslint-disable-line no-useless-escape
   const addonUrl = m && addonUrls[m[1]];
   return addonUrl ? `${addonUrl}${m[2]}` : null;
 }
@@ -67,7 +66,7 @@ function initializeProxy(port) {
     if (
       proxyRes.statusCode === 404 &&
       req.alternativePaths &&
-      req.alternativePaths.length
+      req.alternativePaths.length > 0
     ) {
       req.url = req.alternativePaths.shift();
       return proxy.web(req, res, req.proxyOptions);
@@ -137,7 +136,7 @@ async function startProxy(settings, addonUrls) {
   return `http://localhost:${port}`;
 }
 
-function startDevServer(settings, log, error) {
+function startDevServer(settings, log) {
   if (settings.noCmd) {
     const StaticServer = require("static-server");
 
@@ -195,7 +194,7 @@ class DevCommand extends Command {
         (config.dev && config.dev.publish) ||
         (config.build && config.build.publish);
       if (!dist) {
-        console.log(`${NETLIFYDEVLOG} Using current working directory`);
+        this.log(`${NETLIFYDEVLOG} Using current working directory`);
         this.log(
           `${NETLIFYDEVWARN} Unable to determine public folder to serve files from.`
         );
@@ -220,7 +219,7 @@ class DevCommand extends Command {
 
     let url;
 
-    startDevServer(settings, this.log, this.error);
+    startDevServer(settings, this.log);
 
     // serve functions from zip-it-and-ship-it
     // env variables relies on `url`, careful moving this code
@@ -234,7 +233,9 @@ class DevCommand extends Command {
         functionWatcher.on("unlink", functionBuilder.build);
       }
       const functionsPort = await getPort({ port: 34567 });
-      const fnSettings = await serveFunctions({
+
+      // returns a value but we dont use it
+      await serveFunctions({
         ...settings,
         port: functionsPort,
         functionsDir
@@ -253,13 +254,7 @@ class DevCommand extends Command {
       url = liveSession.session_url;
       process.env.BASE_URL = url;
 
-      await connectTunnel(
-        liveSession,
-        accessToken,
-        settings.port,
-        this.log,
-        this.error
-      );
+      await connectTunnel(liveSession, accessToken, settings.port, this.log);
     }
 
     // Todo hoist this telemetry `command` to CLI hook

src/commands/functions/build.js

@@ -3,15 +3,15 @@ const { flags } = require("@oclif/command");
 const Command = require("@netlify/cli-utils");
 const { zipFunctions } = require("@netlify/zip-it-and-ship-it");
 const {
-  NETLIFYDEV,
+  // NETLIFYDEV,
   NETLIFYDEVLOG,
-  NETLIFYDEVWARN,
+  // NETLIFYDEVWARN,
   NETLIFYDEVERR
 } = require("netlify-cli-logo");
 
 class FunctionsBuildCommand extends Command {
   async run() {
-    const { flags, args } = this.parse(FunctionsBuildCommand);
+    const { flags } = this.parse(FunctionsBuildCommand);
     const { config } = this.netlify;
 
     const src = flags.src || config.build.functionsSource;