Skip to content

internal: Only compare relevant parts in ide::{runnables,inlay_hints} tests #16256

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 9 commits into from
Jan 5, 2024

Conversation

Urhengulas
Copy link
Contributor

This PR limits the data being compared. Therefore the tests should be more readable, as well as being more robust to changes to the data structure.

Part of #14268.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 4, 2024
@Urhengulas Urhengulas changed the title internal: Only compare relevant parts in ide::runnables tests internal: Only compare relevant parts in ide::{runnables,inlay_hints} tests Jan 4, 2024
@Urhengulas
Copy link
Contributor Author

@rustbot r? @Veykril

ignore: false,
},
},
cfg: Some(
Copy link
Member

Choose a reason for hiding this comment

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

This is interesting data (as long as its Some), same for when use_name_in_title is true. Would be good to render those conditionally (or otherwise keep them in the output in some way)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, gonna add that.

@Veykril
Copy link
Member

Veykril commented Jan 5, 2024

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Jan 5, 2024

📌 Commit f8ff4fb has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jan 5, 2024

⌛ Testing commit f8ff4fb with merge e53792b...

@bors
Copy link
Contributor

bors commented Jan 5, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing e53792b to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants