Skip to content

Commit 2f4b463

Browse files
Copilotzkoppert
andcommitted
Simplify test assertions for better clarity
Co-authored-by: zkoppert <[email protected]>
1 parent 5f38ece commit 2f4b463

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

test_env.py

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -257,10 +257,8 @@ def test_get_env_vars_end_date_before_start_date(self):
257257
def test_get_env_vars_equal_start_and_end_date(self):
258258
"""Test that equal START_DATE and END_DATE are allowed"""
259259
result = env.get_env_vars()
260-
start_date = result[8]
261-
end_date = result[9]
262-
self.assertEqual(start_date, "2024-01-01")
263-
self.assertEqual(end_date, "2024-01-01")
260+
self.assertEqual(result[8], "2024-01-01") # start_date
261+
self.assertEqual(result[9], "2024-01-01") # end_date
264262

265263
@patch.dict(
266264
os.environ,
@@ -276,10 +274,8 @@ def test_get_env_vars_equal_start_and_end_date(self):
276274
def test_get_env_vars_valid_date_range(self):
277275
"""Test that valid date range (START_DATE before END_DATE) is accepted"""
278276
result = env.get_env_vars()
279-
start_date = result[8]
280-
end_date = result[9]
281-
self.assertEqual(start_date, "2024-01-01")
282-
self.assertEqual(end_date, "2025-01-01")
277+
self.assertEqual(result[8], "2024-01-01") # start_date
278+
self.assertEqual(result[9], "2025-01-01") # end_date
283279

284280

285281
if __name__ == "__main__":

0 commit comments

Comments
 (0)