browserDevtools: true doesn't make Chromium headful

[simonhaenisch]

Stencil version:

 @stencil/core@1.15.0

I'm submitting a:
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://stencil-worldwide.herokuapp.com/ or https://forum.ionicframework.com/

Current behavior:

Setting testing.browserDevtools = true doesn't run Chromium in headful mode.

Expected behavior:

It does what the doc comment says:

Whether to auto-open a DevTools panel for each tab. If this option is true, the headless option will be set false

Other information:

I think

stencil/src/testing/puppeteer/puppeteer-browser.ts

Lines 17 to 19 in 6d69f53

	if (config.flags.devtools) {
	config.testing.browserDevtools = true;
	config.testing.browserHeadless = false;

should be moved into src/compiler/config/validate-testing.ts.

--

BTW I usually don't want to run e2e tests in headful mode but sometimes it helps to see what's going on, so having a --headful or --devtools flag would be quite convenient.