Mojo: Mojom: Detect (lexically) bad or missing ordinals.

mojo/public/bindings/pylib/parse/mojo_parser_unittest.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import mojo_lexer
 import mojo_parser
 import unittest
 
 
 class MojoParserTest(unittest.TestCase):
@@ skipping 65 matching lines @@
   MY_ENUM_6 = -6 / -2,
   MY_ENUM_7 = 3 | (1 << 2),
   MY_ENUM_8 = 16 >> 1,
   MY_ENUM_9 = 1 ^ 15 & 8,
   MY_ENUM_10 = 110 % 100,
   MY_ENUM_MINUS_1 = ~0
 };
 
 }  // my_module
 """
-    self.maxDiff = 2000
     expected = \
 [('MODULE',
   'my_module',
   [('ENUM',
     'MyEnum',
     [('ENUM_FIELD', 'MY_ENUM_1', ('EXPRESSION', ['1'])),
      ('ENUM_FIELD', 'MY_ENUM_2', ('EXPRESSION', ['1', '+', '1'])),
      ('ENUM_FIELD', 'MY_ENUM_3', ('EXPRESSION', ['1', '*', '3'])),
      ('ENUM_FIELD',
       'MY_ENUM_4',
@@ skipping 29 matching lines @@
   MY_ENUM_1 = 1 ? 2 : 3
 };
 
 }  // my_module
 """
     with self.assertRaisesRegexp(
         mojo_lexer.LexError,
         r"^my_file\.mojom:4: Error: Illegal character '\?'$"):
       mojo_parser.Parse(source, "my_file.mojom")
 
+  def testSimpleOrdinals(self):
+    """Tests that (valid) ordinal values are scanned correctly."""
+    source = """\
+module my_module {
+
+// This isn't actually valid .mojom, but the problem (missing ordinals) should
+// be handled at a different level.
+struct MyStruct {
+  int32 a0 @0;
+  int32 a1 @1;
+  int32 a2 @2;
+  int32 a9 @9;
+  int32 a10 @10;
+  int32 a11 @11;
+  int32 a29 @29;
+  int32 a1234567890 @1234567890;
+};
+
+}  // module my_module
+"""
+    expected = \
+[('MODULE',
+  'my_module',
+  [('STRUCT',
+    'MyStruct',
+    None,
+    [('FIELD', 'int32', 'a0', '@0', None),
+     ('FIELD', 'int32', 'a1', '@1', None),
+     ('FIELD', 'int32', 'a2', '@2', None),
+     ('FIELD', 'int32', 'a9', '@9', None),
+     ('FIELD', 'int32', 'a10', '@10', None),
+     ('FIELD', 'int32', 'a11', '@11', None),
+     ('FIELD', 'int32', 'a29', '@29', None),
+     ('FIELD', 'int32', 'a1234567890', '@1234567890', None)])])]
+    self.assertEquals(mojo_parser.Parse(source, "my_file.mojom"), expected)
+
+  def testInvalidOrdinals(self):
+    """Tests that (lexically) invalid ordinals are correctly detected."""
+    source1 = """\
+module my_module {
+
+struct MyStruct {
+  int32 a_missing @;
+};
+
+}  // module my_module
+"""
+    with self.assertRaisesRegexp(
+        mojo_lexer.LexError,
+        r"^my_file\.mojom:4: Error: Missing ordinal value$"):
+      mojo_parser.Parse(source1, "my_file.mojom")
+
+    source2 = """\
+module my_module {
+
+struct MyStruct {
+  int32 a_octal @01;
+};
+
+}  // module my_module
+"""
+    with self.assertRaisesRegexp(
+        mojo_lexer.LexError,
+        r"^my_file\.mojom:4: Error: "
+            r"Octal and hexadecimal ordinal values not allowed$"):
+      mojo_parser.Parse(source2, "my_file.mojom")
+
+    source3 = """\
+module my_module { struct MyStruct { int32 a_invalid_octal @08; }; }
+"""
+    with self.assertRaisesRegexp(
+        mojo_lexer.LexError,
+        r"^my_file\.mojom:1: Error: "
+            r"Octal and hexadecimal ordinal values not allowed$"):
+      mojo_parser.Parse(source3, "my_file.mojom")
+
+    source4 = """\
+module my_module { struct MyStruct { int32 a_hex @0x1aB9; }; }
+"""
+    with self.assertRaisesRegexp(
+        mojo_lexer.LexError,
+        r"^my_file\.mojom:1: Error: "
+            r"Octal and hexadecimal ordinal values not allowed$"):
+      mojo_parser.Parse(source4, "my_file.mojom")
+
+    source5 = """\
+module my_module { struct MyStruct { int32 a_hex @0X0; }; }
+"""
+    with self.assertRaisesRegexp(
+        mojo_lexer.LexError,
+        r"^my_file\.mojom:1: Error: "
+            r"Octal and hexadecimal ordinal values not allowed$"):
+      mojo_parser.Parse(source5, "my_file.mojom")
 
 if __name__ == "__main__":
   unittest.main()