diff options
author | Nan Zhou <nanzhoumails@gmail.com> | 2022-06-14 01:33:55 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-06-21 18:28:42 +0300 |
commit | 5ad7720717807d6cd25b334c11ad117b46c9ee0d (patch) | |
tree | 5d759e1c497ef39328fc642d1987cb0b613f31cd /src | |
parent | 76686dcc74002493e22d72e24db9d3326b3a63ed (diff) | |
download | bmcweb-5ad7720717807d6cd25b334c11ad117b46c9ee0d.tar.xz |
crow_getroutes_test: revive the test
The test today exists but it isn't enabled. This commit revives the
test, and fixed obsolete interfaces.
Note that the current codes don't return the "/" route correctly. This
commit doesn't fix it but left a TODO.
Tested: unit test passed
Signed-off-by: Nan Zhou <nanzhoumails@gmail.com>
Change-Id: Ie5be7f545f1930ddb2c01b829d8de2e312e936dc
Diffstat (limited to 'src')
-rw-r--r-- | src/crow_getroutes_test.cpp | 23 |
1 files changed, 14 insertions, 9 deletions
diff --git a/src/crow_getroutes_test.cpp b/src/crow_getroutes_test.cpp index eea045c64c..5352165b5b 100644 --- a/src/crow_getroutes_test.cpp +++ b/src/crow_getroutes_test.cpp @@ -8,8 +8,8 @@ using namespace std; TEST(GetRoutes, TestEmptyRoutes) { - SimpleApp app; - decltype(app)::server_t server(&app, "127.0.0.1", 45451); + App app; + app.validate(); EXPECT_THAT(app.getRoutes(), testing::IsEmpty()); } @@ -17,19 +17,20 @@ TEST(GetRoutes, TestEmptyRoutes) // Tests that static urls are correctly passed TEST(GetRoutes, TestOneRoute) { - SimpleApp app; - decltype(app)::server_t server(&app, "127.0.0.1", 45451); + App app; + BMCWEB_ROUTE(app, "/")([]() { return boost::beast::http::status::ok; }); - EXPECT_THAT(app.getRoutes(), - testing::ElementsAre(testing::Pointee(std::string("/")))); + // TODO: "/" doesn't get reported in |getRoutes| today. Uncomment this once + // it is fixed + // EXPECT_THAT(app.getRoutes(), + // testing::ElementsAre(testing::Pointee(std::string("/")))); } // Tests that static urls are correctly passed TEST(GetRoutes, TestlotsOfRoutes) { - SimpleApp app; - decltype(app)::server_t server(&app, "127.0.0.1", 45451); + App app; BMCWEB_ROUTE(app, "/")([]() { return boost::beast::http::status::ok; }); BMCWEB_ROUTE(app, "/foo")([]() { return boost::beast::http::status::ok; }); BMCWEB_ROUTE(app, "/bar")([]() { return boost::beast::http::status::ok; }); @@ -37,8 +38,12 @@ TEST(GetRoutes, TestlotsOfRoutes) BMCWEB_ROUTE(app, "/boo")([]() { return boost::beast::http::status::ok; }); BMCWEB_ROUTE(app, "/moo")([]() { return boost::beast::http::status::ok; }); + app.validate(); + + // TODO: "/" doesn't get reported in |getRoutes| today. Uncomment this once + // it is fixed EXPECT_THAT(app.getRoutes(), testing::UnorderedElementsAre( - testing::Pointee(std::string("/")), + // testing::Pointee(std::string("/")), testing::Pointee(std::string("/foo")), testing::Pointee(std::string("/bar")), testing::Pointee(std::string("/baz")), |