Skip to content

Migrate fixture tests #201

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.

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

Merged
merged 2 commits into from
Apr 28, 2025
Merged

Conversation

Muscraft
Copy link
Member

Compared to source code based tests, the benefits of using toml to specify tests haven't felt worth it to me recently. That, combined with the fixture tests mostly being about the colored output, made me realize that we should probably get rid of the toml fixtures. So I went ahead and did just that. I am happy with the changes, with the only caveat being that the test names all have ::case at the end.

test ann_eof::case ... ok
test ann_insertion::case ... ok
test ann_removed_nl::case ... ok
test ann_multiline2::case ... ok
test ensure_emoji_highlight_width::case ... ok
test ann_multiline::case ... ok
test fold_bad_origin_line::case ... ok
test issue_9::case ... ok
test fold_leading::case ... ok
test fold_ann_multiline::case ... ok
test fold_trailing::case ... ok
test strip_line_char::case ... ok
test strip_line::case ... ok
test multiple_annotations::case ... ok
test simple::case ... ok
test strip_line_non_ws::case ... ok

@Muscraft Muscraft force-pushed the migrate-fixture-tests branch from 9da045c to df05eb7 Compare April 28, 2025 17:39
@epage epage merged commit 44da343 into rust-lang:master Apr 28, 2025
17 checks passed
@Muscraft Muscraft deleted the migrate-fixture-tests branch April 28, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants