linter-eslint-spec.js

'use babel'

import * as path from 'path'
import * as fs from 'fs'
import { tmpdir } from 'os'
import rimraf from 'rimraf'
import linter from '../lib/main'

const fixturesDir = path.join(__dirname, 'fixtures')

const goodPath = path.join(fixturesDir, 'files', 'good.js')
const badPath = path.join(fixturesDir, 'files', 'bad.js')
const emptyPath = path.join(fixturesDir, 'files', 'empty.js')
const fixPath = path.join(fixturesDir, 'files', 'fix.js')
const configPath = path.join(fixturesDir, 'configs', '.eslintrc.yml')
const importingpath = path.join(fixturesDir,
  'import-resolution', 'nested', 'importing.js')
const badImportPath = path.join(fixturesDir,
  'import-resolution', 'nested', 'badImport.js')
const ignoredPath = path.join(fixturesDir, 'eslintignore', 'ignored.js')
const modifiedIgnorePath = path.join(fixturesDir,
  'modified-ignore-rule', 'foo.js')
const modifiedIgnoreSpacePath = path.join(fixturesDir,
  'modified-ignore-rule', 'foo-space.js')
const endRangePath = path.join(fixturesDir, 'end-range', 'no-unreachable.js')

describe('The eslint provider for Linter', () => {
  const { spawnWorker } = require('../lib/helpers')

  const worker = spawnWorker()
  const lint = linter.provideLinter.call(worker).lint
  const fix = textEditor =>
    worker.worker.request('job', {
      type: 'fix',
      config: atom.config.get('linter-eslint'),
      filePath: textEditor.getPath()
    })

  beforeEach(() => {
    atom.config.set('linter-eslint.disableFSCache', false)
    atom.config.set('linter-eslint.disableEslintIgnore', true)

    waitsForPromise(() =>
      Promise.all([
        atom.packages.activatePackage('language-javascript'),
        atom.packages.activatePackage('linter-eslint'),
      ]).then(() =>
        atom.workspace.open(goodPath)
      )
    )
  })

  describe('checks bad.js and', () => {
    let editor = null
    beforeEach(() => {
      waitsForPromise(() =>
        atom.workspace.open(badPath).then((openEditor) => {
          editor = openEditor
        })
      )
    })

    it('finds at least one message', () => {
      waitsForPromise(() =>
        lint(editor).then(messages => expect(messages.length).toBeGreaterThan(0))
      )
    })

    it('verifies that message', () => {
      waitsForPromise(() =>
        lint(editor).then((messages) => {
          const expected = '<a href=http://eslint.org/docs/rules/no-undef ' +
            'class="badge badge-flexible eslint">no-undef</a> ' +
            '&#39;foo&#39; is not defined.'
          expect(messages[0].type).toBe('Error')
          expect(messages[0].text).not.toBeDefined()
          expect(messages[0].html).toBe(expected)
          expect(messages[0].filePath).toBe(badPath)
          expect(messages[0].range).toEqual([[0, 0], [0, 3]])
          expect(Object.hasOwnProperty.call(messages[0], 'fix')).toBeFalsy()
        })
      )
    })
  })

  it('finds nothing wrong with an empty file', () => {
    waitsForPromise(() =>
      atom.workspace.open(emptyPath).then(editor =>
        lint(editor).then(messages => expect(messages.length).toBe(0))
      )
    )
  })

  it('finds nothing wrong with a valid file', () => {
    waitsForPromise(() =>
      atom.workspace.open(goodPath).then(editor =>
        lint(editor).then(messages => expect(messages.length).toBe(0))
      )
    )
  })

  it('reports the fixes for fixable errors', () => {
    waitsForPromise(() =>
      atom.workspace.open(fixPath).then(editor =>
        lint(editor)
      ).then((messages) => {
        expect(messages[0].fix.range).toEqual([[0, 11], [0, 12]])
        expect(messages[0].fix.newText).toBe('')

        expect(messages[1].fix.range).toEqual([[2, 0], [2, 1]])
        expect(messages[1].fix.newText).toBe('  ')
      })
    )
  })

  describe('when resolving import paths using eslint-plugin-import', () => {
    it('correctly resolves imports from parent', () => {
      waitsForPromise(() =>
        atom.workspace.open(importingpath).then(editor =>
          lint(editor).then(messages => expect(messages.length).toBe(0))
        )
      )
    })
    it('shows a message for an invalid import', () => {
      waitsForPromise(() =>
        atom.workspace.open(badImportPath).then(editor =>
          lint(editor).then((messages) => {
            const expected = '<a href=' +
              'https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md ' +
              'class="badge badge-flexible eslint">import/no-unresolved</a> ' +
              'Unable to resolve path to module &#39;../nonexistent&#39;.'

            expect(messages.length).toBeGreaterThan(0)
            expect(messages[0].type).toBe('Error')
            expect(messages[0].text).not.toBeDefined()
            expect(messages[0].html).toBe(expected)
            expect(messages[0].filePath).toBe(badImportPath)
            expect(messages[0].range).toEqual([[0, 24], [0, 39]])
            expect(Object.hasOwnProperty.call(messages[0], 'fix')).toBeFalsy()
          })
        )
      )
    })
  })

  describe('when a file is specified in an .eslintignore file', () => {
    beforeEach(() => {
      atom.config.set('linter-eslint.disableEslintIgnore', false)
    })
    it('will not give warnings for the file', () => {
      waitsForPromise(() =>
        atom.workspace.open(ignoredPath).then(editor =>
          lint(editor).then(messages => expect(messages.length).toBe(0))
        )
      )
    })
  })

  describe('fixes errors', () => {
    let editor
    let doneCheckingFixes
    let tempFixtureDir
    let tempFixturePath
    let tempConfigPath

    beforeEach(() => {
      waitsForPromise(() => {
        tempFixtureDir = fs.mkdtempSync(tmpdir() + path.sep)
        tempFixturePath = path.join(tempFixtureDir, 'fixed.js')
        tempConfigPath = path.join(tempFixtureDir, '.eslintrc.yaml')
        fs.createReadStream(configPath).pipe(fs.createWriteStream(tempConfigPath))

        return atom.workspace.open(fixPath).then((openEditor) => {
          openEditor.saveAs(tempFixturePath)
          editor = openEditor
        })
      })
    })

    afterEach(() => {
      rimraf.sync(tempFixtureDir)
    })

    it('should fix linting errors', () => {
      function firstLint(textEditor) {
        return lint(textEditor)
          .then((messages) => {
            // The original file has two errors
            expect(messages.length).toBe(2)
            return textEditor
          })
      }
      function makeFixes(textEditor) {
        return fix(textEditor)
          .then((messagesAfterSave) => {
            // Linter reports a successful fix
            expect(messagesAfterSave).toBe('Linter-ESLint: Fix Complete')
          })
      }
      // Create a subscription to watch when the editor changes (from the fix)
      editor.onDidChange(() => {
        lint(editor)
          .then((messagesAfterFixing) => {
            // Note: this fires several times, with only the final time resulting in
            // a non-null messagesAfterFixing.  This is the reason for the check here
            // and for the `waitsFor` which makes sure the expectation is tested.
            if (messagesAfterFixing) {
              // After opening the file again, there are no linting errors
              expect(messagesAfterFixing.length).toBe(0)
              doneCheckingFixes = true
            }
          })
      })

      waitsForPromise(() =>
        firstLint(editor)
          .then(makeFixes)
      )
      waitsFor(
        () => doneCheckingFixes,
        'Messages should be checked after fixing'
      )
    })
  })

  describe('Ignores specified rules when editing', () => {
    const expected = '<a href=http://eslint.org/docs/rules/no-trailing-spaces ' +
      'class="badge badge-flexible eslint">no-trailing-spaces</a> ' +
      'Trailing spaces not allowed.'
    it('does nothing on saved files', () => {
      atom.config.set('linter-eslint.rulesToSilenceWhileTyping', ['no-trailing-spaces'])
      waitsForPromise(() =>
        atom.workspace.open(modifiedIgnoreSpacePath).then(editor =>
          lint(editor).then((messages) => {
            expect(messages.length).toBe(1)
            expect(messages[0].type).toBe('Error')
            expect(messages[0].text).not.toBeDefined()
            expect(messages[0].html).toBe(expected)
            expect(messages[0].filePath).toBe(modifiedIgnoreSpacePath)
            expect(messages[0].range).toEqual([[0, 9], [0, 10]])
          })
        )
      )
    })

    it('works when the file is modified', () => {
      let done

      // Set up an observer to check the editor once it is modified
      waitsForPromise(() =>
        atom.workspace.open(modifiedIgnorePath).then((editor) => {
          editor.onDidChange(() => {
            lint(editor).then((messages) => {
              if (messages) {
                // Verify the space is showing an error
                expect(messages.length).toBe(1)
                expect(messages[0].type).toBe('Error')
                expect(messages[0].text).not.toBeDefined()
                expect(messages[0].html).toBe(expected)
                expect(messages[0].filePath).toBe(modifiedIgnorePath)
                expect(messages[0].range).toEqual([[0, 9], [0, 10]])

                // Enable the option under test
                atom.config.set('linter-eslint.rulesToSilenceWhileTyping', ['no-trailing-spaces'])

                // Check the lint results
                lint(editor).then((newMessages) => {
                  expect(newMessages.length).toBe(0)
                  done = true
                })
              }
            })
          })

          // Verify no error before
          return lint(editor).then(messages =>
            expect(messages.length).toBe(0)
          )

          // Insert a space into the editor
          .then(() => {
            editor.getBuffer().insert([0, 9], ' ')
          })
        })
      )

      waitsFor(
        () => done,
        'Messages should be checked after modifying the buffer'
      )
    })
  })

  it('handles ranges in messages', () =>
    waitsForPromise(() =>
      atom.workspace.open(endRangePath).then(editor =>
        lint(editor).then((messages) => {
          const expected = '<a href=http://eslint.org/docs/rules/no-unreachable ' +
            'class="badge badge-flexible eslint">no-unreachable</a> ' +
            'Unreachable code.'
          expect(messages[0].type).toBe('Error')
          expect(messages[0].text).not.toBeDefined()
          expect(messages[0].html).toBe(expected)
          expect(messages[0].filePath).toBe(endRangePath)
          expect(messages[0].range).toEqual([[5, 2], [6, 15]])
        })
      )
    )
  )
})