ale/test/command_callback/test_cs_mcs_command_callbacks.vader

Before:
  call ale#assert#SetUpLinterTest('cs', 'mcs')

After:
  call ale#assert#TearDownLinterTest()

Execute(The default command should be correct):
  AssertLinter 'mcs', 'mcs -unsafe --parse %t'

Execute(The options should be be used in the command):
  let b:ale_cs_mcs_options = '-pkg:dotnet'

  AssertLinter 'mcs', 'mcs -unsafe --parse -pkg:dotnet %t'
Implemented review recommendations Implements suggestions and recommendations suggested by the first review of the "Advance C# linter based on mcs -t:module (#952)" pull request. - Clarifies and simplifies description of linters and options - Added links to help file and marked the mcsc linter as to be run only when file in buffer is saved or loaded. - Added comments to the mcsc.vim file to clarify code - removed type checks considered not necessary be reviewer. - addresses findings by vader - removed call to getcwd and cd in vim script - handler expands file names relative to route of source tree into absolute pathes. Fixes errors not being marked when vim is started from subdirectory of source tree. - implements tests for mcs.vim and mcsc.vim linter 2017-09-26 07:19:53 +00:00			`Before:`
Massively reduce the amount of code needed for linter tests 2018-07-15 17:24:53 +00:00			`call ale#assert#SetUpLinterTest('cs', 'mcs')`
Implemented review recommendations Implements suggestions and recommendations suggested by the first review of the "Advance C# linter based on mcs -t:module (#952)" pull request. - Clarifies and simplifies description of linters and options - Added links to help file and marked the mcsc linter as to be run only when file in buffer is saved or loaded. - Added comments to the mcsc.vim file to clarify code - removed type checks considered not necessary be reviewer. - addresses findings by vader - removed call to getcwd and cd in vim script - handler expands file names relative to route of source tree into absolute pathes. Fixes errors not being marked when vim is started from subdirectory of source tree. - implements tests for mcs.vim and mcsc.vim linter 2017-09-26 07:19:53 +00:00
			`After:`
Massively reduce the amount of code needed for linter tests 2018-07-15 17:24:53 +00:00			`call ale#assert#TearDownLinterTest()`
Implemented review recommendations Implements suggestions and recommendations suggested by the first review of the "Advance C# linter based on mcs -t:module (#952)" pull request. - Clarifies and simplifies description of linters and options - Added links to help file and marked the mcsc linter as to be run only when file in buffer is saved or loaded. - Added comments to the mcsc.vim file to clarify code - removed type checks considered not necessary be reviewer. - addresses findings by vader - removed call to getcwd and cd in vim script - handler expands file names relative to route of source tree into absolute pathes. Fixes errors not being marked when vim is started from subdirectory of source tree. - implements tests for mcs.vim and mcsc.vim linter 2017-09-26 07:19:53 +00:00
Massively reduce the amount of code needed for linter tests 2018-07-15 17:24:53 +00:00			`Execute(The default command should be correct):`
			`AssertLinter 'mcs', 'mcs -unsafe --parse %t'`
Implemented review recommendations Implements suggestions and recommendations suggested by the first review of the "Advance C# linter based on mcs -t:module (#952)" pull request. - Clarifies and simplifies description of linters and options - Added links to help file and marked the mcsc linter as to be run only when file in buffer is saved or loaded. - Added comments to the mcsc.vim file to clarify code - removed type checks considered not necessary be reviewer. - addresses findings by vader - removed call to getcwd and cd in vim script - handler expands file names relative to route of source tree into absolute pathes. Fixes errors not being marked when vim is started from subdirectory of source tree. - implements tests for mcs.vim and mcsc.vim linter 2017-09-26 07:19:53 +00:00
			`Execute(The options should be be used in the command):`
			`let b:ale_cs_mcs_options = '-pkg:dotnet'`

Massively reduce the amount of code needed for linter tests 2018-07-15 17:24:53 +00:00			`AssertLinter 'mcs', 'mcs -unsafe --parse -pkg:dotnet %t'`