Re: [JGIT PATCH] Fix AbstractTreeIterator path comparion betwen 'a' and 'a/b'
From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:46:05
Tor Arne Vestbø [off-list ref] wrote:
The occurance of a '/' as the next character in the longer path does not neccecarily mean the two paths are equal, for example when the longer path has more components following the '/'. Signed-off-by: Tor Arne Vestbø <redacted> --- .../jgit/treewalk/AbstractTreeIteratorTest.java | 93 ++++++++++++++++++++ .../jgit/treewalk/AbstractTreeIterator.java | 4 +- 2 files changed, 95 insertions(+), 2 deletions(-) create mode 100644 org.spearce.jgit.test/tst/org/spearce/jgit/treewalk/AbstractTreeIteratorTest.java
*sigh*
I can't get Eclipse to run this test. Every time I try it comes
up with a CNFE:
Class not found org.spearce.jgit.treewalk.AbstractTreeIteratorTest
java.lang.ClassNotFoundException: org.spearce.jgit.treewalk.AbstractTreeIteratorTest
at java.net.URLClassLoader$1.run(URLClassLoader.java:200)
Aside from this test not running, Eclipse says the tests pass. But
Maven tells another story:
(cd jgit-maven/jgit && mvn clean package)
...
Failed tests:
testNoDF_NoGap(org.spearce.jgit.treewalk.NameConflictTreeWalkTest)
testDF_GapByOne(org.spearce.jgit.treewalk.NameConflictTreeWalkTest)
testDF_SkipsSeenSubtree(org.spearce.jgit.treewalk.NameConflictTreeWalkTest)
Tests run: 773, Failures: 3, Errors: 0, Skipped: 0
Yet under Eclipse this test says its fine. But Maven actually
managed to run your new test case, where Eclipse didn't.
*confused*
--
Shawn.