adding a different test for python2

Author: youben11

test/__init__.py

@@ -8,3 +8,4 @@
 unix_only = pytest.mark.skipif(IS_WIN, reason="Unix only")
 windows_only = pytest.mark.skipif(not IS_WIN, reason="Windows only")
 py3_only = pytest.mark.skipif(not IS_PY3, reason="Python3 only")
+py2_only = pytest.mark.skipif(IS_PY3, reason="Python2 only")

test/plugins/test_pylint_lint.py

@@ -3,7 +3,7 @@
 import os
 import tempfile
 
-from test import py3_only
+from test import py2_only, py3_only
 from pyls import lsp, uris
 from pyls.workspace import Document
 from pyls.plugins import pylint_lint
@@ -51,7 +51,7 @@ def test_pylint(config):
 
 
 @py3_only
-def test_syntax_error_pylint(config):
+def test_syntax_error_pylint_py3(config):
     with temp_document(DOC_SYNTAX_ERR) as doc:
         diag = pylint_lint.pyls_lint(config, doc, True)[0]
 
@@ -61,6 +61,17 @@ def test_syntax_error_pylint(config):
         assert diag['severity'] == lsp.DiagnosticSeverity.Error
 
 
+@py2_only
+def test_syntax_error_pylint_py2(config):
+    with temp_document(DOC_SYNTAX_ERR) as doc:
+        diag = pylint_lint.pyls_lint(config, doc, True)[0]
+
+        assert diag['message'].startswith('[syntax-error] invalid syntax')
+        # Pylint doesn't give column numbers for invalid syntax.
+        assert diag['range']['start'] == {'line': 0, 'character': 0}
+        assert diag['severity'] == lsp.DiagnosticSeverity.Error
+
+
 def test_lint_free_pylint(config):
     # Can't use temp_document because it might give us a file that doesn't
     # match pylint's naming requirements. We should be keeping this file clean