Make it possible to use cargo test, not just x.py test

[jyn514]

This sounds harder than it actually is, I think. We already have default-members = ["bootstrap"] in the root Cargo.toml, so we can implement this by adding an integration test in src/bootstrap/tests which calls builder.ensure(test::X) for all possible values of X; ideally we would reuse the logic from x.py test exactly so that cargo test runs all tests, cargo test src/test/ui runs UI tests, etc.

cc #95253 (comment), #94829
@rustbot label +A-rustbuild +C-enhancement

[jyn514]

@rustbot label +E-mentor +E-medium +E-help-wanted

[bjorn3]

If this isn't possible you could put an alias from cargo x to cargo run -- in .cargo/config.toml.

[jyn514]

@bjorn3 we should do that either way I think :) doesn't hurt and there's other commands we definitely can't support, like cargo doc.

[jihiggins]

@rustbot claim

[jyn514]

Given the new entrypoints we're planning in https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/bootstrap.20user.20survey/near/283897117, I am not sure this still makes sense. cargo test will be the only command that goes through cargo, and it will make it seem like build is happening through cargo when it's still really using bootstrap under the hood.

@estebank I saw you left a ❤️ reaction - do you want to give feedback before I close the issue?

[estebank]

@jyn514 I'm ok with the current direction. Familiarity for new contributors with the normal tooling would be useful, but maybe what is needed is to make cargo flexible enough for rustc, and not the other way around.

[jyn514]

maybe what is needed is to make cargo flexible enough for rustc, and not the other way around.

I don't have a lot of confidence this is possible :( there's a thread in https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/why.20not.20.60cargo.20build.60.20in.20rust-lang.2Frust.3F where I explain some fundamental limitations cargo has.

Anyway, I think having a mismatch between test and the rest of the subcommands isn't great so I'm going to close this.