From 5677e1630ebe63c71651b13ebbc9676beb3767ef Mon Sep 17 00:00:00 2001 From: Dan Goodliffe Date: Fri, 1 Sep 2023 20:49:30 +0100 Subject: Remove unnecessary wrapper from xml tests Fixes weird Clang link problem too. --- slicer/xml/testSpecifics.cpp | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/slicer/xml/testSpecifics.cpp b/slicer/xml/testSpecifics.cpp index 6f94ed4..26a8ee9 100644 --- a/slicer/xml/testSpecifics.cpp +++ b/slicer/xml/testSpecifics.cpp @@ -21,10 +21,6 @@ // IWYU pragma: no_forward_declare Slicer::BadBooleanValue // IWYU pragma: no_forward_declare Slicer::XmlDocumentDeserializer -template -const auto BoostThrowWrapperHelper - = Slicer::DeserializeAny; - template using data = std::tuple; BOOST_FIXTURE_TEST_SUITE(doc, xmlpp::DomParser) @@ -36,7 +32,7 @@ BOOST_DATA_TEST_CASE(good_boolean_values, in, exp) { parse_memory(in); - BOOST_CHECK_EQUAL(exp, BoostThrowWrapperHelper(get_document())); + BOOST_CHECK_EQUAL(exp, (Slicer::DeserializeAny(get_document()))); } BOOST_DATA_TEST_CASE(bad_boolean_values, @@ -48,7 +44,8 @@ BOOST_DATA_TEST_CASE(bad_boolean_values, in) { parse_memory(in); - BOOST_CHECK_THROW(BoostThrowWrapperHelper(get_document()), Slicer::BadBooleanValue); + BOOST_CHECK_THROW( + (Slicer::DeserializeAny(get_document())), Slicer::BadBooleanValue); } BOOST_DATA_TEST_CASE(good_integer_values, @@ -61,7 +58,7 @@ BOOST_DATA_TEST_CASE(good_integer_values, in, exp) { parse_memory(in); - BOOST_CHECK_EQUAL(exp, BoostThrowWrapperHelper(get_document())); + BOOST_CHECK_EQUAL(exp, (Slicer::DeserializeAny(get_document()))); } BOOST_DATA_TEST_CASE(bad_integer_values, @@ -73,7 +70,8 @@ BOOST_DATA_TEST_CASE(bad_integer_values, in) { parse_memory(in); - BOOST_CHECK_THROW(BoostThrowWrapperHelper(get_document()), Slicer::BadNumericValue); + BOOST_CHECK_THROW((Slicer::DeserializeAny(get_document())), + Slicer::BadNumericValue); } BOOST_AUTO_TEST_SUITE_END() -- cgit v1.2.3