Skip to content

Commit

Permalink
[Code] remove requirement for SettingRepository button in tests (#33081)
Browse files Browse the repository at this point in the history
  • Loading branch information
spacedragon authored and mw-ding committed Mar 13, 2019
1 parent cce157a commit 0088511
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions x-pack/test/functional/apps/code/with_security.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,8 @@ export default function testWithSecurity({ getService, getPageObjects }: TestInv
const codeAdmin = 'codeAdmin';
const codeUser = 'codeUser';
const repositoryListSelector = 'codeRepositoryList codeRepositoryItem';
const manageButtonSelectors = [
'settingsRepositoryButton',
'indexRepositoryButton',
'deleteRepositoryButton',
];
const manageButtonSelectors = ['indexRepositoryButton', 'deleteRepositoryButton'];
const log = getService('log');

describe('with security enabled:', () => {
before(async () => {
Expand Down Expand Up @@ -89,9 +86,11 @@ export default function testWithSecurity({ getService, getPageObjects }: TestInv
for (const buttonSelector of manageButtonSelectors) {
const buttons = await testSubjects.findAll(buttonSelector);
expect(buttons).to.have.length(1);
log.debug(`button ${buttonSelector} found.`);
}
const importButton = await testSubjects.findAll('newProjectButton');
expect(importButton).to.have.length(1);
log.debug(`button newProjectButton found.`);
});

await login(codeUser);
Expand Down

0 comments on commit 0088511

Please sign in to comment.