Skip to content

Commit 57bf1c1

Browse files
committed
feat(plugin-typescript): replace @poppinss/cliui with new logger
1 parent d3888a1 commit 57bf1c1

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

packages/plugin-typescript/src/lib/utils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import type { CompilerOptions } from 'typescript';
22
import type { Audit, CategoryConfig, CategoryRef } from '@code-pushup/models';
3-
import { kebabCaseToCamelCase, ui } from '@code-pushup/utils';
3+
import { kebabCaseToCamelCase, logger } from '@code-pushup/utils';
44
import { AUDITS, GROUPS, TYPESCRIPT_PLUGIN_SLUG } from './constants.js';
55
import type {
66
TypescriptPluginConfig,
@@ -146,6 +146,6 @@ export function logSkippedAudits(audits: Audit[]) {
146146
audit => !audits.some(filtered => filtered.slug === audit.slug),
147147
).map(audit => kebabCaseToCamelCase(audit.slug));
148148
if (skippedAudits.length > 0) {
149-
ui().logger.info(`Skipped audits: [${skippedAudits.join(', ')}]`);
149+
logger.info(`Skipped audits: [${skippedAudits.join(', ')}]`);
150150
}
151151
}

packages/plugin-typescript/src/lib/utils.unit.test.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { describe, expect, it } from 'vitest';
22
import { type Audit, categoryRefSchema } from '@code-pushup/models';
3-
import { ui } from '@code-pushup/utils';
3+
import { logger } from '@code-pushup/utils';
44
import { AUDITS } from './constants.js';
55
import {
66
filterAuditsByCompilerOptions,
@@ -124,15 +124,16 @@ describe('logSkippedAudits', () => {
124124
it('should warn about skipped audits', () => {
125125
logSkippedAudits(AUDITS.slice(0, -1));
126126

127-
expect(ui()).toHaveLogged(
128-
'info',
127+
expect(logger.info).toHaveBeenCalledWith(
129128
expect.stringContaining(`Skipped audits: [`),
130129
);
131130
});
132131

133132
it('should camel case the slugs in the audit message', () => {
134133
logSkippedAudits(AUDITS.slice(0, -1));
135134

136-
expect(ui()).toHaveLogged('info', expect.stringContaining(`unknownCodes`));
135+
expect(logger.info).toHaveBeenCalledWith(
136+
expect.stringContaining(`unknownCodes`),
137+
);
137138
});
138139
});

0 commit comments

Comments
 (0)