24773884

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/7de1c31687845dc23d78652e171ea4c4a2fed160">7de1c3168<a href="https://github.com/benmosher/eslint-plugin-import/commit/7de1c31687845dc23d78652e171ea4c4a2fed160">">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/caae65c57b309daac7c54bc5855bdf758d9c198e">caae65c57</a>">caae65c57</a><a href="https://github.com/benmosher/eslint-plugin-import/commit/7de1c31687845dc23d78652e171ea4c4a2fed160"> into eddcfa9ff">eddcfa9ff</a>

Run Details

1310 of 1436 branches covered (91.23%)

30 of 30 new or added lines in 3 files covered. (100.0%)

1906 of 2002 relevant lines covered (95.2%)

49.93 hits per line

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

93.18

/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/exports
      if (node.source == null || node.importKind === 'type' ||
          node.importKind === 'typeof'  || node.exportKind === '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' || im.importKind === 'typeof') 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/exports
16	if (node.source == null \|\| node.importKind === 'type' \|\|	135✔
17	node.importKind === 'typeof' \|\| node.exportKind === 'type') {
18	return	14✔
19	}
20
21	if (!node.specifiers	121✔
22	.some(function (im) { return im.type === type })) {	123✔
23	return // no named imports/exports	6✔
24	}
25
26	const imports = Exports.get(node.source.value, context)	115✔
27	if (imports == null) return	115✔
28
29	if (imports.errors.length) {	105!
30	imports.reportErrors(context, node)	×
31	return	×
32	}
33
34	node.specifiers.forEach(function (im) {	105✔
35	if (im.type !== type) return	132✔
36
37	// ignore type imports
38	if (im.importKind === 'type' \|\| im.importKind === 'typeof') return	129✔
39
40	const deepLookup = imports.hasDeep(im[key].name)	126✔
41
42	if (!deepLookup.found) {	126✔
43	if (deepLookup.path.length > 1) {	34✔
44	const deepPath = deepLookup.path	1✔
45	.map(i => path.relative(path.dirname(context.getFilename()), i.path))	2✔
46	.join(' -> ')
47
48	context.report(im[key],	1✔
49	`${im[key].name} not found via ${deepPath}`)
50	} else {
51	context.report(im[key],	33✔
52	im[key].name + ' not found in \'' + node.source.value + '\'')
53	}
54	}
55	})
56	}
57
58	return {	142✔
59	'ImportDeclaration': checkSpecifiers.bind( null
60	, 'imported'
61	, 'ImportSpecifier'
62	),
63
64	'ExportNamedDeclaration': checkSpecifiers.bind( null
65	, 'local'
66	, 'ExportSpecifier'
67	),
68	}
69
70	},
71	}

benmosher / eslint-plugin-import / 24773884

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