René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"tests: review and fix ineffective error tests","articleBody":"See https://github.com/nodejs/node/pull/26078#pullrequestreview-209542148 for context.\r\n\r\nAfter that, I looked up if we already have similar ineffective tests in our codebase, and I think that I found at least one.\r\n\r\nE.g. using `grep -r ' catch' test/ -A 2 | grep strictEqual`,  this one popped up half through the list: \r\n\r\nhttps://github.com/nodejs/node/blob/453ed053432e0c1ae34457e5c364b7091af2615e/test/parallel/test-stream-readable-async-iterators.js#L355-L361\r\n\r\nThe code above doesn't test that the error is thrown (as it likely should), instead it tests that _another_ error _doesn't_ get thrown.\r\n\r\nNote that there are a lot of false positives in the said grep (actually most matches are false positives), because e.g. this is perfectly fine:\r\nhttps://github.com/nodejs/node/blob/453ed053432e0c1ae34457e5c364b7091af2615e/test/parallel/test-vm-module-errors.js#L266-L272","author":{"url":"https://github.com/ChALkeR","@type":"Person","name":"ChALkeR"},"datePublished":"2019-03-01T23:48:39.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":7},"url":"https://github.com/26385/node/issues/26385"}