From b1d6a7e11d9cc0dc623cfc7e45f10b3d6eef0b8b Mon Sep 17 00:00:00 2001 From: Dale Wijnand Date: Thu, 2 Aug 2018 14:57:19 +0100 Subject: [PATCH] Drop niche Execs::with_no_expected_status .. by gating the nightly-only tests. --- tests/testsuite/build.rs | 4 +++- tests/testsuite/package.rs | 6 ++++-- tests/testsuite/support/mod.rs | 5 ----- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/tests/testsuite/build.rs b/tests/testsuite/build.rs index 99b7e6d69..e9f5ce9cc 100644 --- a/tests/testsuite/build.rs +++ b/tests/testsuite/build.rs @@ -4361,6 +4361,9 @@ fn inferred_benchmarks() { #[test] fn target_edition() { + if !is_nightly() { // --edition is nightly-only + return; + } let p = project() .file( "Cargo.toml", @@ -4380,7 +4383,6 @@ fn target_edition() { assert_that( p.cargo("build").arg("-v").masquerade_as_nightly_cargo(), execs() - .with_no_expected_status() // passes on nightly, fails on stable, b/c --edition is nightly-only .with_stderr_contains("\ [COMPILING] foo v0.0.1 ([..]) [RUNNING] `rustc [..]--edition=2018 [..] diff --git a/tests/testsuite/package.rs b/tests/testsuite/package.rs index 4ae6f8f6f..d4ed25415 100644 --- a/tests/testsuite/package.rs +++ b/tests/testsuite/package.rs @@ -5,7 +5,7 @@ use std::path::{Path, PathBuf}; use git2; use support::{cargo_process, sleep_ms, ChannelChanger}; -use support::{basic_manifest, execs, git, paths, project, registry, path2url}; +use support::{basic_manifest, execs, git, is_nightly, paths, project, registry, path2url}; use support::registry::Package; use flate2::read::GzDecoder; use support::hamcrest::{assert_that, contains, existing_file}; @@ -934,6 +934,9 @@ fn package_two_kinds_of_deps() { #[test] fn test_edition() { + if !is_nightly() { // --edition is nightly-only + return; + } let p = project() .file( "Cargo.toml", @@ -982,7 +985,6 @@ fn test_edition_missing() { assert_that( p.cargo("build").arg("-v").masquerade_as_nightly_cargo(), execs() - .with_no_expected_status() // passes on nightly, fails on stable, b/c --edition is nightly-only // --edition is still in flux and we're not passing -Zunstable-options // from Cargo so it will probably error. Only partially match the output // until stuff stabilizes diff --git a/tests/testsuite/support/mod.rs b/tests/testsuite/support/mod.rs index 8b7198b5b..602e4f8a9 100644 --- a/tests/testsuite/support/mod.rs +++ b/tests/testsuite/support/mod.rs @@ -540,11 +540,6 @@ impl Execs { self } - pub fn with_no_expected_status(mut self) -> Execs { - self.expect_exit_code = None; - self - } - /// Verify that stdout contains the given contiguous lines somewhere in /// its output. /// See `lines_match` for supported patterns. -- 2.39.5