Skip to content
This repository was archived by the owner on Apr 5, 2024. It is now read-only.

Commit a4a1781

Browse files
committed
WIP: Debug why rmpv testcase fails in CI
1 parent 1246e15 commit a4a1781

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

ci/run.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ if [ "${OS}" = "windows" ]; then
1212
fi
1313

1414
cargo build
15-
cargo test --verbose -- --nocapture
1615

1716
case "${OS}" in
1817
*"linux"*)
@@ -26,6 +25,9 @@ case "${OS}" in
2625
;;
2726
esac
2827

28+
# does it work afterwards?
29+
cargo test --verbose -- --nocapture
30+
2931
# FIXME: semververver step fails with:
3032
# error: breaking changes in `old::changes::_::<impl serde::ser::Serialize for old::changes::ChangeCategory>`
3133
# --> /home/runner/.cargo/registry/src/i.8713187.xyz-1ecc6299db9ec823/semverver-0.1.48/src/changes.rs:35:62

tests/full.rs

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,6 @@ mod full {
1010
};
1111

1212
fn test_full(crate_name: &str, old_version: &str, new_version: &str, expected_result: bool) {
13-
// FIXME: CI started to fail since 2022-04-25, ignore the rmpv test on CI for now.
14-
if crate_name == "rmpv" && env::var_os("CI").unwrap_or_default() == "true" {
15-
return;
16-
}
17-
1813
// Add target dir to PATH so cargo-semver will call the right rust-semverver
1914
if let Some(path) = env::var_os("PATH") {
2015
let mut paths = env::split_paths(&path).collect::<Vec<_>>();
@@ -34,16 +29,19 @@ mod full {
3429
&format!("{}:{}", crate_name, new_version),
3530
"-q",
3631
])
37-
.env("RUST_BACKTRACE", "full")
38-
.stdin(Stdio::null())
39-
.stdout(Stdio::piped())
40-
.stderr(Stdio::piped());
32+
.env("RUST_BACKTRACE", "full");
33+
// .stdin(Stdio::null())
34+
// .stdout(Stdio::piped())
35+
// .stderr(Stdio::piped());
4136

4237
if let Ok(target) = std::env::var("TEST_TARGET") {
4338
cmd.args(&["--target", &target]);
4439
}
4540

41+
eprintln!("{:?}", cmd);
42+
4643
let output = cmd.output().expect("could not run cargo semver");
44+
eprintln!("cargo semver exit status: {}", output.status);
4745

4846
// Choose solution depending on the platform
4947
let file_ext = if cfg!(target_os = "macos") {

0 commit comments

Comments
 (0)