Fix RacyGitTests#testRacyGitDetection

This test case assumed file system timestamp resolution of 1 second. On
filesystems with a finer resolution this test fails since the index
entry is only smudged if the file index entry's lastModified and the
lastModified of the git index itself are within the same filesystem
timer tick. Fix this by ensuring that these timestamps are identical
which should work for any filesystem timer resolution.

Bug: 548188
Change-Id: Id84d59e1cfeb48fa008f8f27f2f892c4f73985de
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>
(cherry picked from commit 1159f9dd7c)

org.eclipse.jgit.test/tst/org/eclipse/jgit/lib/RacyGitTests.java

@@ -44,6 +44,7 @@ package org.eclipse.jgit.lib;
 
 import static java.lang.Long.valueOf;
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
 import java.io.File;
@@ -52,10 +53,12 @@ import java.io.IOException;
 import java.util.TreeSet;
 
 import org.eclipse.jgit.api.Git;
+import org.eclipse.jgit.dircache.DirCache;
 import org.eclipse.jgit.junit.RepositoryTestCase;
 import org.eclipse.jgit.treewalk.FileTreeIterator;
 import org.eclipse.jgit.treewalk.FileTreeIteratorWithTimeControl;
 import org.eclipse.jgit.treewalk.NameConflictTreeWalk;
+import org.eclipse.jgit.treewalk.WorkingTreeOptions;
 import org.eclipse.jgit.util.FileUtils;
 import org.junit.Test;
 
@@ -136,8 +139,8 @@ public class RacyGitTests extends RepositoryTestCase {
 		fsTick(db.getIndexFile());
 
 		// create two files
-		File a = addToWorkDir("a", "a");
+		File a = writeToWorkDir("a", "a");
-		File b = addToWorkDir("b", "b");
+		File b = writeToWorkDir("b", "b");
 		assertTrue(a.setLastModified(b.lastModified()));
 		assertTrue(b.setLastModified(b.lastModified()));
 
@@ -157,23 +160,35 @@ public class RacyGitTests extends RepositoryTestCase {
 		fsTick(db.getIndexFile());
 
 		// Create a racy git situation. This is a situation that the index is
-		// updated and then a file is modified within a second. By changing the
+		// updated and then a file is modified within the same tick of the
-		// index file artificially, we create a fake racy situation.
+		// filesystem timestamp resolution. By changing the index file
-		File updatedA = addToWorkDir("a", "a2");
+		// artificially, we create a fake racy situation.
-		assertTrue(updatedA.setLastModified(updatedA.lastModified() + 100));
+		File updatedA = writeToWorkDir("a", "a2");
+		long newLastModified = updatedA.lastModified() + 100;
+		assertTrue(updatedA.setLastModified(newLastModified));
 		resetIndex(new FileTreeIterator(db));
-		assertTrue(db.getIndexFile()
+		assertTrue(db.getIndexFile().setLastModified(newLastModified));
-				.setLastModified(updatedA.lastModified() + 90));
 
-		db.readDirCache();
+		DirCache dc = db.readDirCache();
-		// although racily clean a should not be reported as being dirty
+		// check index state: although racily clean a should not be reported as
+		// being dirty since we forcefully reset the index to match the working
+		// tree
 		assertEquals(
 				"[a, mode:100644, time:t1, smudged, length:0, content:a2]"
 						+ "[b, mode:100644, time:t0, length:1, content:b]",
 				indexState(SMUDGE | MOD_TIME | LENGTH | CONTENT));
+
+		// compare state of files in working tree with index to check that
+		// FileTreeIterator.isModified() works as expected
+		FileTreeIterator f = new FileTreeIterator(db.getWorkTree(), db.getFS(),
+				db.getConfig().get(WorkingTreeOptions.KEY));
+		assertTrue(f.findFile("a"));
+		try (ObjectReader reader = db.newObjectReader()) {
+			assertFalse(f.isModified(dc.getEntry("a"), false, reader));
+		}
 	}
 
-	private File addToWorkDir(String path, String content) throws IOException {
+	private File writeToWorkDir(String path, String content) throws IOException {
 		File f = new File(db.getWorkTree(), path);
 		FileOutputStream fos = new FileOutputStream(f);
 		try {