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

Unified Diff: appengine/findit/waterfall/test/buildbot_test.py

Issue 1149743002: [Findit] Use step level analysis to exclude flaky test failures. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Fix name style nit. Created 5 years, 7 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: appengine/findit/waterfall/test/buildbot_test.py
diff --git a/appengine/findit/waterfall/test/buildbot_test.py b/appengine/findit/waterfall/test/buildbot_test.py
index 99b8c559c0a42ee3c9fd821394e5d4faf5aee5d1..b82f7e48350644c0a1cced5e76e3b679cbc3bb0f 100644
--- a/appengine/findit/waterfall/test/buildbot_test.py
+++ b/appengine/findit/waterfall/test/buildbot_test.py
@@ -3,6 +3,7 @@
# found in the LICENSE file.
from datetime import datetime
+import gzip
import os
import unittest
@@ -120,6 +121,19 @@ class BuildBotTest(unittest.TestCase):
buildbot.CreateStdioLogUrl(
master_name, builder_name, build_number, step_name))
+ def testCreateGtestResultPath(self):
+ master_name = 'a'
+ builder_name = 'Win7 Tests (1)'
+ build_number = 123
+ step_name = '[trigger] abc_tests'
+ expected_stdio_log_url = ('/chrome-gtest-results/buildbot/a/Win7 Tests '
+ '(1)/123/[trigger] abc_tests.json.gz')
+
+ self.assertEqual(
+ expected_stdio_log_url,
+ buildbot.CreateGtestResultPath(
+ master_name, builder_name, build_number, step_name))
+
def testGetBuildDataFromArchiveSuccess(self):
master_name = 'a'
builder_name = 'b c'
« no previous file with comments | « appengine/findit/waterfall/extract_signal_pipeline.py ('k') | appengine/findit/waterfall/test/data/m_b_123_abc_test.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698