Got.js throws ERR_INVALID_ARG_TYPE when mocking with nock on Node 12.x

[yamalight]

Describe the bug

• Node.js version: 12.x
• OS & version: any

Trying to use nock with Got in Node 12.x breaks Got with ERR_INVALID_ARG_TYPE.

Actual behavior

When mocking API with nock.js - on request Got.js throws ERR_INVALID_ARG_TYPE and dies.

Expected behavior

Should correctly handle request to nock server.

Code to reproduce

const nock = require('nock');
const got = require('got');

const thisWorks = await got('https://httpbin.org/get');
console.log(thisWorks.body);

const testServer = nock('http://localhost:8080')
    .get('/')
    .reply(200, {hello: 'world'});

const thisBreaks = await got('http://localhost:8080/');
console.log(thisBreaks.body);

Here's an interactive demo:
https://runkit.com/yamalight/5cf50559b975d2001a4dfaa8

Switching to Node 10.x fixes the issue. Using e.g. node-fetch instead of Got.js also works correctly with nock.

Checklist

• I have read the documentation.
• I have tried my code with the latest version of Node.js and Got.

[sindresorhus]

Are you sure it's not nock/nock#1509?

[yamalight]

@sindresorhus yes, I've tried using other libs and they seem to work fine.
I've added node-fetch and axios to interactive demo to demonstrate that they work fine.

[szmarczak]

Make sure you're running at least 11.0.0-beta.13 (or above): nock/nock#1509 (comment)
The RunKit demo runs 10.0.6 where the issue is not fixed.

Let us know then.

[yamalight]

@szmarczak yep, that does indeed fixes it. Somehow misread it and thought fix was shipped in 10.0.6. Sorry for the bother and thanks for help! 😅

[szmarczak]

No problem :)