github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-06-27 18:51:07 -04:00
Code Issues Projects Releases Packages Wiki Activity

[Security] Fix security privileges tests by removing Security Solution file_operations_all sub-feature from test assertions (#145247)

Browse source 
## Summary

- Remove `file_operations_all` from list of sub-feature of SIEM
(security solution). This sub-feature [was recently placed behind an
experimental feature
flag](https://github.com/elastic/kibana/pull/145042), which is disabled
by default for 8.6.
- Un-skips tests


Fixes: #145134
Fixes: #145135
Fixes: #145136
This commit is contained in:
Paul Tavares 2022-11-15 10:34:50 -05:00 • committed by GitHub
parent 295a267dae
commit 731c8b962b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

7
x-pack/test/api_integration/apis/security/privileges.ts
View file

@ -38,7 +38,6 @@ export default function ({ getService }: FtrProviderContext) {
'actions_log_management_read',
'host_isolation_all',
'process_operations_all',
'file_operations_all',
],
uptime: ['all', 'read', 'minimal_all', 'minimal_read'],
securitySolutionCases: ['all', 'read', 'minimal_all', 'minimal_read', 'cases_delete'],
@ -84,8 +83,7 @@ export default function ({ getService }: FtrProviderContext) {
};
describe('Privileges', () => {
// FLAKY: https://github.com/elastic/kibana/issues/145135
describe.skip('GET /api/security/privileges', () => {
describe('GET /api/security/privileges', () => {
it('should return a privilege map with all known privileges, without actions', async () => {
// If you're adding a privilege to the following, that's great!
// If you're removing a privilege, this breaks backwards compatibility
@ -194,8 +192,7 @@ export default function ({ getService }: FtrProviderContext) {
});
// In this non-Basic case, results should be exactly the same as not supplying the respectLicenseLevel flag
// FLAKY: https://github.com/elastic/kibana/issues/145136
describe.skip('GET /api/security/privileges?respectLicenseLevel=false', () => {
describe('GET /api/security/privileges?respectLicenseLevel=false', () => {
it('should return a privilege map with all known privileges, without actions', async () => {
// If you're adding a privilege to the following, that's great!
// If you're removing a privilege, this breaks backwards compatibility

4
x-pack/test/api_integration/apis/security/privileges_basic.ts
View file

@ -13,8 +13,7 @@ export default function ({ getService }: FtrProviderContext) {
const supertest = getService('supertest');
describe('Privileges', () => {
// FLAKY: https://github.com/elastic/kibana/issues/145134
describe.skip('GET /api/security/privileges', () => {
describe('GET /api/security/privileges', () => {
it('should return a privilege map with all known privileges, without actions', async () => {
// If you're adding a privilege to the following, that's great!
// If you're removing a privilege, this breaks backwards compatibility
@ -101,7 +100,6 @@ export default function ({ getService }: FtrProviderContext) {
'actions_log_management_all',
'actions_log_management_read',
'all',
'file_operations_all',
'host_isolation_all',
'minimal_all',
'minimal_read',