From acfb15dbd451f1605890b5ba6c164c7580ed0bcf Mon Sep 17 00:00:00 2001 From: Devin Hurley Date: Thu, 17 Sep 2020 10:40:11 -0400 Subject: [PATCH] updates comments in e2e tests for import rules route --- .../basic/tests/import_rules.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/x-pack/test/detection_engine_api_integration/basic/tests/import_rules.ts b/x-pack/test/detection_engine_api_integration/basic/tests/import_rules.ts index b290de0d55a7d..1944e856f981d 100644 --- a/x-pack/test/detection_engine_api_integration/basic/tests/import_rules.ts +++ b/x-pack/test/detection_engine_api_integration/basic/tests/import_rules.ts @@ -192,7 +192,7 @@ export default ({ getService }: FtrProviderContext): void => { }); }); - // import is still very slow due to the alerts client find api + // import is very slow in 7.10+ due to the alerts client find api // when importing 100 rules it takes about 30 seconds for this // test to complete so at 10 rules completing in about 10 seconds // I figured this is enough to make sure the import route is doing its job. @@ -212,6 +212,7 @@ export default ({ getService }: FtrProviderContext): void => { }); // uncomment the below test once we speed up the alerts client find api + // in another PR. // it('should be able to import 10000 rules', async () => { // const ruleIds = new Array(10000).fill(undefined).map((_, index) => `rule-${index}`); // const { body } = await supertest