Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(tests): Prepare for wait-for-publish test changes #11210

Merged
merged 5 commits into from
Oct 11, 2022
Merged

refactor(tests): Prepare for wait-for-publish test changes #11210

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0846b8e
refactor(tests): Document case outside the test, rather than in
epage Oct 7, 2022
af82aa0
refactor(tests): Remove redundant with_status(0)
epage Oct 6, 2022
5e1a647
refactor(tests): Consistent predicate ordering
epage Oct 6, 2022
5e5b532
refactor(tests): Be more explicit about publish stderr
epage Oct 7, 2022
08df53b
refactor(tests): Be explicit about publish stderr
epage Oct 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor(tests): Consistent predicate ordering
  • Loading branch information
@epage
epage committed Oct 10, 2022
commit 5e1a6479246d915e050566a0edf039b075550030
4 changes: 2 additions & 2 deletions tests/testsuite/publish.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -921,8 +921,8 @@ fn publish_with_no_default_features() {

p.cargo("publish --no-default-features")
.replace_crates_io(registry.index_url())
.with_stderr_contains("error: This crate requires `required` feature!")
.with_status(101)
.with_stderr_contains("error: This crate requires `required` feature!")
.run();
}

Expand Down Expand Up @@ -964,8 +964,8 @@ fn publish_with_patch() {
// Check that verify fails with patched crate which has new functionality.
p.cargo("publish")
.replace_crates_io(registry.index_url())
.with_stderr_contains("[..]newfunc[..]")
.with_status(101)
.with_stderr_contains("[..]newfunc[..]")
.run();

// Remove the usage of new functionality and try again.
Expand Down