Graph: Stabilize functional test (#52417)

elastic · Dec 9, 2019 · 5a14a7a · 5a14a7a
1 parent 43c55cf
commit 5a14a7a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/x-pack/test/functional/apps/graph/graph.ts b/x-pack/test/functional/apps/graph/graph.ts
@@ -13,8 +13,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
  const esArchiver = getService('esArchiver');
  const browser = getService('browser');
 
- // FLAKY: https:/elastic/kibana/issues/45321
- describe.skip('graph', function() {
+ describe('graph', function() {
  before(async () => {
  await browser.setWindowSize(1600, 1000);
  log.debug('load graph/secrepo data');
@@ -114,6 +113,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
  const circlesText = nodes.map(({ label }) => label);
  expect(circlesText.length).to.equal(expectedNodes.length);
  circlesText.forEach(circleText => {
+ log.debug(`Looking for ${circleText}`);
  expect(expectedNodes.includes(circleText)).to.be(true);
  });
  });

diff --git a/x-pack/test/functional/page_objects/graph_page.ts b/x-pack/test/functional/page_objects/graph_page.ts
@@ -235,6 +235,11 @@ export function GraphPageProvider({ getService, getPageObjects }: FtrProviderCon
  await this.goToListingPage();
  await this.searchForWorkspaceWithName(name);
  await find.clickByLinkText(name);
+ // wait for nodes to show up
+ if (!(await find.existsByCssSelector('.gphNode', 10000))) {
+ throw new Error('nodes did not show up');
+ }
+ // let force simulation settle down before continuing
  await PageObjects.common.sleep(5000);
  }