-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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.
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
test: migrate install* and inheritable_workspace_fields to snapbox #14170
base: master
Are you sure you want to change the base?
Conversation
r? @weihanglo rustbot has assigned @weihanglo. Use |
6810f0d
to
c27d3f9
Compare
@@ -1429,6 +1385,7 @@ fn use_path_workspace() { | |||
assert_eq!(lock, lock2, "different lockfiles"); | |||
} | |||
|
|||
#[allow(deprecated)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with_stderr_does_not_contain
is being used in this function.
// Particularly avoid "Installed package `ws-root v0.1.0 ([..]])` (executable `ws-member`)": | ||
.with_stderr_does_not_contain("ws-member") | ||
.run(); | ||
} | ||
|
||
#[allow(deprecated)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with_stderr_does_not_contain
is being used in this function.
.run(); | ||
} | ||
|
||
#[allow(deprecated)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with_stderr_does_not_contain
is being used in this function.
.with_stderr_data(str![[r#" | ||
[UPDATING] git repository `[ROOTURL]/foo` | ||
[ERROR] invalid type: integer `3`, expected a boolean | ||
--> home/.cargo/git/checkouts/foo-[HASH]/[..]/Cargo.toml:6:27 | ||
| | ||
6 | incremental = 3 | ||
| ^ | ||
| | ||
[ERROR] invalid type: integer `3`, expected a boolean | ||
--> home/.cargo/git/checkouts/foo-[HASH]/[..]/Cargo.toml:6:27 | ||
| | ||
6 | incremental = 3 | ||
| ^ | ||
| | ||
|
||
"#]]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if the duplicated error message shown here is correct. Therefore, I'm currently avoiding redacting with glob intentionally.
What does this PR try to resolve?
Part of #14039.
Migrate following to snapbox:
tests/testsuite/inheritable_workspace_fields.rs
tests/testsuite/install.rs
tests/testsuite/install_upgrade.rs