Used oneof in blimp_message.proto

blimp/common/create_blimp_message_unittest.cc

 // 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.
 
 #include "blimp/common/create_blimp_message.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/common/proto/compositor.pb.h"
 #include "blimp/common/proto/input.pb.h"
 #include "blimp/common/proto/navigation.pb.h"
 #include "blimp/common/proto/render_widget.pb.h"
@@ skipping 64 matching lines @@
   EXPECT_NE(nullptr, message);
   EXPECT_EQ(details, message->mutable_settings()->mutable_engine_settings());
 }
 
 TEST(CreateBlimpMessageTest, StartConnectionMessage) {
   const char* client_token = "token";
   const int protocol_version = 1;
   std::unique_ptr<BlimpMessage> message =
       CreateStartConnectionMessage(client_token, protocol_version);
   EXPECT_NE(nullptr, message);
-  EXPECT_EQ(BlimpMessage::PROTOCOL_CONTROL, message->type());
+  EXPECT_TRUE(message->has_protocol_control());

[Kevin M, 2016/05/02 17:47:48]

EXPECT_EQ(kCase, ...)

[shaktisahu, 2016/05/16 20:19:02]

Done.

   EXPECT_EQ(ProtocolControlMessage::START_CONNECTION,
             message->protocol_control().type());
   EXPECT_EQ(client_token,
             message->protocol_control().start_connection().client_token());
   EXPECT_EQ(protocol_version,
             message->protocol_control().start_connection().protocol_version());
 }
 
 }  // namespace
 }  // namespace blimp