Issue metadata
Sign in to add a comment
|
[Predator] Broken mocking of CrashWrapperPipeline |
||||||||||||||||||||
Issue descriptionIn ./crash/test/crash_pipeline_test.py, there are three tests which currently fail due to bad mocking. All three tests call _TestRunningAnalysisForResult which in turn calls ScheduleNewAnalysis. The problem is, we need _TestRunningAnalysisForResult to mock the CrashWrapperPipeline class it hands to ScheduleNewAnalysis so that (a) we do not go over the wire, but (b) we test all the logic in CrashWrapperPipeline (and the other pipeline classes it calls). I (wrengr) have no idea how to mock this appropriately. Either stgao or katesonia should be able to do it pretty easily though.
,
Oct 27 2016
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by st...@chromium.org
, Oct 25 2016Status: Assigned (was: Available)