Fix equality error in modified filepaths logic (#229)

This check was on the wrong type of object

Author: tinder-maxwellelliott

cli/src/main/kotlin/com/bazel_diff/hash/SourceFileHasher.kt

@@ -74,7 +74,9 @@ class SourceFileHasher : KoinComponent {
                     if (file.isFile) {
                         if (modifiedFilepaths.isEmpty()) {
                             putFile(file)
-                        } else if (modifiedFilepaths.stream().anyMatch { workingDirectory.resolve(it) == file }) {
+                        } else if (modifiedFilepaths.any {
+                            workingDirectory.resolve(it) == absoluteFilePath
+                        }) {
                             putFile(file)
                         }
                     }

cli/src/test/kotlin/com/bazel_diff/hash/SourceFileHasherTest.kt

@@ -18,13 +18,13 @@ import java.nio.file.Paths
 internal class SourceFileHasherTest : KoinTest {
     private val repoAbsolutePath = Paths.get("").toAbsolutePath()
     private val outputBasePath = Files.createTempDirectory("SourceFileHasherTest")
+    private val path = Paths.get("cli/src/test/kotlin/com/bazel_diff/hash/fixture/foo.ts")
     private val fixtureFileTarget = "//cli/src/test/kotlin/com/bazel_diff/hash/fixture:foo.ts"
     private val fixtureFileContent: ByteArray
     private val seed = "seed".toByteArray()
     private val externalRepoResolver = ExternalRepoResolver(repoAbsolutePath, Paths.get("bazel"), outputBasePath)
 
     init {
-        val path = Paths.get("cli/src/test/kotlin/com/bazel_diff/hash/fixture/foo.ts")
         fixtureFileContent = Files.readAllBytes(path)
     }
 
@@ -47,6 +47,37 @@ internal class SourceFileHasherTest : KoinTest {
         assertThat(actual).isEqualTo(expected)
     }
 
+    @Test
+    fun testHashConcreteFileWithModifiedFilepathsEnabled() = runBlocking {
+        val hasher = SourceFileHasher(repoAbsolutePath, null, externalRepoResolver)
+        val bazelSourceFileTarget = BazelSourceFileTarget(fixtureFileTarget, seed)
+        val actual = hasher.digest(
+            bazelSourceFileTarget,
+            setOf(path)
+        ).toHexString()
+        val expected = sha256 {
+            safePutBytes(fixtureFileContent)
+            safePutBytes(seed)
+            safePutBytes(fixtureFileTarget.toByteArray())
+        }.toHexString()
+        assertThat(actual).isEqualTo(expected)
+    }
+
+    @Test
+    fun testHashConcreteFileWithModifiedFilepathsEnabledNoMatch() = runBlocking {
+        val hasher = SourceFileHasher(repoAbsolutePath, null, externalRepoResolver)
+        val bazelSourceFileTarget = BazelSourceFileTarget(fixtureFileTarget, seed)
+        val actual = hasher.digest(
+            bazelSourceFileTarget,
+            setOf(Paths.get("some/other/path"))
+        ).toHexString()
+        val expected = sha256 {
+            safePutBytes(seed)
+            safePutBytes(fixtureFileTarget.toByteArray())
+        }.toHexString()
+        assertThat(actual).isEqualTo(expected)
+    }
+
     @Test
     fun testHashConcreteFileInExternalRepo() = runBlocking {
         val hasher = SourceFileHasher(repoAbsolutePath, null, externalRepoResolver, setOf("external_repo"))