24088240

Build Type

push

appveyor

Committed by GitHub

Commit Message

<a href="https://github.com/benmosher/eslint-plugin-import/commit/<a class=hub.com/benmosher/eslint-plugin-import/commit/70cd4c3658280c1a27a7b3a2d0815e9de123cd5f">70cd4c365<a href="https://github.com/benmosher/eslint-plugin-import/commit/70cd4c3658280c1a27a7b3a2d0815e9de123cd5f">">Merge </a><a class="double-link" href="https://github.com/benmosher/eslint-plugin-import/commit/<a class="double-link" href="https://github.com/benmosher/eslint-plugin-import/commit/b00a20494501f90ca58e2c656102c9dd2b8abbc4">b00a20494</a>">b00a20494</a><a href="https://github.com/benmosher/eslint-plugin-import/commit/70cd4c3658280c1a27a7b3a2d0815e9de123cd5f"> into c17dd73e4">c17dd73e4</a>

Run Details

1333 of 1460 branches covered (91.3%)

61 of 61 new or added lines in 1 file covered. (100.0%)

1953 of 2044 relevant lines covered (95.55%)

44.8 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

92.31

/src/rules/named.js

import * as path from 'path'
import Exports from '../ExportMap'
import docsUrl from '../docsUrl'

module.exports = {
  meta: {
    type: 'problem',
    docs: {
      url: docsUrl('named'),
    },
  },

  create: function (context) {
    function checkSpecifiers(key, type, node) {
      // ignore local exports and type imports
      if (node.source == null || node.importKind === 'type') return

      if (!node.specifiers
            .some(function (im) { return im.type === type })) {
        return // no named imports/exports
      }

      const imports = Exports.get(node.source.value, context)
      if (imports == null) return

      if (imports.errors.length) {
        imports.reportErrors(context, node)
        return
      }

      node.specifiers.forEach(function (im) {
        if (im.type !== type) return

        // ignore type imports
        if (im.importKind === 'type') return

        const deepLookup = imports.hasDeep(im[key].name)

        if (!deepLookup.found) {
          if (deepLookup.path.length > 1) {
            const deepPath = deepLookup.path
              .map(i => path.relative(path.dirname(context.getFilename()), i.path))
              .join(' -> ')

            context.report(im[key],
              `${im[key].name} not found via ${deepPath}`)
          } else {
            context.report(im[key],
              im[key].name + ' not found in \'' + node.source.value + '\'')
          }
        }
      })
    }

    return {
      'ImportDeclaration': checkSpecifiers.bind( null
                                               , 'imported'
                                               , 'ImportSpecifier'
                                               ),

      'ExportNamedDeclaration': checkSpecifiers.bind( null
                                                    , 'local'
                                                    , 'ExportSpecifier'
                                                    ),
    }

  },
}

1	import * as path from 'path'
2	import Exports from '../ExportMap'
3	import docsUrl from '../docsUrl'
4
5	module.exports = {	1✔
6	meta: {
7	type: 'problem',
8	docs: {
9	url: docsUrl('named'),
10	},
11	},
12
13	create: function (context) {
14	function checkSpecifiers(key, type, node) {
15	// ignore local exports and type imports
16	if (node.source == null \|\| node.importKind === 'type') return	92✔
17
18	if (!node.specifiers	84✔
19	.some(function (im) { return im.type === type })) {	86✔
20	return // no named imports/exports	6✔
21	}
22
23	const imports = Exports.get(node.source.value, context)	78✔
24	if (imports == null) return	78✔
25
26	if (imports.errors.length) {	68!
27	imports.reportErrors(context, node)	×
28	return	×
29	}
30
31	node.specifiers.forEach(function (im) {	68✔
32	if (im.type !== type) return	94✔
33
34	// ignore type imports
35	if (im.importKind === 'type') return	91✔
36
37	const deepLookup = imports.hasDeep(im[key].name)	89✔
38
39	if (!deepLookup.found) {	89✔
40	if (deepLookup.path.length > 1) {	26✔
41	const deepPath = deepLookup.path	1✔
42	.map(i => path.relative(path.dirname(context.getFilename()), i.path))	2✔
43	.join(' -> ')
44
45	context.report(im[key],	1✔
46	`${im[key].name} not found via ${deepPath}`)
47	} else {
48	context.report(im[key],	25✔
49	im[key].name + ' not found in \'' + node.source.value + '\'')
50	}
51	}
52	})
53	}
54
55	return {	99✔
56	'ImportDeclaration': checkSpecifiers.bind( null
57	, 'imported'
58	, 'ImportSpecifier'
59	),
60
61	'ExportNamedDeclaration': checkSpecifiers.bind( null
62	, 'local'
63	, 'ExportSpecifier'
64	),
65	}
66
67	},
68	}

benmosher / eslint-plugin-import / 24088240

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous