Incorrect handling of non-json response from the remote node for HttpProvider

[ghost]

Is there an existing issue for this?

• I have searched the existing issues

Current Behavior

Console would warn for unhandled promise rejection when non-json response is received from the remote node, which is a bug.

Expected Behavior

HttpProvider should throw an Invalid JSON RPC response error for non-json response ( Which is usually an error string from the loadbalancer )

Steps to Reproduce

it('should fail for non-json format response', async function () {
            var provider = new HttpProvider('/fetchMock');
            var web3 = new Web3(provider);

            fetchMock.mock('/fetchMock', 'Testing non-json format response');

            await expect(web3.eth.getChainId()).to.be.rejectedWith(Error, /Invalid JSON RPC response/);
            fetchMock.restore();
});

Web3.js Version

1.x

Environment

Fixed by latest commit from PR #5228

Anything Else?

No response