[Predator] Refactor GitRepository to make http_client required argument.
Issue 659791 has been merged into this issue.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/c04e4a7c1d39377b163da6219106b365ca2b3b25 commit c04e4a7c1d39377b163da6219106b365ca2b3b25 Author: katesonia <katesonia@chromium.org> Date: Wed Nov 23 19:31:00 2016 [Culprit-finder] Refactor findit testcase and move it to libs. TBR=stgao@chromium.org, lijeffrey@chromium.org BUG= 659449 Review-Url: https://codereview.chromium.org/2521773004 [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/crash/test/crash_testcase.py [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/crash/test/stacktrace_test_suite.py [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/lib/gitiles/test/gitiles_repository_test.py [rename] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/libs/testcase.py [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/model/crash/test/crash_config_test.py [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/model/flake/test/flake_analysis_request_test.py [modify] https://crrev.com/c04e4a7c1d39377b163da6219106b365ca2b3b25/appengine/findit/waterfall/test/wf_testcase.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/d9cdc8afc8913ea2f73456858bb1bf4d271542ea commit d9cdc8afc8913ea2f73456858bb1bf4d271542ea Author: katesonia <katesonia@chromium.org> Date: Wed Nov 23 21:59:25 2016 [Predator] Move testcase.py from libs/ to gae_libs/ Following https://codereview.chromium.org/2521773004/, since the TestCase includes some appengine local testing set up, it should be in gae_libs/ BUG= 659449 Review-Url: https://codereview.chromium.org/2525003005 [modify] https://crrev.com/d9cdc8afc8913ea2f73456858bb1bf4d271542ea/appengine/findit/crash/test/crash_testcase.py [rename] https://crrev.com/d9cdc8afc8913ea2f73456858bb1bf4d271542ea/appengine/findit/gae_libs/testcase.py [modify] https://crrev.com/d9cdc8afc8913ea2f73456858bb1bf4d271542ea/appengine/findit/lib/gitiles/test/gitiles_repository_test.py [modify] https://crrev.com/d9cdc8afc8913ea2f73456858bb1bf4d271542ea/appengine/findit/model/flake/test/flake_analysis_request_test.py [modify] https://crrev.com/d9cdc8afc8913ea2f73456858bb1bf4d271542ea/appengine/findit/waterfall/test/wf_testcase.py
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/cae4d6478d25b9be3607bb20989713569eae9c49 commit cae4d6478d25b9be3607bb20989713569eae9c49 Author: katesonia <katesonia@chromium.org> Date: Tue Nov 29 19:32:29 2016 [Culprit-Finder] Refactor GitilesRepostory to make http_client required argument. BUG= 659449 Review-Url: https://codereview.chromium.org/2524633002 [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/crash/crash_pipeline.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/crash/test/changelist_classifier_test.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/crash/test/findit_for_chromecrash_test.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/handlers/crash/crash_handler.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/handlers/help_triage.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/lib/gitiles/gitiles_repository.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/lib/gitiles/test/gitiles_repository_test.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/waterfall/build_failure_analysis.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/waterfall/extract_deps_info_pipeline.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/waterfall/identify_try_job_culprit_pipeline.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/waterfall/pull_changelog_pipeline.py [modify] https://crrev.com/cae4d6478d25b9be3607bb20989713569eae9c49/appengine/findit/waterfall/send_notification_for_culprit_pipeline.py
Comment 1 by kateso...@chromium.org
, Nov 17 2016