Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit e7aeb06

Browse files
authoredDec 31, 2020
Rollup merge of #80565 - camelid:fix-not-has, r=GuillaumeGomez
Fix tests that incorrectly used `!@has` instead of `@!has` The command is ``@!has`,` not `!`@has`.` I don't think these checks were doing anything before! Ideally we would accept `!`@has`` as well, or at least fail tests that use `!`@has`.` The current behavior seems to be silently ignoring the check, which is very confusing. r? `@GuillaumeGomez`
2 parents 5801a58 + 2c405ae commit e7aeb06

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed
 

‎src/test/rustdoc/inline_local/trait-vis.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,5 @@ mod asdf {
1414

1515
// @has trait_vis/struct.SomeStruct.html
1616
// @has - '//code' 'impl ThisTrait for SomeStruct'
17-
// !@has - '//code' 'impl PrivateTrait for SomeStruct'
17+
// @!has - '//code' 'impl PrivateTrait for SomeStruct'
1818
pub use asdf::SomeStruct;

‎src/test/rustdoc/issue-74083.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ impl Foo {
77
}
88

99
// @has issue_74083/struct.Bar.html
10-
// !@has - '//div[@class="sidebar-links"]/a[@href="#method.foo"]' 'foo'
10+
// @!has - '//div[@class="sidebar-links"]/a[@href="#method.foo"]' 'foo'
1111
pub struct Bar {
1212
foo: Foo,
1313
}

‎src/test/rustdoc/remove-url-from-headings.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#![crate_name = "foo"]
22

33
// @has foo/fn.foo.html
4-
// !@has - '//a[@href="http://a.a"]'
4+
// @!has - '//a[@href="http://a.a"]'
55
// @has - '//a[@href="#implementing-stuff-somewhere"]' 'Implementing stuff somewhere'
66
// @has - '//a[@href="#another-one-urg"]' 'Another one urg'
77

0 commit comments

Comments
 (0)
Please sign in to comment.