Fix leftover "size" check when deciding whether to use digest or mtime. Since we no longer stored mtime for empty files, this bug meant that we always compared empty files equal (which is good). But we shouldn't be using Metadata based on mtime for them.

A follow-up change will do a refactor to make this impossible.

--
MOS_MIGRATED_REVID=128742054

1 files changed, 0 insertions, 1 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
index 84e455cc34..1ba51c8bde 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
@@ -272,7 +272,6 @@ public class ArtifactFunctionTest extends ArtifactFunctionTestCase {
     path.setLastModifiedTime(1L);
     FileArtifactValue value = create(artifact);
     assertArrayEquals(path.getMD5Digest(), value.getDigest());
-    assertEquals(-1L, value.getModifiedTime());
     assertEquals(0L, value.getSize());
   }