The unit tests in
InitFormTests.pm
can be made to pass by changing just one bit:
- No, that was a wrong path. -- haj, 02 Dec 2006
I would simply check that in, but it points to a case sensitivity in template names where there wasn't one before: The template in question is stored in a topic
MyeditTemplate.txt
, and I seem to recall that such a topic could be found by refering to
'myedit'
, as in the SVN version of the tests.
Could someone (
TW?) please check whether this is an intentional change?
If so, the test case can be fixed as written above.
After running some more tests it looks pretty clear that this is
not intended, and the bug is in the code.
In r12038, (one of the commits of
Item2907), two calls of
ucfirst
have been moved in
lib/TWiki/Templates.pm
. Unfortunately they have been moved into a conditional which only is executed if a
template
parameter has the syntax
template=Web.Template
, but not if it is just
template=myedit
(as in
InitFormTests.pm
and
TemplateTests
).
Since the bug is in Beta2 and
Item2907 is marked "Waiting for Release" I'll no longer commit to that item, but use the current 3226 instead.
Fixed, and closed. It would be nice if
TW could confirm my observations, or undo my patch if appropriate.