[Culprit-Finder] Refactor GitilesRepostory to make http_client required argument.

appengine/findit/crash/test/stacktrace_test_suite.py

Index: appengine/findit/crash/test/stacktrace_test_suite.py
diff --git a/appengine/findit/crash/test/stacktrace_test_suite.py b/appengine/findit/crash/test/stacktrace_test_suite.py
index 16fc3455afba19ca76073ed38435b0f78d7237d4..e617f9a5369560a30a01d5288b08ba863581767c 100644
--- a/appengine/findit/crash/test/stacktrace_test_suite.py
+++ b/appengine/findit/crash/test/stacktrace_test_suite.py
@@ -4,8 +4,10 @@
 
 from testing_utils import testing
 
+from crash.test.crash_testcase import CrashTestCase
 
-class StacktraceTestSuite(testing.AppengineTestCase):  #pragma: no cover.
+
+class StacktraceTestSuite(CrashTestCase):  #pragma: no cover.

[wrengr, 2016/11/22 19:11:53]

Is this change related to the rest of the CL?

[Sharu Jiang, 2016/11/22 20:15:38]

Yes, since the GetMockHttpClient is in TestCase in the dependency cl
https://codereview.chromium.org/2521773004#ps20001

But it's better that this change goes to the dependency cl.

 
   def _VerifyTwoStackFramesEqual(self, frame1, frame2):
     self.assertEqual(str(frame1), str(frame2))