Skip to content

Commit b53be9e

Browse files
committed
change expect calls from truthy/falsy to true/false checks
1 parent b8d2b0c commit b53be9e

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

packages/node/test/handlers.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -193,8 +193,8 @@ describe('tracingHandler', () => {
193193
expect(transaction.parentSpanId).toEqual('1121201211212012');
194194
expect(transaction.sampled).toEqual(false);
195195
expect(transaction.metadata?.baggage).toBeDefined();
196-
expect(isBaggageEmpty(transaction.metadata?.baggage)).toBeTruthy();
197-
expect(isBaggageMutable(transaction.metadata?.baggage)).toBeFalsy();
196+
expect(isBaggageEmpty(transaction.metadata?.baggage)).toBe(true);
197+
expect(isBaggageMutable(transaction.metadata?.baggage)).toBe(false);
198198
});
199199

200200
it("pulls parent's data from tracing and baggage headers on the request", () => {

packages/tracing/test/browser/browsertracing.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ describe('BrowserTracing', () => {
398398

399399
expect(headerContext).toBeDefined();
400400
expect(headerContext?.metadata?.baggage).toBeDefined();
401-
expect(isBaggageEmpty(headerContext?.metadata?.baggage as Baggage)).toBeTruthy();
401+
expect(isBaggageEmpty(headerContext?.metadata?.baggage as Baggage)).toBe(true);
402402
});
403403

404404
it('does not crash if the baggage header is malformed', () => {
@@ -421,7 +421,7 @@ describe('BrowserTracing', () => {
421421

422422
expect(headerContext).toBeDefined();
423423
expect(headerContext?.metadata?.baggage).toBeDefined();
424-
expect(isBaggageEmpty(headerContext?.metadata?.baggage as Baggage)).toBeTruthy();
424+
expect(isBaggageEmpty(headerContext?.metadata?.baggage as Baggage)).toBe(true);
425425
});
426426
});
427427

@@ -475,7 +475,7 @@ describe('BrowserTracing', () => {
475475
expect(transaction.parentSpanId).toEqual('1121201211212012');
476476
expect(transaction.sampled).toBe(false);
477477
expect(baggage).toBeDefined();
478-
expect(isSentryBaggageEmpty(baggage)).toBeTruthy();
478+
expect(isSentryBaggageEmpty(baggage)).toBe(true);
479479
expect(baggage[1]).toBeDefined();
480480
expect(baggage[1]).toEqual('foo=bar');
481481
});

packages/tracing/test/span.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,7 @@ describe('Span', () => {
419419
const baggage = span.getBaggage();
420420

421421
expect(hubSpy).toHaveBeenCalledTimes(0);
422-
expect(baggage && isSentryBaggageEmpty(baggage)).toBeFalsy();
422+
expect(baggage && isSentryBaggageEmpty(baggage)).toBe(false);
423423
expect(baggage && getSentryBaggageItems(baggage)).toStrictEqual({ environment: 'myEnv' });
424424
expect(baggage && getThirdPartyBaggage(baggage)).toStrictEqual('');
425425
});
@@ -440,7 +440,7 @@ describe('Span', () => {
440440
const baggage = span.getBaggage();
441441

442442
expect(hubSpy).toHaveBeenCalledTimes(1);
443-
expect(baggage && isSentryBaggageEmpty(baggage)).toBeFalsy();
443+
expect(baggage && isSentryBaggageEmpty(baggage)).toBe(false);
444444
expect(baggage && getSentryBaggageItems(baggage)).toStrictEqual({ release: '1.0.1', environment: 'production' });
445445
expect(baggage && getThirdPartyBaggage(baggage)).toStrictEqual('');
446446
});

0 commit comments

Comments
 (0)