Skip to content

Reorganisation of UI Tests#153962

Open
rishi-techo-14 wants to merge 5 commits intorust-lang:mainfrom
rishi-techo-14:ui-tests-reorg-v2
Open

Reorganisation of UI Tests#153962
rishi-techo-14 wants to merge 5 commits intorust-lang:mainfrom
rishi-techo-14:ui-tests-reorg-v2

Conversation

@rishi-techo-14
Copy link

@rishi-techo-14 rishi-techo-14 commented Mar 16, 2026

Reorganize 11 tests in tests/ui/issues to appropriate subdirectories

This PR reorganizes tests in tests/ui/issues folder following the workflow suggested in the comment on #149411

Workflow:

  • Move 11 tests to appropriate directories (such as pattern, typeck, traits, borrowck, closures).
  • Rename each test in {reason}-issue-xxxx.rs format along with its corresponding .stderr file
  • Add issue links and short description about what the test checks

All tests pass tidy locally using

python x.py test tidy

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 16, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 16, 2026

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 69 candidates
  • Random selection from 15 candidates

@rishi-techo-14
Copy link
Author

@rustbot r? @Kivooeo

@rustbot rustbot assigned Kivooeo and unassigned JohnTitor Mar 16, 2026
Copy link
Member

@Kivooeo Kivooeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the word "issue" from all test names and remove the issue numbers from those tests whose names are descriptive enough.

View changes since this review

@Kivooeo
Copy link
Member

Kivooeo commented Mar 16, 2026

Also, remove the comments describing what the test is doing if the test is simple enough. However, keep an issue link and add a new line between your comment and the test.

Here's an example of what the test could look like: #152732.

@rust-log-analyzer

This comment has been minimized.

@@ -1,3 +1,5 @@
// https://github.com/rust-lang/rust/issues/4265
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be moved to ui/traits, there are no traits involved at all.

@Kivooeo
Copy link
Member

Kivooeo commented Mar 17, 2026

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 17, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 17, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@rust-bors

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants