non-TTY check for required variables:

Added a check in non-TTY environments for `account_id`, `CLOUDFLARE_ACCOUNT_ID` and `CLOUDFLARE_API_TOKEN`. If `account_id` exists in `wrangler.toml`
then `CLOUDFLARE_ACCOUNT_ID` is not needed in non-TTY scope. The `CLOUDFLARE_API_TOKEN` is necessary in non-TTY scope and will always error if missing.

resolves #827

Author: JacobMGEvans

packages/wrangler/src/__tests__/publish.test.ts

@@ -233,10 +233,7 @@ describe("publish", () => {
         await expect(runWrangler("publish index.js")).rejects.toThrowError();
 
         expect(std.err).toMatchInlineSnapshot(`
-          "X [ERROR] Missing 'CLOUDFLARE_API_TOKEN' from non-TTY environment, please see docs for more info: TBD
-
-
-          X [ERROR] Did not login, quitting...
+          "X [ERROR] Missing 'CLOUDFLARE_API_TOKEN' from non-TTY environment, please see docs for more info
 
           "
         `);

packages/wrangler/src/__tests__/whoami.test.tsx

@@ -4,12 +4,18 @@ import { writeAuthConfigFile } from "../user";
 import { getUserInfo, WhoAmI } from "../whoami";
 import { setMockResponse } from "./helpers/mock-cfetch";
 import { mockConsoleMethods } from "./helpers/mock-console";
+import { useMockIsTTY } from "./helpers/mock-istty";
 import { runInTempDir } from "./helpers/run-in-tmp";
 import type { UserInfo } from "../whoami";
 
 describe("getUserInfo()", () => {
   runInTempDir({ homedir: "./home" });
   const std = mockConsoleMethods();
+  const { setIsTTY } = useMockIsTTY();
+
+  beforeEach(() => {
+    setIsTTY(true);
+  });
 
   it("should return undefined if there is no config file", async () => {
     const userInfo = await getUserInfo();

packages/wrangler/src/user.tsx

@@ -418,11 +418,9 @@ export function getAPIToken(): string | undefined {
     !localAPIToken &&
     !LocalState.accessToken?.value
   ) {
-    logger.error(
-      "Missing 'CLOUDFLARE_API_TOKEN' from non-TTY environment, please see docs for more info: TBD"
+    throw new Error(
+      "Missing 'CLOUDFLARE_API_TOKEN' from non-TTY environment, please see docs for more info"
     );
-
-    return;
   }
 
   return localAPIToken ?? LocalState.accessToken?.value;