tests: Fix tests if the HOST triple is too long

rust-lang / cargo-bisect-rustc

Bisects rustc, either nightlies or CI artifacts

https://rust-lang.github.io/cargo-bisect-rustc/

Apache License 2.0

174 stars 55 forks source link

tests: Fix tests if the HOST triple is too long #230

Closed LingMan closed 1 year ago

LingMan commented 1 year ago

The HOST triple defaults to the current host and gets printed in the help output. The CLI tests currently match it with [..]. However, if the triple is too long clap might introduce a line break, which [..] can't handle. Use ... instead to match arbitrary lines at the position. Not elegant, but these are just smoke tests to ensure there's no major probelm. They are not meant to ensure 100% stable output.

Fixes: https://github.com/rust-lang/cargo-bisect-rustc/issues/226

LingMan commented 1 year ago

@tjni: Could you please test this? Should fix the broken tests you were seeing, but I don't have an aarch64 system available to actually test.

tjni commented 1 year ago

Confirmed that this fixes the issue I was seeing. Much thanks for the fast turnaround on the fix.