New Mojom parser: Don't allow enum variables to be assigned int values.

mojom/mojom_parser/parser/parser_test.go

 // Copyright 2015 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.
 
 package parser
 
 import (
         "fmt"
         "mojom/mojom_parser/lexer"
         "mojom/mojom_parser/mojom"
@@ skipping 693 matching lines @@
         struct Foo {
                 uint8 x = 9999999999;
         };
 
         `
         expectError("Illegal assignment")
         expectError("Field x of type uint8 may not be assigned the value 9999999999 of type int64.")
         endTestCase()
 
         ////////////////////////////////////////////////////////////
-        // Test Case (Assign string to user-defined-type)
-        ////////////////////////////////////////////////////////////
-        startTestCase("")
-        cases[testCaseNum].mojomContents = `
-        struct Foo {
-                Foo x = "hello";
-        };
-
-        `
-        expectError("Illegal assignment")
-        expectError("Field x of type Foo may not be assigned the value \"hello\" of type string.")
-        endTestCase()
-
-        ////////////////////////////////////////////////////////////
-        // Test Case (Assign string to user-defined-type)
-        ////////////////////////////////////////////////////////////
-        startTestCase("")
-        cases[testCaseNum].mojomContents = `
-        struct Foo {
-                Foo x = "hello";
-        };
-
-        `
-        expectError("Illegal assignment")
-        expectError("Field x of type Foo may not be assigned the value \"hello\" of type string.")
-        endTestCase()
-
-        ////////////////////////////////////////////////////////////
         // Group 2: Assign to constant.
         ////////////////////////////////////////////////////////////
 
         ////////////////////////////////////////////////////////////
         // Test Case (Assign boolean to int32)
         ////////////////////////////////////////////////////////////
         startTestCase("")
         cases[testCaseNum].mojomContents = `
         const int32 Foo = true;
         `
         expectError("Illegal assignment")
         expectError("Constant Foo of type int32 may not be assigned the value true of type bool.")
         endTestCase()
 
         ////////////////////////////////////////////////////////////
         // Test Case (Assign string to bool)
         ////////////////////////////////////////////////////////////
         startTestCase("")
         cases[testCaseNum].mojomContents = `
         const bool Foo = "true";
         `
         expectError("Illegal assignment")
         expectError("Constant Foo of type bool may not be assigned the value \"true\" of type string.")
         endTestCase()
 
         ////////////////////////////////////////////////////////////
-        // Test Case (Assign boolean to user-defined type)
-        ////////////////////////////////////////////////////////////
-        startTestCase("")
-        cases[testCaseNum].mojomContents = `
-        const FooType Foo = true;
-        `
-        expectError("Illegal assignment")
-        expectError("Constant Foo of type FooType may not be assigned the value true of type bool.")
-        endTestCase()
-
-        ////////////////////////////////////////////////////////////
         // Execute all of the test cases.
         ////////////////////////////////////////////////////////////
         for i, c := range cases {
                 descriptor := mojom.NewMojomDescriptor()
                 parser := MakeParser(c.fileName, c.mojomContents, descriptor, nil)
                 parser.Parse()
                 if parser.OK() {
                         t.Errorf("Parsing was supposed to fail but did not for test case %d", i)
                 } else {
                         got := parser.GetError().Error()
@@ skipping 180 matching lines @@
                 } else {
                         got := parser.GetError().Error()
                         for _, expected := range c.expectedErrors {
                                 if !strings.Contains(got, expected) {
                                         t.Errorf("%s:\n*****expected to contain:\n%s\n****actual\n%s", c.fileName, expected, got)
                                 }
                         }
                 }
         }
 }