Add --endpointURL flag

src/lib/config/index.ts

@@ -159,7 +159,7 @@ export class ShellConfig {
 
     this.projectConfig?.validate(this.rootConfig);
 
-    const urlFlag = Endpoint.getURLFromConfig(this.flags);
+    const urlFlag = Endpoint.getURLFromFlags(this.flags);
     if (urlFlag !== undefined) {
       try {
         new URL(urlFlag);

src/lib/config/root-config.ts

@@ -88,21 +88,41 @@ export class Endpoint {
   }
 
   static getURLFromConfig = (config: Config): string | undefined => {
-    const url = config.strOpt("url");
-    const scheme = config.strOpt("scheme");
-    const domain = config.strOpt("domain");
-    const port = config.numberOpt("port");
+    return this.getURLInner({
+      url: config.strOpt("url"),
+      scheme: config.strOpt("scheme"),
+      domain: config.strOpt("domain"),
+      port: config.numberOpt("port"),
+    });
+  };
+
+  static getURLFromFlags = (flags: Config): string | undefined => {
+    return this.getURLInner({
+      url: flags.strOpt("endpointURL"),
+      scheme: flags.strOpt("scheme"),
+      domain: flags.strOpt("domain"),
+      port: flags.numberOpt("port"),
+    });
+  };
 
+  static getURLInner = (opts: {
+    url?: string;
+    scheme?: string;
+    domain?: string;
+    port?: number;
+  }): string | undefined => {
     if (
-      url === undefined &&
-      (domain !== undefined || port !== undefined || scheme !== undefined)
+      opts.url === undefined &&
+      (opts.domain !== undefined ||
+        opts.port !== undefined ||
+        opts.scheme !== undefined)
     ) {
-      const scheme0 = scheme ?? "https";
-      const domain0 = domain ?? "db.fauna.com";
-      const port0 = port ? `:${port}` : "";
-      return `${scheme0}://${domain0}${port0}`;
+      const scheme = opts.scheme ?? "https";
+      const domain = opts.domain ?? "db.fauna.com";
+      const port = opts.port ? `:${opts.port}` : "";
+      return `${scheme}://${domain}${port}`;
     } else {
-      return url;
+      return opts.url;
     }
   };
 }

src/lib/fauna-command.js

@@ -206,22 +206,33 @@ FaunaCommand.flags = {
   ...Command.flags,
   domain: Flags.string({
     description: "Fauna server domain",
+    // Emits a warning if this flag is used.
+    deprecated: { to: "endpointURL" },
+    // Hides the flag in `--help`
+    hidden: true,
   }),
   scheme: Flags.string({
     description: "Connection scheme",
     options: ["https", "http"],
+    deprecated: { to: "endpointURL" },
+    hidden: true,
   }),
   port: Flags.string({
     description: "Connection port",
+    deprecated: { to: "endpointURL" },
+    hidden: true,
+  }),
+  endpointURL: Flags.string({
+    description: "Database URL. Overrides the `url` in ~/.fauna-shell",
   }),
   timeout: Flags.string({
     description: "Connection timeout in milliseconds",
   }),
   secret: Flags.string({
-    description: "Fauna secret key",
+    description: "Secret key. Overrides the `secret` in ~/.fauna-shell",
   }),
   endpoint: Flags.string({
-    description: "Fauna server endpoint",
+    description: "Connection endpoint, from ~/.fauna-shell",
   }),
   graphqlHost: Flags.string({
     description: "The Fauna GraphQL API host",

test/commands/eval.test.js

@@ -1,5 +1,5 @@
 const { expect, test } = require("@oclif/test");
-const { withOpts, getEndpoint, matchFqlReq } = require("../helpers/utils.js");
+const { withOpts, getEndpoint, matchFqlReq, withLegacyOpts } = require("../helpers/utils.js");
 const { query: q } = require("faunadb");
 
 describe("eval", () => {
@@ -20,6 +20,23 @@ describe("eval", () => {
       expect(JSON.parse(ctx.stdout).data[0].targetDb).to.equal("root");
     });
 
+  test
+    .nock(getEndpoint(), { allowUnmocked: true }, mockQuery)
+    .stdout()
+    .command(
+      withLegacyOpts([
+        "eval",
+        "--version",
+        "4",
+        "--format",
+        "json",
+        "Paginate(Collections())",
+      ])
+    )
+    .it("works with legacy --domain, --schema, and --port opts", (ctx) => {
+      expect(JSON.parse(ctx.stdout).data[0].targetDb).to.equal("root");
+    });
+
   test
     .nock(getEndpoint(), { allowUnmocked: true }, (api) => {
       api

test/helpers/utils.js

@@ -2,7 +2,10 @@ const url = require("url");
 const { query: q } = require("faunadb");
 const env = process.env;
 
-module.exports.withOpts = (cmd) => {
+/**
+ * Sets --domain, --secret, --scheme, and --port.
+ */
+module.exports.withLegacyOpts = (cmd) => {
   const opts = [
     "--secret",
     env.FAUNA_SECRET,
@@ -16,6 +19,19 @@ module.exports.withOpts = (cmd) => {
   return cmd.concat(opts);
 };
 
+/**
+ * Sets --secret and --endpointURL
+ */
+module.exports.withOpts = (cmd) => {
+  const opts = [
+    "--secret",
+    env.FAUNA_SECRET,
+    "--endpointURL",
+    `${env.FAUNA_SCHEME}://${env.FAUNA_DOMAIN}:${env.FAUNA_PORT}`,
+  ];
+  return cmd.concat(opts);
+};
+
 module.exports.getEndpoint = () =>
   url.format({
     protocol: env.FAUNA_SCHEME,