Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(234)

Unified Diff: third_party/buildbot_7_12/buildbot/process/subunitlogobserver.py

Issue 12207158: Bye bye buildbot 0.7.12. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/buildbot_7_12/buildbot/process/subunitlogobserver.py
diff --git a/third_party/buildbot_7_12/buildbot/process/subunitlogobserver.py b/third_party/buildbot_7_12/buildbot/process/subunitlogobserver.py
deleted file mode 100644
index 32e2afe41eedd452ef45924135ee08b37a310020..0000000000000000000000000000000000000000
--- a/third_party/buildbot_7_12/buildbot/process/subunitlogobserver.py
+++ /dev/null
@@ -1,64 +0,0 @@
-# -*- test-case-name: buildbot.test.test_buildstep -*-
-
-from unittest import TestResult
-
-from buildbot.steps.shell import ShellCommand
-from buildbot.process import buildstep
-
-
-class DiscardStream:
- """A trivial thunk used to discard passthrough content."""
-
- def write(self, bytes):
- pass
-
-
-class SubunitLogObserver(buildstep.LogLineObserver, TestResult):
- """Observe a log that may contain subunit output.
-
- This class extends TestResult to receive the callbacks from the subunit
- parser in the most direct fashion.
- """
-
- def __init__(self):
- buildstep.LogLineObserver.__init__(self)
- TestResult.__init__(self)
- try:
- from subunit import TestProtocolServer
- except ImportError:
- raise ImportError("subunit is not importable, but is required for "
- "SubunitLogObserver support.")
- self.protocol = TestProtocolServer(self, DiscardStream())
-
- def outLineReceived(self, line):
- """Process a received line."""
- # Impedance mismatch: subunit wants lines, observers get lines-no\n
- self.protocol.lineReceived(line + '\n')
-
- def startTest(self, test):
- TestResult.startTest(self, test)
- self.step.setProgress('tests', self.testsRun)
-
- def addError(self, test, err):
- TestResult.addError(self, test, err)
- self.issue()
-
- def addFailure(self, test, err):
- TestResult.addFailure(self, test, err)
- self.issue()
-
- def issue(self):
- """An issue - failing, erroring etc test."""
- self.step.setProgress('tests failed', len(self.failures) + len(self.errors))
-
-
-class SubunitShellCommand(ShellCommand):
- """A ShellCommand that sniffs subunit output.
-
- Ideally not needed, and thus here to be trivially deleted. See issue #615
- """
-
- def __init__(self, *args, **kwargs):
- ShellCommand.__init__(self, *args, **kwargs)
- self.addLogObserver('stdio', SubunitLogObserver())
- self.progressMetrics = self.progressMetrics + ('tests', 'tests failed')
« no previous file with comments | « third_party/buildbot_7_12/buildbot/process/step_twisted2.py ('k') | third_party/buildbot_7_12/buildbot/scheduler.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698