mirror of
https://github.com/pnpm/pnpm.git
synced 2026-01-19 12:28:44 -05:00
b217edbfd8
* test: move expect blocks (without other changes) to end of test This is required for the refactor in the next commit. The log statements have to run before consuming the output stream, otherwise the output stream will be empty. * test: fix swallowed Jest expect errors in `default-reporter` package * test: fix expected to match actual values in default-reporter tests * refactor: remove redundant `.pipe(take(1))` With the refactor to use `firstValueFrom`, the `take(1)` is now redundant in many places. ```ts firstValueFrom(output$.pipe(take(1))) ``` ```ts firstValueFrom(output$) ```
49 lines
1.2 KiB
TypeScript
49 lines
1.2 KiB
TypeScript
import { setTimeout } from 'node:timers/promises'
|
|
import { executionTimeLogger } from '@pnpm/core-loggers'
|
|
import { packageManager } from '@pnpm/cli-meta'
|
|
import { toOutput$ } from '@pnpm/default-reporter'
|
|
import { createStreamParser } from '@pnpm/logger'
|
|
import { firstValueFrom } from 'rxjs'
|
|
|
|
const NO_OUTPUT = Symbol('test should not log anything')
|
|
|
|
test('does not print execution time for help command', async () => {
|
|
const output$ = toOutput$({
|
|
context: {
|
|
argv: ['help'],
|
|
},
|
|
streamParser: createStreamParser(),
|
|
})
|
|
|
|
executionTimeLogger.debug({
|
|
startedAt: 1665279402859,
|
|
endedAt: 1665279413671,
|
|
})
|
|
|
|
const output = await Promise.race([
|
|
firstValueFrom(output$),
|
|
setTimeout(10).then(() => NO_OUTPUT),
|
|
])
|
|
|
|
expect(output).toEqual(NO_OUTPUT)
|
|
})
|
|
|
|
test('prints execution time for install command', async () => {
|
|
const output$ = toOutput$({
|
|
context: {
|
|
argv: ['install'],
|
|
},
|
|
streamParser: createStreamParser(),
|
|
})
|
|
|
|
executionTimeLogger.debug({
|
|
startedAt: 1665279402859,
|
|
endedAt: 1665279413671,
|
|
})
|
|
|
|
expect.assertions(1)
|
|
|
|
const output = await firstValueFrom(output$)
|
|
expect(output).toBe(`Done in 10.8s using ${packageManager.name} v${packageManager.version}`)
|
|
})
|