test(osv): Rename patchFields()

The function actually does two things, patching ignorable fields and normalize this. So, rename the function to just `patch()` so that it better fits also the normalization and uses a shorter name. Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · Jan 17, 2025 · dcd71a5 · dcd71a5
1 parent 78edc76
commit dcd71a5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/clients/osv/src/funTest/kotlin/OsvServiceWrapperFunTest.kt b/clients/osv/src/funTest/kotlin/OsvServiceWrapperFunTest.kt
@@ -60,7 +60,7 @@ private fun Vulnerability.normalizeUrls(): Vulnerability {
 
 private val emptyJsonObject = JsonObject(emptyMap())
 
-private fun List<Vulnerability>.patchFields() = map { it.patchIgnorableFields().normalizeUrls() }
+private fun List<Vulnerability>.patch() = map { it.patchIgnorableFields().normalizeUrls() }
 
 class OsvServiceWrapperFunTest : StringSpec({
     "getVulnerabilitiesForPackage() returns the expected vulnerability when queried by commit" {
@@ -70,7 +70,7 @@ class OsvServiceWrapperFunTest : StringSpec({
 
         result.shouldBeSuccess { actualData ->
             val expectedData = OsvService.JSON.decodeFromString<List<Vulnerability>>(expectedResult)
-            actualData.patchFields() shouldContainExactlyInAnyOrder expectedData.patchFields()
+            actualData.patch() shouldContainExactlyInAnyOrder expectedData.patch()
         }
     }
 
@@ -81,7 +81,7 @@ class OsvServiceWrapperFunTest : StringSpec({
 
         result.shouldBeSuccess { actualData ->
             val expectedData = OsvService.JSON.decodeFromString<List<Vulnerability>>(expectedResult)
-            actualData.patchFields() shouldContainExactlyInAnyOrder expectedData.patchFields()
+            actualData.patch() shouldContainExactlyInAnyOrder expectedData.patch()
         }
     }