0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Disabled MP Change Control test

Browse source 
This dialog is rather useless without a game_board object which the tests don't have. The dialog refactor
(5effd5338) did expose a few invalid cases assuming resources::gameboard was always valid (when it's not -
see the tests), but even if that were fixed the test would be rather useless since there's nothing there
without said game_board.
This commit is contained in:
Charles Dang 2017-04-28 11:00:10 +11:00
parent f5918996df
commit 2ddb305844
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/tests/gui/test_gui2.cpp
View file

@ -440,7 +440,7 @@ BOOST_AUTO_TEST_CASE(test_gui2)
//test<lua_interpreter>(& lua_kernel_base());
test<message>();
test<mp_alerts_options>();
test<mp_change_control>();
//test<mp_change_control>();
test<mp_cmd_wrapper>();
test<mp_connect>();
//test<mp_create_game>();
@ -518,6 +518,7 @@ BOOST_AUTO_TEST_CASE(test_gui2)
"help_browser",
"story_viewer",
"outro",
"mp_change_control", // Basically useless without a game_board object, so disabling
};
std::sort(list.begin(), list.end());
std::sort(omitted.begin(), omitted.end());
@ -893,7 +894,7 @@ struct dialog_tester<message>
return new message("Title", "Message", false, false);
}
};
#if 0
template<>
struct dialog_tester<mp_change_control>
{
@ -902,7 +903,7 @@ struct dialog_tester<mp_change_control>
return new mp_change_control(nullptr);
}
};
#endif
template<>
struct dialog_tester<mp_cmd_wrapper>
{