Re: [JGIT PATCH] Fix AbstractTreeIterator path comparion betwen 'a' and 'a/b'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Tor Arne Vestbø <torarnv@xxxxxxxxx> 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ø <torarnv@xxxxxxxxx>
> ---
>  .../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.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux