Skip to content

Commit 6a7b15a

Browse files
committed
test: Auto-redact host target and alt target
1 parent d7de937 commit 6a7b15a

File tree

20 files changed

+46
-36
lines changed

20 files changed

+46
-36
lines changed

crates/cargo-test-support/src/compare.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,9 @@
3636
//! a problem.
3737
//! - Carriage returns are removed, which can help when running on Windows.
3838
39-
use crate::diff;
39+
use crate::cross_compile::try_alternate;
4040
use crate::paths;
41+
use crate::{diff, rustc_host};
4142
use anyhow::{bail, Context, Result};
4243
use serde_json::Value;
4344
use std::fmt;
@@ -173,6 +174,10 @@ fn add_common_redactions(subs: &mut snapbox::Redactions) {
173174
regex!("home/\\.cargo/registry/src/-(?<redacted>[a-z0-9]+)"),
174175
)
175176
.unwrap();
177+
subs.insert("[HOST_TARGET]", rustc_host()).unwrap();
178+
if let Some(alt_target) = try_alternate() {
179+
subs.insert("[ALT_TARGET]", alt_target).unwrap();
180+
}
176181
}
177182

178183
static MIN_LITERAL_REDACTIONS: &[(&str, &str)] = &[

crates/cargo-test-support/src/cross_compile.rs

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,18 +209,23 @@ pub fn native_arch() -> &'static str {
209209
///
210210
/// Only use this function on tests that check `cross_compile::disabled`.
211211
pub fn alternate() -> &'static str {
212+
try_alternate().expect("This test should be gated on cross_compile::disabled.")
213+
}
214+
215+
/// A possible alternate target-triple to build with.
216+
pub(crate) fn try_alternate() -> Option<&'static str> {
212217
if cfg!(all(target_os = "macos", target_arch = "aarch64")) {
213-
"x86_64-apple-darwin"
218+
Some("x86_64-apple-darwin")
214219
} else if cfg!(target_os = "macos") {
215-
"x86_64-apple-ios"
220+
Some("x86_64-apple-ios")
216221
} else if cfg!(target_os = "linux") {
217-
"i686-unknown-linux-gnu"
222+
Some("i686-unknown-linux-gnu")
218223
} else if cfg!(all(target_os = "windows", target_env = "msvc")) {
219-
"i686-pc-windows-msvc"
224+
Some("i686-pc-windows-msvc")
220225
} else if cfg!(all(target_os = "windows", target_env = "gnu")) {
221-
"i686-pc-windows-gnu"
226+
Some("i686-pc-windows-gnu")
222227
} else {
223-
panic!("This test should be gated on cross_compile::disabled.");
228+
None
224229
}
225230
}
226231

tests/testsuite/cargo_add/target/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ fn case() {
2828

2929
snapbox::cmd::Command::cargo_ui()
3030
.arg("add")
31-
.arg_line("my-package1 my-package2 --target i686-unknown-linux-gnu")
31+
.arg_line("my-package1 my-package2 --target wasm32-unknown-unknown")
3232
.current_dir(cwd)
3333
.assert()
3434
.success()

tests/testsuite/cargo_add/target/out/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ name = "cargo-list-test-fixture"
55
version = "0.0.0"
66
edition = "2015"
77

8-
[target.i686-unknown-linux-gnu.dependencies]
8+
[target.wasm32-unknown-unknown.dependencies]
99
my-package1 = "99999.0.0"
1010
my-package2 = "99999.0.0"

tests/testsuite/cargo_add/target/stderr.term.svg

Lines changed: 2 additions & 2 deletions
Loading

tests/testsuite/cargo_remove/invalid_target/out/Cargo.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ path = "src/main.rs"
1010
[target.x86_64-unknown-freebsd.build-dependencies]
1111
semver = "0.1.0"
1212

13-
[target.x86_64-unknown-linux-gnu.build-dependencies]
13+
[target.wasm32-unknown-unknown.build-dependencies]
1414
semver = "0.1.0"
1515

1616
[dependencies]
@@ -20,14 +20,14 @@ semver = "0.1"
2020
toml = "0.1"
2121
clippy = "0.4"
2222

23-
[target.x86_64-unknown-linux-gnu.dependencies]
23+
[target.wasm32-unknown-unknown.dependencies]
2424
dbus = "0.6.2"
2525

2626
[dev-dependencies]
2727
regex = "0.1.1"
2828
serde = "1.0.90"
2929

30-
[target.x86_64-unknown-linux-gnu.dev-dependencies]
30+
[target.wasm32-unknown-unknown.dev-dependencies]
3131
ncurses = "20.0"
3232

3333
[features]

tests/testsuite/cargo_remove/invalid_target/stderr.term.svg

Lines changed: 1 addition & 1 deletion
Loading

tests/testsuite/cargo_remove/invalid_target_dep/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ fn case() {
2828

2929
snapbox::cmd::Command::cargo_ui()
3030
.arg("remove")
31-
.args(["--target", "x86_64-unknown-linux-gnu", "toml"])
31+
.args(["--target", "wasm32-unknown-unknown", "toml"])
3232
.current_dir(cwd)
3333
.assert()
3434
.code(101)

tests/testsuite/cargo_remove/invalid_target_dep/out/Cargo.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ path = "src/main.rs"
1010
[target.x86_64-unknown-freebsd.build-dependencies]
1111
semver = "0.1.0"
1212

13-
[target.x86_64-unknown-linux-gnu.build-dependencies]
13+
[target.wasm32-unknown-unknown.build-dependencies]
1414
semver = "0.1.0"
1515

1616
[dependencies]
@@ -20,14 +20,14 @@ semver = "0.1"
2020
toml = "0.1"
2121
clippy = "0.4"
2222

23-
[target.x86_64-unknown-linux-gnu.dependencies]
23+
[target.wasm32-unknown-unknown.dependencies]
2424
dbus = "0.6.2"
2525

2626
[dev-dependencies]
2727
regex = "0.1.1"
2828
serde = "1.0.90"
2929

30-
[target.x86_64-unknown-linux-gnu.dev-dependencies]
30+
[target.wasm32-unknown-unknown.dev-dependencies]
3131
ncurses = "20.0"
3232

3333
[features]

tests/testsuite/cargo_remove/invalid_target_dep/stderr.term.svg

Lines changed: 2 additions & 2 deletions
Loading

0 commit comments

Comments
 (0)