Menu
▾
▴
[Jtidy-devel] [ jtidy-Bugs-3038697 ] Remove lots of the reasons for failing junit-tests
|
From: SourceForge.net <no...@so...> - 2010-08-17 00:36:13
|
Bugs item #3038697, was opened at 2010-08-03 16:48 Message generated for change (Comment added) made by aditsu You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Johannes Weberhofer (weberho) Assigned to: Adrian Sandor (aditsu) Summary: Remove lots of the reasons for failing junit-tests Initial Comment: Most junit tests were failing because there were jtidy-version information in the resulting messages definied. I have fixed those issues plus some general issues, so that there should be 'only' around 90 failing junit tests at the end. ---------------------------------------------------------------------- >Comment By: Adrian Sandor (aditsu) Date: 2010-08-17 08:36 Message: Um.. did you create this patch just by running the tests and assuming the current output is correct? If so, why do you think it is good to make that assumption? ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 18:14 Message: Ok. I'll send you a patch so that maven produces 1.4 code,too (in trunk) and have a look on the new sources. ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:43 Message: I don't use maven so I can't comment on the pom, but afaik, the DOM part of the java api was added in 1.4, so JTidy couldn't run in 1.3 (or maybe only part of it). Also, recent versions of JTidy were compiled for 1.4. Yes, that branch is usable and I think better than trunk at this point, but I'm not sure about organizing the new classes, maybe you can take a look and give some feedback. The new test suite is TidyTests. Thanks for your contributions. ---------------------------------------------------------------------- Comment By: Johannes Weberhofer (weberho) Date: 2010-08-03 17:28 Message: Java compatibility is set to 1.3 in the pom.xml in trunk. I think, it whould be set to 1.4, as it 1.3 is outdated for a long time. Is the current CodeUpdateAndJava5 branch useable? Then I would possibly to switch to that branch and help improving jtidy (as my time allows me so). ---------------------------------------------------------------------- Comment By: Adrian Sandor (aditsu) Date: 2010-08-03 17:07 Message: I kind of gave up on those test cases, and made a new test suite on the CodeUpdateAndJava5 branch, where "only" 73 tests fail now (meaning the result is not 100% byte-by-byte identical with Tidy). But I think it's good to fix these tests too, especially if we plan to maintain the trunk version (compatible with java 1.4) longer. I'll check your patch when I find some time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=113153&aid=3038697&group_id=13153 |
