blob: fcb2f0718858ea0067d3ab4480f4e585b0617911 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
|
[[!template id=gitbranch branch=smcv/trail author=smcv]] [[!tag patch]]
`t/trail.t` has some test suite failures. This is after applying
[[smcv]]'s patch that fixed some races that caused it to fail
sometimes. These remaining failures may also be intermittant,
although I can get them reliably on my laptop. I've added some debugging
output, which seems to point to an actual bug in the plugin AFAICS. --[[Joey]]
> I can reproduce this reliably at 0a23666ddd but not 3.20120203. Bisecting
> indicates that it regressed in aaa72a3a80f, "inline: When the pagenames list
> includes pages that do not exist, skip them".
>
> I don't think this is the bug noted in the commit message - the inline
> containing `sorting/new` uses `pages`, not `pagenames`. --[[smcv]]
>> It seems you removed `trail` support from `inline` in that commit.
>> Assuming that wasn't intentional, this is fixed in `smcv/trail`.
>> --[[smcv]]
<pre>
ok 71 - expected n=sorting/end p=sorting/beginning in sorting/middle.html
not ok 72 - expected n=sorting/new p=sorting/middle in sorting/end.html
# Failed test 'expected n=sorting/new p=sorting/middle in sorting/end.html'
# at t/trail.t line 13.
# got: 'n=sorting/linked2 p=sorting/middle'
# expected: 'n=sorting/new p=sorting/middle'
not ok 73 - expected n=sorting/old p=sorting/end in sorting/new.html
# Failed test 'expected n=sorting/old p=sorting/end in sorting/new.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/old p=sorting/end'
not ok 74 - expected n=sorting/ancient p=sorting/new in sorting/old.html
# Failed test 'expected n=sorting/ancient p=sorting/new in sorting/old.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/ancient p=sorting/new'
not ok 75 - expected n=sorting/linked2 p=sorting/old in sorting/ancient.html
# Failed test 'expected n=sorting/linked2 p=sorting/old in sorting/ancient.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/linked2 p=sorting/old'
not ok 76 - expected n= p=sorting/ancient in sorting/linked2.html
# Failed test 'expected n= p=sorting/ancient in sorting/linked2.html'
# at t/trail.t line 13.
# got: 'n= p=sorting/end'
# expected: 'n= p=sorting/ancient'
ok 77
</pre>
Here, the "new" page does not seem to be included into the trail as expected.
Looking at the rendered page, there is no trail directive output on it either.
--[[Joey]]
<pre>
ok 90
not ok 91 - expected n=sorting/new p= in sorting/old.html
# Failed test 'expected n=sorting/new p= in sorting/old.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/new p='
not ok 92 - expected n=sorting/middle p=sorting/old in sorting/new.html
# Failed test 'expected n=sorting/middle p=sorting/old in sorting/new.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/middle p=sorting/old'
not ok 93 - expected n=sorting/linked2 p=sorting/new in sorting/middle.html
# Failed test 'expected n=sorting/linked2 p=sorting/new in sorting/middle.html'
# at t/trail.t line 13.
# got: 'n=sorting/linked2 p='
# expected: 'n=sorting/linked2 p=sorting/new'
ok 94 - expected n=sorting/linked p=sorting/middle in sorting/linked2.html
ok 95 - expected n=sorting/end p=sorting/linked2 in sorting/linked.html
ok 96 - expected n=sorting/a/c p=sorting/linked in sorting/end.html
ok 97 - expected n=sorting/beginning p=sorting/end in sorting/a/c.html
ok 98 - expected n=sorting/a/b p=sorting/a/c in sorting/beginning.html
not ok 99 - expected n=sorting/ancient p=sorting/beginning in sorting/a/b.html
# Failed test 'expected n=sorting/ancient p=sorting/beginning in sorting/a/b.html'
# at t/trail.t line 13.
# got: 'n=sorting/z/a p=sorting/beginning'
# expected: 'n=sorting/ancient p=sorting/beginning'
not ok 100 - expected n=sorting/z/a p=sorting/a/b in sorting/ancient.html
# Failed test 'expected n=sorting/z/a p=sorting/a/b in sorting/ancient.html'
# at t/trail.t line 13.
# got: undef
# expected: 'n=sorting/z/a p=sorting/a/b'
not ok 101 - expected n= p=sorting/ancient in sorting/z/a.html
# Failed test 'expected n= p=sorting/ancient in sorting/z/a.html'
# at t/trail.t line 13.
# got: 'n= p=sorting/a/b'
# expected: 'n= p=sorting/ancient'
ok 102
</pre>
Haven't investigated, but looks like the same sort of problem, a
page expected to be in the trail isn't. --[[Joey]]
|