Mojo: Reformat parser_unittest.py.

mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py

Index: mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py
diff --git a/mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py b/mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py
index 7e479947b28589a443e13f7f0d399e51655a7c14..e4e92b8e03b2f3881a3797662c6cbc152cc49d36 100644
--- a/mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py
+++ b/mojo/public/tools/bindings/pylib/mojom_tests/parse/parser_unittest.py
@@ -34,28 +34,31 @@ class ParserTest(unittest.TestCase):
 
   def testTrivialValidSource(self):
     """Tests a trivial, but valid, .mojom source."""
+
     source = """\
-// This is a comment.
+        // This is a comment.
 
-module my_module {
-}
-"""
+        module my_module {
+        }
+        """
     self.assertEquals(parser.Parse(source, "my_file.mojom"),
                       [("MODULE", "my_module", None, None)])
 
   def testSourceWithCrLfs(self):
     """Tests a .mojom source with CR-LFs instead of LFs."""
+
     source = "// This is a comment.\r\n\r\nmodule my_module {\r\n}\r\n"
     self.assertEquals(parser.Parse(source, "my_file.mojom"),
                       [("MODULE", "my_module", None, None)])
 
   def testUnexpectedEOF(self):
     """Tests a "truncated" .mojom source."""
+
     source = """\
-// This is a comment.
+        // This is a comment.
 
-module my_module {
-"""
+        module my_module {
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
         r"^my_file\.mojom: Error: Unexpected end of file$"):
@@ -64,195 +67,202 @@ module my_module {
   def testCommentLineNumbers(self):
     """Tests that line numbers are correctly tracked when comments are
     present."""
+
     source1 = """\
-// Isolated C++-style comments.
+        // Isolated C++-style comments.
 
-// Foo.
-asdf1
-"""
+        // Foo.
+        asdf1
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:4: Error: Unexpected 'asdf1':\nasdf1$"):
+        r"^my_file\.mojom:4: Error: Unexpected 'asdf1':\n *asdf1$"):
       parser.Parse(source1, "my_file.mojom")
 
     source2 = """\
-// Consecutive C++-style comments.
-// Foo.
-  // Bar.
+        // Consecutive C++-style comments.
+        // Foo.
+          // Bar.
 
-struct Yada {  // Baz.
-// Quux.
-  int32 x;
-};
+        struct Yada {  // Baz.
+        // Quux.
+          int32 x;
+        };
 
-asdf2
-"""
+        asdf2
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:10: Error: Unexpected 'asdf2':\nasdf2$"):
+        r"^my_file\.mojom:10: Error: Unexpected 'asdf2':\n *asdf2$"):
       parser.Parse(source2, "my_file.mojom")
 
     source3 = """\
-/* Single-line C-style comments. */
-/* Foobar. */
+        /* Single-line C-style comments. */
+        /* Foobar. */
 
-/* Baz. */
-asdf3
-"""
+        /* Baz. */
+        asdf3
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:5: Error: Unexpected 'asdf3':\nasdf3$"):
+        r"^my_file\.mojom:5: Error: Unexpected 'asdf3':\n *asdf3$"):
       parser.Parse(source3, "my_file.mojom")
 
     source4 = """\
-/* Multi-line C-style comments.
-*/
-/*
-Foo.
-Bar.
-*/
-
-/* Baz
-   Quux. */
-asdf4
-"""
+        /* Multi-line C-style comments.
+        */
+        /*
+        Foo.
+        Bar.
+        */
+
+        /* Baz
+           Quux. */
+        asdf4
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:10: Error: Unexpected 'asdf4':\nasdf4$"):
+        r"^my_file\.mojom:10: Error: Unexpected 'asdf4':\n *asdf4$"):
       parser.Parse(source4, "my_file.mojom")
 
 
   def testSimpleStruct(self):
     """Tests a simple .mojom source that just defines a struct."""
+
     source = """\
-module my_module {
+        module my_module {
 
-struct MyStruct {
-  int32 a;
-  double b;
-};
+        struct MyStruct {
+          int32 a;
+          double b;
+        };
 
-}  // module my_module
-"""
+        }  // module my_module
+        """
     expected = \
-[('MODULE',
-  'my_module',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32', 'a', ast.Ordinal(None), None),
-     ('FIELD', 'double', 'b', ast.Ordinal(None), None)])])]
+        [('MODULE',
+          'my_module',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32', 'a', ast.Ordinal(None), None),
+             ('FIELD', 'double', 'b', ast.Ordinal(None), None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testSimpleStructWithoutModule(self):
     """Tests a simple struct without an enclosing module."""
+
     source = """\
-struct MyStruct {
-  int32 a;
-  double b;
-};
-"""
+        struct MyStruct {
+          int32 a;
+          double b;
+        };
+        """
     expected = \
-[('MODULE',
-  '',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32', 'a', ast.Ordinal(None), None),
-     ('FIELD', 'double', 'b', ast.Ordinal(None), None)])])]
+        [('MODULE',
+          '',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32', 'a', ast.Ordinal(None), None),
+             ('FIELD', 'double', 'b', ast.Ordinal(None), None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testMissingModuleName(self):
     """Tests an (invalid) .mojom with a missing module name."""
+
     source1 = """\
-// Missing module name.
-module {
-struct MyStruct {
-  int32 a;
-};
-}
-"""
+        // Missing module name.
+        module {
+        struct MyStruct {
+          int32 a;
+        };
+        }
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:2: Error: Unexpected '{':\nmodule {$"):
+        r"^my_file\.mojom:2: Error: Unexpected '{':\n *module {$"):
       parser.Parse(source1, "my_file.mojom")
 
     # Another similar case, but make sure that line-number tracking/reporting
     # is correct.
     source2 = """\
-module
-// This line intentionally left unblank.
+        module
+        // This line intentionally left unblank.
 
-{
-}
-"""
+        {
+        }
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:4: Error: Unexpected '{':\n{$"):
+        r"^my_file\.mojom:4: Error: Unexpected '{':\n *{$"):
       parser.Parse(source2, "my_file.mojom")
 
   def testEnumInitializers(self):
     """Tests an enum with simple initialized values."""
+
     source = """\
-module my_module {
+        module my_module {
 
-enum MyEnum {
-  MY_ENUM_NEG1 = -1,
-  MY_ENUM_ZERO = 0,
-  MY_ENUM_1 = +1,
-  MY_ENUM_2,
-};
+        enum MyEnum {
+          MY_ENUM_NEG1 = -1,
+          MY_ENUM_ZERO = 0,
+          MY_ENUM_1 = +1,
+          MY_ENUM_2,
+        };
 
-}  // my_module
-"""
+        }  // my_module
+        """
     expected = \
-[('MODULE',
-  'my_module',
-  None,
-  [('ENUM',
-    'MyEnum',
-    [('ENUM_FIELD', 'MY_ENUM_NEG1', '-1'),
-     ('ENUM_FIELD', 'MY_ENUM_ZERO', '0'),
-     ('ENUM_FIELD', 'MY_ENUM_1', '+1'),
-     ('ENUM_FIELD', 'MY_ENUM_2', None)])])]
+        [('MODULE',
+          'my_module',
+          None,
+          [('ENUM',
+            'MyEnum',
+            [('ENUM_FIELD', 'MY_ENUM_NEG1', '-1'),
+             ('ENUM_FIELD', 'MY_ENUM_ZERO', '0'),
+             ('ENUM_FIELD', 'MY_ENUM_1', '+1'),
+             ('ENUM_FIELD', 'MY_ENUM_2', None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testConst(self):
     """Tests some constants and struct memebers initialized with them."""
+
     source = """\
-module my_module {
+        module my_module {
 
-struct MyStruct {
-  const int8 kNumber = -1;
-  int8 number@0 = kNumber;
-};
+        struct MyStruct {
+          const int8 kNumber = -1;
+          int8 number@0 = kNumber;
+        };
 
-}  // my_module
-"""
+        }  // my_module
+        """
     expected = \
-[('MODULE',
-  'my_module',
-  None,
-  [('STRUCT',
-    'MyStruct', None,
-    [('CONST', 'int8', 'kNumber', '-1'),
-     ('FIELD', 'int8', 'number',
-        ast.Ordinal(0), ('IDENTIFIER', 'kNumber'))])])]
+        [('MODULE',
+          'my_module',
+          None,
+          [('STRUCT',
+            'MyStruct', None,
+            [('CONST', 'int8', 'kNumber', '-1'),
+             ('FIELD', 'int8', 'number',
+                ast.Ordinal(0), ('IDENTIFIER', 'kNumber'))])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testNoConditionals(self):
     """Tests that ?: is not allowed."""
+
     source = """\
-module my_module {
+        module my_module {
 
-enum MyEnum {
-  MY_ENUM_1 = 1 ? 2 : 3
-};
+        enum MyEnum {
+          MY_ENUM_1 = 1 ? 2 : 3
+        };
 
-}  // my_module
-"""
+        }  // my_module
+        """
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:4: Error: Illegal character '\?'$"):
@@ -260,66 +270,69 @@ enum MyEnum {
 
   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
-"""
+        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',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32', 'a0', ast.Ordinal(0), None),
-     ('FIELD', 'int32', 'a1', ast.Ordinal(1), None),
-     ('FIELD', 'int32', 'a2', ast.Ordinal(2), None),
-     ('FIELD', 'int32', 'a9', ast.Ordinal(9), None),
-     ('FIELD', 'int32', 'a10', ast.Ordinal(10), None),
-     ('FIELD', 'int32', 'a11', ast.Ordinal(11), None),
-     ('FIELD', 'int32', 'a29', ast.Ordinal(29), None),
-     ('FIELD', 'int32', 'a1234567890', ast.Ordinal(1234567890), None)])])]
+        [('MODULE',
+          'my_module',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32', 'a0', ast.Ordinal(0), None),
+             ('FIELD', 'int32', 'a1', ast.Ordinal(1), None),
+             ('FIELD', 'int32', 'a2', ast.Ordinal(2), None),
+             ('FIELD', 'int32', 'a9', ast.Ordinal(9), None),
+             ('FIELD', 'int32', 'a10', ast.Ordinal(10), None),
+             ('FIELD', 'int32', 'a11', ast.Ordinal(11), None),
+             ('FIELD', 'int32', 'a29', ast.Ordinal(29), None),
+             ('FIELD', 'int32', 'a1234567890', ast.Ordinal(1234567890),
+              None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testInvalidOrdinals(self):
     """Tests that (lexically) invalid ordinals are correctly detected."""
+
     source1 = """\
-module my_module {
+        module my_module {
 
-struct MyStruct {
-  int32 a_missing@;
-};
+        struct MyStruct {
+          int32 a_missing@;
+        };
 
-}  // module my_module
-"""
+        }  // module my_module
+        """
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:4: Error: Missing ordinal value$"):
       parser.Parse(source1, "my_file.mojom")
 
     source2 = """\
-module my_module {
+        module my_module {
 
-struct MyStruct {
-  int32 a_octal@01;
-};
+        struct MyStruct {
+          int32 a_octal@01;
+        };
 
-}  // module my_module
-"""
+        }  // module my_module
+        """
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:4: Error: "
@@ -327,26 +340,22 @@ struct MyStruct {
       parser.Parse(source2, "my_file.mojom")
 
     source3 = """\
-module my_module { struct MyStruct { int32 a_invalid_octal@08; }; }
-"""
+        module my_module { struct MyStruct { int32 a_invalid_octal@08; }; }
+        """
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:1: Error: "
             r"Octal and hexadecimal ordinal values not allowed$"):
       parser.Parse(source3, "my_file.mojom")
 
-    source4 = """\
-module my_module { struct MyStruct { int32 a_hex@0x1aB9; }; }
-"""
+    source4 = "module my_module { struct MyStruct { int32 a_hex@0x1aB9; }; }"
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:1: Error: "
             r"Octal and hexadecimal ordinal values not allowed$"):
       parser.Parse(source4, "my_file.mojom")
 
-    source5 = """\
-module my_module { struct MyStruct { int32 a_hex@0X0; }; }
-"""
+    source5 = "module my_module { struct MyStruct { int32 a_hex@0X0; }; }"
     with self.assertRaisesRegexp(
         lexer.LexError,
         r"^my_file\.mojom:1: Error: "
@@ -354,277 +363,284 @@ module my_module { struct MyStruct { int32 a_hex@0X0; }; }
       parser.Parse(source5, "my_file.mojom")
 
     source6 = """\
-struct MyStruct {
-  int32 a_too_big@999999999999;
-};
-"""
+        struct MyStruct {
+          int32 a_too_big@999999999999;
+        };
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
         r"^my_file\.mojom:2: Error: "
             r"Ordinal value 999999999999 too large:\n"
-            r"  int32 a_too_big@999999999999;$"):
+            r" *int32 a_too_big@999999999999;$"):
       parser.Parse(source6, "my_file.mojom")
 
   def testNestedNamespace(self):
     """Tests that "nested" namespaces work."""
+
     source = """\
-module my.mod {
+        module my.mod {
 
-struct MyStruct {
-  int32 a;
-};
+        struct MyStruct {
+          int32 a;
+        };
 
-}  // module my.mod
-"""
+        }  // module my.mod
+        """
     expected = \
-[('MODULE',
-  'my.mod',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32', 'a', ast.Ordinal(None), None)])])]
+        [('MODULE',
+          'my.mod',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32', 'a', ast.Ordinal(None), None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testValidHandleTypes(self):
     """Tests (valid) handle types."""
+
     source = """\
-struct MyStruct {
-  handle a;
-  handle<data_pipe_consumer> b;
-  handle <data_pipe_producer> c;
-  handle < message_pipe > d;
-  handle
-    < shared_buffer
-    > e;
-};
-"""
+        struct MyStruct {
+          handle a;
+          handle<data_pipe_consumer> b;
+          handle <data_pipe_producer> c;
+          handle < message_pipe > d;
+          handle
+            < shared_buffer
+            > e;
+        };
+        """
     expected = \
-[('MODULE',
-  '',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'handle', 'a', ast.Ordinal(None), None),
-     ('FIELD', 'handle<data_pipe_consumer>', 'b', ast.Ordinal(None), None),
-     ('FIELD', 'handle<data_pipe_producer>', 'c', ast.Ordinal(None), None),
-     ('FIELD', 'handle<message_pipe>', 'd', ast.Ordinal(None), None),
-     ('FIELD', 'handle<shared_buffer>', 'e', ast.Ordinal(None), None)])])]
+        [('MODULE',
+          '',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'handle', 'a', ast.Ordinal(None), None),
+             ('FIELD', 'handle<data_pipe_consumer>', 'b', ast.Ordinal(None),
+              None),
+             ('FIELD', 'handle<data_pipe_producer>', 'c', ast.Ordinal(None),
+              None),
+             ('FIELD', 'handle<message_pipe>', 'd', ast.Ordinal(None), None),
+             ('FIELD', 'handle<shared_buffer>', 'e', ast.Ordinal(None),
+              None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testInvalidHandleType(self):
     """Tests an invalid (unknown) handle type."""
+
     source = """\
-struct MyStruct {
-  handle<wtf_is_this> foo;
-};
-"""
+        struct MyStruct {
+          handle<wtf_is_this> foo;
+        };
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
         r"^my_file\.mojom:2: Error: "
             r"Invalid handle type 'wtf_is_this':\n"
-            r"  handle<wtf_is_this> foo;$"):
+            r" *handle<wtf_is_this> foo;$"):
       parser.Parse(source, "my_file.mojom")
 
   def testValidDefaultValues(self):
     """Tests default values that are valid (to the parser)."""
+
     source = """\
-struct MyStruct {
-  int16 a0 = 0;
-  uint16 a1 = 0x0;
-  uint16 a2 = 0x00;
-  uint16 a3 = 0x01;
-  uint16 a4 = 0xcd;
-  int32 a5 = 12345;
-  int64 a6 = -12345;
-  int64 a7 = +12345;
-  uint32 a8 = 0x12cd3;
-  uint32 a9 = -0x12cD3;
-  uint32 a10 = +0x12CD3;
-  bool a11 = true;
-  bool a12 = false;
-  float a13 = 1.2345;
-  float a14 = -1.2345;
-  float a15 = +1.2345;
-  float a16 = 123.;
-  float a17 = .123;
-  double a18 = 1.23E10;
-  double a19 = 1.E-10;
-  double a20 = .5E+10;
-  double a21 = -1.23E10;
-  double a22 = +.123E10;
-};
-"""
+        struct MyStruct {
+          int16 a0 = 0;
+          uint16 a1 = 0x0;
+          uint16 a2 = 0x00;
+          uint16 a3 = 0x01;
+          uint16 a4 = 0xcd;
+          int32 a5 = 12345;
+          int64 a6 = -12345;
+          int64 a7 = +12345;
+          uint32 a8 = 0x12cd3;
+          uint32 a9 = -0x12cD3;
+          uint32 a10 = +0x12CD3;
+          bool a11 = true;
+          bool a12 = false;
+          float a13 = 1.2345;
+          float a14 = -1.2345;
+          float a15 = +1.2345;
+          float a16 = 123.;
+          float a17 = .123;
+          double a18 = 1.23E10;
+          double a19 = 1.E-10;
+          double a20 = .5E+10;
+          double a21 = -1.23E10;
+          double a22 = +.123E10;
+        };
+        """
     expected = \
-[('MODULE',
-  '',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int16', 'a0', ast.Ordinal(None), '0'),
-     ('FIELD', 'uint16', 'a1', ast.Ordinal(None), '0x0'),
-     ('FIELD', 'uint16', 'a2', ast.Ordinal(None), '0x00'),
-     ('FIELD', 'uint16', 'a3', ast.Ordinal(None), '0x01'),
-     ('FIELD', 'uint16', 'a4', ast.Ordinal(None), '0xcd'),
-     ('FIELD', 'int32', 'a5' , ast.Ordinal(None), '12345'),
-     ('FIELD', 'int64', 'a6', ast.Ordinal(None), '-12345'),
-     ('FIELD', 'int64', 'a7', ast.Ordinal(None), '+12345'),
-     ('FIELD', 'uint32', 'a8', ast.Ordinal(None), '0x12cd3'),
-     ('FIELD', 'uint32', 'a9', ast.Ordinal(None), '-0x12cD3'),
-     ('FIELD', 'uint32', 'a10', ast.Ordinal(None), '+0x12CD3'),
-     ('FIELD', 'bool', 'a11', ast.Ordinal(None), 'true'),
-     ('FIELD', 'bool', 'a12', ast.Ordinal(None), 'false'),
-     ('FIELD', 'float', 'a13', ast.Ordinal(None), '1.2345'),
-     ('FIELD', 'float', 'a14', ast.Ordinal(None), '-1.2345'),
-     ('FIELD', 'float', 'a15', ast.Ordinal(None), '+1.2345'),
-     ('FIELD', 'float', 'a16', ast.Ordinal(None), '123.'),
-     ('FIELD', 'float', 'a17', ast.Ordinal(None), '.123'),
-     ('FIELD', 'double', 'a18', ast.Ordinal(None), '1.23E10'),
-     ('FIELD', 'double', 'a19', ast.Ordinal(None), '1.E-10'),
-     ('FIELD', 'double', 'a20', ast.Ordinal(None), '.5E+10'),
-     ('FIELD', 'double', 'a21', ast.Ordinal(None), '-1.23E10'),
-     ('FIELD', 'double', 'a22', ast.Ordinal(None), '+.123E10')])])]
+        [('MODULE',
+          '',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int16', 'a0', ast.Ordinal(None), '0'),
+             ('FIELD', 'uint16', 'a1', ast.Ordinal(None), '0x0'),
+             ('FIELD', 'uint16', 'a2', ast.Ordinal(None), '0x00'),
+             ('FIELD', 'uint16', 'a3', ast.Ordinal(None), '0x01'),
+             ('FIELD', 'uint16', 'a4', ast.Ordinal(None), '0xcd'),
+             ('FIELD', 'int32', 'a5' , ast.Ordinal(None), '12345'),
+             ('FIELD', 'int64', 'a6', ast.Ordinal(None), '-12345'),
+             ('FIELD', 'int64', 'a7', ast.Ordinal(None), '+12345'),
+             ('FIELD', 'uint32', 'a8', ast.Ordinal(None), '0x12cd3'),
+             ('FIELD', 'uint32', 'a9', ast.Ordinal(None), '-0x12cD3'),
+             ('FIELD', 'uint32', 'a10', ast.Ordinal(None), '+0x12CD3'),
+             ('FIELD', 'bool', 'a11', ast.Ordinal(None), 'true'),
+             ('FIELD', 'bool', 'a12', ast.Ordinal(None), 'false'),
+             ('FIELD', 'float', 'a13', ast.Ordinal(None), '1.2345'),
+             ('FIELD', 'float', 'a14', ast.Ordinal(None), '-1.2345'),
+             ('FIELD', 'float', 'a15', ast.Ordinal(None), '+1.2345'),
+             ('FIELD', 'float', 'a16', ast.Ordinal(None), '123.'),
+             ('FIELD', 'float', 'a17', ast.Ordinal(None), '.123'),
+             ('FIELD', 'double', 'a18', ast.Ordinal(None), '1.23E10'),
+             ('FIELD', 'double', 'a19', ast.Ordinal(None), '1.E-10'),
+             ('FIELD', 'double', 'a20', ast.Ordinal(None), '.5E+10'),
+             ('FIELD', 'double', 'a21', ast.Ordinal(None), '-1.23E10'),
+             ('FIELD', 'double', 'a22', ast.Ordinal(None), '+.123E10')])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testValidFixedSizeArray(self):
     """Tests parsing a fixed size array."""
+
     source = """\
-struct MyStruct {
-  int32[] normal_array;
-  int32[1] fixed_size_array_one_entry;
-  int32[10] fixed_size_array_ten_entries;
-};
-"""
+        struct MyStruct {
+          int32[] normal_array;
+          int32[1] fixed_size_array_one_entry;
+          int32[10] fixed_size_array_ten_entries;
+        };
+        """
     expected = \
-[('MODULE',
-  '',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32[]', 'normal_array', ast.Ordinal(None), None),
-     ('FIELD', 'int32[1]', 'fixed_size_array_one_entry',
-      ast.Ordinal(None), None),
-     ('FIELD', 'int32[10]', 'fixed_size_array_ten_entries',
-      ast.Ordinal(None), None)])])]
+        [('MODULE',
+          '',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32[]', 'normal_array', ast.Ordinal(None), None),
+             ('FIELD', 'int32[1]', 'fixed_size_array_one_entry',
+              ast.Ordinal(None), None),
+             ('FIELD', 'int32[10]', 'fixed_size_array_ten_entries',
+              ast.Ordinal(None), None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testValidNestedArray(self):
     """Tests parsing a nested array."""
-    source = """\
-struct MyStruct {
-  int32[][] nested_array;
-};
-"""
+
+    source = "struct MyStruct { int32[][] nested_array; };"
     expected = \
-[('MODULE',
-  '',
-  None,
-  [('STRUCT',
-    'MyStruct',
-    None,
-    [('FIELD', 'int32[][]', 'nested_array', ast.Ordinal(None), None)])])]
+        [('MODULE',
+          '',
+          None,
+          [('STRUCT',
+            'MyStruct',
+            None,
+            [('FIELD', 'int32[][]', 'nested_array', ast.Ordinal(None),
+              None)])])]
     self.assertEquals(parser.Parse(source, "my_file.mojom"), expected)
 
   def testInvalidFixedArraySize(self):
     """Tests that invalid fixed array bounds are correctly detected."""
+
     source1 = """\
-struct MyStruct {
-  int32[0] zero_size_array;
-};
-"""
+        struct MyStruct {
+          int32[0] zero_size_array;
+        };
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom: Error: Fixed array size 0 invalid$"):
+        r"^my_file\.mojom:2: Error: Fixed array size 0 invalid\n"
+            r" *int32\[0\] zero_size_array;$"):
       parser.Parse(source1, "my_file.mojom")
 
     source2 = """\
-struct MyStruct {
-  int32[999999999999] too_big_array;
-};
-"""
+        struct MyStruct {
+          int32[999999999999] too_big_array;
+        };
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom: Error: Fixed array size 999999999999 invalid$"):
+        r"^my_file\.mojom:2: Error: Fixed array size 999999999999 invalid\n"
+            r" *int32\[999999999999\] too_big_array;$"):
       parser.Parse(source2, "my_file.mojom")
 
     source3 = """\
-struct MyStruct {
-  int32[abcdefg] not_a_number;
-};
-"""
+        struct MyStruct {
+          int32[abcdefg] not_a_number;
+        };
+        """
     with self.assertRaisesRegexp(
         parser.ParseError,
-        r"^my_file\.mojom:2: Error: Unexpected 'abcdefg':"):
+        r"^my_file\.mojom:2: Error: Unexpected 'abcdefg':\n"
+        r" *int32\[abcdefg\] not_a_number;"):
       parser.Parse(source3, "my_file.mojom")
 
   def testValidMethod(self):
     """Tests parsing method declarations."""
-    source1 = """\
-interface MyInterface {
-  MyMethod(int32 a);
-};
-"""
+
+    source1 = "interface MyInterface { MyMethod(int32 a); };"
     expected1 = \
-[('MODULE',
-  '',
-  None,
-  [('INTERFACE',
-    'MyInterface',
-    None,
-    [('METHOD',
-      'MyMethod',
-      [ast.Parameter('int32', 'a', ast.Ordinal(None))],
-      ast.Ordinal(None),
-      None)])])]
+        [('MODULE',
+          '',
+          None,
+          [('INTERFACE',
+            'MyInterface',
+            None,
+            [('METHOD',
+              'MyMethod',
+              [ast.Parameter('int32', 'a', ast.Ordinal(None))],
+              ast.Ordinal(None),
+              None)])])]
     self.assertEquals(parser.Parse(source1, "my_file.mojom"), expected1)
 
     source2 = """\
-interface MyInterface {
-  MyMethod1@0(int32 a@0, int64 b@1);
-  MyMethod2@1() => ();
-};
-"""
+        interface MyInterface {
+          MyMethod1@0(int32 a@0, int64 b@1);
+          MyMethod2@1() => ();
+        };
+        """
     expected2 = \
-[('MODULE',
-  '',
-  None,
-  [('INTERFACE',
-    'MyInterface',
-    None,
-    [('METHOD',
-      'MyMethod1',
-      [ast.Parameter('int32', 'a', ast.Ordinal(0)),
-       ast.Parameter('int64', 'b', ast.Ordinal(1))],
-      ast.Ordinal(0),
-      None),
-     ('METHOD',
-      'MyMethod2',
-      [],
-      ast.Ordinal(1),
-      [])])])]
+        [('MODULE',
+          '',
+          None,
+          [('INTERFACE',
+            'MyInterface',
+            None,
+            [('METHOD',
+              'MyMethod1',
+              [ast.Parameter('int32', 'a', ast.Ordinal(0)),
+               ast.Parameter('int64', 'b', ast.Ordinal(1))],
+              ast.Ordinal(0),
+              None),
+             ('METHOD',
+              'MyMethod2',
+              [],
+              ast.Ordinal(1),
+              [])])])]
     self.assertEquals(parser.Parse(source2, "my_file.mojom"), expected2)
 
     source3 = """\
-interface MyInterface {
-  MyMethod(string a) => (int32 a, bool b);
-};
-"""
+        interface MyInterface {
+          MyMethod(string a) => (int32 a, bool b);
+        };
+        """
     expected3 = \
-[('MODULE',
-  '',
-  None,
-  [('INTERFACE',
-    'MyInterface',
-    None,
-    [('METHOD',
-      'MyMethod',
-      [ast.Parameter('string', 'a', ast.Ordinal(None))],
-      ast.Ordinal(None),
-      [ast.Parameter('int32', 'a', ast.Ordinal(None)),
-       ast.Parameter('bool', 'b', ast.Ordinal(None))])])])]
+        [('MODULE',
+          '',
+          None,
+          [('INTERFACE',
+            'MyInterface',
+            None,
+            [('METHOD',
+              'MyMethod',
+              [ast.Parameter('string', 'a', ast.Ordinal(None))],
+              ast.Ordinal(None),
+              [ast.Parameter('int32', 'a', ast.Ordinal(None)),
+               ast.Parameter('bool', 'b', ast.Ordinal(None))])])])]
     self.assertEquals(parser.Parse(source3, "my_file.mojom"), expected3)
 
 if __name__ == "__main__":