Gracefully handle invalid Location redirect URLs

[brandon93s]

• Move new URL into existing try...catch block to gracefully handle failures
• Add test for ERR_INVALID_URL

h/t to @alextes for his initial research.

---

Closes #604

[alextes]

Nice work Brandon93. Lots of little improvements over what I wrote too. Unfortunately it does still suffer from the issue I flagged with the test. This test, like mine, is half functional. It lights up green beautifully, but imagine someone refactors and moves the new URL line out of the try/catch , the test won't show up red 😥 . It'll hang ava indefinitely which is something, but if possible I'd like the test red.

Let's see if the other maintainers have an idea of how to fix it, if not, let's merge this and open an issue.

To reproduce try moving the line back out of the try/catch construct.

[szmarczak]

To reproduce try moving the line back out of the try/catch construct.

So don't move that. :>

[alextes]

🙈

[brandon93s]

See avajs/ava#1931 for related issue.