Revert logs and isolate

connectrpc · Nov 30, 2022 · 332f03b · 332f03b
1 parent b728df1
commit 332f03b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 31 deletions.
diff --git a/packages/connect-node-test/src/badweather/unavailable.spec.ts b/packages/connect-node-test/src/badweather/unavailable.spec.ts
@@ -30,7 +30,7 @@ describe("unavailable", () => {
       baseUrl: "https://host.invalid",
     });
     const client = createPromiseClient(TestService, transport);
-    fit("should raise code unavailable", async function () {
+    it("should raise code unavailable", async function () {
       try {
         await client.unaryCall({});
         fail("expected an error");

diff --git a/packages/connect-node/src/private/connect-error-from-node.ts b/packages/connect-node/src/private/connect-error-from-node.ts
@@ -40,44 +40,15 @@ function isSyscallError(
   // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-explicit-any
   const r = reason as any;
   if (reason instanceof Error) {
-    // if ("code" in r) {
-    //   // TODO
-    //   // eslint-disable-next-line no-console
-    //   console.error(r);
-    //   // eslint-disable-next-line no-console
-    //   console.error("want syscall", syscall, "want code", code);
-    //   // eslint-disable-next-line no-console
-    //   console.log("syscall in r", "syscall" in r)
-    //   if ("syscall" in r) {
-    //     // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access,no-console
-    //     console.log("r.syscall", r.syscall)
-    //   }
-    //   // eslint-disable-next-line no-console
-    //   console.log("code in r", "code" in r)
-    //   if ("code" in r) {
-    //     // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access,no-console
-    //     console.log("r.code", r.code)
-    //   }
-    //   // eslint-disable-next-line no-console
-    //   console.log("syscall = " + syscall, "syscall" in r)
-    // }
     if ("code" in r && "syscall" in r) {
       // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-unsafe-member-access
       if (syscall === r.syscall && code === r.code) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-explicit-any,@typescript-eslint/no-unsafe-member-access,no-console
-        console.log(`isSyscallError ${syscall} ${code} OK`, reason)
         return true;
-      } else {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-explicit-any,@typescript-eslint/no-unsafe-member-access,no-console
-        console.log(`isSyscallError ${syscall} ${code} NO`, reason)
       }
     }
     if ("cause" in reason) {
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-explicit-any,@typescript-eslint/no-unsafe-member-access
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
       return isSyscallError(r.cause, syscall, code);
-    } else {
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment,@typescript-eslint/no-explicit-any,@typescript-eslint/no-unsafe-member-access,no-console
-      console.log(`isSyscallError ${syscall} ${code} NO`, reason)
     }
   }
   return false;