Skip to content

Commit

Permalink
[Security Solution] Unskip tests for Detection Engine and Security So…
Browse files Browse the repository at this point in the history
…lution API Integration tests (elastic#169262)

## Summary

### Changes

-
`x-pack/test/detection_engine_api_integration/security_and_spaces/group4/telemetry/task_based/detection_rules.ts`
**unskipped**
-
`x-pack/test/detection_engine_api_integration/security_and_spaces/rule_execution_logic/machine_learning.ts`
**unskipped**

### Related flaky test issues

**Tests already moved to
`x-pack/test/security_solution_api_integration`**

1. elastic#169055
2. elastic#168775
This issue is reported for the legacy test, but is now moved to
`x-pack/test/security_solution_api_integration/test_suites/detections_response/default_license/exceptions/operators_data_types/ips_text_array/text_array.ts`
3. elastic#163511
This issue is reported for the legacy test, but is now moved to
`x-pack/test/security_solution_api_integration/test_suites/detections_response/default_license/exceptions/operators_data_types/keyword_text_long/keyword.ts`

- Flaky test runner for above tests:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3631

**Test still in `x-pack/test/detection_engine_api_integration`**
1. elastic#168415
2. elastic#164318
3. elastic#164313
4. elastic#145776

**Other**
1. elastic#151636

## Flaky test runner link

Part 1 - `x-pack/test/detection_engine_api_integration` - tests not yet
migrated
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3597
Part 2 - `x-pack/test/security_solution_api_integration` - tests already
moved
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3631
  • Loading branch information
jpdjere authored Oct 31, 2023
1 parent c054a2d commit 9eff64a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ export default ({ getService }: FtrProviderContext) => {
const log = getService('log');
const retry = getService('retry');

// Failing: See https://github.com/elastic/kibana/issues/164318
// FLAKY: https://github.com/elastic/kibana/issues/164313
describe.skip('Detection rule task telemetry', async () => {
describe('Detection rule task telemetry', async () => {
before(async () => {
await esArchiver.load('x-pack/test/functional/es_archives/security_solution/telemetry');
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,7 @@ export default ({ getService }: FtrProviderContext) => {
rule_id: 'ml-rule-id',
};

// FLAKY: https://github.com/elastic/kibana/issues/145776
describe.skip('Machine learning type rules', () => {
describe('Machine learning type rules', () => {
before(async () => {
// Order is critical here: auditbeat data must be loaded before attempting to start the ML job,
// as the job looks for certain indices on start
Expand Down

0 comments on commit 9eff64a

Please sign in to comment.