Add diag kind to /detailed_diagnostic response

ycm-core · Nov 5, 2023 · 63a9aaa · 63a9aaa
1 parent 62cfc45
commit 63a9aaa
Show file tree

Hide file tree

Showing 5 changed files with 57 additions and 4 deletions.
diff --git a/ycmd/completers/language_server/language_server_completer.py b/ycmd/completers/language_server/language_server_completer.py
@@ -1670,6 +1670,12 @@ def GetDetailedDiagnostic( self, request_data ):
       distance = _DistanceOfPointToRange( point, diagnostic[ 'range' ] )
       if minimum_distance is None or distance < minimum_distance:
         message = diagnostic[ 'message' ]
+        try:
+          code = diagnostic[ 'code' ]
+          message += f' [{ code }]'
+        except KeyError:
+          pass
+
         if distance == 0:
           break
         minimum_distance = distance

diff --git a/ycmd/tests/clangd/diagnostics_test.py b/ycmd/tests/clangd/diagnostics_test.py
@@ -21,10 +21,12 @@
                        contains_exactly,
                        contains_string,
                        empty,
+                       ends_with,
                        equal_to,
                        has_entries,
                        has_entry,
-                       has_items )
+                       has_items,
+                       is_not )
 from unittest.mock import patch
 from unittest import TestCase
 from pprint import pprint
@@ -398,6 +400,49 @@ def test_Diagnostics_MaximumDiagnosticsNumberExceeded( self, app ):
     ) )
 
 
+  @IsolatedYcmd( { 'max_diagnostics_to_display': 0 } )
+  def test_Diagnostics_MultilineNoKind( self, app ):
+    filepath = PathToTestFile( 'foo.cc' )
+    contents = """int main () {
+const int &&
+        /* */
+    rd = 1;
+rd = 4;
+}
+"""
+    request = { 'contents': contents,
+                'filepath': filepath,
+                'filetype': 'cpp' }
+
+    test = { 'request': request, 'route': '/receive_messages' }
+    response = RunAfterInitialized( app, test )
+
+    pprint( response )
+
+    assert_that( response, contains_exactly(
+      has_entries( { 'diagnostics': has_items(
+        has_entries( {
+          'kind': equal_to( 'INFORMATION' ),
+          'location': LocationMatcher( filepath, 2, 1 ),
+          'location_extent': RangeMatcher( filepath, ( 2, 1 ), ( 4, 11 ) ),
+          'ranges': contains_exactly(
+            RangeMatcher( filepath, ( 2, 1 ), ( 4, 11 ) ) ),
+          'text': is_not( ends_with( ']' ) ), # No diagnostic kind for INFO.
+          'fixit_available': False
+        } ),
+        has_entries( {
+          'kind': equal_to( 'ERROR' ),
+          'location': LocationMatcher( filepath, 5, 4 ),
+          'location_extent': RangeMatcher( filepath, ( 5, 4 ), ( 5, 5 ) ),
+          'ranges': contains_exactly(
+            RangeMatcher( filepath, ( 5, 4 ), ( 5, 5 ) ) ),
+          'text': contains_string( "typecheck_assign_const" ),
+          'fixit_available': False
+        } )
+      ) } )
+    ) )
+
+
   @IsolatedYcmd( { 'max_diagnostics_to_display': 0 } )
   def test_Diagnostics_NoLimitToNumberOfDiagnostics( self, app ):
     filepath = PathToTestFile( 'max_diagnostics.cc' )

diff --git a/ycmd/tests/go/diagnostics_test.py b/ycmd/tests/go/diagnostics_test.py
@@ -74,7 +74,8 @@ def test_Diagnostics_DetailedDiags( self, app ):
                  any_of( has_entry( 'message',
                                     'undeclared name: diagnostics_test' ),
                          has_entry( 'message',
-                                    'undefined: diagnostics_test' ) ) )
+                                    'undefined: diagnostics_test'
+                                    ' [UndeclaredName]' ) ) )
 
 
   @WithRetry()

diff --git a/ycmd/tests/java/diagnostics_test.py b/ycmd/tests/java/diagnostics_test.py
@@ -301,7 +301,8 @@ def test_Diagnostics_DetailedDiags( self, app ):
     results = app.post_json( '/detailed_diagnostic', request_data ).json
     assert_that( results, has_entry(
         'message',
-        'The value of the field TestFactory.Bar.testString is not used' ) )
+        'The value of the field TestFactory.Bar.testString '
+        'is not used [570425421]' ) )
 
 
   @WithRetry()

diff --git a/ycmd/tests/rust/diagnostics_test.py b/ycmd/tests/rust/diagnostics_test.py
@@ -79,7 +79,7 @@ def test_Diagnostics_DetailedDiags( self, app ):
     results = app.post_json( '/detailed_diagnostic', request_data ).json
     assert_that( results, has_entry(
         'message',
-        'no field `build_` on type `test::Builder`\nunknown field' ) )
+        'no field `build_` on type `test::Builder`\nunknown field [E0609]' ) )
 
 
   @WithRetry()