Skip to content

Commit 0d928b6

Browse files
GuillaumeGomezGuillaumeGomez
committed
Update rustdoc tests to adapt to changes to the sidebar
1 parent 5136b54 commit 0d928b6

17 files changed

+59
-59
lines changed

‎src/test/rustdoc-gui/hash-item-expansion.goml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ assert-attribute: ("#blanket-implementations-list > details:nth-child(2)", {"ope
55
// We first check that the impl block is open by default.
66
assert-attribute: ("#implementations + details", {"open": ""})
77
// To ensure that we will click on the currently hidden method.
8-
assert-text: (".sidebar-links > a", "must_use")
9-
click: ".sidebar-links > a"
8+
assert-text: (".sidebar-elems section .block li > a", "must_use")
9+
click: ".sidebar-elems section .block li > a"
1010
// We check that the impl block was opened as expected so that we can see the method.
1111
assert-attribute: ("#implementations + details", {"open": ""})

‎src/test/rustdoc-gui/sidebar-mobile.goml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ assert-property: (".mobile-topbar", {"clientHeight": "45"})
3333
// Check that clicking an element from the sidebar scrolls to the right place
3434
// so the target is not obscured by the topbar.
3535
click: ".sidebar-menu-toggle"
36-
click: ".sidebar-links a"
36+
click: ".sidebar-elems section .block li > a"
3737
assert-position: ("#method\.must_use", {"y": 45})
3838

3939
// Check that the bottom-most item on the sidebar menu can be scrolled fully into view.

‎src/test/rustdoc-gui/sidebar.goml‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,15 @@ assert-css: ("#all-types", {"color": "rgb(53, 109, 164)"})
1313
// We check that we have the crates list and that the "current" on is "test_docs".
1414
assert-text: (".sidebar-elems .crate > ul > li > a.current", "test_docs")
1515
// And we're also supposed to have the list of items in the current module.
16-
assert-text: (".sidebar-elems .items > ul > li:nth-child(1)", "Modules")
17-
assert-text: (".sidebar-elems .items > ul > li:nth-child(2)", "Macros")
18-
assert-text: (".sidebar-elems .items > ul > li:nth-child(3)", "Structs")
19-
assert-text: (".sidebar-elems .items > ul > li:nth-child(4)", "Enums")
20-
assert-text: (".sidebar-elems .items > ul > li:nth-child(5)", "Traits")
21-
assert-text: (".sidebar-elems .items > ul > li:nth-child(6)", "Functions")
22-
assert-text: (".sidebar-elems .items > ul > li:nth-child(7)", "Type Definitions")
23-
assert-text: (".sidebar-elems .items > ul > li:nth-child(8)", "Unions")
24-
assert-text: (".sidebar-elems .items > ul > li:nth-child(9)", "Keywords")
16+
assert-text: (".sidebar-elems section ul > li:nth-child(1)", "Modules")
17+
assert-text: (".sidebar-elems section ul > li:nth-child(2)", "Macros")
18+
assert-text: (".sidebar-elems section ul > li:nth-child(3)", "Structs")
19+
assert-text: (".sidebar-elems section ul > li:nth-child(4)", "Enums")
20+
assert-text: (".sidebar-elems section ul > li:nth-child(5)", "Traits")
21+
assert-text: (".sidebar-elems section ul > li:nth-child(6)", "Functions")
22+
assert-text: (".sidebar-elems section ul > li:nth-child(7)", "Type Definitions")
23+
assert-text: (".sidebar-elems section ul > li:nth-child(8)", "Unions")
24+
assert-text: (".sidebar-elems section ul > li:nth-child(9)", "Keywords")
2525
assert-text: ("#structs + .item-table .item-left > a", "Foo")
2626
click: "#structs + .item-table .item-left > a"
2727

@@ -30,7 +30,7 @@ assert-count: (".sidebar .location", 2)
3030
// We check that there is no crate listed outside of the top level.
3131
assert-false: ".sidebar-elems > .crate"
3232

33-
click: ".sidebar-links a"
33+
click: ".sidebar-elems section .block li > a"
3434
assert-property-false: ("html", {"scrollTop": "0"})
3535

3636
click: ".sidebar h2.location a"
@@ -47,11 +47,11 @@ assert-text: (".sidebar > .location", "Crate lib2")
4747
// We check that we have the crates list and that the "current" on is now "lib2".
4848
assert-text: (".sidebar-elems .crate > ul > li > a.current", "lib2")
4949
// We now go to the "foobar" function page.
50-
assert-text: (".sidebar-elems > .items > ul > li:nth-child(1)", "Modules")
51-
assert-text: (".sidebar-elems > .items > ul > li:nth-child(2)", "Structs")
52-
assert-text: (".sidebar-elems > .items > ul > li:nth-child(3)", "Traits")
53-
assert-text: (".sidebar-elems > .items > ul > li:nth-child(4)", "Functions")
54-
assert-text: (".sidebar-elems > .items > ul > li:nth-child(5)", "Type Definitions")
50+
assert-text: (".sidebar-elems > section .block ul > li:nth-child(1)", "Modules")
51+
assert-text: (".sidebar-elems > section .block ul > li:nth-child(2)", "Structs")
52+
assert-text: (".sidebar-elems > section .block ul > li:nth-child(3)", "Traits")
53+
assert-text: (".sidebar-elems > section .block ul > li:nth-child(4)", "Functions")
54+
assert-text: (".sidebar-elems > section .block ul > li:nth-child(5)", "Type Definitions")
5555
assert-text: ("#functions + .item-table .item-left > a", "foobar")
5656
click: "#functions + .item-table .item-left > a"
5757

@@ -72,12 +72,12 @@ goto: ./sub_module/sub_sub_module/index.html
7272
assert-text: (".sidebar > .location", "Module sub_sub_module")
7373
// We check that we don't have the crate list.
7474
assert-false: ".sidebar-elems .crate"
75-
assert-text: (".sidebar-elems .items > ul > li:nth-child(1)", "Functions")
75+
assert-text: (".sidebar-elems > section ul > li:nth-child(1)", "Functions")
7676
assert-text: ("#functions + .item-table .item-left > a", "foo")
7777

7878
// Links to trait implementations in the sidebar should not wrap even if they are long.
7979
goto: file://|DOC_PATH|/lib2/struct.HasALongTraitWithParams.html
80-
assert-property: (".sidebar-links a", {"offsetHeight": 29})
80+
assert-property: (".sidebar-elems section .block li > a", {"offsetHeight": 29})
8181

8282
// Test that clicking on of the "In <module>" headings in the sidebar links to the
8383
// appropriate anchor in index.html.

‎src/test/rustdoc-gui/trait-sidebar-item-order.goml‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
// Checks that the elements in the sidebar are alphabetically sorted.
22
goto: file://|DOC_PATH|/test_docs/trait.AnotherOne.html
3-
assert-text: (".sidebar-links a:nth-of-type(1)", "another")
4-
assert-text: (".sidebar-links a:nth-of-type(2)", "func1")
5-
assert-text: (".sidebar-links a:nth-of-type(3)", "func2")
6-
assert-text: (".sidebar-links a:nth-of-type(4)", "func3")
7-
assert-text: (".sidebar-links a:nth-of-type(5)", "hello")
8-
assert-text: (".sidebar-links a:nth-of-type(6)", "why_not")
3+
assert-text: (".sidebar-elems section .block li:nth-of-type(1) > a", "another")
4+
assert-text: (".sidebar-elems section .block li:nth-of-type(2) > a", "func1")
5+
assert-text: (".sidebar-elems section .block li:nth-of-type(3) > a", "func2")
6+
assert-text: (".sidebar-elems section .block li:nth-of-type(4) > a", "func3")
7+
assert-text: (".sidebar-elems section .block li:nth-of-type(5) > a", "hello")
8+
assert-text: (".sidebar-elems section .block li:nth-of-type(6) > a", "why_not")

‎src/test/rustdoc/associated-consts.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ pub struct Bar;
1010

1111
// @has 'foo/struct.Bar.html'
1212
// @has - '//h3[@class="sidebar-title"]' 'Associated Constants'
13-
// @has - '//div[@class="sidebar-elems"]//div[@class="sidebar-links"]/a' 'FOO'
13+
// @has - '//div[@class="sidebar-elems"]//a' 'FOO'
1414
impl Trait for Bar {
1515
const FOO: u32 = 1;
1616

@@ -23,7 +23,7 @@ pub enum Foo {
2323

2424
// @has 'foo/enum.Foo.html'
2525
// @has - '//h3[@class="sidebar-title"]' 'Associated Constants'
26-
// @has - '//div[@class="sidebar-elems"]//div[@class="sidebar-links"]/a' 'FOO'
26+
// @has - '//div[@class="sidebar-elems"]//a' 'FOO'
2727
impl Trait for Foo {
2828
const FOO: u32 = 1;
2929

‎src/test/rustdoc/deref-mut-methods.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ impl Foo {
99
}
1010

1111
// @has foo/struct.Bar.html
12-
// @has - '//div[@class="sidebar-links"]/a[@href="#method.foo"]' 'foo'
12+
// @has - '//*[@class="sidebar-elems"]//*[@class="block"]//a[@href="#method.foo"]' 'foo'
1313
pub struct Bar {
1414
foo: Foo,
1515
}

‎src/test/rustdoc/deref-recursive-pathbuf.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
// @has '-' '//*[@id="deref-methods-Path"]' 'Methods from Deref<Target = Path>'
99
// @has '-' '//*[@class="impl-items"]//*[@id="method.exists"]' 'pub fn exists(&self)'
1010
// @has '-' '//*[@class="sidebar-title"]/a[@href="#deref-methods-PathBuf"]' 'Methods from Deref<Target=PathBuf>'
11-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.as_path"]' 'as_path'
11+
// @has '-' '//*[@class="sidebar-elems"]//*[@class="block"]//a[@href="#method.as_path"]' 'as_path'
1212
// @has '-' '//*[@class="sidebar-title"]/a[@href="#deref-methods-Path"]' 'Methods from Deref<Target=Path>'
13-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.exists"]' 'exists'
13+
// @has '-' '//*[@class="sidebar-elems"]//*[@class="block"]//a[@href="#method.exists"]' 'exists'
1414

1515
#![crate_name = "foo"]
1616

‎src/test/rustdoc/deref-recursive.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
// @has '-' '//*[@id="deref-methods-Baz"]' 'Methods from Deref<Target = Baz>'
99
// @has '-' '//*[@class="impl-items"]//*[@id="method.baz"]' 'pub fn baz(&self)'
1010
// @has '-' '//*[@class="sidebar-title"]/a[@href="#deref-methods-Bar"]' 'Methods from Deref<Target=Bar>'
11-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.bar"]' 'bar'
11+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.bar"]' 'bar'
1212
// @has '-' '//*[@class="sidebar-title"]/a[@href="#deref-methods-Baz"]' 'Methods from Deref<Target=Baz>'
13-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.baz"]' 'baz'
13+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.baz"]' 'baz'
1414

1515
#![crate_name = "foo"]
1616

‎src/test/rustdoc/deref-typedef.rs‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77
// @has '-' '//*[@class="impl-items"]//*[@id="method.foo_c"]' 'pub fn foo_c(&self)'
88
// @has '-' '//*[@class="impl-items"]//*[@id="method.foo_j"]' 'pub fn foo_j(&self)'
99
// @has '-' '//*[@class="sidebar-title"]/a[@href="#deref-methods-FooJ"]' 'Methods from Deref<Target=FooJ>'
10-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.foo_a"]' 'foo_a'
11-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.foo_b"]' 'foo_b'
12-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.foo_c"]' 'foo_c'
13-
// @has '-' '//*[@class="sidebar-links"]/a[@href="#method.foo_j"]' 'foo_j'
10+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.foo_a"]' 'foo_a'
11+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.foo_b"]' 'foo_b'
12+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.foo_c"]' 'foo_c'
13+
// @has '-' '//*[@class="sidebar-elems"]//section//a[@href="#method.foo_j"]' 'foo_j'
1414

1515
pub struct FooA;
1616
pub type FooB = FooA;

‎src/test/rustdoc/double-quote-escape.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ pub trait Foo<T> {
88
pub struct Bar;
99

1010
// @has foo/struct.Bar.html
11-
// @has - '//*[@class="sidebar-links"]/a[@href="#impl-Foo%3Cunsafe%20extern%20%22C%22%20fn()%3E"]' 'Foo<unsafe extern "C" fn()>'
11+
// @has - '//*[@class="sidebar-elems"]//section//a[@href="#impl-Foo%3Cunsafe%20extern%20%22C%22%20fn()%3E"]' 'Foo<unsafe extern "C" fn()>'
1212
impl Foo<unsafe extern "C" fn()> for Bar {}

0 commit comments

Comments
 (0)