git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Test framework regressions - spelling and usertrack




Le 22/10/2018 à 16:56, Jim Jagielski a écrit :
The latest update to usertrack works. Thx! speling still bad:

On httpd-2.4 HEAD:

   t/modules/speling.t ................. 1/48 # Failed test 11 in t/modules/speling.t at line 46 fail #6
   # Failed test 12 in t/modules/speling.t at line 50 fail #5
   # Failed test 35 in t/modules/speling.t at line 46 fail #18
   # Failed test 36 in t/modules/speling.t at line 50 fail #9
   t/modules/speling.t ................. Failed 4/48 subtests
   	(less 13 skipped subtests: 31 okay)

On trunk:

   t/modules/speling.t ................. 1/48 # Failed test 11 in t/modules/speling.t at line 46 fail #6
   # Failed test 12 in t/modules/speling.t at line 50 fail #5
   # Failed test 13 in t/modules/speling.t at line 46 fail #7
   # Failed test 14 in t/modules/speling.t at line 50 fail #6
   # Failed test 15 in t/modules/speling.t at line 46 fail #8
   # Failed test 16 in t/modules/speling.t at line 50 fail #7
   # Failed test 35 in t/modules/speling.t at line 46 fail #18
   # Failed test 36 in t/modules/speling.t at line 50 fail #9
   # Failed test 37 in t/modules/speling.t at line 46 fail #19
   # Failed test 38 in t/modules/speling.t at line 50 fail #10
   # Failed test 39 in t/modules/speling.t at line 46 fail #20
   # Failed test 40 in t/modules/speling.t at line 50 fail #11
   t/modules/speling.t ................. Failed 12/48 subtests
	  (less 13 skipped subtests: 23 okay)
I've not tested on trunk :(.
It is expected to give some different result because of r1557580 which has changed the default behavior. I plan to restore the previous behavior (just need to change the default value of 'check_basename_match', I think) and then propose for backport.
Consider this ~5 years few lines patch still a WIP...
(apologies for neither finishing, nor reverting the changes)

This is on macOS. The speling tests show:

# testing : Checking case. Expecting: 301
# expected: 301
# received: '200'
not ok 11
# testing : Redirect ok
# expected: qr/good\.html|several1\.html/
# received: '<html></html
# '
not ok 12
If the system is not case-sensitive, it is likely that this test must be tweaked.

# Failed test 11 in t/modules/speling.t at line 46 fail #6
# Failed test 12 in t/modules/speling.t at line 50 fail #5
# testing : Checking wrong extension. Expecting: 300
# expected: 300
# received: '404'
not ok 13
This is the expected change of behavior.

CJ