Skip to content

Add missing generic test coverage for #[splat]#158684

Open
Ajay-singh1 wants to merge 1 commit into
rust-lang:mainfrom
Ajay-singh1:ui-tests-for-splat
Open

Add missing generic test coverage for #[splat]#158684
Ajay-singh1 wants to merge 1 commit into
rust-lang:mainfrom
Ajay-singh1:ui-tests-for-splat

Conversation

@Ajay-singh1

@Ajay-singh1 Ajay-singh1 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This PR adds missing test coverage for #[splat] that was marked as FIXME in splat-generics-everywhere.rs:

  • Add generic parameter to BarTrait (BarTrait<T>)
  • Add generic_in_tuple , tests generics inside the splatted tuple
  • Add generic_tuple_assoc , tests assoc functions with splatted generic tuple traits
  • Add trait_generic_in_tuple , tests generics inside splatted tuple on trait methods
  • Add trait_generic_tuple , tests splatted generic tuple trait bounds on trait methods

Removes the resolved FIXME comments:

  • also add generics to the trait
  • also add assoc/method with splatted generic tuple traits
  • also add generics inside the splatted tuple

CC:- @teor2345

r? @teor2345

Signed-off-by: Ajay Singh <ajaykripa8736968359@gmail.com>
@rustbot

rustbot commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Failed to set assignee to teor2345: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@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 Jul 2, 2026
@teor2345

teor2345 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Still haven't got the triage or member role yet, so I have to comment to assign myself

r? @teor2345

@teor2345 teor2345 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@rustbot label +F-splat +T-compiler

View changes since this review

@rustbot rustbot added the F-splat `#![feature(splat)]` label Jul 2, 2026
@Kivooeo

Kivooeo commented Jul 2, 2026

Copy link
Copy Markdown
Member

@bors r=teor2345 rollup

@rust-bors

rust-bors Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 8020a9b has been approved by teor2345

It is now in the queue for this repository.

🌲 The tree is currently closed for pull requests below priority 1000. This pull request will be tested once the tree is reopened.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 2, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 2, 2026
…teor2345

Add missing generic test coverage for ```#[splat]```

This PR adds missing test coverage for `#[splat]` that was marked as FIXME in `splat-generics-everywhere.rs`:

- Add generic parameter to `BarTrait` (`BarTrait<T>`)
- Add `generic_in_tuple` , tests generics inside the splatted tuple
- Add `generic_tuple_assoc` , tests assoc functions with splatted generic tuple traits
- Add `trait_generic_in_tuple` , tests generics inside splatted tuple on trait methods
- Add `trait_generic_tuple` , tests splatted generic tuple trait bounds on trait methods

Removes the resolved FIXME comments:
- also add generics to the trait
- also add assoc/method with splatted generic tuple traits
- also add generics inside the splatted tuple

CC:- @teor2345

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

Labels

F-splat `#![feature(splat)]` S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

4 participants