@@ -42,25 +42,20 @@ def test_colorized_detection_checks_for_environment_variables(self):
4242 stdout_mock .isatty .return_value = True
4343 stderr_mock .fileno .return_value = 2
4444 stderr_mock .isatty .return_value = True
45- with unittest .mock .patch ("os.environ" , {'TERM' : 'dumb' }):
46- self .assertEqual (_colorize .can_colorize (), False )
47- with unittest .mock .patch ("os.environ" , {'PYTHON_COLORS' : '1' }):
48- self .assertEqual (_colorize .can_colorize (), True )
49- with unittest .mock .patch ("os.environ" , {'PYTHON_COLORS' : '0' }):
50- self .assertEqual (_colorize .can_colorize (), False )
51- with unittest .mock .patch ("os.environ" , {'NO_COLOR' : '1' }):
52- self .assertEqual (_colorize .can_colorize (), False )
53- with unittest .mock .patch ("os.environ" ,
54- {'NO_COLOR' : '1' , "PYTHON_COLORS" : '1' }):
55- self .assertEqual (_colorize .can_colorize (), True )
56- with unittest .mock .patch ("os.environ" , {'FORCE_COLOR' : '1' }):
57- self .assertEqual (_colorize .can_colorize (), True )
58- with unittest .mock .patch ("os.environ" ,
59- {'FORCE_COLOR' : '1' , 'NO_COLOR' : '1' }):
60- self .assertEqual (_colorize .can_colorize (), False )
61- with unittest .mock .patch ("os.environ" ,
62- {'FORCE_COLOR' : '1' , "PYTHON_COLORS" : '0' }):
63- self .assertEqual (_colorize .can_colorize (), False )
45+
46+ for env_vars , expected in [
47+ ({"TERM" : "dumb" }, False ),
48+ ({"PYTHON_COLORS" : "1" }, True ),
49+ ({"PYTHON_COLORS" : "0" }, False ),
50+ ({"NO_COLOR" : "1" }, False ),
51+ ({"NO_COLOR" : "1" , "PYTHON_COLORS" : "1" }, True ),
52+ ({"FORCE_COLOR" : "1" }, True ),
53+ ({"FORCE_COLOR" : "1" , "NO_COLOR" : "1" }, False ),
54+ ({"FORCE_COLOR" : "1" , "PYTHON_COLORS" : "0" }, False ),
55+ ]:
56+ with self .subTest (env_vars = env_vars , expected = expected ):
57+ with unittest .mock .patch ("os.environ" , env_vars ):
58+ self .assertEqual (_colorize .can_colorize (), expected )
6459
6560 with unittest .mock .patch ("os.environ" , {}):
6661 if sys .platform == "win32" :
0 commit comments