chore(api-logs)!: remove ProxyLoggerProvider export (#6322)

Author: david-luna

experimental/CHANGELOG.md

@@ -10,6 +10,7 @@ For notes on migrating to 2.x / 0.200.x see [the upgrade guide](doc/upgrade-to-2
 
 * fix(otlp-exporter-base)!: remove xhr transport [#6317](https://github.com/open-telemetry/opentelemetry-js/pull/6317) @cjihrig
   * (user-facing) The deprecated XHR-based transport has been removed and replaced with `fetch()`. This change affects users who relied on `XmlHttpRequest` instead of `fetch()` for sending headers with OTLP exports. To maintain compatibility on browsers without a `fetch()` implementation, include a `fetch()` polyfill.
+chore(api-logs)!: remove `ProxyLoggerProvider` export [#6322](https://github.com/open-telemetry/opentelemetry-js/pull/6322) @david-luna
 
 ### :rocket: Features
 

experimental/packages/api-logs/src/index.ts

@@ -21,7 +21,6 @@ export type { LogAttributes, LogBody, LogRecord } from './types/LogRecord';
 export type { LoggerOptions } from './types/LoggerOptions';
 export type { AnyValue, AnyValueMap } from './types/AnyValue';
 export { NOOP_LOGGER, NoopLogger } from './NoopLogger';
-export { ProxyLoggerProvider } from './ProxyLoggerProvider';
 
 import { LogsAPI } from './api/logs';
 export const logs = LogsAPI.getInstance();

experimental/packages/api-logs/test/api/api.test.ts

@@ -15,7 +15,8 @@
  */
 
 import * as assert from 'assert';
-import { Logger, ProxyLoggerProvider, logs } from '../../src';
+import { Logger, logs } from '../../src';
+import { ProxyLoggerProvider } from '../../src/ProxyLoggerProvider';
 import { NoopLogger } from '../../src/NoopLogger';
 import { NoopLoggerProvider } from '../../src/NoopLoggerProvider';
 

experimental/packages/api-logs/test/internal/global.test.ts

@@ -17,7 +17,7 @@
 import * as assert from 'assert';
 import { _global, GLOBAL_LOGS_API_KEY } from '../../src/internal/global-utils';
 import { NoopLoggerProvider } from '../../src/NoopLoggerProvider';
-import { ProxyLoggerProvider } from '../../src';
+import { ProxyLoggerProvider } from '../../src/ProxyLoggerProvider';
 
 const api1 = require('../../src') as typeof import('../../src');
 

experimental/packages/api-logs/test/proxy-implementations/proxy-logger.test.ts

@@ -16,9 +16,10 @@
 
 import * as assert from 'assert';
 import * as sinon from 'sinon';
-import { Logger, LoggerProvider, ProxyLoggerProvider } from '../../src';
+import { Logger, LoggerProvider } from '../../src';
 import { NoopLogger } from '../../src/NoopLogger';
 import { ProxyLogger } from '../../src/ProxyLogger';
+import { ProxyLoggerProvider } from '../../src/ProxyLoggerProvider';
 
 describe('ProxyLogger', () => {
   let provider: ProxyLoggerProvider;

packages/opentelemetry-sdk-trace-node/test/registration.test.ts

@@ -59,8 +59,10 @@ describe('API registration', function () {
       propagation['_getGlobalPropagator'](),
       CompositePropagator
     );
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should register configured implementations', function () {
@@ -80,8 +82,10 @@ describe('API registration', function () {
     assert.strictEqual(context['_getContextManager'](), mockContextManager);
     assert.strictEqual(propagation['_getGlobalPropagator'](), mockPropagator);
 
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should skip null context manager', function () {
@@ -102,8 +106,10 @@ describe('API registration', function () {
       CompositePropagator
     );
 
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should skip null propagator', function () {
@@ -121,7 +127,9 @@ describe('API registration', function () {
       AsyncLocalStorageContextManager
     );
 
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 });

packages/opentelemetry-sdk-trace-web/test/NodeGlobalsFoolProofing.test.ts

@@ -57,8 +57,10 @@ describe('Node Globals Foolproofing', function () {
     );
 
     assert.ok(context['_getContextManager']() instanceof StackContextManager);
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('Can get TraceProvider with custom id generator and without node globals such as process', function () {

packages/opentelemetry-sdk-trace-web/test/registration.test.ts

@@ -42,8 +42,10 @@ describe('API registration', function () {
     assert.ok(
       propagation['_getGlobalPropagator']() instanceof CompositePropagator
     );
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should register configured implementations', function () {
@@ -60,8 +62,10 @@ describe('API registration', function () {
     assert.ok(context['_getContextManager']() === contextManager);
     assert.ok(propagation['_getGlobalPropagator']() === propagator);
 
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should skip null context manager', function () {
@@ -80,8 +84,10 @@ describe('API registration', function () {
     assert.ok(
       propagation['_getGlobalPropagator']() instanceof CompositePropagator
     );
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 
   it('should skip null propagator', function () {
@@ -98,7 +104,9 @@ describe('API registration', function () {
     );
 
     assert.ok(context['_getContextManager']() instanceof StackContextManager);
-    assert.strictEqual(setGlobalTracerProviderSpy.callCount, 1);
-    assert.ok(setGlobalTracerProviderSpy.lastCall.args[0] === tracerProvider);
+    sinon.assert.calledOnceWithMatch(
+      setGlobalTracerProviderSpy,
+      (provider: any) => provider === tracerProvider
+    );
   });
 });