no-restricted-imports doesn't restrict export * when importNames are configured

[mdjermanovic]

Tell us about your environment

• ESLint Version: v6.8.0
• Node Version: v12.14.0
• npm Version: v6.13.4

What parser (default, Babel-ESLint, etc.) are you using?

default

Please show your full configuration:

Configuration

module.exports = {
    parserOptions: {
        ecmaVersion: 2015,
        sourceType: "module"
    },
};

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

Online Demo Link

/* eslint no-restricted-imports: ["error",
  {
    name: "mod1"  
  },
  {
    name: "mod2",
    importNames: ["a"]
  }
]*/

import * as foo from "mod1"; // error
export * from "mod1"; // error

import { a } from "mod2"; // error
export { a } from "mod2"; // error
import * as bar from "mod2"; // error
export * from "mod2"; // no error, is this a bug?

eslint index.js

What did you expect to happen?

Errors for the first 5 declarations.

I'm not sure what should be expected behavior for export * from "mod2". That is the question of this issue.

What actually happened? Please include the actual, raw output from ESLint.

Errors for the first 5 declarations, as expected.

No error for export * from "mod2".

  11:1  error  'mod1' import is restricted from being used                no-restricted-imports
  12:1  error  'mod1' import is restricted from being used                no-restricted-imports
  14:1  error  'mod2' import is restricted from being used                no-restricted-imports
  15:1  error  'mod2' import is restricted from being used                no-restricted-imports
  16:1  error  * import is invalid because 'a' from 'mod2' is restricted  no-restricted-imports

Are you willing to submit a pull request to fix this bug?

Yes.

[kaicataldo]

It makes sense to me to treat import * as bar from "mod2"; and export * from "mod2"; similarly and error on the latter as well.

[mdjermanovic]

As for the PR, seems best to finish and merge #12711 first and then do the PR for this fix on top of the changes from #12711.

[mdjermanovic]

I'm working on this.