@ogcio/o11y-sdk-node 0.4.1 → 0.4.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.4.2](https://github.com/ogcio/o11y/compare/@ogcio/o11y-sdk-node@v0.4.1...@ogcio/o11y-sdk-node@v0.4.2) (2025-09-03)
+
+
+### Bug Fixes
+
+* withSpan function throws error if sdk is not initialized AB[#30828](https://github.com/ogcio/o11y/issues/30828) ([#200](https://github.com/ogcio/o11y/issues/200)) ([407b616](https://github.com/ogcio/o11y/commit/407b616590024ca4131610c7a69bacd9699f23c5))
+
 ## [0.4.1](https://github.com/ogcio/o11y/compare/@ogcio/o11y-sdk-node@v0.4.0...@ogcio/o11y-sdk-node@v0.4.1) (2025-09-03)
 
 

package/dist/lib/config-manager.d.ts

@@ -1,3 +1,3 @@
 import { NodeSDKConfig } from "./index.js";
 export declare const setNodeSdkConfig: (config: NodeSDKConfig) => void;
-export declare const getNodeSdkConfig: () => NodeSDKConfig;
+export declare const getNodeSdkConfig: () => NodeSDKConfig | undefined;

package/dist/lib/config-manager.js

@@ -3,9 +3,6 @@ export const setNodeSdkConfig = (config) => {
     nodeSDKConfig = config;
 };
 export const getNodeSdkConfig = () => {
-    if (!nodeSDKConfig) {
-        throw new Error("Node SDK Config was not initialized.");
-    }
     // Ensure getters do not edit config.
-    return JSON.parse(JSON.stringify(nodeSDKConfig));
+    return nodeSDKConfig ? JSON.parse(JSON.stringify(nodeSDKConfig)) : undefined;
 };

package/dist/lib/exporter/pii-exporter-decorator.d.ts

@@ -8,7 +8,7 @@ export declare class PIIExporterDecorator extends OTLPExporterBase<(ReadableSpan
     private readonly _exporter;
     private readonly _config;
     private readonly _redactors;
-    constructor(exporter: OTLPExporterBase<(ReadableSpan | ReadableLogRecord)[] | ResourceMetrics>, config: NodeSDKConfig);
+    constructor(exporter: OTLPExporterBase<(ReadableSpan | ReadableLogRecord)[] | ResourceMetrics>, config: NodeSDKConfig | undefined);
     forceFlush(): Promise<void>;
     shutdown(): Promise<void>;
     export(items: (ReadableSpan | ReadableLogRecord)[] | ResourceMetrics, resultCallback: (result: ExportResult) => void): void;

package/dist/lib/exporter/pii-exporter-decorator.js

@@ -9,7 +9,7 @@ export class PIIExporterDecorator extends OTLPExporterBase {
         super(exporter["_delegate"]);
         this._exporter = exporter;
         this._config = config;
-        this._redactors = this._buildRedactors(config.detection);
+        this._redactors = this._buildRedactors(config?.detection);
     }
     forceFlush() {
         return this._exporter.forceFlush();

package/dist/lib/traces.js

@@ -45,6 +45,6 @@ export function getActiveSpan() {
 }
 export function withSpan({ traceName, spanName, spanOptions = {}, fn, }) {
     const sdkConfig = getNodeSdkConfig();
-    const tracer = trace.getTracer(traceName ?? sdkConfig.serviceName ?? "o11y-sdk", sdkConfig.serviceVersion);
+    const tracer = trace.getTracer(traceName ?? sdkConfig?.serviceName ?? "o11y-sdk", sdkConfig?.serviceVersion);
     return tracer.startActiveSpan(spanName, spanOptions, selfContainedSpanHandlerGenerator(fn));
 }

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@ogcio/o11y-sdk-node",
-    "version": "0.4.1",
+    "version": "0.4.2",
     "description": "Opentelemetry standard instrumentation SDK for NodeJS based project",
     "main": "dist/index.js",
     "type": "module",

package/dist/vitest.config.js

@@ -36,7 +36,7 @@ export default defineConfig({
             },
             {
                 test: {
-                    include: ["**/test/integration/*.test.ts"],
+                    include: ["**/test/integration/**/*.test.ts"],
                     name: "integration",
                 },
             },

package/lib/config-manager.ts

@@ -6,11 +6,7 @@ export const setNodeSdkConfig = (config: NodeSDKConfig) => {
   nodeSDKConfig = config;
 };
 
-export const getNodeSdkConfig = (): NodeSDKConfig => {
-  if (!nodeSDKConfig) {
-    throw new Error("Node SDK Config was not initialized.");
-  }
-
+export const getNodeSdkConfig = (): NodeSDKConfig | undefined => {
   // Ensure getters do not edit config.
-  return JSON.parse(JSON.stringify(nodeSDKConfig));
+  return nodeSDKConfig ? JSON.parse(JSON.stringify(nodeSDKConfig)) : undefined;
 };

package/lib/exporter/pii-exporter-decorator.ts

@@ -31,12 +31,12 @@ export class PIIExporterDecorator
     exporter: OTLPExporterBase<
       (ReadableSpan | ReadableLogRecord)[] | ResourceMetrics
     >,
-    config: NodeSDKConfig,
+    config: NodeSDKConfig | undefined,
   ) {
     super(exporter["_delegate"]);
     this._exporter = exporter;
     this._config = config;
-    this._redactors = this._buildRedactors(config.detection);
+    this._redactors = this._buildRedactors(config?.detection);
   }
 
   forceFlush(): Promise<void> {

package/lib/traces.ts

@@ -67,8 +67,8 @@ export function withSpan<T>({
 }: WithSpanParams<T>) {
   const sdkConfig = getNodeSdkConfig();
   const tracer = trace.getTracer(
-    traceName ?? sdkConfig.serviceName ?? "o11y-sdk",
-    sdkConfig.serviceVersion,
+    traceName ?? sdkConfig?.serviceName ?? "o11y-sdk",
+    sdkConfig?.serviceVersion,
   );
   return tracer.startActiveSpan(
     spanName,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ogcio/o11y-sdk-node",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Opentelemetry standard instrumentation SDK for NodeJS based project",
   "main": "dist/index.js",
   "type": "module",

package/test/config-manager.test.ts

@@ -3,8 +3,8 @@ import { getNodeSdkConfig, setNodeSdkConfig } from "../lib/config-manager";
 import { NodeSDKConfig } from "../lib";
 
 describe("Config Manager", () => {
-  it("throws if getConfig is called before initialization", () => {
-    expect(() => getNodeSdkConfig()).toThrow();
+  it("does not throw if getConfig is called before initialization", () => {
+    expect(() => getNodeSdkConfig()).not.toThrow();
   });
 
   it("sdk defined config is not pollutable", () => {

package/test/integration/README.md

@@ -1,26 +1,74 @@
-# Integration Test
+# Integration Test Runner
 
-This folder contains a setup for integration test with o11y node sdk.
+This script is a **Bash-based test runner** that discovers and executes integration test scripts for the Node SDK (`./packages/sdk-node/test/integration`).  
+It ensures tests run inside an isolated Docker network and provides a summary of results.
 
-## Workflow
+---
 
-- Docker must be in running state
-- Run the sh script `sh ./packages/sdk-node/test/integration/run.sh 1 .` from project root with following params
-  1. pipeline build number, for local development, any number or string is fine
-  2. root folder for docker context
-- Change dir to `packages/sdk-node/`
-- Run full test suite with `pnpm test`
+## Usage
 
-## Script
+```bash
+./run-tests.sh <build-id> <root-path> <is-ci?>
+```
+
+**Arguments**
+
+<build-id>
+A unique identifier for the test run. Used to name the Docker network (<build-id>_testnetwork).
+
+<root-path>
+The root path for the project or build environment.
+
+[ci-flag] (optional)
+Defaults to false.
+If set to true, the script will remove all Docker images after the tests complete (intended for CI pipelines to save disk space).
+
+## How It Works
+
+- Ensures at least two arguments are provided. Exits with usage instructions otherwise.
+- Creates a temporary Docker network named <build-id>\_testnetwork.
+- Finds all integration test scripts under: ./packages/sdk-node/test/integration/test\*/run.sh
+- Runs each test script.
+- Captures exit codes and tracks failures.
+- Prints results per test.
+- Removes the Docker network after all tests finish.
+- If running in CI mode (ci-flag = true), removes all Docker images.
+- Reports whether all tests passed or how many failed.
+  Exits with:
+  - 0 if all tests succeeded
+  - 1 if one or more tests failed
+
+**Example**
+
+```
+# Run tests locally
+./run-tests.sh my-build-id /path/to/project
+
+# Run tests in CI mode (cleanup Docker images after)
+./run-tests.sh my-build-id /path/to/project true
+```
+
+## Tests
+
+### test_fastify-o11y-pii-enabled
 
 The `run.sh` script performs the following steps:
 
 - build a docker image of a fastify app `/examples/fastify`
-- setup an temporary test docker network
 - run grafana alloy inside a docker container with a test configuration `/alloy/integration-test.alloy`
 - ensure is running otherwise exit process
 - run fastify app in a docker container
 - ensure is running otherwise exit process
 - execute some curl to the fastify microservice
 - persist alloy log to a file and save to following path `/packages/sdk-node/test/integration/`
+- verify pii data are removed and metrics are increased
 - docker turn down process (containers/network/image)
+
+### test_without-o11y
+
+The `run.sh` script performs the following steps:
+
+- build a docker image of a fastify app `/examples/fastify`
+- verify service execution with o11y sdk disabled (otel collector empty)
+- execute http requests for dummy route with otel trace customization function calls
+- assert 200 http status without any error

package/test/integration/docker-utils.sh

@@ -0,0 +1,214 @@
+#!/bin/bash
+
+: '
+Build a new docker image of a given Dockerfile, if already exists skip build.
+
+@param container_name docker container name
+
+@return int error code
+0. success
+1. validation error
+'
+build_image() {
+    if [ "$#" -lt 3 ] || [ "$#" -gt 4 ]; then
+        echo "Error: build_image expected 3 or 4 parameters, got $#." >&2
+        echo "Usage: build_image <image_name> <dockerfile_path> <context_path> [max_age_minutes]" >&2
+        return 1
+    fi
+    
+    
+    local image_name="$1"
+    local dockerfile_path="$2"
+    local context_path="$3"
+    local max_age_minutes="${4:-30}"  # default threshold = 30 minutes
+    
+    if docker image inspect "$image_name" > /dev/null 2>&1; then
+        echo "Image $image_name already exists. Checking age..."
+        
+        # Extract the Created timestamp from docker inspect (ISO 8601 format)
+        local created_at
+        created_at=$(docker image inspect --format '{{.Created}}' "$image_name")
+        if [ -z "$created_at" ]; then
+            echo "Warning: Could not determine creation date. Proceeding without rebuild check." >&2
+            return 0
+        fi
+        
+        # Convert created_at to epoch seconds
+        local created_epoch
+        created_epoch=$(date -d "$created_at" +%s 2>/dev/null)
+        
+        # Current time in epoch seconds
+        local now_epoch
+        now_epoch=$(date +%s)
+        
+        # Calculate age in minutes
+        local age_minutes=$(( (now_epoch - created_epoch) / 60 ))
+        
+        echo "Image $image_name is $age_minutes minutes old (threshold: $max_age_minutes minutes)."
+        
+        if [ "$age_minutes" -ge "$max_age_minutes" ]; then
+            echo "Image is too old. Rebuilding..."
+            docker build -t $image_name -f $dockerfile_path $context_path
+            return $?
+        else
+            echo "Image is recent. Skipping rebuild."
+            return 0
+        fi
+    else
+        echo "Image $image_name not found. Building..."
+        docker build -t "$image_name" -f "$dockerfile_path" "$context_path"
+        return $?
+    fi
+}
+
+: '
+Stop and remove a docker container for the given input container name.
+
+@param container_name docker container name
+
+@return int error code
+0. success
+1. validation error
+'
+container_stop_and_rm() {
+    local container_name="$1"
+    
+    if [[ -z "$container_name" ]]; then
+        echo "Error: container_name must have a value."
+        return 1
+    fi
+    
+    docker container stop $container_name
+    docker container rm -f $container_name
+    
+    return $?
+}
+
+: '
+Run a new alloy container and check if the status is running.
+
+@param root_path volume root path for alloy file config
+@param network_name docker network name
+@param container_name docker container name
+
+@optional max_retries
+@default 10
+
+@return int error code
+0. success
+1. validation error
+2. execution failed
+'
+run_and_check_alloy() {
+    
+    if [ "$#" -lt 3 ]; then
+        echo "Error: run_and_check_alloy expected 3 parameters, got $#." >&2
+        return 1
+    fi
+    
+    local root_path="$1"
+    local network_name="$2"
+    local container_name="$3"
+    local max_retries="${4:-10}"
+    local counter_retries=0
+    
+    if [[ -z "$root_path" || -z "$network_name" || -z "$container_name" ]]; then
+        echo "Error: root_path, network_name, and container_name must all have values."
+        return 1
+    fi
+    
+    if ! [[ "$max_retries" =~ ^[0-9]+$ ]]; then
+        echo "Error: max_retries parameter must be a number." >&2
+        return 1
+    fi
+    
+    docker run -d \
+    -v "$root_path/alloy/integration-test.alloy:/etc/alloy/config.alloy:ro" \
+    --network $network_name \
+    --name $container_name \
+    -p 4317:4317 \
+    -p 4318:4318 \
+    grafana/alloy:v1.9.2 \
+    run --server.http.listen-addr=0.0.0.0:12345 --stability.level=experimental /etc/alloy/config.alloy
+    
+    echo "$container_name container status"
+    until [ "$(docker inspect -f {{.State.Running}} $container_name)" = true ]; do
+        sleep 2
+        docker inspect -f {{.State.Running}} $container_name
+        counter_retries=$((counter_retries + 1))
+        if [ $counter_retries -ge $max_retries ]; then
+            echo "Exceeded maximum retries. Exiting."
+            return 2
+        fi
+    done
+    
+    return 0
+}
+
+: '
+Run a new fastify microservice container and check if the status is running.
+
+@param network_name docker network name
+@param collector_url otel collector grpc endpoint
+@param container_name docker container name
+@param build_id current pipeline build id
+
+@optional max_retries
+@default 10
+
+@return int error code
+0. success
+1. validation error
+2. execution failed
+'
+run_and_check_fastify() {
+    
+    if [ "$#" -lt 4 ]; then
+        echo "Error: run_and_check_alloy expected 3 parameters, got $#." >&2
+        return 1
+    fi
+    
+    local network_name="$1"
+    local collector_url="$2"
+    local container_name="$3"
+    local build_id="$4"
+    local max_retries="${5:-10}"
+    local counter_retries=0
+    
+    if [[ -z "$build_id" || -z "$network_name" || -z "$container_name" ]]; then
+        echo "Error: build_id, network_name, and container_name must all have values."
+        return 1
+    fi
+    
+    if ! [[ "$max_retries" =~ ^[0-9]+$ ]]; then
+        echo "Error: max_retries parameter must be a number." >&2
+        return 1
+    fi
+    
+    docker run --detach \
+    --network $network_name \
+    --name $container_name \
+    -e DB_DISABLED="true" \
+    -e SERVER_HOST="0.0.0.0" \
+    -e OTEL_COLLECTOR_URL=$collector_url \
+    --health-cmd="curl -f http://${container_name}:9091/api/health > /dev/null || exit 1" \
+    --health-start-period=1s \
+    --health-retries=10 \
+    --health-interval=1s \
+    -p 9091:9091 \
+    $container_name:$build_id
+    
+    counter_retries=0
+    echo "$container_name container status"
+    until [ "$(docker inspect -f {{.State.Health.Status}} $container_name)" = "healthy" ]; do
+        sleep 1
+        docker inspect -f {{.State.Health.Status}} $container_name
+        counter_retries=$((counter_retries + 1))
+        if [ $counter_retries -ge $max_retries ]; then
+            echo "Exceeded maximum retries. Exiting."
+            return 2
+        fi
+    done
+    
+    return 0
+}

package/test/integration/main.sh

@@ -0,0 +1,52 @@
+#!/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

@@ -0,0 +1,7 @@
+#!/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/{http-tracing.integration.test.ts → test_fastify-o11y-pii-enabled/http-tracing.integration.test.ts}

@@ -1,5 +1,5 @@
 import { describe, test, assert } from "vitest";
-import { parseLog } from "../utils/alloy-log-parser";
+import { parseLog } from "../../utils/alloy-log-parser";
 import { readFile } from "node:fs/promises";
 import { join } from "node:path";
 

package/test/integration/{pii.integration.test.ts → test_fastify-o11y-pii-enabled/pii.integration.test.ts}

@@ -1,5 +1,5 @@
 import { describe, test, assert } from "vitest";
-import { parseLog } from "../utils/alloy-log-parser";
+import { parseLog } from "../../utils/alloy-log-parser";
 import { readFile } from "node:fs/promises";
 import { join } from "node:path";
 

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

@@ -0,0 +1,42 @@
+#!/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

@@ -0,0 +1,30 @@
+#!/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

@@ -0,0 +1,32 @@
+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/traces/active-span.test.ts

@@ -1,8 +1,6 @@
-import { beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
-import * as piiDetection from "../../lib/internals/redaction/pii-detection.js";
-import { getActiveSpan } from "../../lib/traces.js";
-import { MockSpan } from "../utils/mock-signals.js";
+import { describe, expect, it, vi } from "vitest";
 import { setNodeSdkConfig } from "../../lib/config-manager.js";
+import { getActiveSpan } from "../../lib/traces.js";
 
 describe("getActiveSpan", () => {
   it("returns undefined if no active span", async () => {

package/test/traces/with-span.test.ts

@@ -10,6 +10,22 @@ import { afterAll, afterEach, beforeAll, describe, expect, it } from "vitest";
 import { setNodeSdkConfig } from "../../lib/config-manager.js";
 import { getActiveSpan, withSpan } from "../../lib/traces.js";
 
+describe("withSpan with opentelemetry not initialized", () => {
+  it("should handle noop span", async ({}) => {
+    let capturedSpan: Span;
+
+    await withSpan({
+      spanName: "test-sync-span",
+      fn: (span: Span) => {
+        capturedSpan = span;
+      },
+    });
+
+    expect(capturedSpan).not.toBeNull();
+    expect(capturedSpan.constructor.name).toBe("NonRecordingSpan");
+  });
+});
+
 describe("withSpan", () => {
   let memoryExporter: InMemorySpanExporter;
   let spanProcessor: SpanProcessor;

package/vitest.config.ts

@@ -37,7 +37,7 @@ export default defineConfig({
       },
       {
         test: {
-          include: ["**/test/integration/*.test.ts"],
+          include: ["**/test/integration/**/*.test.ts"],
           name: "integration",
         },
       },

package/test/integration/run.sh

@@ -1,88 +0,0 @@
-BUILD_ID=$1
-ROOT_PATH=$2
-
-NETWORK_NAME="${BUILD_ID}_testnetwork"
-ALLOY_CONTAINER_NAME="integrationalloy"
-NODE_CONTAINER_NAME="${BUILD_ID}_fastify_app"
-ERROR_CODE=0
-
-docker build -t ${NODE_CONTAINER_NAME}:${BUILD_ID} -f $ROOT_PATH/examples/fastify/Dockerfile $ROOT_PATH/
-
-docker network create $NETWORK_NAME
-
-docker run -d \
-    -v "$ROOT_PATH/alloy/integration-test.alloy:/etc/alloy/config.alloy:ro" \
-    --network $NETWORK_NAME \
-    --name $ALLOY_CONTAINER_NAME \
-    -p 4317:4317 \
-    -p 4318:4318 \
-    grafana/alloy:v1.9.2 \
-    run --server.http.listen-addr=0.0.0.0:12345 --stability.level=experimental /etc/alloy/config.alloy
-
-MAX_RETRIES=10
-COUNTER=0
-echo "$ALLOY_CONTAINER_NAME container status"
-until [ "$(docker inspect -f {{.State.Running}} $ALLOY_CONTAINER_NAME)" = true ]; do
-    sleep 2
-    docker inspect -f {{.State.Running}} $ALLOY_CONTAINER_NAME
-    COUNTER=$((COUNTER + 1))
-    if [ $COUNTER -ge $MAX_RETRIES ]; then
-        echo "Exceeded maximum retries. Exiting."
-        ERROR_CODE=1
-        break
-    fi
-done
-
-if [[ $ERROR_CODE -eq 0 ]]; then
-    docker run --detach \
-        --network $NETWORK_NAME \
-        --name $NODE_CONTAINER_NAME \
-        -e DB_DISABLED="true" \
-        -e SERVER_HOST="0.0.0.0" \
-        -e OTEL_COLLECTOR_URL="http://integrationalloy:4317" \
-        --health-cmd="curl -f http://${NODE_CONTAINER_NAME}:9091/api/health > /dev/null || exit 1" \
-        --health-start-period=1s \
-        --health-retries=10 \
-        --health-interval=1s \
-        -p 9091:9091 \
-        ${NODE_CONTAINER_NAME}:${BUILD_ID}
-
-    COUNTER=0
-    echo "$NODE_CONTAINER_NAME container status"
-    until [ "$(docker inspect -f {{.State.Health.Status}} $NODE_CONTAINER_NAME)" = "healthy" ]; do
-        sleep 1
-        docker inspect -f {{.State.Health.Status}} $NODE_CONTAINER_NAME
-        COUNTER=$((COUNTER + 1))
-        if [ $COUNTER -ge $MAX_RETRIES ]; then
-            echo "Exceeded maximum retries. Exiting."
-            ERROR_CODE=1
-            break
-        fi
-    done
-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 >&$ROOT_PATH/packages/sdk-node/test/integration/logs.txt
-echo "log file at $ROOT_PATH/packages/sdk-node/test/integration/logs.txt"
-
-docker container stop $ALLOY_CONTAINER_NAME
-docker container stop $NODE_CONTAINER_NAME
-
-docker container rm -f $ALLOY_CONTAINER_NAME
-docker container rm -f $NODE_CONTAINER_NAME
-
-docker image rm ${NODE_CONTAINER_NAME}:${BUILD_ID}
-
-docker network rm $NETWORK_NAME
-
-exit $ERROR_CODE