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

Unified Diff: build/android/gyp/java_cpp_enum_tests.py

Issue 1273173002: Added Network Quality Estimator Real-time interface to Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed enum parser Created 5 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
Index: build/android/gyp/java_cpp_enum_tests.py
diff --git a/build/android/gyp/java_cpp_enum_tests.py b/build/android/gyp/java_cpp_enum_tests.py
index 44f9766c82c98d6c6bab67764d89ee4bc6640282..8a55795b776e570bb3a4e6366cc5b481a2939583 100755
--- a/build/android/gyp/java_cpp_enum_tests.py
+++ b/build/android/gyp/java_cpp_enum_tests.py
@@ -310,20 +310,24 @@ public class ClassName {
def testEnumValueAssignmentNoneDefined(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', None)
- definition.AppendEntry('C', None)
+ definition.AppendEntry('A', None, ['// foo'])
+ definition.AppendEntry('B', None, [])
+ definition.AppendEntry('C', None, ['// bar', '// goo'])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
('B', 1),
('C', 2)]),
definition.entries)
+ self.assertEqual(collections.OrderedDict([('A', ['foo']),
+ ('B', []),
+ ('C', ['bar', 'goo'])]),
+ definition.comments)
def testEnumValueAssignmentAllDefined(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', '1')
- definition.AppendEntry('B', '2')
- definition.AppendEntry('C', '3')
+ definition.AppendEntry('A', '1', [])
+ definition.AppendEntry('B', '2', [])
+ definition.AppendEntry('C', '3', [])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', '1'),
('B', '2'),
@@ -332,10 +336,10 @@ public class ClassName {
def testEnumValueAssignmentReferences(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', 'A')
- definition.AppendEntry('C', None)
- definition.AppendEntry('D', 'C')
+ definition.AppendEntry('A', None, [])
+ definition.AppendEntry('B', 'A', [])
+ definition.AppendEntry('C', None, [])
+ definition.AppendEntry('D', 'C', [])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
('B', 0),
@@ -345,9 +349,9 @@ public class ClassName {
def testEnumValueAssignmentSet(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', '2')
- definition.AppendEntry('C', None)
+ definition.AppendEntry('A', None, [])
+ definition.AppendEntry('B', '2', [])
+ definition.AppendEntry('C', None, [])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
('B', 2),
@@ -356,10 +360,10 @@ public class ClassName {
def testEnumValueAssignmentSetReferences(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', 'A')
- definition.AppendEntry('C', 'B')
- definition.AppendEntry('D', None)
+ definition.AppendEntry('A', None, [])
+ definition.AppendEntry('B', 'A', [])
+ definition.AppendEntry('C', 'B', [])
+ definition.AppendEntry('D', None, [])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
('B', 0),
@@ -369,18 +373,18 @@ public class ClassName {
def testEnumValueAssignmentRaises(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', 'foo')
- definition.AppendEntry('C', None)
+ definition.AppendEntry('A', None, [])
+ definition.AppendEntry('B', 'foo', [])
+ definition.AppendEntry('C', None, [])
with self.assertRaises(Exception):
definition.Finalize()
def testExplicitPrefixStripping(self):
definition = EnumDefinition(original_enum_name='c', enum_package='p')
- definition.AppendEntry('P_A', None)
- definition.AppendEntry('B', None)
- definition.AppendEntry('P_C', None)
- definition.AppendEntry('P_LAST', 'P_C')
+ definition.AppendEntry('P_A', None, [])
+ definition.AppendEntry('B', None, [])
+ definition.AppendEntry('P_C', None, [])
+ definition.AppendEntry('P_LAST', 'P_C', [])
definition.prefix_to_strip = 'P_'
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
@@ -392,10 +396,10 @@ public class ClassName {
def testImplicitPrefixStripping(self):
definition = EnumDefinition(original_enum_name='ClassName',
enum_package='p')
- definition.AppendEntry('CLASS_NAME_A', None)
- definition.AppendEntry('CLASS_NAME_B', None)
- definition.AppendEntry('CLASS_NAME_C', None)
- definition.AppendEntry('CLASS_NAME_LAST', 'CLASS_NAME_C')
+ definition.AppendEntry('CLASS_NAME_A', None, [])
+ definition.AppendEntry('CLASS_NAME_B', None, [])
+ definition.AppendEntry('CLASS_NAME_C', None, [])
+ definition.AppendEntry('CLASS_NAME_LAST', 'CLASS_NAME_C', [])
definition.Finalize()
self.assertEqual(collections.OrderedDict([('A', 0),
('B', 1),
@@ -406,9 +410,9 @@ public class ClassName {
def testImplicitPrefixStrippingRequiresAllConstantsToBePrefixed(self):
definition = EnumDefinition(original_enum_name='Name',
enum_package='p')
- definition.AppendEntry('A', None)
- definition.AppendEntry('B', None)
- definition.AppendEntry('NAME_LAST', None)
+ definition.AppendEntry('A', None, [])
+ definition.AppendEntry('B', None, [])
+ definition.AppendEntry('NAME_LAST', None, [])
definition.Finalize()
self.assertEqual(['A', 'B', 'NAME_LAST'], definition.entries.keys())

Powered by Google App Engine
This is Rietveld 408576698