[Controls] Pause fetch until initialized

[ThomThomson]

Summary

This PR changes the Dashboard to render all of its panels at once - even when the control group is not initialized.

To prevent double refetches the rendered panels will have their fetches paused until all controls and their initial values are ready.

How to test

• Create a Dashboard with one Lens chart (disable the other bucket)
• Add a control to that Dashboard and select an option
• Empty cache and hard refresh that Dashboard.
• You should see only one ese request in the network tab.

[ThomThomson]

/ci

[elasticmachine]

Pinging @elastic/kibana-presentation (Team:Presentation)

[ThomThomson]

@elasticmachine merge upstream

[nreese]

How about breaking this file into blocks with a few describes. For instance, this test and next should be in a describe block for getStateForControlGroup

[nreese]

How about moving this test and the next into a describe block for serializeControlGroup

[drewdaemon]

I'm not sure how far we want to go with testing, but I'll share here a way to check the runtime request count in a Scout functional test in case it is helpful:

test('request count', async ({ page, context }) => {
  const cdp = await context.newCDPSession(page);
  await cdp.send('Network.enable');
  cdp.on('Network.requestWillBeSent', (event) => { // doc: https://chromedevtools.github.io/devtools-protocol/tot/Network/#event-requestWillBeSent
    if (/* check event.request.url */) {
      // count requests
    }
  });

  // load page and wait for something relevant

  // assert
});

(from #237949 (comment))

[ThomThomson]

Thanks for the snippet @drewdaemon! It's a great idea to write this kind of scout test, but I'd prefer to do so in a followup if that is okay for you!

[drewdaemon]

@ThomThomson ++ definitely not a blocker, just a flyby advertisement 😁

[elasticmachine]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: e647299

Failed CI Steps

• FTR Configs #15

Test Failures

• [job] [logs] FTR Configs #15 / Cases - group 1 View case Assignees field logs in with default user assigns a user from the popover

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
dashboard	664.1KB	664.0KB	-164.0B

History

• 💔 Build #352724 failed d0e58c5
• 💚 Build #351493 succeeded 9ed13de

[nreese]

kibana-presentation changes LGTM
code review and tested in chrome