fix(rule-finder): Support scoped packages with dashes

[chrisblossom]

Use eslint's internal functions to support shorthand eslint plugin naming.

fix #303

[chrisblossom]

Are there any other requested changes to get this PR moving along?

[ljharb]

Thanks, i like the improvements

[ljharb]

@chrisblossom I'm afraid that now that this is rebased on top of eslint 6 support, the eslint 6 tests are failing.

[chrisblossom]

I'm afraid that now that this is rebased on top of eslint 6 support, the eslint 6 tests are failing.

I’ll look into this soon.

[chrisblossom]

Looks like one of the issues is that eslint moved the naming file in 6.1.0.

v6.0.1: lib/cli-engine/naming.js
v6.1.0: lib/shared/naming.js

[ljharb]

Sigh. Let’s do another try/catch require on that then.

[chrisblossom]

Tests pass now. Also increased the mocha timeout in test/lib/rule-finder.js because tests would randomly fail.

...
    ✓ no specifiedFile - all available rules including deprecated
    1) specifiedFile (relative path) - unused rules
    ✓ specifiedFile (relative path) - unused rules including deprecated (1756ms)
    ✓ specifiedFile (relative path) - current rules (1697ms)
    ✓ specifiedFile (relative path) - current rule config (1722ms)
    ✓ specifiedFile (relative path) - plugin rules (1717ms)

...

  64 passing (39s)
  1 failing

  1) rule-finder specifiedFile (relative path) - unused rules:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
      at processImmediate (internal/timers.js:439:21)

[chrisblossom]

Who else needs to review this PR to get it merged and released? I'd like to be able to start using this with my typescript eslint config.