Skip to content

Commit

Permalink
Fix spaces test flakyness (#100605)
Browse files Browse the repository at this point in the history
  • Loading branch information
legrego authored and kibanamachine committed May 26, 2021
1 parent 181ef83 commit 570025c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,6 @@ export class NavControlPopover extends Component<Props, State> {

public render() {
const button = this.getActiveSpaceButton();
if (!button) {
return null;
}

let element: React.ReactNode;
if (!this.state.loading && this.state.spaces.length < 2) {
Expand Down Expand Up @@ -102,7 +99,7 @@ export class NavControlPopover extends Component<Props, State> {
return (
<EuiPopover
id={'spcMenuPopover'}
data-test-subj={`spacesNavSelector`}
data-test-subj={`${this.state.loading ? 'spacesNavSelectorLoading' : 'spacesNavSelector'}`}
button={button}
isOpen={this.state.showSpaceSelector}
closePopover={this.closeSpaceSelector}
Expand Down
3 changes: 1 addition & 2 deletions x-pack/test/functional/apps/spaces/spaces_selection.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@ export default function spaceSelectorFunctionalTests({
'spaceSelector',
]);

// FLAKY: https://github.com/elastic/kibana/issues/99581
describe.skip('Spaces', function () {
describe('Spaces', function () {
this.tags('includeFirefox');
describe('Space Selector', () => {
before(async () => {
Expand Down

0 comments on commit 570025c

Please sign in to comment.