Skip to content

Commit c618339

Browse files
committed
Drop dependency on ansicolors
pytest doesn't emit colors if there is no terminal, so there is no need to filter out ANSI sequences. This drops the only remaining user of ansicolors, so the dpeendency can be dropped. Signed-off-by: Uwe Kleine-König <[email protected]>
1 parent b1b4021 commit c618339

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

pyproject.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ classifiers = [
3232
"License :: OSI Approved :: GNU Lesser General Public License v2 or later (LGPLv2+)",
3333
]
3434
dependencies = [
35-
"ansicolors>=1.1.8",
3635
"attrs>=21.4.0",
3736
"grpcio>=1.64.1, <2.0.0",
3837
"grpcio-reflection>=1.64.1, <2.0.0",

tests/test_steplogger.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,15 +29,14 @@ def test(target):
2929
return t
3030

3131
def test_step_logger(power_env, pw_cycle_test):
32-
from colors import strip_color
3332
# step reporter is called when -vv is given
3433
# -s is necessary for manual power driver confirmation
3534
with pexpect.spawn('pytest -vvs --lg-env {env} {test}'
3635
.format(env=power_env, test=pw_cycle_test)) as spawn:
3736

3837
# rough match
3938
spawn.expect("→.*?ManualPowerDriver.*?cycle.*?\r\n".encode("utf-8"))
40-
step_line = strip_color(spawn.after.decode("utf-8")).rstrip()
39+
step_line = spawn.after.decode("utf-8").rstrip()
4140
# exact match
4241
assert step_line.endswith("→ ManualPowerDriver.cycle()"), f"'{step_line}' did not match"
4342

@@ -47,7 +46,7 @@ def test_step_logger(power_env, pw_cycle_test):
4746

4847
# rough match
4948
spawn.expect("←.*?ManualPowerDriver.*?cycle.*?\r\n".encode("utf-8"))
50-
step_line = strip_color(spawn.after.decode("utf-8")).rstrip()
49+
step_line = spawn.after.decode("utf-8").rstrip()
5150
# exact match
5251

5352
assert re.match(r"← ManualPowerDriver.cycle\(\) \[[\d.]+s\]$", step_line), f"'{step_line}' did not match"

0 commit comments

Comments
 (0)