changes to the test_api_keys role

elastic · Mar 25, 2020 · 7079fe1 · 7079fe1
1 parent d415596
commit 7079fe1
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/x-pack/test/functional/apps/api_keys/home_page.ts b/x-pack/test/functional/apps/api_keys/home_page.ts
@@ -31,7 +31,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
     });
 
     it('Loads the app', async () => {
-      await security.testUser.setRoles(['kibana_admin', 'test_api_keys']);
+      await security.testUser.setRoles(['test_api_keys']);
       log.debug('Checking for section header');
       const headerText = await pageObjects.apiKeys.noAPIKeysHeading();
       expect(headerText).to.be('No API keys');

diff --git a/x-pack/test/functional/config.js b/x-pack/test/functional/config.js
@@ -234,10 +234,19 @@ export default async function({ readConfigFile }) {
           kibana: [],
         },
 
+        //Kibana feature privilege isn't specific to advancedSetting. It can be anything. https://github.com/elastic/kibana/issues/35965
         test_api_keys: {
           elasticsearch: {
             cluster: ['manage_security', 'manage_api_key'],
           },
+          kibana: [
+            {
+              feature: {
+                advancedSettings: ['read'],
+              },
+              spaces: ['default'],
+            },
+          ],
         },
       },
       defaultRoles: ['superuser'],

diff --git a/x-pack/test/functional/page_objects/api_keys_page.ts b/x-pack/test/functional/page_objects/api_keys_page.ts
@@ -21,9 +21,7 @@ export function ApiKeysPageProvider({ getService }: FtrProviderContext) {
     },
 
     async apiKeysPermissionDeniedMessage() {
-      const messageElement = await testSubjects.find('apiKeysPermissionDeniedMessage');
-      const message = await messageElement.getVisibleText();
-      return message;
+      return await testSubjects.getVisibleText('apiKeysPermissionDeniedMessage');
     },
   };
 }