fix tests using assert

open-telemetry · Jan 8, 2024 · 7847db3 · 7847db3
1 parent c17af35
commit 7847db3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/packages/opentelemetry-core/test/utils/callback.test.ts b/packages/opentelemetry-core/test/utils/callback.test.ts
@@ -17,7 +17,6 @@
 import * as assert from 'assert';
 import * as sinon from 'sinon';
 import { BindOnceFuture } from '../../src/utils/callback.js';
-import { assertRejects } from '../test-utils.js';
 
 describe('callback', () => {
   describe('BindOnceFuture', () => {
@@ -34,27 +33,27 @@ describe('callback', () => {
       assert.deepStrictEqual(stub.firstCall.args, [1]);
       assert.deepStrictEqual(stub.firstCall.thisValue, that);
 
-      assert(future.isCalled);
+      assert.ok(future.isCalled);
     });
 
     it('should handle thrown errors', async () => {
       const stub = sinon.stub();
       stub.throws(new Error('foo'));
       const future = new BindOnceFuture(stub, undefined);
 
-      await assertRejects(future.call(), /foo/);
-      await assertRejects(future.call(), /foo/);
-      await assertRejects(future.promise, /foo/);
+      await assert.rejects(future.call(), /foo/);
+      await assert.rejects(future.call(), /foo/);
+      await assert.rejects(future.promise, /foo/);
     });
 
     it('should handle rejections', async () => {
       const stub = sinon.stub();
       stub.rejects(new Error('foo'));
       const future = new BindOnceFuture(stub, undefined);
 
-      await assertRejects(future.call(), /foo/);
-      await assertRejects(future.call(), /foo/);
-      await assertRejects(future.promise, /foo/);
+      await assert.rejects(future.call(), /foo/);
+      await assert.rejects(future.call(), /foo/);
+      await assert.rejects(future.promise, /foo/);
     });
   });
 });
diff --git a/packages/opentelemetry-core/test/utils/promise.test.ts b/packages/opentelemetry-core/test/utils/promise.test.ts
@@ -16,7 +16,6 @@
 
 import * as assert from 'assert';
 import { Deferred } from '../../src/utils/promise.js';
-import { assertRejects } from '../test-utils.js';
 
 describe('promise', () => {
   describe('Deferred', () => {
@@ -35,7 +34,7 @@ describe('promise', () => {
       deferred.reject(new Error('foo'));
       deferred.reject(new Error('bar'));
 
-      await assertRejects(deferred.promise, /foo/);
+      await assert.rejects(deferred.promise, /foo/);
     });
   });
 });