Skip to content

Move cross crate tests into the appropriate folder#157447

Open
danieljofficial wants to merge 2 commits into
rust-lang:mainfrom
danieljofficial:move-tests-cross-crate
Open

Move cross crate tests into the appropriate folder#157447
danieljofficial wants to merge 2 commits into
rust-lang:mainfrom
danieljofficial:move-tests-cross-crate

Conversation

@danieljofficial
Copy link
Copy Markdown
Contributor

@danieljofficial danieljofficial commented Jun 4, 2026

View all comments

Hi, I've returned with a few more reorganized tests.
r? @jieyouxu
This is part of 133895

try-job: armhf-gnu

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Jun 4, 2026

This PR modifies tests/ui/issues/. If this PR is adding new tests to tests/ui/issues/,
please refrain from doing so, and instead add it to more descriptive subdirectories.

@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 Jun 4, 2026
@Kivooeo
Copy link
Copy Markdown
Member

Kivooeo commented Jun 5, 2026

cc @zedddie to avoid gsoc work conflicts

@danieljofficial
Copy link
Copy Markdown
Contributor Author

cc @zedddie to avoid gsoc work conflicts

Noted.

Copy link
Copy Markdown
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Comment thread tests/ui/issues/issue-2170-exe.rs
Comment thread tests/ui/cross-crate/mut-ref-write-visible-after-unwind.rs Outdated
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 5, 2026

📌 Commit 2b55fda has been approved by jieyouxu

It is now in the queue for this repository.

@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 Jun 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 5, 2026
…rate, r=jieyouxu

Move cross crate tests into the appropriate folder

Hi, I've returned with a few more reorganized tests.
r? @jieyouxu
This is part of [133895](rust-lang#133895)
rust-bors Bot pushed a commit that referenced this pull request Jun 5, 2026
…uwer

Rollup of 8 pull requests

Successful merges:

 - #157467 (stdarch subtree update)
 - #156798 (delegation: do not always generate first argument)
 - #157450 (mark `Encode`, `Decode`, `Mark` impls as `#[inline]`)
 - #157190 (Silence recursive RUSTC_LOG_FORMAT_JSON messages)
 - #157447 (Move cross crate tests into the appropriate folder)
 - #157470 (Avoid ICE when emitting TargetMachine config errors)
 - #157474 (Forbid optimize(none) with inline(always) or inline.)
 - #157475 (Add a smoke test for the optimize attribute.)
@JonathanBrouwer
Copy link
Copy Markdown
Contributor

💔 I suspect this PR failed tests as part of a rollup
@bors r-

After fixing the problem, consider running a try job for the failed job before re-approving.

Link to failure: #157481 (comment)

@rust-bors rust-bors Bot 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 5, 2026
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 5, 2026

This pull request was unapproved.

This PR was contained in a rollup (#157481), which was unapproved.

View changes since this unapproval

@danieljofficial
Copy link
Copy Markdown
Contributor Author

💔 I suspect this PR failed tests as part of a rollup @bors r-

After fixing the problem, consider running a try job for the failed job before re-approving.

Link to failure: #157481 (comment)

I can't tell what might've caused this, locally the tests are passing. So I'm a bit confused about what I should change to fix this.

@jieyouxu

This comment was marked as outdated.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 5, 2026
Move cross crate tests into the appropriate folder


try-job: armhf-gnu
@jieyouxu
Copy link
Copy Markdown
Member

jieyouxu commented Jun 5, 2026

---- [ui] tests/ui/cross-crate/stable-symbol-hash-for-trait-object-projections.rs stdout ----

error: test did not exit with success! code=Some(127) so test would pass with `run-fail`
status: exit status: 127
command: RUSTC="/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" RUST_TEST_THREADS="4" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage1-tools-bin/remote-test-client" "run" "1" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/cross-crate/stable-symbol-hash-for-trait-object-projections/a" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/cross-crate/stable-symbol-hash-for-trait-object-projections/auxiliary/libstable_symbol_hash_for_trait_object_projections.so"
--- stdout -------------------------------
uploaded "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/cross-crate/stable-symbol-hash-for-trait-object-projections/a", waiting for result
------------------------------------------
--- stderr -------------------------------
/tmp/work/test1271/a: error while loading shared libraries: libstable_symbol_hash_for_trait_object_projections.so: cannot open shared object file: No such file or directory
------------------------------------------

---- [ui] tests/ui/cross-crate/stable-symbol-hash-for-trait-object-projections.rs stdout end ----

@danieljofficial danieljofficial force-pushed the move-tests-cross-crate branch from 2b55fda to dc17feb Compare June 5, 2026 15:11
@rust-bors

This comment was marked as outdated.

@rust-log-analyzer

This comment was marked as outdated.

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

This comment was marked as outdated.

@Kivooeo
Copy link
Copy Markdown
Member

Kivooeo commented Jun 6, 2026

@bors r- because of conflict

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

This pull request was not previously approved.

@jieyouxu
Copy link
Copy Markdown
Member

jieyouxu commented Jun 6, 2026

Ugh. Please rebase. I will bump the p on this PR.
@bors p=1 (conflict-prone)

@danieljofficial danieljofficial force-pushed the move-tests-cross-crate branch from dc17feb to 1e1560b Compare June 6, 2026 09:15
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Jun 6, 2026

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@jieyouxu
Copy link
Copy Markdown
Member

jieyouxu commented Jun 6, 2026

@bors r+

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

📌 Commit 1e1560b has been approved by jieyouxu

It is now in the queue for this repository.

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jun 6, 2026
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

⌛ Testing commit 1e1560b with merge 554b619...

Workflow: https://github.com/rust-lang/rust/actions/runs/27060515262

rust-bors Bot pushed a commit that referenced this pull request Jun 6, 2026
…youxu

Move cross crate tests into the appropriate folder



Hi, I've returned with a few more reorganized tests. 
r? @jieyouxu 
This is part of [133895](#133895)

try-job: armhf-gnu
@JonathanBrouwer
Copy link
Copy Markdown
Contributor

Stuck
@bors yield

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #157275.

@JonathanBrouwer
Copy link
Copy Markdown
Contributor

@bors try jobs=dist-i686-msvc

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 6, 2026
Move cross crate tests into the appropriate folder


try-job: dist-i686-msvc
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

⌛ Testing commit 1e1560b with merge e97a4f2...

Workflow: https://github.com/rust-lang/rust/actions/runs/27071779922

rust-bors Bot pushed a commit that referenced this pull request Jun 6, 2026
…youxu

Move cross crate tests into the appropriate folder



Hi, I've returned with a few more reorganized tests. 
r? @jieyouxu 
This is part of [133895](#133895)

try-job: armhf-gnu
@JonathanBrouwer
Copy link
Copy Markdown
Contributor

@bors yield

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #157541.

@JonathanBrouwer
Copy link
Copy Markdown
Contributor

Try job got stuck two times in a row now.
Gonna give it one more try to see if it is spurious, but unapproving for now
@bors r-
@bors try cancel
@bors try jobs=dist-i686-msvc

@rust-bors rust-bors Bot 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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 6, 2026
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 6, 2026

Try build cancelled. Cancelled workflows:

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 6, 2026
Move cross crate tests into the appropriate folder


try-job: dist-i686-msvc
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 7, 2026

💔 Test for 30ae555 failed: CI

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