Commit 06968cb7 authored by mithro's avatar mithro Committed by Commit bot

Make EXPECT_ACTION_UPDATE_STATE failure output easier to understand.

Changes;
-------------------------------------------------------------------------------
Value of: state.NextAction()
  Actual: 1
Expected: SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT
Which is: 8
-------------------------------------------------------------------------------

to

-------------------------------------------------------------------------------
Value of: SchedulerStateMachine::ActionToString(state.NextAction())
  Actual: "ACTION_NONE"
Expected: SchedulerStateMachine::ActionToString( SchedulerStateMachine::ACTION_DRAW_AND_SWAP_ABORT)
Which is: "ACTION_DRAW_AND_SWAP_ABORT"
-------------------------------------------------------------------------------

BUG=416749

Review URL: https://codereview.chromium.org/639963006

Cr-Commit-Position: refs/heads/master@{#299282}
parent cd6462df
...@@ -10,7 +10,9 @@ ...@@ -10,7 +10,9 @@
#include "testing/gtest/include/gtest/gtest.h" #include "testing/gtest/include/gtest/gtest.h"
#define EXPECT_ACTION_UPDATE_STATE(action) \ #define EXPECT_ACTION_UPDATE_STATE(action) \
EXPECT_EQ(action, state.NextAction()) << state.AsValue()->ToString(); \ EXPECT_STREQ(SchedulerStateMachine::ActionToString(action), \
SchedulerStateMachine::ActionToString(state.NextAction())) \
<< state.AsValue()->ToString(); \
if (action == SchedulerStateMachine::ACTION_DRAW_AND_SWAP_IF_POSSIBLE || \ if (action == SchedulerStateMachine::ACTION_DRAW_AND_SWAP_IF_POSSIBLE || \
action == SchedulerStateMachine::ACTION_DRAW_AND_SWAP_FORCED) { \ action == SchedulerStateMachine::ACTION_DRAW_AND_SWAP_FORCED) { \
EXPECT_EQ(SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_INSIDE_DEADLINE, \ EXPECT_EQ(SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_INSIDE_DEADLINE, \
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment