Fixed failing e2e tests

Author: sacOO7

src/base-command.ts

@@ -472,7 +472,7 @@ export abstract class AblyBaseCommand extends InteractiveBaseCommand {
     // Make sure we have authentication after potentially modifying options
     if (!clientOptions.key && !clientOptions.token) {
       this.fail(
-        "Authentication required. Please provide either an API key, a token, or log in first.",
+        'Authentication required. Please provide either an API key, a token, or log in first. Run "ably accounts login" to configure authentication.',
         flags,
         "auth",
       );

src/spaces-base-command.ts

@@ -186,7 +186,11 @@ export abstract class SpacesBaseCommand extends AblyBaseCommand {
     // First create an Ably client
     this.realtimeClient = await this.createAblyRealtimeClient(flags);
     if (!this.realtimeClient) {
-      this.fail("Failed to create Ably client", flags, "client");
+      this.fail(
+        'Failed to create Ably client. Run "ably accounts login" to configure authentication, or set the ABLY_API_KEY environment variable.',
+        flags,
+        "client",
+      );
     }
 
     // Create a Spaces client using the Ably client
@@ -235,7 +239,11 @@ export abstract class SpacesBaseCommand extends AblyBaseCommand {
       this.logCliEvent(flags, "connection", "failed", errorMsg, {
         state: connection.state,
       });
-      this.fail(errorMsg, flags, "connection");
+      this.fail(
+        `${errorMsg}. Use --verbose to see detailed connection logs.`,
+        flags,
+        "connection",
+      );
     }
 
     await new Promise<void>((resolve, reject) => {

test/e2e/auth/auth-keys-e2e.test.ts

@@ -100,9 +100,9 @@ describe.skipIf(SHOULD_SKIP_CONTROL_E2E)("Auth Keys E2E Tests", () => {
     const createdKey = createRecord.key as Record<string, unknown>;
     const keyFullName = createdKey.keyName as string;
 
-    // Now get that key by its name
+    // Now get that key by its name (appId is embedded in keyFullName)
     const getResult = await runCommand(
-      ["auth", "keys", "get", keyFullName, "--app", testAppId, "--json"],
+      ["auth", "keys", "get", keyFullName, "--json"],
       {
         env: { ABLY_ACCESS_TOKEN: E2E_ACCESS_TOKEN || "" },
       },
@@ -139,18 +139,9 @@ describe.skipIf(SHOULD_SKIP_CONTROL_E2E)("Auth Keys E2E Tests", () => {
 
     // Update the key name
     const updatedName = `updated-key-${Date.now()}`;
+    // appId is embedded in keyFullName, no --app needed
     const updateResult = await runCommand(
-      [
-        "auth",
-        "keys",
-        "update",
-        keyFullName,
-        "--app",
-        testAppId,
-        "--name",
-        updatedName,
-        "--json",
-      ],
+      ["auth", "keys", "update", keyFullName, "--name", updatedName, "--json"],
       {
         env: { ABLY_ACCESS_TOKEN: E2E_ACCESS_TOKEN || "" },
       },