Clean up YCM object in PostComplete tests

Move PostComplete tests inside a class that defines setUp and tearDown
methods. Clean YCM object in tearDown method. This fixes the error
"OSError: [WinError 6] The handle is invalid" on Windows with
Python 3.5.

python/ycm/tests/postcomplete_tests.py

@@ -26,13 +26,14 @@ from builtins import *  # noqa
 from ycm.test_utils import MockVimModule
 MockVimModule()
 
-from mock import ( MagicMock, DEFAULT, patch )
+import contextlib
-from nose.tools import eq_
 from hamcrest import assert_that, empty
+from mock import MagicMock, DEFAULT, patch
+from nose.tools import eq_, ok_
+
 from ycm import vimsupport
 from ycm.youcompleteme import YouCompleteMe
 
-import contextlib
 
 def GetVariableValue_CompleteItemIs( word, abbr = None, menu = None,
                                      info = None, kind = None ):
@@ -45,24 +46,11 @@ def GetVariableValue_CompleteItemIs( word, abbr = None, menu = None,
         'info': info,
         'kind': kind,
       }
-    else:
     return DEFAULT
   return MagicMock( side_effect = Result )
 
 
-@contextlib.contextmanager
+def BuildCompletion( namespace = None, insertion_text = 'Test',
-def _SetupForCsharpCompletionDone( completions ):
-  with patch( 'ycm.vimsupport.InsertNamespace' ):
-    with patch( 'ycm.vimsupport.TextBeforeCursor', return_value = '   Test' ):
-      ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
-      request = MagicMock();
-      request.Done = MagicMock( return_value = True )
-      request.RawResponse = MagicMock( return_value = completions )
-      ycm_state._latest_completion_request = request
-      yield ycm_state
-
-
-def _BuildCompletion( namespace = None, insertion_text = 'Test',
                      menu_text = None, extra_menu_info = None,
                      detailed_info = None, kind = None ):
   return {
@@ -75,45 +63,61 @@ def _BuildCompletion( namespace = None, insertion_text = 'Test',
   }
 
 
+class PostComplete_test():
+
+  def setUp( self ):
+    self.ycm = YouCompleteMe( MagicMock( spec_set = dict ) )
+
+
+  def tearDown( self ):
+    self.ycm.OnVimLeave()
+
+
+  @contextlib.contextmanager
+  def _SetupForCsharpCompletionDone( self, completions ):
+    with patch( 'ycm.vimsupport.InsertNamespace' ):
+      with patch( 'ycm.vimsupport.TextBeforeCursor',
+                  return_value = '   Test' ):
+        request = MagicMock()
+        request.Done = MagicMock( return_value = True )
+        request.RawResponse = MagicMock( return_value = completions )
+        self.ycm._latest_completion_request = request
+        yield
+
+
   @patch( 'ycm.vimsupport.CurrentFiletypes', return_value = [ "cs" ] )
-def GetCompleteDoneHooks_ResultOnCsharp_test( *args ):
+  def GetCompleteDoneHooks_ResultOnCsharp_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    result = self.ycm.GetCompleteDoneHooks()
-  result = ycm_state.GetCompleteDoneHooks()
     eq_( 1, len( list( result ) ) )
 
 
   @patch( 'ycm.vimsupport.CurrentFiletypes', return_value = [ "txt" ] )
-def GetCompleteDoneHooks_EmptyOnOtherFiletype_test( *args ):
+  def GetCompleteDoneHooks_EmptyOnOtherFiletype_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    result = self.ycm.GetCompleteDoneHooks()
-  result = ycm_state.GetCompleteDoneHooks()
     eq_( 0, len( list( result ) ) )
 
 
   @patch( 'ycm.vimsupport.CurrentFiletypes', return_value = [ "txt" ] )
-def OnCompleteDone_WithActionCallsIt_test( *args ):
+  def OnCompleteDone_WithActionCallsIt_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
     action = MagicMock()
-  ycm_state._complete_done_hooks[ "txt" ] = action
+    self.ycm._complete_done_hooks[ "txt" ] = action
-  ycm_state.OnCompleteDone()
+    self.ycm.OnCompleteDone()
 
-  assert action.called
+    ok_( action.called )
 
 
   @patch( 'ycm.vimsupport.CurrentFiletypes', return_value = [ "txt" ] )
-def OnCompleteDone_NoActionNoError_test( *args ):
+  def OnCompleteDone_NoActionNoError_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self.ycm.OnCompleteDone()
-
-  ycm_state.OnCompleteDone()
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True )
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( 'Test' ) )
-def FilterToCompletedCompletions_NewVim_MatchIsReturned_test( *args ):
+  def FilterToCompletedCompletions_NewVim_MatchIsReturned_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    completions = [ BuildCompletion( "Test" ) ]
-  completions = [ _BuildCompletion( "Test" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     eq_( list( result ), completions )
 
@@ -121,21 +125,21 @@ def FilterToCompletedCompletions_NewVim_MatchIsReturned_test( *args ):
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True )
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( 'A' ) )
-def FilterToCompletedCompletions_NewVim_ShortTextDoesntRaise_test( *args ):
+  def FilterToCompletedCompletions_NewVim_ShortTextDoesntRaise_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "AAA" ) ]
+    completions = [ BuildCompletion( "AAA" ) ]
 
-  ycm_state._FilterToMatchingCompletions( completions, False )
+    self.ycm._FilterToMatchingCompletions( completions, False )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True )
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( 'Test' ) )
-def FilterToCompletedCompletions_NewVim_ExactMatchIsReturned_test( *args ):
+  def FilterToCompletedCompletions_NewVim_ExactMatchIsReturned_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "Test" ) ]
+    completions = [ BuildCompletion( "Test" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     eq_( list( result ), completions )
 
@@ -143,53 +147,52 @@ def FilterToCompletedCompletions_NewVim_ExactMatchIsReturned_test( *args ):
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True )
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( '   Quote' ) )
-def FilterToCompletedCompletions_NewVim_NonMatchIsntReturned_test( *args ):
+  def FilterToCompletedCompletions_NewVim_NonMatchIsntReturned_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "A" ) ]
+    completions = [ BuildCompletion( "A" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     assert_that( list( result ), empty() )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Test" )
-def FilterToCompletedCompletions_OldVim_MatchIsReturned_test( *args ):
+  def FilterToCompletedCompletions_OldVim_MatchIsReturned_test( self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    completions = [ BuildCompletion( "Test" ) ]
-  completions = [ _BuildCompletion( "Test" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     eq_( list( result ), completions )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "X" )
-def FilterToCompletedCompletions_OldVim_ShortTextDoesntRaise_test( *args ):
+  def FilterToCompletedCompletions_OldVim_ShortTextDoesntRaise_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "AAA" ) ]
+    completions = [ BuildCompletion( "AAA" ) ]
 
-  ycm_state._FilterToMatchingCompletions( completions, False )
+    self.ycm._FilterToMatchingCompletions( completions, False )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "Test" )
-def FilterToCompletedCompletions_OldVim_ExactMatchIsReturned_test( *args ):
+  def FilterToCompletedCompletions_OldVim_ExactMatchIsReturned_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "Test" ) ]
+    completions = [ BuildCompletion( "Test" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     eq_( list( result ), completions )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Quote" )
-def FilterToCompletedCompletions_OldVim_NonMatchIsntReturned_test( *args ):
+  def FilterToCompletedCompletions_OldVim_NonMatchIsntReturned_test( self,
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+                                                                     *args ):
-  completions = [ _BuildCompletion( "A" ) ]
+    completions = [ BuildCompletion( "A" ) ]
 
-  result = ycm_state._FilterToMatchingCompletions( completions, False )
+    result = self.ycm._FilterToMatchingCompletions( completions, False )
 
     assert_that( list( result ), empty() )
 
@@ -197,41 +200,41 @@ def FilterToCompletedCompletions_OldVim_NonMatchIsntReturned_test( *args ):
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Te" )
   def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_MatchIsReturned_test(
-    *args ):
+    self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    completions = [ BuildCompletion( "Test" ) ]
-  completions = [ _BuildCompletion( "Test" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, True )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "X" )
-def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_ShortTextDoesntRaise_test( *args ) :
+  def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_ShortTextDoesntRaise_test(
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self, *args ):
-  completions = [ _BuildCompletion( "AAA" ) ]
+    completions = [ BuildCompletion( "AAA" ) ]
+
+    self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
-  ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "Test" )
-def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_ExactMatchIsntReturned_test( *args ):
+  def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_ExactMatchIsntReturned_test(
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self, *args ):
-  completions = [ _BuildCompletion( "Test" ) ]
+    completions = [ BuildCompletion( "Test" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, False )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Quote" )
-def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_NonMatchIsntReturned_test( *args ):
+  def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_NonMatchIsntReturned_test(
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self, *args ):
-  completions = [ _BuildCompletion( "A" ) ]
+    completions = [ BuildCompletion( "A" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, False )
 
@@ -241,11 +244,10 @@ def HasCompletionsThatCouldBeCompletedWithMoreText_OldVim_NonMatchIsntReturned_t
           GetVariableValue_CompleteItemIs( "Te") )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Quote" )
   def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_MatchIsReturned_test(
-    *args ):
+    self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    completions = [ BuildCompletion( "Test" ) ]
-  completions = [ _BuildCompletion( "Test" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, True )
 
@@ -254,11 +256,11 @@ def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_MatchIsReturned_test(
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( "X") )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = "   Quote" )
-def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_ShortTextDoesntRaise_test( *args ):
+  def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_ShortTextDoesntRaise_test(
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self, *args ):
-  completions = [ _BuildCompletion( "AAA" ) ]
+    completions = [ BuildCompletion( "AAA" ) ]
 
-  ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
 
   @patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True )
@@ -266,11 +268,10 @@ def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_ShortTextDoesntRaise_t
           GetVariableValue_CompleteItemIs( "Test" ) )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = '   Quote' )
   def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_ExactMatchIsntReturned_test(
-    *args ):
+    self, *args ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    completions = [ BuildCompletion( "Test" ) ]
-  completions = [ _BuildCompletion( "Test" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, False )
 
@@ -279,144 +280,146 @@ def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_ExactMatchIsntReturned
   @patch( 'ycm.vimsupport.GetVariableValue',
           GetVariableValue_CompleteItemIs( "   Quote" ) )
   @patch( 'ycm.vimsupport.TextBeforeCursor', return_value = '   Quote' )
-def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_NonMatchIsntReturned_test( *args ):
+  def HasCompletionsThatCouldBeCompletedWithMoreText_NewVim_NonMatchIsntReturned_test(
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    self, *args ):
-  completions = [ _BuildCompletion( "A" ) ]
+    completions = [ BuildCompletion( "A" ) ]
 
-  result = ycm_state._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
+    result = self.ycm._HasCompletionsThatCouldBeCompletedWithMoreText( completions )
 
     eq_( result, False )
 
 
-def GetRequiredNamespaceImport_ReturnNoneForNoExtraData_test():
+  def GetRequiredNamespaceImport_ReturnNoneForNoExtraData_test( self ):
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
+    eq_( None, self.ycm._GetRequiredNamespaceImport( {} ) )
-
-  eq_( None, ycm_state._GetRequiredNamespaceImport( {} ) )
 
 
-def GetRequiredNamespaceImport_ReturnNamespaceFromExtraData_test():
+  def GetRequiredNamespaceImport_ReturnNamespaceFromExtraData_test( self ):
     namespace = "A_NAMESPACE"
-  ycm_state = YouCompleteMe( MagicMock( spec_set = dict ) )
 
-  eq_( namespace, ycm_state._GetRequiredNamespaceImport(
+    eq_( namespace, self.ycm._GetRequiredNamespaceImport(
-    _BuildCompletion( namespace )
+      BuildCompletion( namespace )
     ) )
 
 
-def GetCompletionsUserMayHaveCompleted_ReturnEmptyIfNotDone_test():
+  def GetCompletionsUserMayHaveCompleted_ReturnEmptyIfNotDone_test( self ):
-  with _SetupForCsharpCompletionDone( [] ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( [] ):
-    ycm_state._latest_completion_request.Done = MagicMock( return_value = False )
+      self.ycm._latest_completion_request.Done = MagicMock(
+        return_value = False )
 
-    eq_( [], ycm_state.GetCompletionsUserMayHaveCompleted() )
+      eq_( [], self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def GetCompletionsUserMayHaveCompleted_ReturnEmptyIfPendingMatches_NewVim_test():
+  def GetCompletionsUserMayHaveCompleted_ReturnEmptyIfPendingMatches_NewVim_test(
-  completions = [ _BuildCompletion( None ) ]
+    self ):
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    completions = [ BuildCompletion( None ) ]
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( 'Te' ) ):
-        eq_( [], ycm_state.GetCompletionsUserMayHaveCompleted() )
+          eq_( [], self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
   def GetCompletionsUserMayHaveCompleted_ReturnEmptyIfPendingMatches_OldVim_test(
-    *args ):
+    self, *args ):
-  completions = [ _BuildCompletion( None ) ]
+    completions = [ BuildCompletion( None ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( 'Te' ) ):
-        eq_( [], ycm_state.GetCompletionsUserMayHaveCompleted() )
+          eq_( [], self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
   def GetCompletionsUserMayHaveCompleted_ReturnMatchIfExactMatches_NewVim_test(
-    *args ):
+    self, *args ):
     info = [ "NS", "Test", "Abbr", "Menu", "Info", "Kind" ]
-  completions = [ _BuildCompletion( *info ) ]
+    completions = [ BuildCompletion( *info ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( *info[ 1: ] ) ):
-        eq_( completions, ycm_state.GetCompletionsUserMayHaveCompleted() )
+          eq_( completions, self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def GetCompletionsUserMayHaveCompleted_ReturnMatchIfExactMatchesEvenIfPartial_NewVim_test( *args ):
+  def GetCompletionsUserMayHaveCompleted_ReturnMatchIfExactMatchesEvenIfPartial_NewVim_test(
+    self, *args ):
     info = [ "NS", "Test", "Abbr", "Menu", "Info", "Kind" ]
-  completions = [ _BuildCompletion( *info ),
+    completions = [ BuildCompletion( *info ),
-                  _BuildCompletion( insertion_text = "TestTest" ) ]
+                    BuildCompletion( insertion_text = "TestTest" ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( *info[ 1: ] ) ):
           eq_( [ completions[ 0 ] ],
-             ycm_state.GetCompletionsUserMayHaveCompleted() )
+               self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def GetCompletionsUserMayHaveCompleted_DontReturnMatchIfNontExactMatchesAndPartial_NewVim_test():
+  def GetCompletionsUserMayHaveCompleted_DontReturnMatchIfNontExactMatchesAndPartial_NewVim_test(
+    self ):
     info = [ "NS", "Test", "Abbr", "Menu", "Info", "Kind" ]
-  completions = [ _BuildCompletion( insertion_text = info[ 0 ] ),
+    completions = [ BuildCompletion( insertion_text = info[ 0 ] ),
-                  _BuildCompletion( insertion_text = "TestTest" ) ]
+                    BuildCompletion( insertion_text = "TestTest" ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( *info[ 1: ] ) ):
-        eq_( [], ycm_state.GetCompletionsUserMayHaveCompleted() )
+          eq_( [], self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def GetCompletionsUserMayHaveCompleted_ReturnMatchIfMatches_NewVim_test( *args ):
+  def GetCompletionsUserMayHaveCompleted_ReturnMatchIfMatches_NewVim_test(
-  completions = [ _BuildCompletion( None ) ]
+    self, *args ):
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    completions = [ BuildCompletion( None ) ]
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = True ):
         with patch( 'ycm.vimsupport.GetVariableValue',
                     GetVariableValue_CompleteItemIs( "Test" ) ):
-        eq_( completions, ycm_state.GetCompletionsUserMayHaveCompleted() )
+          eq_( completions, self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def GetCompletionsUserMayHaveCompleted_ReturnMatchIfMatches_OldVim_test( *args ):
+  def GetCompletionsUserMayHaveCompleted_ReturnMatchIfMatches_OldVim_test(
-  completions = [ _BuildCompletion( None ) ]
+    self, *args ):
-  ycm_state = _SetupForCsharpCompletionDone( completions )
+    completions = [ BuildCompletion( None ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False ):
-      eq_( completions, ycm_state.GetCompletionsUserMayHaveCompleted() )
+        eq_( completions, self.ycm.GetCompletionsUserMayHaveCompleted() )
 
 
-def PostCompleteCsharp_EmptyDoesntInsertNamespace_test( *args ):
+  def PostCompleteCsharp_EmptyDoesntInsertNamespace_test( self, *args ):
-  with _SetupForCsharpCompletionDone( [] ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( [] ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False ):
-      ycm_state._OnCompleteDone_Csharp()
+        self.ycm._OnCompleteDone_Csharp()
 
-    assert not vimsupport.InsertNamespace.called
+      ok_( not vimsupport.InsertNamespace.called )
 
 
-def PostCompleteCsharp_ExistingWithoutNamespaceDoesntInsertNamespace_test( *args
+  def PostCompleteCsharp_ExistingWithoutNamespaceDoesntInsertNamespace_test( 
-    ):
+    self, *args ):
-  completions = [ _BuildCompletion( None ) ]
+    completions = [ BuildCompletion( None ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False ):
-      ycm_state._OnCompleteDone_Csharp()
+        self.ycm._OnCompleteDone_Csharp()
 
-    assert not vimsupport.InsertNamespace.called
+      ok_( not vimsupport.InsertNamespace.called )
 
 
-def PostCompleteCsharp_ValueDoesInsertNamespace_test( *args ):
+  def PostCompleteCsharp_ValueDoesInsertNamespace_test( self, *args ):
     namespace = "A_NAMESPACE"
-  completions = [ _BuildCompletion( namespace ) ]
+    completions = [ BuildCompletion( namespace ) ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False ):
-      ycm_state._OnCompleteDone_Csharp()
+        self.ycm._OnCompleteDone_Csharp()
 
         vimsupport.InsertNamespace.assert_called_once_with( namespace )
 
-def PostCompleteCsharp_InsertSecondNamespaceIfSelected_test( *args ):
+  def PostCompleteCsharp_InsertSecondNamespaceIfSelected_test( self, *args ):
     namespace = "A_NAMESPACE"
     namespace2 = "ANOTHER_NAMESPACE"
     completions = [
-    _BuildCompletion( namespace ),
+      BuildCompletion( namespace ),
-    _BuildCompletion( namespace2 ),
+      BuildCompletion( namespace2 ),
     ]
-  with _SetupForCsharpCompletionDone( completions ) as ycm_state:
+    with self._SetupForCsharpCompletionDone( completions ):
       with patch( 'ycm.vimsupport.VimVersionAtLeast', return_value = False ):
         with patch( 'ycm.vimsupport.PresentDialog', return_value = 1 ):
-        ycm_state._OnCompleteDone_Csharp()
+          self.ycm._OnCompleteDone_Csharp()
 
           vimsupport.InsertNamespace.assert_called_once_with( namespace2 )