From 53a6076d05e264d6bed743032a63fd782e7caae9 Mon Sep 17 00:00:00 2001 From: Egor Tensin Date: Sun, 4 Oct 2020 22:26:13 +0300 Subject: unit_tests: reorder code a little --- test/unit_tests/lexer.cpp | 66 +++++++++++++++++++++++----------------------- test/unit_tests/parser.cpp | 4 +-- 2 files changed, 35 insertions(+), 35 deletions(-) diff --git a/test/unit_tests/lexer.cpp b/test/unit_tests/lexer.cpp index c43b879..1a8a29d 100644 --- a/test/unit_tests/lexer.cpp +++ b/test/unit_tests/lexer.cpp @@ -18,8 +18,6 @@ #include #include -BOOST_AUTO_TEST_SUITE(lexer_tests) - namespace bdata = boost::unit_test::data; using math::server::Lexer; using math::server::LexerError; @@ -27,37 +25,6 @@ using math::server::lexer::Token; using math::server::lexer::token::Type; namespace details = math::server::lexer::details; -BOOST_AUTO_TEST_CASE(test_parse_number) { - // These are valid numbers: - BOOST_TEST(details::parse_number("0").value() == 0); - BOOST_TEST(details::parse_number("1.").value() == 1.); - BOOST_TEST(details::parse_number(".1").value() == .1); - // parse_* functions only consume a single token: - BOOST_TEST(details::parse_number(".1+1").value() == .1); - BOOST_TEST(details::parse_number("1e3").value() == 1e3); - BOOST_TEST(details::parse_number(".123e1").value() == .123e1); - BOOST_TEST(details::parse_number("123e-1").value() == 123e-1); - BOOST_TEST(details::parse_number("123e+1").value() == 123e+1); - BOOST_TEST(details::parse_number("1.e6").value() == 1.e6); - BOOST_TEST(details::parse_number("2.3e-4").value() == 2.3e-4); - // These are not numbers, but perhaps something else? - BOOST_TEST(!details::parse_number(".").has_value()); - BOOST_TEST(!details::parse_number(".e3").has_value()); - BOOST_TEST(!details::parse_number("e12").has_value()); - // This is definitely a number, but a malformed one (an exponent must be - // followed by some digits). - BOOST_CHECK_THROW(details::parse_number("12e"), LexerError); -} - -BOOST_AUTO_TEST_CASE(test_parse_const_token) { - BOOST_TEST(details::parse_const_token("+").value() == Type::PLUS); - // parse_* functions only consume a single token: - BOOST_TEST(details::parse_const_token("+/*").value() == Type::PLUS); - BOOST_TEST(details::parse_const_token("-").value() == Type::MINUS); - BOOST_TEST(details::parse_const_token("^^").value() == Type::CARET); - BOOST_TEST(!details::parse_const_token("&+").has_value()); -} - namespace { namespace get_tokens::valid { @@ -152,6 +119,39 @@ const std::vector error_msg{ } // namespace get_tokens::invalid } // namespace +BOOST_AUTO_TEST_SUITE(lexer_tests) + +BOOST_AUTO_TEST_CASE(test_parse_number) { + // These are valid numbers: + BOOST_TEST(details::parse_number("0").value() == 0); + BOOST_TEST(details::parse_number("1.").value() == 1.); + BOOST_TEST(details::parse_number(".1").value() == .1); + // parse_* functions only consume a single token: + BOOST_TEST(details::parse_number(".1+1").value() == .1); + BOOST_TEST(details::parse_number("1e3").value() == 1e3); + BOOST_TEST(details::parse_number(".123e1").value() == .123e1); + BOOST_TEST(details::parse_number("123e-1").value() == 123e-1); + BOOST_TEST(details::parse_number("123e+1").value() == 123e+1); + BOOST_TEST(details::parse_number("1.e6").value() == 1.e6); + BOOST_TEST(details::parse_number("2.3e-4").value() == 2.3e-4); + // These are not numbers, but perhaps something else? + BOOST_TEST(!details::parse_number(".").has_value()); + BOOST_TEST(!details::parse_number(".e3").has_value()); + BOOST_TEST(!details::parse_number("e12").has_value()); + // This is definitely a number, but a malformed one (an exponent must be + // followed by some digits). + BOOST_CHECK_THROW(details::parse_number("12e"), LexerError); +} + +BOOST_AUTO_TEST_CASE(test_parse_const_token) { + BOOST_TEST(details::parse_const_token("+").value() == Type::PLUS); + // parse_* functions only consume a single token: + BOOST_TEST(details::parse_const_token("+/*").value() == Type::PLUS); + BOOST_TEST(details::parse_const_token("-").value() == Type::MINUS); + BOOST_TEST(details::parse_const_token("^^").value() == Type::CARET); + BOOST_TEST(!details::parse_const_token("&+").has_value()); +} + BOOST_DATA_TEST_CASE(test_get_tokens_valid, bdata::make(get_tokens::valid::input) ^ get_tokens::valid::expected, input, diff --git a/test/unit_tests/parser.cpp b/test/unit_tests/parser.cpp index 2e90992..655ba39 100644 --- a/test/unit_tests/parser.cpp +++ b/test/unit_tests/parser.cpp @@ -14,8 +14,6 @@ #include #include -BOOST_AUTO_TEST_SUITE(parser_tests) - namespace bdata = boost::unit_test::data; using math::server::Parser; using math::server::ParserError; @@ -114,6 +112,8 @@ const std::vector error_msg{ } // namespace exec::invalid } // namespace +BOOST_AUTO_TEST_SUITE(parser_tests) + BOOST_DATA_TEST_CASE(test_exec_valid, bdata::make(exec::valid::input) ^ exec::valid::expected, input, -- cgit v1.2.3