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

Unified Diff: PRESUBMIT_test.py

Issue 2296783002: Adds new logging type SYSLOG which logs to the system log. (Closed)
Patch Set: Copy/paste error. Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« PRESUBMIT.py ('K') | « PRESUBMIT.py ('k') | PRESUBMIT_test_mocks.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: PRESUBMIT_test.py
diff --git a/PRESUBMIT_test.py b/PRESUBMIT_test.py
index c45378b7f53f46211dc2ff42822e7cf4c738c14f..da00dbe6de573206bf60321e8135c30cd74f8160 100755
--- a/PRESUBMIT_test.py
+++ b/PRESUBMIT_test.py
@@ -866,6 +866,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
checker_for_tests=self.checker)
def testAddedPydep(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile('new.pydeps', [], action='A'),
]
@@ -875,6 +879,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
self.assertTrue('PYDEPS_FILES' in str(results[0]))
def testRemovedPydep(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile(PRESUBMIT._ALL_PYDEPS_FILES[0], [], action='D'),
]
@@ -884,6 +892,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
self.assertTrue('PYDEPS_FILES' in str(results[0]))
def testRandomPyIgnored(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile('random.py', []),
]
@@ -892,6 +904,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
self.assertEqual(0, len(results), 'Unexpected results: %r' % results)
def testRelevantPyNoChange(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile('A.py', []),
]
@@ -906,6 +922,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
self.assertEqual(0, len(results), 'Unexpected results: %r' % results)
def testRelevantPyOneChange(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile('A.py', []),
]
@@ -921,6 +941,10 @@ class PydepsNeedsUpdatingTest(unittest.TestCase):
self.assertTrue('File is stale' in str(results[0]))
def testRelevantPyTwoChanges(self):
+ # PRESUBMIT._CheckPydepsNeedsUpdating is only implemented for Android.
+ if self.mock_input_api.platform != 'linux2':
+ return []
+
self.mock_input_api.files = [
MockAffectedFile('C.py', []),
]
« PRESUBMIT.py ('K') | « PRESUBMIT.py ('k') | PRESUBMIT_test_mocks.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698