Skip to content

Commit

Permalink
Increase the timeout for the shutdown tests for our CI environment
Browse files Browse the repository at this point in the history
  • Loading branch information
puremourning committed Oct 19, 2017
1 parent 507f00d commit 326bfcc
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions ycmd/tests/shutdown_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@

from ycmd.tests.client_test import Client_test

# Time to wait for all the servers to shutdown. Tweak for the CI environment.
SUBSERVER_SHUTDOWN_TIMEOUT = 10


class Shutdown_test( Client_test ):

Expand All @@ -37,7 +40,7 @@ def FromHandlerWithoutSubserver_test( self ):
response = self.PostRequest( 'shutdown' )
self.AssertResponse( response )
assert_that( response.json(), equal_to( True ) )
self.AssertServersShutDown( timeout = 5 )
self.AssertServersShutDown( timeout = SUBSERVER_SHUTDOWN_TIMEOUT )
self.AssertLogfilesAreRemoved()


Expand All @@ -59,7 +62,7 @@ def FromHandlerWithSubservers_test( self ):
response = self.PostRequest( 'shutdown' )
self.AssertResponse( response )
assert_that( response.json(), equal_to( True ) )
self.AssertServersShutDown( timeout = 5 )
self.AssertServersShutDown( timeout = SUBSERVER_SHUTDOWN_TIMEOUT )
self.AssertLogfilesAreRemoved()


Expand All @@ -68,7 +71,7 @@ def FromWatchdogWithoutSubserver_test( self ):
self.Start( idle_suicide_seconds = 2, check_interval_seconds = 1 )
self.AssertServersAreRunning()

self.AssertServersShutDown( timeout = 5 )
self.AssertServersShutDown( timeout = SUBSERVER_SHUTDOWN_TIMEOUT )
self.AssertLogfilesAreRemoved()


Expand All @@ -87,5 +90,5 @@ def FromWatchdogWithSubservers_test( self ):
self.StartSubserverForFiletype( filetype )
self.AssertServersAreRunning()

self.AssertServersShutDown( timeout = 15 )
self.AssertServersShutDown( timeout = SUBSERVER_SHUTDOWN_TIMEOUT + 10 )
self.AssertLogfilesAreRemoved()

0 comments on commit 326bfcc

Please sign in to comment.