Implement specs

[Arcanemagus]

It looks like this linter does not currently have any specs written for it, adding it to the list being maintained here for tracking.

Specs do not need to be super detailed, as they should mainly test whether the package is properly running the linter plugin and parsing its messages. Leave testing of the linter functionality to the author of the linter.

If you need any help feel free to ask!

[florianb]

Fundamental specs implemented.

The whole test-arrangement could be improved f.e. by mocking more the javac-dependency (see #70).

[Arcanemagus]

Thanks!

[florianb]

@Arcanemagus you're welcome - 👍 I'm merging it in with the upcoming release..

[florianb]

@steelbrain - i am still working on the tests and i have a question (which i already posted on Slack), is there a way to obtain the linted messages from within a running atom-environment?

[steelbrain]

Usually the way we test linter providers is,

linter = getProvider()
const results = await linter.lint(textEditor)
expect(results.length).toBe(1)
expect(results[0]).toEqual({type: 'Error', text: 'something', filePath: '/path/to/file'})

[florianb]

Hmm, getProvider() is currently unavailable since i tried to not instantiate the linter on my own but use the instance created by atom. But i guess that's just not possible.

[florianb]

Thanks anyways! I'm going to instantiate the linter again.. 🍬