Skip to content

Commit b41e6c1

Browse files
committed
refactor: wip
1 parent b5a0c7b commit b41e6c1

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

packages/utils/src/lib/profiler/profiler.unit.test.ts

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ describe('Profiler', () => {
2121
new Profiler({
2222
prefix: 'cp',
2323
track: 'test-track',
24+
enabled: true,
2425
...overrides,
2526
});
2627

@@ -44,7 +45,6 @@ describe('Profiler', () => {
4445

4546
it('constructor should use defaults for measure', () => {
4647
const customProfiler = getProfiler({ color: 'secondary' });
47-
customProfiler.setEnabled(true);
4848

4949
const result = customProfiler.measure('test-operation', () => 'success');
5050

@@ -129,7 +129,7 @@ describe('Profiler', () => {
129129
});
130130

131131
it('isEnabled should set and get enabled state', () => {
132-
const profiler = getProfiler();
132+
const profiler = getProfiler({ enabled: false });
133133
expect(profiler.isEnabled()).toBe(false);
134134

135135
profiler.setEnabled(true);
@@ -154,7 +154,6 @@ describe('Profiler', () => {
154154

155155
it('marker should execute without error when enabled', () => {
156156
const enabledProfiler = getProfiler();
157-
enabledProfiler.setEnabled(true);
158157
expect(() => {
159158
enabledProfiler.marker('test-marker', {
160159
color: 'primary',
@@ -183,7 +182,6 @@ describe('Profiler', () => {
183182
performance.clearMarks();
184183

185184
const profilerWithColor = getProfiler({ color: 'primary' });
186-
profilerWithColor.setEnabled(true);
187185

188186
expect(() => {
189187
profilerWithColor.marker('test-marker-default-color', {
@@ -208,7 +206,6 @@ describe('Profiler', () => {
208206

209207
it('marker should execute without error when enabled with no default color', () => {
210208
const profilerNoColor = getProfiler();
211-
profilerNoColor.setEnabled(true);
212209

213210
expect(() => {
214211
profilerNoColor.marker('test-marker-no-color', {
@@ -253,7 +250,6 @@ describe('Profiler', () => {
253250
performance.clearMeasures();
254251

255252
const enabledProfiler = getProfiler();
256-
enabledProfiler.setEnabled(true);
257253
const workFn = vi.fn(() => 'result');
258254
const result = enabledProfiler.measure('test-event', workFn, {
259255
color: 'primary',
@@ -333,7 +329,6 @@ describe('Profiler', () => {
333329

334330
it('measure should propagate errors when enabled and call error callback', () => {
335331
const enabledProfiler = getProfiler();
336-
enabledProfiler.setEnabled(true);
337332
const error = new Error('Enabled test error');
338333
const workFn = vi.fn(() => {
339334
throw error;
@@ -372,7 +367,6 @@ describe('Profiler', () => {
372367

373368
it('measureAsync should handle async operations correctly when enabled', async () => {
374369
const enabledProfiler = getProfiler();
375-
enabledProfiler.setEnabled(true);
376370
const workFn = vi.fn(async () => {
377371
await Promise.resolve();
378372
return 'async-result';
@@ -446,7 +440,6 @@ describe('Profiler', () => {
446440

447441
it('measureAsync should propagate async errors when enabled and call error callback', async () => {
448442
const enabledProfiler = getProfiler();
449-
enabledProfiler.setEnabled(true);
450443
const error = new Error('Enabled async test error');
451444
const workFn = vi.fn(async () => {
452445
await Promise.resolve();

0 commit comments

Comments
 (0)