doc/bugs/trail_test_suite_failures.mdwn

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