Skip to content

Commit b340f65

Browse files
committed
add more detailed failure reasons to get_bool tests
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
1 parent aeb05b5 commit b340f65

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/test_classes/test_environment.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def test_get_bool_trues(self):
5858

5959
for case in cases:
6060
os.environ["OTHER_BOOL"] = case
61-
assert SystemEnv.get_bool("OTHER_BOOL", False, prefixes) == True
61+
assert SystemEnv.get_bool("OTHER_BOOL", False, prefixes) == True, f"value '{case}' should resolve to True"
6262
del os.environ["OTHER_BOOL"]
6363

6464
def test_get_bool_falses(self):
@@ -67,7 +67,7 @@ def test_get_bool_falses(self):
6767

6868
for case in cases:
6969
os.environ["OTHER_BOOL"] = case
70-
assert SystemEnv.get_bool("OTHER_BOOL", True, prefixes) == False
70+
assert SystemEnv.get_bool("OTHER_BOOL", True, prefixes) == False, f"value '{case}' should resolve to False"
7171
del os.environ["OTHER_BOOL"]
7272

7373
def test_get_bool_default(self):
@@ -76,7 +76,7 @@ def test_get_bool_default(self):
7676

7777
for case in cases:
7878
os.environ["OTHER_BOOL"] = case
79-
assert SystemEnv.get_bool("OTHER_BOOL", False, prefixes) == False
79+
assert SystemEnv.get_bool("OTHER_BOOL", False, prefixes) == False, f"value '{case}' should resolve to Default value"
8080
del os.environ["OTHER_BOOL"]
8181

8282

0 commit comments

Comments
 (0)