Skip to content

Commit a092d26

Browse files
authored
Build and test test/env.cpp (#6786)
1 parent 8ab87d0 commit a092d26

File tree

2 files changed

+28
-20
lines changed

2 files changed

+28
-20
lines changed

CMakeLists.txt

+5
Original file line numberDiff line numberDiff line change
@@ -651,6 +651,11 @@ if(BUILD_TESTS)
651651
if(BUILD_UNIT_TESTS)
652652
add_unit_test(map_test ${CMAKE_CURRENT_SOURCE_DIR}/src/ds/test/map_test.cpp)
653653

654+
add_unit_test(
655+
env_test ${CMAKE_CURRENT_SOURCE_DIR}/src/host/test/env.cpp
656+
${CCF_DIR}/src/host/env.cpp
657+
)
658+
654659
add_unit_test(
655660
json_schema ${CMAKE_CURRENT_SOURCE_DIR}/src/ds/test/json_schema.cpp
656661
)

src/host/test/env.cpp

+23-20
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,12 @@
33

44
#include "host/env.h"
55

6+
#define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN
67
#include <doctest/doctest.h>
78

9+
#define FMT_HEADER_ONLY
10+
#include <fmt/format.h>
11+
812
TEST_CASE("envvars" * doctest::test_suite("env"))
913
{
1014
{
@@ -13,15 +17,14 @@ TEST_CASE("envvars" * doctest::test_suite("env"))
1317
std::string test_value("test_value");
1418
::setenv("TEST_ENV_VAR", test_value.c_str(), 1);
1519

16-
REQUIRE("" == ccf::nonstd::expand_envvar(""));
17-
REQUIRE("not an env var" == ccf::nonstd::expand_envvar("not an env var"));
18-
REQUIRE(
19-
"$ENV_VAR_NOT_SET" == ccf::nonstd::expand_envvar("$ENV_VAR_NOT_SET"));
20-
REQUIRE(test_value == ccf::nonstd::expand_envvar("$TEST_ENV_VAR"));
20+
REQUIRE("" == ccf::env::expand_envvar(""));
21+
REQUIRE("not an env var" == ccf::env::expand_envvar("not an env var"));
22+
REQUIRE("$ENV_VAR_NOT_SET" == ccf::env::expand_envvar("$ENV_VAR_NOT_SET"));
23+
REQUIRE(test_value == ccf::env::expand_envvar("$TEST_ENV_VAR"));
2124

2225
// ${} syntax is not supported
2326
REQUIRE(
24-
"${ENV_VAR_NOT_SET}" == ccf::nonstd::expand_envvar("${ENV_VAR_NOT_SET}"));
27+
"${ENV_VAR_NOT_SET}" == ccf::env::expand_envvar("${ENV_VAR_NOT_SET}"));
2528
}
2629
{
2730
INFO("Expand path");
@@ -31,33 +34,33 @@ TEST_CASE("envvars" * doctest::test_suite("env"))
3134
std::string test_value2("test_value2");
3235
::setenv("TEST_ENV_VAR2", test_value2.c_str(), 1);
3336

34-
REQUIRE("" == ccf::nonstd::expand_envvars_in_path(""));
35-
REQUIRE("foo" == ccf::nonstd::expand_envvars_in_path("foo"));
36-
REQUIRE("foo/" == ccf::nonstd::expand_envvars_in_path("foo/"));
37-
REQUIRE("foo/bar" == ccf::nonstd::expand_envvars_in_path("foo/bar"));
38-
REQUIRE("/" == ccf::nonstd::expand_envvars_in_path("/"));
39-
REQUIRE("/foo" == ccf::nonstd::expand_envvars_in_path("/foo"));
40-
REQUIRE("/foo/" == ccf::nonstd::expand_envvars_in_path("/foo/"));
41-
REQUIRE("/foo/bar" == ccf::nonstd::expand_envvars_in_path("/foo/bar"));
37+
REQUIRE("" == ccf::env::expand_envvars_in_path(""));
38+
REQUIRE("foo" == ccf::env::expand_envvars_in_path("foo"));
39+
REQUIRE("foo/" == ccf::env::expand_envvars_in_path("foo/"));
40+
REQUIRE("foo/bar" == ccf::env::expand_envvars_in_path("foo/bar"));
41+
REQUIRE("/" == ccf::env::expand_envvars_in_path("/"));
42+
REQUIRE("/foo" == ccf::env::expand_envvars_in_path("/foo"));
43+
REQUIRE("/foo/" == ccf::env::expand_envvars_in_path("/foo/"));
44+
REQUIRE("/foo/bar" == ccf::env::expand_envvars_in_path("/foo/bar"));
4245

4346
REQUIRE(
4447
fmt::format("{}", test_value1) ==
45-
ccf::nonstd::expand_envvars_in_path("$TEST_ENV_VAR1"));
48+
ccf::env::expand_envvars_in_path("$TEST_ENV_VAR1"));
4649
REQUIRE(
4750
fmt::format("{}/", test_value1) ==
48-
ccf::nonstd::expand_envvars_in_path("$TEST_ENV_VAR1/"));
51+
ccf::env::expand_envvars_in_path("$TEST_ENV_VAR1/"));
4952
REQUIRE(
5053
fmt::format("{}/{}", test_value1, test_value2) ==
51-
ccf::nonstd::expand_envvars_in_path("$TEST_ENV_VAR1/$TEST_ENV_VAR2"));
54+
ccf::env::expand_envvars_in_path("$TEST_ENV_VAR1/$TEST_ENV_VAR2"));
5255

5356
REQUIRE(
5457
fmt::format("/{}", test_value1) ==
55-
ccf::nonstd::expand_envvars_in_path("/$TEST_ENV_VAR1"));
58+
ccf::env::expand_envvars_in_path("/$TEST_ENV_VAR1"));
5659
REQUIRE(
5760
fmt::format("/{}/", test_value1) ==
58-
ccf::nonstd::expand_envvars_in_path("/$TEST_ENV_VAR1/"));
61+
ccf::env::expand_envvars_in_path("/$TEST_ENV_VAR1/"));
5962
REQUIRE(
6063
fmt::format("/{}/{}", test_value1, test_value2) ==
61-
ccf::nonstd::expand_envvars_in_path("/$TEST_ENV_VAR1/$TEST_ENV_VAR2"));
64+
ccf::env::expand_envvars_in_path("/$TEST_ENV_VAR1/$TEST_ENV_VAR2"));
6265
}
6366
}

0 commit comments

Comments
 (0)