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

Unified Diff: appengine/findit/lib/gitiles/test/gitiles_repository_test.py

Issue 2521773004: [Culprit-finder] Refactor findit testcase and move it to libs. (Closed)
Patch Set: . Created 4 years, 1 month 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/lib/gitiles/test/gitiles_repository_test.py
diff --git a/appengine/findit/lib/gitiles/test/gitiles_repository_test.py b/appengine/findit/lib/gitiles/test/gitiles_repository_test.py
index ad37f28074832c87b14342f555b8e1abc5e79f2a..1a2c67963743a1e64da49ff04b3d7e6fcb6e2d9c 100644
--- a/appengine/findit/lib/gitiles/test/gitiles_repository_test.py
+++ b/appengine/findit/lib/gitiles/test/gitiles_repository_test.py
@@ -12,6 +12,7 @@ from testing_utils import testing
from lib.gitiles import gitiles_repository
from lib.gitiles.change_log import ChangeLog
from libs.http import retry_http_client
+from libs.testcase import TestCase
COMMIT_MESSAGE = ('Add popover for snapshot canvas log.\n\n'
@@ -254,49 +255,21 @@ DUMMY_CHANGELOG_JSON = {
}
-class HttpClientForGit(retry_http_client.RetryHttpClient):
-
- def __init__(self):
- super(HttpClientForGit, self).__init__()
- self.response_for_url = {}
-
- def SetResponseForUrl(self, url, response):
- self.response_for_url[url] = response
-
- def GetBackoff(self, *_): # pragma: no cover
- """Override to avoid sleep."""
- return 0
-
- def _Get(self, url, *_):
- response = self.response_for_url.get(url)
- if response is None:
- return 404, 'Not Found'
- else:
- return 200, response
-
- def _Post(self, *_): # pragma: no cover
- pass
-
- def _Put(self, *_): # pragma: no cover
- pass
-
-
-class GitRepositoryTest(testing.AppengineTestCase):
+class GitRepositoryTest(TestCase):
def setUp(self):
super(GitRepositoryTest, self).setUp()
- self.http_client_for_git = HttpClientForGit()
+ self.http_client_for_git = self.GetMockHttpClient()
self.repo_url = 'https://repo.test'
- self.git_repo = gitiles_repository.GitilesRepository(self.repo_url,
- self.http_client_for_git)
+ self.git_repo = gitiles_repository.GitilesRepository(
+ self.repo_url, self.http_client_for_git)
def testGitRepositoryPropertySetters(self):
- http_client_for_git = HttpClientForGit()
git_repo = gitiles_repository.GitilesRepository(
- http_client=http_client_for_git)
+ http_client=self.http_client_for_git)
git_repo.repo_url = 'https://repo'
self.assertEqual(git_repo.repo_url, 'https://repo')
- self.assertEqual(git_repo.http_client, http_client_for_git)
+ self.assertEqual(git_repo.http_client, self.http_client_for_git)
def testEndingSlashInRepoUrl(self):
git_repo1 = gitiles_repository.GitilesRepository(
@@ -473,7 +446,6 @@ class GitRepositoryTest(testing.AppengineTestCase):
def testGetWrappedGitRepositoryClass(self):
repo = gitiles_repository.GitilesRepository(
- 'http://repo_url', HttpClientForGit())
+ 'http://repo_url', self.http_client_for_git)
self.assertEqual(repo.repo_url, 'http://repo_url')
- self.assertTrue(isinstance(repo.http_client, HttpClientForGit))

Powered by Google App Engine
This is Rietveld 408576698