Skip to content

Commit

Permalink
tools,test: remove unused config hooks
Browse files Browse the repository at this point in the history
Removes a couple of unused/empty functions inside of `tools/test.py`

PR-URL: #22010
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
maclover7 authored and targos committed Aug 1, 2018
1 parent b5ae339 commit 02e665c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
1 change: 0 additions & 1 deletion test/pseudo-tty/testcfg.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,6 @@ def RunCommand(self, command, env):
self.context.GetTimeout(self.mode),
env,
True)
self.Cleanup()
return test.TestOutput(self,
full_command,
output,
Expand Down
13 changes: 0 additions & 13 deletions tools/test.py
Original file line number Diff line number Diff line change
Expand Up @@ -517,21 +517,12 @@ def RunCommand(self, command, env):
self.context.GetTimeout(self.mode),
env,
disable_core_files = self.disable_core_files)
self.Cleanup()
return TestOutput(self,
full_command,
output,
self.context.store_unexpected_output)

def BeforeRun(self):
pass

def AfterRun(self, result):
pass

def Run(self):
self.BeforeRun()

try:
result = self.RunCommand(self.GetCommand(), {
"TEST_THREAD_ID": "%d" % self.thread_id,
Expand All @@ -547,12 +538,8 @@ def Run(self):
from os import O_NONBLOCK
for fd in 0,1,2: fcntl(fd, F_SETFL, ~O_NONBLOCK & fcntl(fd, F_GETFL))

self.AfterRun(result)
return result

def Cleanup(self):
return


class TestOutput(object):

Expand Down

0 comments on commit 02e665c

Please sign in to comment.