refactor: use CHECK_THROW macro to check it throws

This commit is contained in:
ToruNiina
2019-06-21 14:47:27 +09:00
parent 1694f74510
commit 713b42e589

View File

@@ -12,50 +12,19 @@
BOOST_AUTO_TEST_CASE(test_detect_empty_key) BOOST_AUTO_TEST_CASE(test_detect_empty_key)
{ {
std::istringstream stream(std::string("= \"value\"")); std::istringstream stream(std::string("= \"value\""));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_empty_key");
}
catch(const toml::syntax_error& syn)
{
// to see the error message
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_missing_value) BOOST_AUTO_TEST_CASE(test_detect_missing_value)
{ {
std::istringstream stream(std::string("a =")); std::istringstream stream(std::string("a ="));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_missing_value");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_too_many_value) BOOST_AUTO_TEST_CASE(test_detect_too_many_value)
{ {
std::istringstream stream(std::string("a = 1 = \"value\"")); std::istringstream stream(std::string("a = 1 = \"value\""));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_too_many_value");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_duplicate_table) BOOST_AUTO_TEST_CASE(test_detect_duplicate_table)
@@ -66,17 +35,7 @@ BOOST_AUTO_TEST_CASE(test_detect_duplicate_table)
"[table]\n" "[table]\n"
"b = 42\n" "b = 42\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_duplicate_table");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_conflict_array_table) BOOST_AUTO_TEST_CASE(test_detect_conflict_array_table)
@@ -87,17 +46,7 @@ BOOST_AUTO_TEST_CASE(test_detect_conflict_array_table)
"[table]\n" "[table]\n"
"b = 42\n" "b = 42\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_conflict_array_table");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_conflict_table_array) BOOST_AUTO_TEST_CASE(test_detect_conflict_table_array)
@@ -108,17 +57,7 @@ BOOST_AUTO_TEST_CASE(test_detect_conflict_table_array)
"[[table]]\n" "[[table]]\n"
"b = 42\n" "b = 42\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_conflict_table_array");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_duplicate_value) BOOST_AUTO_TEST_CASE(test_detect_duplicate_value)
@@ -127,17 +66,7 @@ BOOST_AUTO_TEST_CASE(test_detect_duplicate_value)
"a = 1\n" "a = 1\n"
"a = 2\n" "a = 2\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_duplicate_value");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_conflicting_value) BOOST_AUTO_TEST_CASE(test_detect_conflicting_value)
@@ -146,17 +75,7 @@ BOOST_AUTO_TEST_CASE(test_detect_conflicting_value)
"a.b = 1\n" "a.b = 1\n"
"a.b.c = 2\n" "a.b.c = 2\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_conflicting_value");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_inhomogeneous_array) BOOST_AUTO_TEST_CASE(test_detect_inhomogeneous_array)
@@ -164,17 +83,7 @@ BOOST_AUTO_TEST_CASE(test_detect_inhomogeneous_array)
std::istringstream stream(std::string( std::istringstream stream(std::string(
"a = [1, 1.0]\n" "a = [1, 1.0]\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_inhomogeneous_array");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }
BOOST_AUTO_TEST_CASE(test_detect_appending_array_of_table) BOOST_AUTO_TEST_CASE(test_detect_appending_array_of_table)
@@ -184,16 +93,5 @@ BOOST_AUTO_TEST_CASE(test_detect_appending_array_of_table)
"[[a]]\n" "[[a]]\n"
"b = 2\n" "b = 2\n"
)); ));
bool exception_thrown = false; BOOST_CHECK_THROW(toml::parse(stream), toml::syntax_error);
try
{
toml::parse(stream, "test_detect_appending_array_of_table");
}
catch(const toml::syntax_error& syn)
{
std::cerr << syn.what() << std::endl;
exception_thrown = true;
}
BOOST_CHECK(exception_thrown);
} }