Tidy up some logging in the tests

Author: puremourning

ycmd/completers/completer_utils.py

@@ -18,7 +18,7 @@
 # Must not import ycm_core here! Vim imports completer, which imports this file.
 # We don't want ycm_core inside Vim.
 from collections import defaultdict
-from ycmd.utils import LOGGER, ToUnicode, re, ReadFile, SplitLines
+from ycmd.utils import ToUnicode, re, ReadFile, SplitLines
 
 
 class PreparedTriggers:
@@ -227,7 +227,6 @@ def GetFileContents( request_data, filename ):
   try:
     return ToUnicode( ReadFile( filename ) )
   except ( OSError, UnicodeError ):
-    LOGGER.warning( 'Error reading file "%s"', filename )
     return ''
 
 

ycmd/completers/language_server/language_server_completer.py

@@ -2023,7 +2023,8 @@ def HandleNotificationInPollThread( self, notification ):
         uri = lsp.FilePathToUri( lsp.UriToFilePath( params[ 'uri' ] ) )
       except lsp.InvalidUriException:
         # Ignore diagnostics for URIs we don't recognise
-        LOGGER.debug( f'Ignoring diagnostics for unrecognized URI: { uri }' )
+        LOGGER.debug(
+          f'Ignoring diagnostics for unrecognized URI: { params[ "uri" ] }' )
         return
       with self._latest_diagnostics_mutex:
         self._latest_diagnostics[ uri ] = params[ 'diagnostics' ]