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

Unified Diff: appengine/findit/model/flake/test/flake_analysis_request_test.py

Issue 2394013002: [Findit] Hacky solution to map a CQ trybot step to a Waterfall buildbot step. (Closed)
Patch Set: Fix nit. Created 4 years, 2 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/model/flake/test/flake_analysis_request_test.py
diff --git a/appengine/findit/model/flake/test/flake_analysis_request_test.py b/appengine/findit/model/flake/test/flake_analysis_request_test.py
index 531d4167bba7c7c16c2101710558b9851872ac25..63e58eea46684309f55ffb9b69a5f79314e56513 100644
--- a/appengine/findit/model/flake/test/flake_analysis_request_test.py
+++ b/appengine/findit/model/flake/test/flake_analysis_request_test.py
@@ -22,6 +22,15 @@ class FlakeAnalysisRequestTest(unittest.TestCase):
self.assertEqual(expected_name,
BuildStep._StripMasterPrefix(original_name))
+ def testBuildStapHasMatchingWaterfallStep(self):
+ build_step = BuildStep.Create('m', 'b', 0, 's', datetime.utcnow())
+ self.assertFalse(build_step.has_matching_waterfall_step)
+ build_step.wf_master_name = 'm'
+ build_step.wf_builder_name = 'b'
+ build_step.wf_build_number = 0
+ build_step.wf_step_name = 's'
+ self.assertTrue(build_step.has_matching_waterfall_step)
+
def testAddBuildStep(self):
t1 = datetime(2016, 10, 1, 0, 0, 0)
t2 = datetime(2016, 10, 2, 0, 0, 0)
« no previous file with comments | « appengine/findit/model/flake/flake_analysis_request.py ('k') | appengine/findit/waterfall/flake/step_mapper.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698