chore(exporter-otlp-*-http): clean up tests (#5198)

experimental/CHANGELOG.md

@@ -32,6 +32,7 @@ All notable changes to experimental packages in this project will be documented
 
 ### :house: (Internal)
 
+* chore(otlp-exporter-\*-http): clean up tests [#5196](https://github.com/open-telemetry/opentelemetry-js/pull/5198) @pichlermarc
 * chore(otlp-exporter-\*-proto): clean up tests [#5196](https://github.com/open-telemetry/opentelemetry-js/pull/5199) @pichlermarc
 
 ## 0.55.0

experimental/packages/exporter-logs-otlp-http/test/browser/OTLPLogExporter.test.ts

@@ -17,16 +17,78 @@ import * as assert from 'assert';
 import * as sinon from 'sinon';
 
 import { OTLPLogExporter } from '../../src/platform/browser';
+import {
+  LoggerProvider,
+  SimpleLogRecordProcessor,
+} from '@opentelemetry/sdk-logs';
 
-describe('OTLPLogExporter', () => {
+/*
+ * NOTE: Tests here are not intended to test the underlying components directly. They are intended as a quick
+ * check if the correct components are used. Use the following packages to test details:
+ * - `@opentelemetry/oltp-exporter-base`: OTLP common exporter logic (handling of concurrent exports, ...), HTTP transport code
+ * - `@opentelemetry/otlp-transformer`: Everything regarding serialization and transforming internal representations to OTLP
+ */
+
+describe('OTLPLogExporter', function () {
   afterEach(() => {
     sinon.restore();
   });
 
-  describe('constructor', () => {
-    it('should create an instance', () => {
-      const exporter = new OTLPLogExporter();
-      assert.ok(exporter instanceof OTLPLogExporter);
+  describe('export', function () {
+    describe('when sendBeacon is available', function () {
+      it('should successfully send data using sendBeacon', async function () {
+        // arrange
+        const stubBeacon = sinon.stub(navigator, 'sendBeacon');
+        const loggerProvider = new LoggerProvider();
+        loggerProvider.addLogRecordProcessor(
+          new SimpleLogRecordProcessor(new OTLPLogExporter())
+        );
+
+        // act
+        loggerProvider.getLogger('test-logger').emit({ body: 'test-body' });
+        await loggerProvider.shutdown();
+
+        // assert
+        const args = stubBeacon.args[0];
+        const blob: Blob = args[1] as unknown as Blob;
+        const body = await blob.text();
+        assert.doesNotThrow(
+          () => JSON.parse(body),
+          'expected requestBody to be in JSON format, but parsing failed'
+        );
+      });
+    });
+
+    describe('when sendBeacon is not available', function () {
+      beforeEach(function () {
+        // fake sendBeacon not being available
+        (window.navigator as any).sendBeacon = false;
+      });
+
+      it('should successfully send data using XMLHttpRequest', async function () {
+        // arrange
+        const server = sinon.fakeServer.create();
+        const loggerProvider = new LoggerProvider();
+        loggerProvider.addLogRecordProcessor(
+          new SimpleLogRecordProcessor(new OTLPLogExporter())
+        );
+
+        // act
+        loggerProvider.getLogger('test-logger').emit({ body: 'test-body' });
+        queueMicrotask(() => {
+          // simulate success response
+          server.requests[0].respond(200, {}, '');
+        });
+        await loggerProvider.shutdown();
+
+        // assert
+        const request = server.requests[0];
+        const body = request.requestBody as unknown as Uint8Array;
+        assert.doesNotThrow(
+          () => JSON.parse(new TextDecoder().decode(body)),
+          'expected requestBody to be in JSON format, but parsing failed'
+        );
+      });
     });
   });
 });