Comments assertion in Flaky transform runner test(#581)

stevanbz · web-flow · commit 1856295792e5 · 2022-10-28T14:28:06.000-07:00
The transform runner test completes the search in sub-millisecond, thus failing. This is specific to Ubuntu 17 &amp; 11 distribution.  

Signed-off-by: Stevan Buzejic &lt;buzejic.stevan@gmail.com&gt;

Signed-off-by: Stevan Buzejic &lt;buzejic.stevan@gmail.com&gt;
diff --git a/src/test/kotlin/org/opensearch/indexmanagement/transform/TransformRunnerIT.kt b/src/test/kotlin/org/opensearch/indexmanagement/transform/TransformRunnerIT.kt
@@ -108,7 +108,8 @@ class TransformRunnerIT : TransformRestTestCase() {
         assertEquals("More than expected documents indexed", 1L, metadata.stats.documentsIndexed)
         assertEquals("More than expected documents processed", 4977L, metadata.stats.documentsProcessed)
         assertTrue("Doesn't capture indexed time", metadata.stats.indexTimeInMillis > 0)
-        assertTrue("Didn't capture search time", metadata.stats.searchTimeInMillis > 0)
+        // In some cases it seems that the search time is less than 1ms - which causes fails on ubuntu instances (at least that was detected)
+        // assertTrue("Didn't capture search time", metadata.stats.searchTimeInMillis > 0)
     }
 
     fun `test invalid transform`() {

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,8 @@ class TransformRunnerIT : TransformRestTestCase() {`
`108`	`108`	`assertEquals("More than expected documents indexed", 1L, metadata.stats.documentsIndexed)`
`109`	`109`	`assertEquals("More than expected documents processed", 4977L, metadata.stats.documentsProcessed)`
`110`	`110`	`assertTrue("Doesn't capture indexed time", metadata.stats.indexTimeInMillis > 0)`
`111`		`- assertTrue("Didn't capture search time", metadata.stats.searchTimeInMillis > 0)`
	`111`	`+ // In some cases it seems that the search time is less than 1ms - which causes fails on ubuntu instances (at least that was detected)`
	`112`	`+ // assertTrue("Didn't capture search time", metadata.stats.searchTimeInMillis > 0)`
`112`	`113`	`}`
`113`	`114`
`114`	`115`	fun `test invalid transform`() {