Skip to content

Commit

Permalink
[Security Solution] Skip flaky Rule Management tests (#173730)
Browse files Browse the repository at this point in the history
**Resolves: #173469

## Summary

This PR:

- Skips flaky Rule Management tests that occurred recently and haven't
been already skipped by Kibana Operations.
- Adds `TODO` comments with links to the corresponding tickets.

We will be working on fixing and unskipping these and a few other tests
in January as part of #173731.
  • Loading branch information
banderror authored Dec 20, 2023
1 parent bada436 commit cdb1047
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ export default ({ getService }: FtrProviderContext): void => {
* Unlike other tests that use mocks, this test uses actual rules from the
* package storage and checks that they are installed.
*/
it('should install prebuilt rules from the package storage', async () => {
// TODO: Fix and unskip https://github.com/elastic/kibana/issues/172107
it.skip('should install prebuilt rules from the package storage', async () => {
// Verify that status is empty before package installation
const statusBeforePackageInstallation = await getPrebuiltRulesAndTimelinesStatus(supertest);
expect(statusBeforePackageInstallation.rules_installed).toBe(0);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ const ruleNameToAssert = 'Custom rule name with actions';
const expectedExistingSlackMessage = 'Existing slack action';
const expectedSlackMessage = 'Slack action test message';

// TODO: Fix and unskip in Serverless https://github.com/elastic/kibana/issues/171101
describe(
'Detection rules, bulk edit of rule actions',
{ tags: ['@ess', '@serverless', '@brokenInServerless', '@brokenInServerlessQA'] },
Expand Down

0 comments on commit cdb1047

Please sign in to comment.