tools: add eslint rule for inspector checking

[danbev]

The motivation for this commit is to pick up early on missing checks for
inspector support (when Node is built --without-inspector).

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• commit message follows commit guidelines

Affected core subsystem(s)

tools

[danbev]

CI: https://ci.nodejs.org/job/node-test-pull-request/8754/

[cjihrig]

It would be nice to combine this with the rule in #13813.

[danbev]

It would be nice to combine this with the rule in #13813.

I thought it might make it easier to review and maintain having them as separate rules. But I can merge them together if that is preferred.

[cjihrig]

I think they are similar enough that the rule can be parameterized. They're both checking "if module X is required, make sure there is an if statement to skip the test."

[danbev]

I think they are similar enough that the rule can be parameterized. They're both checking "if module X is required, make sure there is an if statement to skip the test."

I'll give it a try, closing this for now.