@ogcio/o11y-sdk-node 0.4.2 → 0.6.1

package/test/integration/main.sh

@@ -1,52 +0,0 @@
-#!/bin/bash
-
-set -e
-set -o pipefail
-
-if [ $# -lt 2 ]; then
-    echo "Usage: $0 <build-id> <root-path>"
-    exit 1
-fi
-
-BUILD_ID=$1
-ROOT_PATH=$2
-
-NETWORK_NAME="${BUILD_ID}_testnetwork"
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-docker network create $NETWORK_NAME
-
-TEST_SCRIPTS=( $(find . -type f -path "./packages/sdk-node/test/integration/test*/run.sh" | sort) )
-
-FAILURES=0
-
-for test_script in "${TEST_SCRIPTS[@]}"; do
-    TEST_DIR=$(dirname "$test_script")
-    TEST_NAME=$(basename "$TEST_DIR")
-    
-    echo "Running test: $TEST_NAME ($test_script)"
-    
-    # Run the test
-    bash "$test_script" "$BUILD_ID" "$ROOT_PATH" "$NETWORK_NAME"
-    EXIT_CODE=$?
-    
-    if [ $EXIT_CODE -eq 0 ]; then
-        echo "$test_script completed"
-    else
-        echo "❌ $test_script failed with exit code $EXIT_CODE"
-        FAILURES=$((FAILURES + 1))
-    fi
-    echo "------------------------------"
-done
-
-# Remove network and images from docker host
-bash "$SCRIPT_DIR/teardown.sh" "$BUILD_ID" "$NETWORK_NAME"
-
-# Final summary
-if [ $FAILURES -eq 0 ]; then
-    echo "All tests ready!"
-    exit 0
-else
-    echo "$FAILURES test(s) failed."
-    exit 1
-fi

package/test/integration/teardown.sh

@@ -1,7 +0,0 @@
-#!/bin/bash
-
-BUILD_ID=$1
-NETWORK_NAME=$2
-
-docker network rm $NETWORK_NAME
-docker rmi -f $(docker images --filter "reference=*:$BUILD_ID" -q)

package/test/integration/test_fastify-o11y-pii-enabled/http-tracing.integration.test.ts

@@ -1,56 +0,0 @@
-import { describe, test, assert } from "vitest";
-import { parseLog } from "../../utils/alloy-log-parser";
-import { readFile } from "node:fs/promises";
-import { join } from "node:path";
-
-describe("instrumentation integration test", () => {
-  test("should exclude health url and process only dummy calls", async () => {
-    const data = await readFile(join(__dirname, "logs.txt"), "utf-8");
-
-    let health_traces_counter = 0;
-    let dummy_traces_counter = 0;
-
-    for (const line of data.split(/\nts=/)) {
-      const parsedLine: Record<string, object | string | number> =
-        parseLog(line);
-
-      if (
-        parsedLine["attributes"] &&
-        parsedLine["attributes"]["span_kind"] &&
-        parsedLine["attributes"]["span_kind"] === "trace"
-      ) {
-        if (parsedLine["attributes"]["http.target"] === "/api/dummy") {
-          dummy_traces_counter++;
-
-          // verify global sdk span resource
-          assert.equal(
-            parsedLine["resource_attributes"]["team.infra.pod"],
-            "01",
-          );
-          assert.equal(
-            parsedLine["resource_attributes"]["team.service.type"],
-            "fastify",
-          );
-
-          // verify global sdk span attributes
-          assert.equal(parsedLine["attributes"]["signal.namespace"], "example");
-
-          // verify runtime custom span inside dev application
-          assert.equal(parsedLine["attributes"]["business.info"], "dummy");
-          assert.equal(
-            parsedLine["attributes"]["business.request_type"],
-            "application/json",
-          );
-
-          continue;
-        }
-        if (parsedLine["attributes"]["http.target"] === "/api/health") {
-          health_traces_counter++;
-        }
-      }
-    }
-
-    assert.equal(health_traces_counter, 0);
-    assert.equal(dummy_traces_counter, 2);
-  });
-});

package/test/integration/test_fastify-o11y-pii-enabled/pii.integration.test.ts

@@ -1,68 +0,0 @@
-import { describe, test, assert } from "vitest";
-import { parseLog } from "../../utils/alloy-log-parser";
-import { readFile } from "node:fs/promises";
-import { join } from "node:path";
-
-describe("pii integration test", () => {
-  test("should check redacted attributes and detect zero emails in logs", async () => {
-    const data = await readFile(join(__dirname, "logs.txt"), "utf-8");
-
-    let health_traces_counter = 0;
-    let dummy_traces_counter = 0;
-    let email_not_redacted = 0;
-
-    for (const line of data.split(/\nts=/)) {
-      const parsedLine: Record<string, object | string | number> =
-        parseLog(line);
-
-      if (
-        parsedLine["attributes"] &&
-        parsedLine["attributes"]["span_kind"] &&
-        parsedLine["attributes"]["span_kind"] === "log"
-      ) {
-        const matches = (parsedLine["log_body"] as string).match(
-          /[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,}/g,
-        );
-        if (matches) {
-          email_not_redacted++;
-        }
-      }
-
-      if (
-        parsedLine["attributes"] &&
-        parsedLine["attributes"]["span_kind"] &&
-        parsedLine["attributes"]["span_kind"] === "trace"
-      ) {
-        if (parsedLine["attributes"]["http.target"] === "/api/dummy") {
-          dummy_traces_counter++;
-
-          // verify global sdk span resource
-          assert.equal(
-            parsedLine["resource_attributes"]["email"],
-            "[REDACTED EMAIL]",
-          );
-
-          // verify global sdk span attributes
-          assert.equal(parsedLine["attributes"]["email"], "[REDACTED EMAIL]");
-          // verify custom runtime span attributes
-          assert.equal(
-            parsedLine["attributes"]["multiple.email"],
-            "[REDACTED EMAIL]",
-          );
-
-          // verify global sdk span attributes
-          assert.equal(parsedLine["events"]["email"], "[REDACTED EMAIL]");
-
-          continue;
-        }
-        if (parsedLine["attributes"]["http.target"] === "/api/health") {
-          health_traces_counter++;
-        }
-      }
-    }
-
-    assert.equal(email_not_redacted, 0);
-    assert.equal(health_traces_counter, 0);
-    assert.equal(dummy_traces_counter, 2);
-  });
-});

package/test/integration/test_fastify-o11y-pii-enabled/run.sh

@@ -1,42 +0,0 @@
-#!/bin/bash
-# @docs: packages/sdk-node/test/integration/README.md#test_fastify-o11y-pii-enabled
-BUILD_ID=$1
-ROOT_PATH=$2
-NETWORK_NAME=$3
-
-ALLOY_CONTAINER_NAME="integrationalloy"
-NODE_CONTAINER_NAME="${BUILD_ID}_fastify_app"
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-LOG_FILE="$SCRIPT_DIR/logs.txt"
-ERROR_CODE=0
-
-source "$SCRIPT_DIR/../docker-utils.sh"
-
-build_image "${NODE_CONTAINER_NAME}:${BUILD_ID}" "$ROOT_PATH/examples/fastify/Dockerfile" "$ROOT_PATH/"
-
-run_and_check_alloy $ROOT_PATH $NETWORK_NAME $ALLOY_CONTAINER_NAME
-ERROR_CODE=$?
-
-if [[ $ERROR_CODE -eq 0 ]]; then
-    run_and_check_fastify $NETWORK_NAME "http://integrationalloy:4317" $NODE_CONTAINER_NAME $BUILD_ID
-    ERROR_CODE=$?
-fi
-
-if [[ $ERROR_CODE -eq 0 ]]; then
-    sleep 2
-    curl -X GET -f http://localhost:9091/api/dummy -H 'Content-Type: application/json'
-    sleep 2
-    curl -X GET -f http://localhost:9091/api/dummy -H 'Content-Type: application/json'
-fi
-
-# sleep N seconds to await instrumentation flow send and receiving signals
-sleep 1
-
-# Copy logs from container to file
-docker container logs $ALLOY_CONTAINER_NAME >&$LOG_FILE
-echo "log file at $LOG_FILE"
-
-container_stop_and_rm $ALLOY_CONTAINER_NAME
-container_stop_and_rm $NODE_CONTAINER_NAME
-
-exit $ERROR_CODE

package/test/integration/test_without-o11y/run.sh

@@ -1,30 +0,0 @@
-#!/bin/bash
-# @docs: packages/sdk-node/test/integration/README.md#test_without-o11y
-BUILD_ID=$1
-ROOT_PATH=$2
-NETWORK_NAME=$3
-
-NODE_CONTAINER_NAME="${BUILD_ID}_fastify_app"
-ERROR_CODE=0
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-LOG_FILE="$SCRIPT_DIR/logs.txt"
-
-source "$SCRIPT_DIR/../docker-utils.sh"
-
-build_image "${NODE_CONTAINER_NAME}:${BUILD_ID}" "$ROOT_PATH/examples/fastify/Dockerfile" "$ROOT_PATH/"
-
-run_and_check_fastify $NETWORK_NAME "" $NODE_CONTAINER_NAME $BUILD_ID
-ERROR_CODE=$?
-
-if [[ $ERROR_CODE -eq 0 ]]; then
-    curl -X GET -f http://localhost:9091/api/dummy -H 'Content-Type: application/json'
-    sleep 1
-    curl -X GET -f http://localhost:9091/api/dummy -H 'Content-Type: application/json'
-fi
-
-docker container logs $NODE_CONTAINER_NAME >&$LOG_FILE
-echo "log file at $LOG_FILE"
-
-container_stop_and_rm $NODE_CONTAINER_NAME
-
-exit $ERROR_CODE

package/test/integration/test_without-o11y/verify-status.integration.test.ts

@@ -1,32 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { readFile } from "node:fs/promises";
-import { join } from "node:path";
-
-describe("Log validation for /api/dummy with instrumentation disabled", async () => {
-  const log = await readFile(join(__dirname, "logs.txt"), "utf-8");
-
-  it("should contain message that collector URL is not set", () => {
-    expect(log).toContain(
-      "collectorUrl not set. Skipping NodeJS OpenTelemetry instrumentation.",
-    );
-  });
-
-  it("should have all /api/dummy responses with status code 200", () => {
-    // Extract all request IDs for /api/dummy requests
-    const dummyRequestIds = [
-      ...log.matchAll(/"reqId":"(.*?)","req":\{[^}]*"url":"\/api\/dummy"/g),
-    ].map((match) => match[1]);
-
-    expect(dummyRequestIds.length).toBeGreaterThan(0);
-
-    // For each /api/dummy request ID, verify that its response status is 200
-    const non200Responses = dummyRequestIds.filter((reqId) => {
-      const responseMatch = log.match(
-        new RegExp(`"reqId":"${reqId}".*"statusCode":(\\d+)`),
-      );
-      return responseMatch && Number(responseMatch[1]) !== 200;
-    });
-
-    expect(non200Responses).toEqual([]); // No non-200 responses allowed
-  });
-});

package/test/internals/hooks.test.ts

@@ -1,45 +0,0 @@
-import { describe, it, expect, vi } from "vitest";
-import { _shutdownHook } from "../../lib/internals/hooks";
-
-describe("shutdown hook", () => {
-  it("should call sdk.shutdown and log success on SIGTERM", async () => {
-    const shutdownMock = vi.fn().mockResolvedValue(undefined);
-    const consoleLogMock = vi
-      .spyOn(console, "log")
-      .mockImplementation(() => {});
-
-    _shutdownHook({ shutdown: shutdownMock });
-
-    process.emit("SIGTERM");
-    // Wait for async logic
-    await Promise.resolve();
-
-    expect(shutdownMock).toHaveBeenCalled();
-    expect(consoleLogMock).toHaveBeenCalledWith(
-      "NodeJS OpenTelemetry instrumentation shutdown successfully",
-    );
-
-    consoleLogMock.mockRestore();
-  });
-
-  it("should catch and log errors on shutdown failure", async () => {
-    const error = new Error("Shutdown failed");
-    const shutdownMock = vi.fn().mockRejectedValue(error);
-    const consoleErrorMock = vi
-      .spyOn(console, "error")
-      .mockImplementation(() => {});
-
-    _shutdownHook({ shutdown: shutdownMock });
-
-    process.emit("SIGTERM");
-    // Wait for async logic
-    await Promise.resolve();
-
-    expect(consoleErrorMock).toHaveBeenCalledWith(
-      "Error shutting down NodeJS OpenTelemetry instrumentation:",
-      error,
-    );
-
-    consoleErrorMock.mockRestore();
-  });
-});

package/test/internals/pii-detection.test.ts

@@ -1,265 +0,0 @@
-import { describe, expect, it, vi, beforeEach } from "vitest";
-import {
-  _cleanStringPII,
-  _containsEncodedComponents,
-  _recursiveObjectClean,
-} from "../../lib/internals/redaction/pii-detection.js";
-import * as sharedMetrics from "../../lib/internals/shared-metrics.js";
-import { emailRedactor } from "../../lib/internals/redaction/redactors/email";
-import { ipRedactor } from "../../lib/internals/redaction/redactors/ip";
-
-describe("PII Detection Utils", () => {
-  const mockMetricAdd = vi.fn();
-
-  beforeEach(() => {
-    vi.restoreAllMocks();
-    vi.spyOn(sharedMetrics, "_getPIICounterRedactionMetric").mockReturnValue({
-      add: mockMetricAdd,
-    });
-  });
-
-  describe("_cleanStringPII", () => {
-    it("redacts plain PII", () => {
-      const input = "admin@example.com";
-      const output = _cleanStringPII(input, "log", [emailRedactor]);
-
-      expect(output).toBe("[REDACTED EMAIL]");
-      expect(mockMetricAdd).toHaveBeenCalledWith(
-        1,
-        expect.objectContaining({
-          pii_email_domain: "example.com",
-          pii_type: "email",
-          redaction_source: "log",
-        }),
-      );
-    });
-
-    it("redacts email in URL-encoded string", () => {
-      const input = "user%40gmail.com";
-      const output = _cleanStringPII(input, "log", [emailRedactor]);
-
-      expect(output).toBe("[REDACTED EMAIL]");
-      expect(mockMetricAdd).toHaveBeenCalledWith(
-        1,
-        expect.objectContaining({
-          pii_format: "url",
-          pii_email_domain: "gmail.com",
-        }),
-      );
-    });
-
-    it("redacts ip in URL-encoded string", () => {
-      const input = "%20127.0.0.1";
-      const output = _cleanStringPII(input, "log", [ipRedactor]);
-
-      expect(output).toBe(" [REDACTED IPV4]");
-      expect(mockMetricAdd).toHaveBeenCalledWith(
-        1,
-        expect.objectContaining({
-          pii_format: "url",
-          pii_type: "IPv4",
-          redaction_source: "log",
-        }),
-      );
-    });
-
-    it("handles strings without PII unchanged", () => {
-      const input = "hello world";
-      const output = _cleanStringPII(input, "log", [emailRedactor]);
-
-      expect(output).toBe("hello world");
-      expect(mockMetricAdd).not.toHaveBeenCalled();
-    });
-
-    it("ignores non-string input", () => {
-      // @ts-expect-error
-      expect(_cleanStringPII(1234, "trace", [emailRedactor])).toBe(1234);
-      // @ts-expect-error
-      expect(_cleanStringPII(true, "trace", [emailRedactor])).toBe(true);
-      expect(
-        _cleanStringPII(undefined, "trace", [emailRedactor]),
-      ).toBeUndefined();
-      expect(mockMetricAdd).not.toHaveBeenCalled();
-    });
-  });
-
-  describe("_recursiveObjectClean", () => {
-    it("cleans string PII", () => {
-      const result = _recursiveObjectClean("demo@abc.com", "log", [
-        emailRedactor,
-      ]);
-      expect(result).toBe("[REDACTED EMAIL]");
-    });
-
-    it("cleans array of strings", () => {
-      const input = ["one@gmail.com", "two@example.com"];
-      const output = _recursiveObjectClean(input, "log", [emailRedactor]);
-
-      expect(output).toEqual(["[REDACTED EMAIL]", "[REDACTED EMAIL]"]);
-      expect(mockMetricAdd).toHaveBeenCalledTimes(2);
-    });
-
-    it("cleans deeply nested object", () => {
-      const input = {
-        user: {
-          email: "test@gmail.com",
-          profile: {
-            contact: "foo@example.com",
-          },
-        },
-        status: "active",
-      };
-
-      const result = _recursiveObjectClean(input, "log", [emailRedactor]);
-
-      expect(result).toEqual({
-        user: {
-          email: "[REDACTED EMAIL]",
-          profile: {
-            contact: "[REDACTED EMAIL]",
-          },
-        },
-        status: "active",
-      });
-    });
-
-    it("cleans Uint8Array input", () => {
-      const str = "admin@gmail.com";
-      const buffer = new TextEncoder().encode(str);
-      const result = _recursiveObjectClean(buffer, "log", [emailRedactor]);
-      const decoded = new TextDecoder().decode(result as Uint8Array);
-
-      expect(decoded).toBe("[REDACTED EMAIL]");
-    });
-
-    it("skips malformed Uint8Array decode", () => {
-      const corrupted = new Uint8Array([0xff, 0xfe, 0xfd]);
-      const result = _recursiveObjectClean(corrupted, "log", [emailRedactor]);
-
-      // Should return a Uint8Array, but unmodified/redaction should not happen
-      expect(result).toBeInstanceOf(Uint8Array);
-      expect(result).not.toEqual(expect.arrayContaining([91, 82, 69]));
-    });
-
-    it("cleans arrays of values", () => {
-      const result = _recursiveObjectClean(
-        ["bob@abc.com", 123, { nested: "jane@example.com" }],
-        "log",
-        [emailRedactor],
-      );
-
-      expect(result).toEqual([
-        "[REDACTED EMAIL]",
-        123,
-        { nested: "[REDACTED EMAIL]" },
-      ]);
-    });
-
-    it("passes null and boolean through", () => {
-      expect(_recursiveObjectClean(null, "log", [emailRedactor])).toBeNull();
-      expect(
-        _recursiveObjectClean(undefined, "log", [emailRedactor]),
-      ).toBeUndefined();
-      expect(_recursiveObjectClean(true, "log", [emailRedactor])).toBe(true);
-      expect(_recursiveObjectClean(false, "log", [emailRedactor])).toBe(false);
-    });
-  });
-
-  describe("_containsEncodedComponents", () => {
-    describe("should return true for properly URL encoded strings", () => {
-      it.each([
-        ["hello%20world", "Space encoded as %20"],
-        ["test%2Bvalue", "Plus sign encoded as %2B"],
-        ["path%2Fto%2Ffile", "Forward slashes encoded"],
-        ["user%40domain.com", "@ symbol encoded"],
-        ["100%25%20off", "Percent and space encoded"],
-        ["a%3Db%26c%3Dd", "Query parameters (a=b&c=d)"],
-        ["caf%C3%A9", "UTF-8 encoded (café)"],
-        ["price%3A%20%2410", "Colon, space, dollar ($10)"],
-        ["%22quoted%22", "Double quotes encoded"],
-        ["https%3A%2F%2Fexample.com", "Full URL encoded"],
-        ["file%20name.txt", "filename encoded"],
-        ["search%3Fq%3Dhello%20world", "Query string encoded"],
-        ["%3C%3E%26%22%27", "HTML special chars encoded (<>&\"')"],
-        ["%E2%9C%93", "UTF-8 checkmark (✓) encoded"],
-        ["test%2b", "Lowercase hex"],
-        ["%E4%B8%AD%E6%96%87", "Chinese characters encoded"],
-      ])('should detect "%s" as URL encoded (%s)', (input, description) => {
-        expect(_containsEncodedComponents(input)).toBe(true);
-      });
-    });
-
-    describe("should return false for non-URL encoded strings", () => {
-      it.each([
-        ["test", "Simple ASCII string"],
-        ["hello world", "Unencoded space"],
-        ["user@domain.com", "Unencoded email"],
-        ["simple123", "Alphanumeric only"],
-        ["", "Empty string"],
-        ["25%%", "Literal percent signs"],
-        ["100% off", "Percent without hex digits"],
-        ["test%2", "Incomplete percent encoding"],
-        ["hello%ZZ", "Invalid hex digits"],
-        ["test%2G", "Invalid hex digit G"],
-        ["bad%encoding%here", "Percent without hex pairs"],
-        ["hello%20world and more", "Partially encoded string"],
-        ["hello%20world%21%20how%20are%20you", "Overly encoded string"],
-        ["café", "Unicode characters (unencoded)"],
-        ["hello+world", "Plus sign (form encoding style)"],
-        ["test%", "Trailing percent"],
-        ["hello%20world%", "Encoded content with trailing percent"],
-        ["%", "Single percent"],
-        ["%%", "Double percent"],
-        ["normal text with % symbols", "Text with percent but no encoding"],
-        ["price: $100%", "Currency with percent"],
-        ["file.txt", "Simple filename"],
-        ["path/to/file", "Unencoded path"],
-        ["query?param=value", "Unencoded query string"],
-        ["hello%world", "Percent without following hex"],
-        ["test%1", "Single hex digit after percent"],
-        ["hello%zz", "Non-hex characters after percent"],
-      ])('should not detect "%s" as URL encoded (%s)', (input, description) => {
-        expect(_containsEncodedComponents(input)).toBe(false);
-      });
-    });
-
-    describe("Error handling", () => {
-      it.each([
-        ["%C0%80", "Overlong UTF-8 encoding (security concern)"],
-        ["%ED%A0%80", "UTF-8 surrogate (invalid)"],
-        ["%FF%FE", "Invalid UTF-8 sequence"],
-        ["test%20%ZZ", "Mix of valid and invalid encoding"],
-      ])('should handle "%s" (%s)', (input, description) => {
-        // These should not throw errors
-        expect(() => _containsEncodedComponents(input)).not.toThrow();
-
-        // Most of these should return false due to invalid sequences
-        // or security-related encoding issues
-        const result = _containsEncodedComponents(input);
-        expect(typeof result).toBe("boolean");
-      });
-    });
-
-    describe("real-world url examples", () => {
-      it.each([
-        [
-          "https%3A%2F%2Fgoogle.com%2Fsearch%3Fq%3Djavascript",
-          "Encoded Google search URL",
-        ],
-        ["The%20quick%20brown%20fox", "Sentence with spaces encoded"],
-        [
-          "/api/user?body=here%20are%20all%20my%20secrets",
-          "contextually encoded API path",
-        ],
-        ["redirect_uri=https%3A%2F%2Fapp.com%2Fcallback", "OAuth redirect URI"],
-        ["data%3Atext%2Fplain%3Bbase64%2CSGVsbG8%3D", "Data URL encoded"],
-      ])('real-world case: "%s" (%s)', (input, description) => {
-        const result = _containsEncodedComponents(input);
-
-        // Verify the function doesn't crash
-        expect(typeof result).toBe("boolean");
-        expect(result).toBe(true);
-      });
-    });
-  });
-});

package/test/internals/redactors/email.test.ts

@@ -1,81 +0,0 @@
-import {
-  describe,
-  expect,
-  it,
-  vi,
-  beforeEach,
-  beforeAll,
-  afterAll,
-} from "vitest";
-
-import * as sharedMetrics from "../../../lib/internals/shared-metrics.js";
-import { ipRedactor } from "../../../lib/internals/redaction/redactors/ip";
-import { emailRedactor } from "../../../lib/internals/redaction/redactors/email";
-
-describe("Email Redaction utils", () => {
-  describe("tracks metrics", () => {
-    const mockMetricAdd = vi.fn();
-
-    beforeEach(() => {
-      vi.restoreAllMocks();
-      vi.spyOn(sharedMetrics, "_getPIICounterRedactionMetric").mockReturnValue({
-        add: mockMetricAdd,
-      });
-    });
-
-    it("redacts plain PII and tracks redaction with metric", () => {
-      const input = "admin@example.com";
-      const output = emailRedactor(input, "log", "string");
-
-      expect(output).toBe("[REDACTED EMAIL]");
-      expect(mockMetricAdd).toHaveBeenCalledWith(
-        1,
-        expect.objectContaining({
-          pii_email_domain: "example.com",
-          pii_type: "email",
-          redaction_source: "log",
-        }),
-      );
-    });
-
-    it("handles strings without PII unchanged", () => {
-      const input = "hello world";
-      const output = ipRedactor(input, "log", "string");
-
-      expect(output).toBe("hello world");
-      expect(mockMetricAdd).not.toHaveBeenCalled();
-    });
-  });
-
-  describe("Redacts email addresses", () => {
-    beforeAll(() => {
-      vi.spyOn(sharedMetrics, "_getPIICounterRedactionMetric").mockReturnValue({
-        add: vi.fn(),
-      });
-    });
-
-    afterAll(() => {
-      vi.restoreAllMocks();
-    });
-
-    it.each`
-      value                                  | expectedRedactedValue
-      ${"user+tag@example.com"}              | ${"[REDACTED EMAIL]"}
-      ${"user.name+tag+sorting@example.com"} | ${"[REDACTED EMAIL]"}
-      ${"x@example.museum"}                  | ${"[REDACTED EMAIL]"}
-      ${"a.b-c_d@example.co.uk"}             | ${"[REDACTED EMAIL]"}
-      ${"üser@example.de"}                   | ${"[REDACTED EMAIL]"}
-      ${"john.doe@xn--exmple-cua.com"}       | ${"[REDACTED EMAIL]"}
-      ${"üser@example.de"}                   | ${"[REDACTED EMAIL]"}
-      ${"plainaddress"}                      | ${"plainaddress"}
-      ${"@missinglocal.org"}                 | ${"@missinglocal.org"}
-      ${"user@invalid_domain.com"}           | ${"user@invalid_domain.com"}
-    `(
-      "returns $expectedRedactedValue for value '$value'",
-      async ({ value, expectedRedactedValue }: Record<string, string>) => {
-        const result = emailRedactor(value, "log", "string");
-        expect(result).toBe(expectedRedactedValue);
-      },
-    );
-  });
-});