Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: add an option to hide skipped test lines #2745

Merged
merged 9 commits into from May 3, 2023
Merged

feat: add an option to hide skipped test lines #2745

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
33bc41f
feat: Hide skipped test lines
g4rry420 Jan 25, 2023
d302c14
Merge branch 'main' into issue_2647
g4rry420 Jan 25, 2023
7bd704e
chore: Changed option name from silentSkip to hideSkippedTests
g4rry420 Feb 5, 2023
210bf0c
Merge branch 'main' into issue_2647
g4rry420 Feb 5, 2023
b14bf82
Merge branch 'main' into issue_2647
g4rry420 Feb 10, 2023
7da03c7
Merge branch 'main' into issue_2647
sheremet-va Mar 7, 2023
68dbbb5
chore: resolved conflicts
g4rry420 Apr 17, 2023
b5b61cd
Merge branch 'issue_2647' of https://github.com/g4rry420/vitest into …
g4rry420 Apr 17, 2023
6a7e71a
Merge branch 'main' into issue_2647
sheremet-va May 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/vitest/src/defaults.ts
View file
Open in desktop
Expand Up @@ -76,6 +76,7 @@ const config = {
update: false,
reporters: [],
silent: false,
hideSkippedTests: false,
api: false,
ui: false,
uiBase: '/__vitest__/',
Expand Down
1 change: 1 addition & 0 deletions packages/vitest/src/node/cli.ts
View file
Open in desktop
Expand Up @@ -22,6 +22,7 @@ cli
.option('--api [api]', 'Serve API, available options: --api.port <port>, --api.host [host] and --api.strictPort')
.option('--threads', 'Enabled threads (default: true)')
.option('--silent', 'Silent console output from tests')
.option('--hideSkippedTests', 'Hide logs for skipped tests')
.option('--isolate', 'Isolate environment for each test file (default: true)')
.option('--reporter <name>', 'Specify reporters')
.option('--outputDiffMaxSize <length>', 'Object diff output max size (default: 10000)')
Expand Down
27 changes: 23 additions & 4 deletions packages/vitest/src/node/reporters/renderers/listRenderer.ts
View file
Open in desktop
Expand Up @@ -143,8 +143,15 @@ export function renderTree(tasks: Task[], options: ListRendererOptions, level =
output = output.concat(renderHookState(task, 'beforeAll', level + 1))
output = output.concat(renderHookState(task, 'beforeEach', level + 1))
if (task.type === 'suite' && task.tasks.length > 0) {
if ((task.result?.state === 'fail' || task.result?.state === 'run' || options.renderSucceed))
output = output.concat(renderTree(task.tasks, options, level + 1))
if ((task.result?.state === 'fail' || task.result?.state === 'run' || options.renderSucceed)) {
if (options.logger.ctx.config.hideSkippedTests) {
const filteredTasks = task.tasks.filter(t => t.mode !== 'skip' && t.mode !== 'todo')
output = output.concat(renderTree(filteredTasks, options, level + 1))
}
else {
output = output.concat(renderTree(task.tasks, options, level + 1))
}
}
}
output = output.concat(renderHookState(task, 'afterAll', level + 1))
output = output.concat(renderHookState(task, 'afterEach', level + 1))
Expand All @@ -161,7 +168,13 @@ export const createListRenderer = (_tasks: Task[], options: ListRendererOptions)
const log = options.logger.logUpdate

function update() {
log(renderTree(tasks, options))
if (options.logger.ctx.config.hideSkippedTests) {
const filteredTasks = tasks.filter(t => t.mode !== 'skip' && t.mode !== 'todo')
log(renderTree(filteredTasks, options))
}
else {
log(renderTree(tasks, options))
}
}

return {
Expand All @@ -182,7 +195,13 @@ export const createListRenderer = (_tasks: Task[], options: ListRendererOptions)
timer = undefined
}
log.clear()
options.logger.log(renderTree(tasks, options))
if (options.logger.ctx.config.hideSkippedTests) {
const filteredTasks = tasks.filter(t => t.mode !== 'skip' && t.mode !== 'todo')
options.logger.log(renderTree(filteredTasks, options))
}
else {
options.logger.log(renderTree(tasks, options))
}
return this
},
clear() {
Expand Down
7 changes: 7 additions & 0 deletions packages/vitest/src/types/config.ts
View file
Open in desktop
Expand Up @@ -272,6 +272,13 @@ export interface InlineConfig {
*/
silent?: boolean

/**
* Hide logs for skipped tests
*
* @default false
*/
hideSkippedTests?: boolean

/**
* Path to setup files
*/
Expand Down