diff options
author | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2017-11-20 10:33:36 +0100 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2017-11-22 11:45:12 +0000 |
commit | be59a35641616a4cf23c4a13fa0632624b021c1b (patch) | |
tree | 9da183258bdf9cc413f7562079d25ace6955467f /chromium/PRESUBMIT_test.py | |
parent | d702e4b6a64574e97fc7df8fe3238cde70242080 (diff) | |
download | qtwebengine-chromium-be59a35641616a4cf23c4a13fa0632624b021c1b.tar.gz |
BASELINE: Update Chromium to 62.0.3202.101
Change-Id: I2d5eca8117600df6d331f6166ab24d943d9814ac
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
Diffstat (limited to 'chromium/PRESUBMIT_test.py')
-rwxr-xr-x | chromium/PRESUBMIT_test.py | 143 |
1 files changed, 142 insertions, 1 deletions
diff --git a/chromium/PRESUBMIT_test.py b/chromium/PRESUBMIT_test.py index eac824bf5a8..649872cb9d5 100755 --- a/chromium/PRESUBMIT_test.py +++ b/chromium/PRESUBMIT_test.py @@ -742,7 +742,6 @@ class PydepsNeedsUpdatingTest(unittest.TestCase): self.assertTrue('File is stale' in str(results[0])) self.assertTrue('File is stale' in str(results[1])) - class AndroidDeprecatedTestAnnotationTest(unittest.TestCase): def testCheckAndroidTestAnnotationUsage(self): mock_input_api = MockInputApi() @@ -788,7 +787,95 @@ class AndroidDeprecatedTestAnnotationTest(unittest.TestCase): self.assertTrue('UsedDeprecatedSmokeAnnotation.java:1' in msgs[0].items, 'UsedDeprecatedSmokeAnnotation not found in errors') +class AndroidDeprecatedJUnitFrameworkTest(unittest.TestCase): + def testCheckAndroidTestAnnotationUsage(self): + mock_input_api = MockInputApi() + mock_output_api = MockOutputApi() + + mock_input_api.files = [ + MockAffectedFile('LalaLand.java', [ + 'random stuff' + ]), + MockAffectedFile('CorrectUsage.java', [ + 'import org.junit.ABC', + 'import org.junit.XYZ;', + ]), + MockAffectedFile('UsedDeprecatedJUnit.java', [ + 'import junit.framework.*;', + ]), + MockAffectedFile('UsedDeprecatedJUnitAssert.java', [ + 'import junit.framework.Assert;', + ]), + ] + msgs = PRESUBMIT._CheckAndroidTestJUnitFrameworkImport( + mock_input_api, mock_output_api) + self.assertEqual(1, len(msgs), + 'Expected %d items, found %d: %s' + % (1, len(msgs), msgs)) + self.assertEqual(2, len(msgs[0].items), + 'Expected %d items, found %d: %s' + % (2, len(msgs[0].items), msgs[0].items)) + self.assertTrue('UsedDeprecatedJUnit.java:1' in msgs[0].items, + 'UsedDeprecatedJUnit.java not found in errors') + self.assertTrue('UsedDeprecatedJUnitAssert.java:1' + in msgs[0].items, + 'UsedDeprecatedJUnitAssert not found in errors') +class AndroidJUnitBaseClass(unittest.TestCase): + def testCheckAndroidTestAnnotationUsage(self): + mock_input_api = MockInputApi() + mock_output_api = MockOutputApi() + + mock_input_api.files = [ + MockAffectedFile('LalaLand.java', [ + 'random stuff' + ]), + MockAffectedFile('CorrectTest.java', [ + '@RunWith(ABC.class);' + 'public class CorrectTest {', + '}', + ]), + MockAffectedFile('HistoricallyIncorrectTest.java', [ + 'public class Test extends BaseCaseA {', + '}', + ], old_contents=[ + 'public class Test extends BaseCaseB {', + '}', + ]), + MockAffectedFile('CorrectTestWithInterface.java', [ + '@RunWith(ABC.class);' + 'public class CorrectTest implement Interface {', + '}', + ]), + MockAffectedFile('IncorrectTest.java', [ + 'public class IncorrectTest extends TestCase {', + '}', + ]), + MockAffectedFile('IncorrectTestWithInterface.java', [ + 'public class Test implements X extends BaseClass {', + '}', + ]), + MockAffectedFile('IncorrectTestMultiLine.java', [ + 'public class Test implements X, Y, Z', + ' extends TestBase {', + '}', + ]), + ] + msgs = PRESUBMIT._CheckAndroidTestJUnitInheritance( + mock_input_api, mock_output_api) + self.assertEqual(1, len(msgs), + 'Expected %d items, found %d: %s' + % (1, len(msgs), msgs)) + self.assertEqual(3, len(msgs[0].items), + 'Expected %d items, found %d: %s' + % (3, len(msgs[0].items), msgs[0].items)) + self.assertTrue('IncorrectTest.java:1' in msgs[0].items, + 'IncorrectTest not found in errors') + self.assertTrue('IncorrectTestWithInterface.java:1' + in msgs[0].items, + 'IncorrectTestWithInterface not found in errors') + self.assertTrue('IncorrectTestMultiLine.java:2' in msgs[0].items, + 'IncorrectTestMultiLine not found in errors') class LogUsageTest(unittest.TestCase): @@ -1084,6 +1171,18 @@ class RiskyJsTest(unittest.TestCase): mock_input_api, MockOutputApi()) self.assertEqual(0, len(warnings)) + def testConstLetWarningIos9Code(self): + mock_input_api = MockInputApi() + mock_output_api = MockOutputApi() + + mock_input_api.files = [ + MockAffectedFile('components/blah.js', [" const foo = 'bar';"]), + MockAffectedFile('ui/webui/resources/blah.js', [" let foo = 3;"]), + ] + warnings = PRESUBMIT._CheckForRiskyJsFeatures( + mock_input_api, mock_output_api) + self.assertEqual(2, len(warnings)) + class RelativeIncludesTest(unittest.TestCase): def testThirdPartyNotWebKitIgnored(self): mock_input_api = MockInputApi() @@ -1148,5 +1247,47 @@ class RelativeIncludesTest(unittest.TestCase): mock_input_api, mock_output_api) self.assertEqual(1, len(errors)) + +class MojoManifestOwnerTest(unittest.TestCase): + def testMojoManifestChangeNeedsSecurityOwner(self): + mock_input_api = MockInputApi() + mock_input_api.files = [ + MockAffectedFile('services/goat/manifest.json', + [ + '{', + ' "name": "teleporter",', + ' "display_name": "Goat Teleporter",' + ' "interface_provider_specs": {', + ' }', + '}', + ]) + ] + mock_output_api = MockOutputApi() + errors = PRESUBMIT._CheckIpcOwners( + mock_input_api, mock_output_api) + self.assertEqual(1, len(errors)) + self.assertEqual( + 'Found OWNERS files that need to be updated for IPC security review ' + + 'coverage.\nPlease update the OWNERS files below:', errors[0].message) + + # No warning if already covered by an OWNERS rule. + + def testNonManifestChangesDoNotRequireSecurityOwner(self): + mock_input_api = MockInputApi() + mock_input_api.files = [ + MockAffectedFile('services/goat/species.json', + [ + '[', + ' "anglo-nubian",', + ' "angora"', + ']', + ]) + ] + mock_output_api = MockOutputApi() + errors = PRESUBMIT._CheckIpcOwners( + mock_input_api, mock_output_api) + self.assertEqual([], errors) + + if __name__ == '__main__': unittest.main() |