Direct electron's console (log and error) to test runner's console, as an option about spectron

[kevinsawicki]

There are two APIs currently for log details:

https://github.com/electron/spectron#clientgetmainprocesslogs
https://github.com/electron/spectron#clientgetrenderprocesslogs

There isn't a way to forward them to the test process console though currently, seems like a nice option though 👍

[3n-mb]

Frankly, I am puzzled. Quoting lib/launcher.js, lines 30-34:

var appProcess = ChildProcess.spawn(executablePath, args)
appProcess.on('exit', function (code) { process.exit(code) })
appProcess.stderr.pipe(process.stdout)
appProcess.stdout.pipe(process.stdout)

Last two lines should already pipe child's things to parent from app's script is not showing up.
In fact, if we add in lib/launcher.js console.log('whatever'), it doesn't show either. Nor, process.stdout.write('whatever') works!
Something must have been done to sdout, stderr. What is it? Can you tell?

[3n-mb]

Yah. console.log('whatever'), placed in launcher.js, before spawning child with an app, shows in the output of app.client.getMainProcessLogs()!
Oh, boy. Chrome's log is noisy. Keeping app's stdout out of chrome's log would be nice.

[3n-mb]

app.client.getRenderProcessLogs() sounds promising, so, naturally I do client.execute(... console.log('whatever') ...), and, it does not appear in render process log.
Are me expectations incorrect?

[kevinsawicki]

Are me expectations incorrect?

Yeah, I would think that would show up there.

Here are the tests for it:

https://github.com/electron/spectron/blob/a9b1dba0be76033b3e407de6e319b4bbf2bada55/test/fixtures/app/index.html#L7-L9

https://github.com/electron/spectron/blob/a9b1dba0be76033b3e407de6e319b4bbf2bada55/test/application-test.js#L124-L146

[3n-mb]

Never mind with getRenderProcessLogs(), I had an error in my code.

[3n-mb]

I guess, main process logs can be used, skipping chrome's things that start with '['.

UPDATE: this trick does not work, when console output is multi-line, cause duplicates to app.client.getMainProcessLogs() will me chopped

[3n-mb]

@kevinsawicki I missed, that you labeled this for enhancement, so I'll reopen this.

Browser's console gets echoed into app.client.getMainProcessLogs(). Wonder, if this duplication can turned off.

[Manunarasingh]

using app.client.getMainProcessLogs() or app.client.getRenderProcessLogs() we won't get entire errors stack it is showing string of "ERROR" . i need entire browser console errors. how to achieve this??