From 0f9766591fc441ebaa9226a718ffb58ec744749d Mon Sep 17 00:00:00 2001 From: Frank Hassanabad Date: Tue, 18 May 2021 12:03:49 -0600 Subject: [PATCH] [Security Solution] Re-adds a test that was skipped (#100273) ## Summary Re-adds a test that was skipped. If it goes bonkers again, I will add more debugging information to it. I will keep an eye on the operations channel to see when/if this fails again. Originally this looked to be timeouts waiting, so I increased the global timeout to be 20 seconds instead of the original 10 seconds. Resolves: https://github.com/elastic/kibana/issues/89389 ### 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 --- .../tests/exception_operators_data_types/ip.ts | 3 +-- x-pack/test/detection_engine_api_integration/utils.ts | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/x-pack/test/detection_engine_api_integration/security_and_spaces/tests/exception_operators_data_types/ip.ts b/x-pack/test/detection_engine_api_integration/security_and_spaces/tests/exception_operators_data_types/ip.ts index 058ae16dac8a..521a5c01a120 100644 --- a/x-pack/test/detection_engine_api_integration/security_and_spaces/tests/exception_operators_data_types/ip.ts +++ b/x-pack/test/detection_engine_api_integration/security_and_spaces/tests/exception_operators_data_types/ip.ts @@ -626,8 +626,7 @@ export default ({ getService }: FtrProviderContext) => { expect(ips).to.eql(['127.0.0.1', '127.0.0.3']); }); - // flaky https://github.com/elastic/kibana/issues/89389 - it.skip('will return 4 results if we have a list that excludes all ips', async () => { + it('will return 4 results if we have a list that excludes all ips', async () => { await importFile( supertest, 'ip', diff --git a/x-pack/test/detection_engine_api_integration/utils.ts b/x-pack/test/detection_engine_api_integration/utils.ts index d8466f013a11..0f888c3ee515 100644 --- a/x-pack/test/detection_engine_api_integration/utils.ts +++ b/x-pack/test/detection_engine_api_integration/utils.ts @@ -721,7 +721,7 @@ export const getSimpleRuleOutputWithWebHookAction = (actionId: string): Partial< export const waitFor = async ( functionToTest: () => Promise, functionName: string, - maxTimeout: number = 10000, + maxTimeout: number = 20000, timeoutWait: number = 10 ): Promise => { await new Promise(async (resolve, reject) => {