In Node.js 12 or newer, add hasColors to the fake TTY methods

[novemberborn]

We're missing hasColors() in our fake TTY implementation: https://github.com/avajs/ava/blob/533ee4bc54cbcd1c2b6ab5c67f3ec8a699965269/lib/worker/fake-tty.js

It was introduced in Node.js 11, so we should check if the worker is running in Node.js 12 before adding it (Node.js 11 is end of life, see #2169). We should be able to piggy-back off of the colorDepth value that's already available. I don't think we can make it work with arbitrary env variables though. We can deal with that as a follow-up.

Currently this is causing CI to fail with Node.js 12.5.0: https://travis-ci.org/avajs/ava/jobs/552425902

---

IssueHunt Summary

Backers (Total: $40.00)

• issuehunt ($40.00)

Become a backer now!

Or submit a pull request to get the deposits!

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

---

IssueHunt has been backed by the following sponsors. Become a sponsor

[oantoro]

in simulateTTY() requires colorDepth option to define the getColorDepth() method. Should hasColors() be defined the same way as getColorDepth()? @novemberborn

[novemberborn]

@okyantoro it also requires colorDepth, yes, but getColorDepth() is available in Node.js 10 and hasColors() should only be made available in Node.js 12 and newer. Perhaps a typeof process.stderr.hasColors() is sufficient to see if it needs to be shimmed.

[issuehunt-oss]

@issuehunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.