Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[question] How to see log of launcher? #2465

Open
jeanbza opened this issue Nov 21, 2016 · 5 comments
Open

[question] How to see log of launcher? #2465

jeanbza opened this issue Nov 21, 2016 · 5 comments

Comments

@jeanbza
Copy link

jeanbza commented Nov 21, 2016

e.x. karma-phantomjs-launcher / karma-chrome-launcher / karma-script-launcher

@jeanbza
Copy link
Author

jeanbza commented Nov 21, 2016

Am I right in assuming that this piece of code means that there's no way to see launcher output?

@jeanbza
Copy link
Author

jeanbza commented Nov 21, 2016

I was able to replace that code with the following to make it work, but I'd really prefer to have a flag to do this. Would it make sense to PR this?

  var spawnWithoutOutput = function () {
      var proc = spawn.apply(null, arguments)
      proc.stdout.pipe(process.stdout)
      proc.stderr.pipe(process.stderr)

      return proc
    }

    ProcessLauncher.call(launcher, spawnWithoutOutput, require('../temp_dir'), timer)

@maksimr
Copy link
Contributor

maksimr commented Nov 25, 2016

This was introduced in #2267 Pull request

@maksimr
Copy link
Contributor

maksimr commented Nov 25, 2016

@jadekler

Would it make sense to PR this?

Yes. Maybe better print browser log by default(as before) and hide behind the falg feature from Pull Request #2267 . Because case from #2267

  1. Rare
  2. Hide errors from the browser and I think it's not so obviously for user, more obviously look at flag which resume browser log (IMHO)

@dignifiedquire what do you think?

@dignifiedquire
Copy link
Member

It should be possible to pipe the output from the spawn to the console, without creating the buffer issue. It might be interesting to try out for spawning https://github.com/sindresorhus/execa which I have made good experiences in the past.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants