tests for #210.

tests/files/issue210.config.js

@@ -0,0 +1,3 @@
+exports.parserOptions = {
+  sourceType: 'module',
+}

tests/files/issue210.js

@@ -0,0 +1 @@
+export { test } from './issue210.js'

tests/files/narcissist.js

@@ -0,0 +1,2 @@
+export const me = 'awesome'
+export { me as soGreat } from './narcissist'

tests/src/cli.js

@@ -0,0 +1,25 @@
+/**
+ * tests that require fully booting up ESLint
+ */
+import { expect } from 'chai'
+import { CLIEngine } from 'eslint'
+
+describe('CLI regression tests', function () {
+  describe('issue #210', function () {
+    this.timeout(10000)
+    let cli
+    before(function () {
+      cli = new CLIEngine({
+        useEslintrc: false,
+        configFile: './tests/files/issue210.config.js',
+        rulePaths: ['./lib/rules'],
+        rules: {
+          'named': 2,
+        },
+      })
+    })
+    it("doesn't throw an error on gratuitous, erroneous self-reference", function () {
+      expect(() => cli.executeOnFiles(['./tests/files/issue210.js'])).not.to.throw(Error)
+    })
+  })
+})

tests/src/core/getExports.js

@@ -264,4 +264,18 @@ describe('getExports', function () {
     })
   })
 
+  context('issue #210: self-reference', function () {
+    it("doesn't crash", function () {
+      expect(() => ExportMap.get('./narcissist', fakeContext)).not.to.throw(Error)
+    })
+    it("'has' circular reference", function () {
+      expect(ExportMap.get('./narcissist', fakeContext))
+        .to.exist.and.satisfy(m => m.has('soGreat'))
+    })
+    it("can 'get' circular reference", function () {
+      expect(ExportMap.get('./narcissist', fakeContext))
+        .to.exist.and.satisfy(m => m.get('soGreat') != null)
+    })
+  })
+
 })

tests/src/rules/named.js

@@ -86,6 +86,9 @@ ruleTester.run('named', rule, {
     // ignore is ignored if exports are found
     test({ code: 'import { foo } from "es6-module"' }),
 
+    // issue #210: shameless self-reference
+    test({ code: 'import { me, soGreat } from "./narcissist"' }),
+
   ],
 
   invalid: [