Thread (7 messages) 7 messages, 2 authors, 2016-06-15

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help