Add more tests

Author: flevi29

src/token.ts

@@ -28,7 +28,7 @@ function encodeToBase64(data: unknown): string {
 
 // missing crypto global for Node.js 18 https://nodejs.org/api/globals.html#crypto_1
 let cryptoPonyfill: Promise<Crypto | typeof webcrypto> | undefined;
-function getCrypto() {
+function getCrypto(): NonNullable<typeof cryptoPonyfill> {
   if (cryptoPonyfill === undefined) {
     cryptoPonyfill =
       typeof crypto === "undefined"
@@ -143,11 +143,8 @@ function tryDetectEnvironment(): void {
 
   // Node.js prior to v21.1.0 doesn't have the above global
   // https://nodejs.org/api/globals.html#navigatoruseragent
-  if (
-    Object.hasOwn(globalThis, "process") &&
-    Object.hasOwn(globalThis.process, "versions") &&
-    Object.hasOwn(globalThis.process.versions, "node")
-  ) {
+  const versions = globalThis.process?.versions;
+  if (versions !== undefined && Object.hasOwn(versions, "node")) {
     return;
   }
 

tests/token.test.ts

@@ -1,5 +1,13 @@
 import * as assert from "node:assert";
-import { afterAll, beforeEach, describe, expect, test } from "vitest";
+import {
+  afterAll,
+  beforeEach,
+  describe,
+  expect,
+  test,
+  vi,
+  type MockInstance,
+} from "vitest";
 import {
   getClient,
   decode64,
@@ -21,6 +29,48 @@ afterAll(() => {
   return clearAllIndexes(config);
 });
 
+test("Should throw error for invalid UID", async () => {
+  await assert.rejects(
+    generateTenantToken({
+      apiKey: "wrong",
+      apiKeyUid: "stuff",
+    }),
+    /^Error: the uid of your key is not a valid UUIDv4$/,
+  );
+});
+
+test("Should throw error for non-server-side environment", async () => {
+  using _ = (() => {
+    let userAgentSpy: MockInstance<() => string> | undefined;
+    if (typeof navigator !== "undefined" && "userAgent" in navigator) {
+      userAgentSpy = vi
+        .spyOn(navigator, "userAgent", "get")
+        .mockImplementation(() => "ProbablySomeBrowserUA");
+    }
+
+    const nodeEvnSpy = vi.spyOn(process, "versions", "get").mockImplementation(
+      () =>
+        // @ts-expect-error
+        undefined,
+    );
+
+    return {
+      [Symbol.dispose]() {
+        userAgentSpy?.mockRestore();
+        nodeEvnSpy.mockRestore();
+      },
+    };
+  })();
+
+  await assert.rejects(
+    generateTenantToken({
+      apiKey: "wrong",
+      apiKeyUid: "stuff",
+    }),
+    /^Error: failed to detect a server-side environment;/,
+  );
+});
+
 describe.each([{ permission: "Admin" }])(
   "Tests on token generation",
   ({ permission }) => {