]> source.dussan.org Git - jgit.git/commit
Revert "Add tests for FileUtils.delete and EMPTY_DIREECTORIES_ONLY" 76/12076/1
authorShawn Pearce <spearce@spearce.org>
Fri, 19 Apr 2013 19:01:58 +0000 (12:01 -0700)
committerShawn Pearce <spearce@spearce.org>
Fri, 19 Apr 2013 19:04:47 +0000 (12:04 -0700)
commit1b4320f1fa46169ad08a1d3d40ca00a4224881cd
treedfb524bf4183ed53ad730826a99bfa2de05350a7
parentee222a3be1e8b902667a739625047eef29154924
Revert "Add tests for FileUtils.delete and EMPTY_DIREECTORIES_ONLY"

This reverts commit 7aa54967a26cb027fe390ad1c624ebb30f9ac6d5.
The unit test dependend upon the specific order of names that
listFiles() returned members in. The order is completely undefined
and may differ even on different versions of Linux based systems.

A proper unit test for this code would have considered both cases,
where the deletion function was able to remove an empty subdirectory,
or fail to remove a subdirectory because a file was still present
within. This is not such a test.

Change-Id: Ib0a706fea01e4b1ed8c8e859247d247a1279b4bc
org.eclipse.jgit.test/tst/org/eclipse/jgit/util/FileUtilTest.java