Bug 43640 - Move the tests package to JUnit
Summary: Move the tests package to JUnit
Alias: None
Product: Taglibs
Classification: Unclassified
Component: Standard Taglib (show other bugs)
Version: 1.1
Hardware: Other other
: P2 normal (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
Depends on:
Reported: 2007-10-16 23:04 UTC by Henri Yandell
Modified: 2010-10-10 17:45 UTC (History)
0 users


Note You need to log in before you can comment on or make changes to this bug.
Description Henri Yandell 2007-10-16 23:04:30 UTC
There is a test package within the source tree that should be rewritten as JUnit
tests and moved into the tests tree.
Comment 1 Henri Yandell 2007-12-30 22:21:46 UTC
svn ci -m "Moving the JSTL main() 'tests' from the src tree to the test tree as
per #43640" src test

Deleting       src/org/apache/taglibs/standard/lang/jstl/test
Adding         test/org/apache/taglibs/standard/lang/jstl/test

Committed revision 607685.
Comment 2 Henri Yandell 2007-12-30 22:55:17 UTC
One down:

svn ci -m "Transferred one of the old tests over to JUnit - #43640"

Transmitting file data .
Committed revision 607689.
Comment 3 Henri Yandell 2008-11-03 14:24:31 UTC
svn ci -m "Transferred the ParserTest and EvaluationTest over from the old code to JUnit. #43640. EvaluationTest fails currently - needs investigation but not an issue with the JUnit-izing. Currently turned off in the build. "

Sending        build-tests.xml
Sending        test/org/apache/taglibs/standard/lang/jstl/test/EvaluationTest.java
Sending        test/org/apache/taglibs/standard/lang/jstl/test/ParserTest.java
Sending        test/org/apache/taglibs/standard/lang/jstl/test/parserTests.txt
Transmitting file data ....
Committed revision 710191.
Comment 4 Henri Yandell 2008-11-07 10:12:23 UTC
Need to get the EvaluationTest passing before closing this.
Comment 5 Henri Yandell 2009-05-25 02:32:42 UTC
ParserTest failing since the move to JSTL 1.2.
Comment 6 Jeremy Boynes 2010-10-10 17:42:26 UTC
Fixed ParserTest by setting javax.servlet.jsp.functions.allowed property true (as per previous command line invocation)
Comment 7 Jeremy Boynes 2010-10-10 17:44:07 UTC
EvaluationTest failed on JDK1.5 because Boolean was changed to implement Comparable which changes the behaviour per section 1.8.1 of the EL specification
Comment 8 Jeremy Boynes 2010-10-10 17:45:02 UTC
Resolving as all tests now pass and none need to be excluded.