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

Side by Side Diff: appengine/findit/crash/findit_for_clusterfuzz.py

Issue 2605943002: Removing the mutation in the factories for getting dep repositories (Closed)
Patch Set: Added the Factory method to CachedGitilesRepository 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 logging 5 import logging
6 6
7 from google.appengine.ext import ndb 7 from google.appengine.ext import ndb
8 8
9 from crash.findit import Findit 9 from crash.findit import Findit
10 from crash.type_enums import CrashClient 10 from crash.type_enums import CrashClient
11 11
12 # TODO(katesonia): Implement this class. 12 # TODO(katesonia): Implement this class.
13 class FinditForClusterfuzz(Findit): # pragma: no cover 13 class FinditForClusterfuzz(Findit): # pragma: no cover
14 @classmethod 14 @classmethod
15 def _ClientID(cls): 15 def _ClientID(cls):
16 return CrashClient.CLUSTERFUZZ 16 return CrashClient.CLUSTERFUZZ
17 17
18 def __init__(self, repository): 18 def __init__(self, get_repository):
19 super(FinditForClusterfuzz, self).__init__(repository) 19 super(FinditForClusterfuzz, self).__init__(get_repository)
20 logging.info('Client %s is not supported by findit right now', 20 logging.info('Client %s is not supported by findit right now',
21 self.client_id) 21 self.client_id)
22 raise NotImplementedError() 22 raise NotImplementedError()
23 23
24 def CreateAnalysis(self, crash_identifiers): 24 def CreateAnalysis(self, crash_identifiers):
25 raise NotImplementedError() 25 raise NotImplementedError()
26 26
27 def GetAnalysis(self, crash_identifiers): 27 def GetAnalysis(self, crash_identifiers):
28 raise NotImplementedError() 28 raise NotImplementedError()
29 29
30 def _InitializeAnalysis(self, model, crash_data): 30 def _InitializeAnalysis(self, model, crash_data):
31 super(FinditForClusterfuzz, self)._InitializeAnalysis(model, crash_data) 31 super(FinditForClusterfuzz, self)._InitializeAnalysis(model, crash_data)
32 raise NotImplementedError() 32 raise NotImplementedError()
33 33
34 @ndb.transactional 34 @ndb.transactional
35 def _NeedsNewAnalysis(self, crash_data): 35 def _NeedsNewAnalysis(self, crash_data):
36 raise NotImplementedError() 36 raise NotImplementedError()
37 37
38 def CheckPolicy(self, crash_data): 38 def CheckPolicy(self, crash_data):
39 return crash_data 39 return crash_data
40 40
41 def FindCulprit(self, model): 41 def FindCulprit(self, model):
42 raise NotImplementedError() 42 raise NotImplementedError()
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698