From e38fa24d2e1968e233cb0a18d02c7d29b3730898 Mon Sep 17 00:00:00 2001 From: Georgii Gorbachev <georgii.gorbachev@elastic.co> Date: Mon, 23 Dec 2024 12:36:28 +0100 Subject: [PATCH] [Security Solution] Fix flaky test for multiLineStringDiffAlgorithm (#205038) **Fixes: https://github.com/elastic/kibana/issues/205014** ## Summary This test on CI runs at least 10x slower than locally, and apparently even 1000ms timeout is not enough. Bumping it to 2000ms and hopefully that will be it. Not sure if it makes sense to bump it even higher, because we need this threshold to be reasonably low, and more than 2 seconds doesn't sound low for local test runs. ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [ ] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed --- .../algorithms/multi_line_string_diff_algorithm.test.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/x-pack/solutions/security/plugins/security_solution/server/lib/detection_engine/prebuilt_rules/logic/diff/calculation/algorithms/multi_line_string_diff_algorithm.test.ts b/x-pack/solutions/security/plugins/security_solution/server/lib/detection_engine/prebuilt_rules/logic/diff/calculation/algorithms/multi_line_string_diff_algorithm.test.ts index 8f0b3586066fa..4ecb828a77d4c 100644 --- a/x-pack/solutions/security/plugins/security_solution/server/lib/detection_engine/prebuilt_rules/logic/diff/calculation/algorithms/multi_line_string_diff_algorithm.test.ts +++ b/x-pack/solutions/security/plugins/security_solution/server/lib/detection_engine/prebuilt_rules/logic/diff/calculation/algorithms/multi_line_string_diff_algorithm.test.ts @@ -174,11 +174,11 @@ describe('multiLineStringDiffAlgorithm', () => { const result = multiLineStringDiffAlgorithm(mockVersions); const endTime = performance.now(); - // If the regex merge in this function takes over 1 sec, this test fails + // If the regex merge in this function takes over 2 sec, this test fails // Performance measurements: https://github.com/elastic/kibana/pull/199388 // NOTE: despite the fact that this test runs in ~50ms locally, on CI it - // runs slower and can be flaky even with a 500ms threshold. - expect(endTime - startTime).toBeLessThan(1000); + // runs ~10x slower and can be flaky even with a 1000ms threshold. + expect(endTime - startTime).toBeLessThan(2000); expect(result).toEqual( expect.objectContaining({