feat(allow-modules): include virtual: in the modules pattern

lib/util/get-allow-modules.js

@@ -53,7 +53,7 @@ module.exports.schema = {
  type: "array",
  items: {
  type: "string",
- pattern: "^(?:@[a-zA-Z0-9_\\-.]+/)?[a-zA-Z0-9_\\-.]+$",
+ pattern: "^(?:virtual:)?(?:@[a-zA-Z0-9_\\-.]+/)?[a-zA-Z0-9_\\-.]+$",
  },
  uniqueItems: true,
 }

tests/lib/rules/no-extraneous-import.js

@@ -98,6 +98,16 @@ ruleTester.run("no-extraneous-import", rule, {
  filename: fixture("tsconfig-paths/index.ts"),
  code: "import foo from '#configurations/foo'",
  },
+
+ // virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-name';",
+ },
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-scope/name';",
+ },
  ],
  invalid: [
  {

tests/lib/rules/no-extraneous-require.js

@@ -67,6 +67,16 @@ tester.run("no-extraneous-require", rule, {
  filename: fixture("dependencies/a.js"),
  code: "require('ccc')",
  },
+
+ // virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-name');",
+ },
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-scope/name');",
+ },
  ],
  invalid: [
  {

tests/lib/rules/no-missing-import.js

@@ -167,6 +167,17 @@ ruleTester.run("no-missing-import", rule, {
  code: "import electron from 'electron';",
  options: [{ allowModules: ["electron"] }],
  },
+ // allow virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-name';",
+ options: [{ allowModules: ["virtual:package-name"] }],
+ },
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-scope/name';",
+ options: [{ allowModules: ["virtual:package-scope"] }],
+ },
 
  // resolvePaths
  {
@@ -480,5 +491,17 @@ ruleTester.run("no-missing-import", rule, {
  },
  ]
  : []),
+
+ // virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-name';",
+ errors: cantResolve("virtual:package-name"),
+ },
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-scope/name';",
+ errors: cantResolve("virtual:package-scope/name"),
+ },
  ],
 })

tests/lib/rules/no-missing-require.js

@@ -240,6 +240,17 @@ ruleTester.run("no-missing-require", rule, {
  code: "require('jquery.cookie');",
  options: [{ allowModules: ["jquery.cookie"] }],
  },
+ // allow virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-name');",
+ options: [{ allowModules: ["virtual:package-name"] }],
+ },
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-scope/name');",
+ options: [{ allowModules: ["virtual:package-scope"] }],
+ },
 
  // typescriptExtensionMap
  {
@@ -440,6 +451,18 @@ ruleTester.run("no-missing-require", rule, {
  code: "require.resolve('no-exist-package-0');",
  errors: cantResolve("no-exist-package-0"),
  },
+
+ // Virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-name');",
+ errors: cantResolve("virtual:package-name"),
+ },
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-scope/name');",
+ errors: cantResolve("virtual:package-scope/name"),
+ },
  ],
 })
 

tests/lib/rules/no-unpublished-import.js

@@ -123,11 +123,23 @@ ruleTester.run("no-unpublished-import", rule, {
  code: "import a from './a';",
  },
 
+ // allow module
  {
  filename: fixture("1/test.js"),
  code: "import electron from 'electron';",
  options: [{ allowModules: ["electron"] }],
  },
+ // allow virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-name';",
+ options: [{ allowModules: ["virtual:package-name"] }],
+ },
+ {
+ filename: fixture("test.js"),
+ code: "import a from 'virtual:package-scope/name';",
+ options: [{ allowModules: ["virtual:package-scope"] }],
+ },
 
  // Auto-published files only apply to root package directory
  {

tests/lib/rules/no-unpublished-require.js

@@ -224,6 +224,17 @@ ruleTester.run("no-unpublished-require", rule, {
  code: "require('electron');",
  options: [{ allowModules: ["electron"] }],
  },
+ // allow virtual modules
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-name');",
+ options: [{ allowModules: ["virtual:package-name"] }],
+ },
+ {
+ filename: fixture("test.js"),
+ code: "require('virtual:package-scope/name');",
+ options: [{ allowModules: ["virtual:package-scope"] }],
+ },
 
  // Auto-published files only apply to root package directory
  {