Skip to content

Commit

Permalink
testsuite/doc: Check the arguments used to invoke the browser
Browse files Browse the repository at this point in the history
Previously we apparently weren't doing this as we weren't checking
stdout.
  • Loading branch information
lf- committed May 16, 2021
1 parent ed46a9a commit 50486be
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/testsuite/doc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1184,6 +1184,7 @@ fn doc_workspace_open_different_library_and_package_names() {
.env("BROWSER", "echo")
.with_stderr_contains("[..] Documenting foo v0.1.0 ([..])")
.with_stderr_contains("[..] [CWD]/target/doc/foolib/index.html")
.with_stdout_contains("[CWD]/target/doc/foolib/index.html")
.run();

p.change_file(
Expand All @@ -1197,7 +1198,7 @@ fn doc_workspace_open_different_library_and_package_names() {
// check that the cargo config overrides the browser env var
p.cargo("doc --open")
.env("BROWSER", "true")
.with_stdout_contains("[..]a [CWD]/target/doc/foolib/index.html")
.with_stdout_contains("a [CWD]/target/doc/foolib/index.html")
.run();
}

Expand Down

0 comments on commit 50486be

Please sign in to comment.