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

Side by Side Diff: appengine/findit/crash/test/findit_test.py

Issue 2605943002: Removing the mutation in the factories for getting dep repositories (Closed)
Patch Set: Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import copy 5 import copy
6 import logging 6 import logging
7 7
8 from google.appengine.api import app_identity 8 from google.appengine.api import app_identity
9 9
10 from crash.findit import Findit 10 from crash.findit import Findit
11 from crash.type_enums import CrashClient 11 from crash.type_enums import CrashClient
12 from crash.test.crash_pipeline_test import DummyCrashData 12 from crash.test.crash_pipeline_test import DummyCrashData
13 from crash.test.predator_testcase import PredatorTestCase 13 from crash.test.predator_testcase import PredatorTestCase
14 from model.crash.fracas_crash_analysis import FracasCrashAnalysis 14 from model.crash.fracas_crash_analysis import FracasCrashAnalysis
15 15
16 MOCK_REPOSITORY = None 16
17 MOCK_GET_REPOSITORY = lambda _: None # pragma: no cover
18
17 19
18 class UnsupportedClient(Findit): # pylint: disable=W0223 20 class UnsupportedClient(Findit): # pylint: disable=W0223
19 # TODO(http://crbug.com/659346): this isn't being called for some reason. 21 # TODO(http://crbug.com/659346): this isn't being called for some reason.
20 @property 22 @property
21 def client_id(self): # pragma: no cover 23 def client_id(self): # pragma: no cover
22 return self._client_id 24 return self._client_id
23 25
24 @property 26 @property
25 def config(self): # pragma: no cover 27 def config(self): # pragma: no cover
26 """Don't return None, so that PlatformRename doesn't crash.""" 28 """Don't return None, so that PlatformRename doesn't crash."""
27 return {} 29 return {}
28 30
29 def __init__(self, client_id=None): 31 def __init__(self, client_id=None):
30 super(UnsupportedClient, self).__init__(MOCK_REPOSITORY) 32 super(UnsupportedClient, self).__init__(MOCK_GET_REPOSITORY)
31 if client_id is None: 33 if client_id is None:
32 client_id = 'unsupported_client' 34 client_id = 'unsupported_client'
33 self._client_id = client_id 35 self._client_id = client_id
34 36
35 37
36 class MockFindit(Findit): # pylint: disable = W 38 class MockFindit(Findit): # pylint: disable = W
37 """Overwrite abstract method of Findit for testing.""" 39 """Overwrite abstract method of Findit for testing."""
38 40
39 def __init__(self): 41 def __init__(self):
40 super(MockFindit, self).__init__(MOCK_REPOSITORY) 42 super(MockFindit, self).__init__(MOCK_GET_REPOSITORY)
41 43
42 @classmethod 44 @classmethod
43 def _ClientID(cls): 45 def _ClientID(cls):
44 return CrashClient.FRACAS 46 return CrashClient.FRACAS
45 47
46 def ProcessResultForPublishing(self, result, key): 48 def ProcessResultForPublishing(self, result, key):
47 return result 49 return result
48 50
49 51
50 class FinditTest(PredatorTestCase): 52 class FinditTest(PredatorTestCase):
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 'client_id': self.findit.client_id, 122 'client_id': self.findit.client_id,
121 'result': copy.deepcopy(analysis_result), 123 'result': copy.deepcopy(analysis_result),
122 } 124 }
123 125
124 analysis = FracasCrashAnalysis.Create(crash_identifiers) 126 analysis = FracasCrashAnalysis.Create(crash_identifiers)
125 analysis.result = analysis_result 127 analysis.result = analysis_result
126 128
127 self.assertDictEqual(self.findit.GetPublishableResult(crash_identifiers, 129 self.assertDictEqual(self.findit.GetPublishableResult(crash_identifiers,
128 analysis), 130 analysis),
129 expected_processed_result) 131 expected_processed_result)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698