Skip to content

Commit

Permalink
Migrated Snapshot and Restore tests to use test user (#126011) (#126637)
Browse files Browse the repository at this point in the history
* Migrated Snapshot and Restore tests to use test user.

* added another permission to address unauthorized failure.

* Added metadata perms for kibana administration.

* Testing out different permissions.

* Edited test to not refresh page after role is set. Causing errors.

* Adjusted perms per documentation.

* Addded link to the documentation specifying the permissions needed for snapshot restore.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit d746bd4)
  • Loading branch information
cuff-links committed Mar 1, 2022
1 parent eace746 commit 55b9cf7
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 3 deletions.
5 changes: 3 additions & 2 deletions x-pack/test/functional/apps/snapshot_restore/home_page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,11 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
const pageObjects = getPageObjects(['common', 'snapshotRestore']);
const log = getService('log');
const es = getService('es');
const security = getService('security');

describe('Home page', function () {
before(async () => {
await security.testUser.setRoles(['snapshot_restore_user'], false);
await pageObjects.common.navigateToApp('snapshotRestore');
});

Expand Down Expand Up @@ -46,9 +48,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {

it('cleanup repository', async () => {
await pageObjects.snapshotRestore.viewRepositoryDetails('my-repository');
await pageObjects.common.sleep(25000);
const cleanupResponse = await pageObjects.snapshotRestore.performRepositoryCleanup();
await pageObjects.common.sleep(25000);
expect(cleanupResponse).to.contain('results');
expect(cleanupResponse).to.contain('deleted_bytes');
expect(cleanupResponse).to.contain('deleted_blobs');
Expand All @@ -57,6 +57,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
await es.snapshot.deleteRepository({
name: 'my-repository',
});
await security.testUser.restoreDefaults();
});
});
});
Expand Down
19 changes: 19 additions & 0 deletions x-pack/test/functional/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -551,6 +551,25 @@ export default async function ({ readConfigFile }) {
},
],
},
// https://www.elastic.co/guide/en/elasticsearch/reference/master/snapshots-register-repository.html#snapshot-repo-prereqs
snapshot_restore_user: {
elasticsearch: {
cluster: [
'monitor',
'manage_slm',
'cluster:admin/snapshot',
'cluster:admin/repository',
],
},
kibana: [
{
feature: {
advancedSettings: ['read'],
},
spaces: ['*'],
},
],
},

ingest_pipelines_user: {
elasticsearch: {
Expand Down
5 changes: 4 additions & 1 deletion x-pack/test/functional/page_objects/snapshot_restore_page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -49,12 +49,15 @@ export function SnapshotRestorePageProvider({ getService }: FtrProviderContext)
const repoToView = repos.filter((r) => (r.repoName = name))[0];
await repoToView.repoLink.click();
}
await retry.waitForWithTimeout(`Repo title should be ${name}`, 10000, async () => {
await retry.waitForWithTimeout(`Repo title should be ${name}`, 25000, async () => {
return (await testSubjects.getVisibleText('title')) === name;
});
},
async performRepositoryCleanup() {
await testSubjects.click('cleanupRepositoryButton');
await retry.waitForWithTimeout(`wait for code block to be visible`, 25000, async () => {
return await testSubjects.isDisplayed('cleanupCodeBlock');
});
return await testSubjects.getVisibleText('cleanupCodeBlock');
},
};
Expand Down

0 comments on commit 55b9cf7

Please sign in to comment.