Refactoring the server tests to use a helper func

This makes the tests smaller, less repetitive and easier to maintain.

python/ycm/completers/all/identifier_completer.py

@@ -133,6 +133,7 @@ class IdentifierCompleter( GeneralCompleter ):
                                              ToUtf8IfNeeded( filetypes[ 0 ] ),
                                              ToUtf8IfNeeded( filepath ) )
 
+
   def OnFileReadyToParse( self, request_data ):
     self.AddBufferIdentifiers( request_data )
     if 'tag_files' in request_data:

python/ycm/server/tests/basic_test.py

@@ -25,39 +25,38 @@ import bottle
 
 bottle.debug( True )
 
+# 'contents' should be just one line of text
+def RequestDataForFileWithContents( filename, contents ):
+  return {
+    'filetypes': ['foo'],
+    'filepath': filename,
+    'line_value': contents,
+    'file_data': {
+      filename: {
+        'contents': contents,
+        'filetypes': ['foo']
+      }
+    }
+  }
+
 
 def GetCompletions_IdentifierCompleterWorks_test():
   app = TestApp( server.app )
-  event_data = {
+  event_data = RequestDataForFileWithContents( '/foo/bar', 'foo foogoo ba' )
+  event_data.update( {
     'event_name': 'FileReadyToParse',
-    'filetypes': ['foo'],
+  } )
-    'filepath': '/foo/bar',
-    'file_data': {
-      '/foo/bar': {
-        'contents': 'foo foogoo ba',
-        'filetypes': ['foo']
-      }
-    }
-  }
 
   app.post_json( '/event_notification', event_data )
 
-  line_value = 'oo foo foogoo ba'
+  completion_data = RequestDataForFileWithContents( '/foo/bar',
-  completion_data = {
+                                                    'oo foo foogoo ba' )
+  completion_data.update( {
     'query': 'oo',
-    'filetypes': ['foo'],
-    'filepath': '/foo/bar',
     'line_num': 0,
     'column_num': 2,
     'start_column': 0,
-    'line_value': line_value,
+  } )
-    'file_data': {
-      '/foo/bar': {
-        'contents': line_value,
-        'filetypes': ['foo']
-      }
-    }
-  }
 
   eq_( [ BuildCompletionData( 'foo' ),
          BuildCompletionData( 'foogoo' ) ],
@@ -66,37 +65,22 @@ def GetCompletions_IdentifierCompleterWorks_test():
 
 def GetCompletions_IdentifierCompleter_SyntaxKeywordsAdded_test():
   app = TestApp( server.app )
-  event_data = {
+  event_data = RequestDataForFileWithContents( '/foo/bar', '' )
+  event_data.update( {
     'event_name': 'FileReadyToParse',
-    'filetypes': ['foo'],
-    'filepath': '/foo/bar',
-    'file_data': {
-      '/foo/bar': {
-        'contents': '',
-        'filetypes': ['foo']
-      }
-    },
     'syntax_keywords': ['foo', 'bar', 'zoo']
-  }
+  } )
 
   app.post_json( '/event_notification', event_data )
 
-  line_value = 'oo '
+  completion_data = RequestDataForFileWithContents( '/foo/bar',
-  completion_data = {
+                                                    'oo ' )
+  completion_data.update( {
     'query': 'oo',
-    'filetypes': ['foo'],
-    'filepath': '/foo/bar',
     'line_num': 0,
     'column_num': 2,
     'start_column': 0,
-    'line_value': line_value,
+  } )
-    'file_data': {
-      '/foo/bar': {
-        'contents': line_value,
-        'filetypes': ['foo']
-      }
-    }
-  }
 
   eq_( [ BuildCompletionData( 'foo' ),
          BuildCompletionData( 'zoo' ) ],