Skip to content

Commit 5799177

Browse files
committed
replace usages of py.log
1 parent e407a33 commit 5799177

File tree

11 files changed

+27
-18
lines changed

11 files changed

+27
-18
lines changed

changelog/822.trivial.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Replace internal usage of ``py.log``.

setup.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ project_urls =
4040
packages = find:
4141
package_dir = =src
4242
zip_safe = False
43-
python_requires = >=3.6
43+
python_requires = >=3.6.1
4444
install_requires =
4545
execnet>=1.1
4646
pytest>=6.2.0

src/xdist/_producer.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import sys
2+
from typing import Any, NamedTuple
3+
4+
5+
class Producer(NamedTuple):
6+
name: str
7+
enabled: bool = True
8+
9+
def __call__(self, *a: Any, **k: Any) -> None:
10+
if self.enabled:
11+
print(f"[{self.name}]", *a, **k, file=sys.stderr)
12+
13+
def __getattr__(self, name: str) -> "Producer":
14+
return self._replace(name=name)

src/xdist/dsession.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
import py
21
import pytest
32

3+
from xdist._producer import Producer
44
from xdist.workermanage import NodeManager
55
from xdist.scheduler import (
66
EachScheduling,
@@ -34,9 +34,7 @@ class DSession:
3434

3535
def __init__(self, config):
3636
self.config = config
37-
self.log = py.log.Producer("dsession")
38-
if not config.option.debug:
39-
py.log.setconsumer(self.log._keywords, None)
37+
self.log = Producer("dsession", enabled=config.option.debug)
4038
self.nodemanager = None
4139
self.sched = None
4240
self.shuttingdown = False

src/xdist/remote.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,11 @@
1010
import os
1111
import time
1212

13-
import py
1413
import pytest
1514
from execnet.gateway_base import dumps, DumpError
1615

1716
from _pytest.config import _prepareconfig, Config
17+
from xdist._producer import Producer
1818

1919
try:
2020
from setproctitle import setproctitle
@@ -37,9 +37,7 @@ def __init__(self, config, channel):
3737
self.config = config
3838
self.workerid = config.workerinput.get("workerid", "?")
3939
self.testrunuid = config.workerinput["testrunuid"]
40-
self.log = py.log.Producer("worker-%s" % self.workerid)
41-
if not config.option.debug:
42-
py.log.setconsumer(self.log._keywords, None)
40+
self.log = Producer(f"worker-{self.workerid}", enabled=config.option.debug)
4341
self.channel = channel
4442
config.pluginmanager.register(self)
4543

src/xdist/scheduler/each.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
from py.log import Producer
2-
1+
from xdist._producer import Producer
32
from xdist.workermanage import parse_spec_config
43
from xdist.report import report_collection_diff
54

src/xdist/scheduler/load.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
from itertools import cycle
22

3-
from py.log import Producer
43
from _pytest.runner import CollectReport
54

5+
from xdist._producer import Producer
66
from xdist.workermanage import parse_spec_config
77
from xdist.report import report_collection_diff
88

src/xdist/scheduler/loadfile.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from .loadscope import LoadScopeScheduling
2-
from py.log import Producer
2+
from xdist._producer import Producer
33

44

55
class LoadFileScheduling(LoadScopeScheduling):

src/xdist/scheduler/loadgroup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from .loadscope import LoadScopeScheduling
2-
from py.log import Producer
2+
from xdist._producer import Producer
33

44

55
class LoadGroupScheduling(LoadScopeScheduling):

src/xdist/scheduler/loadscope.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
from collections import OrderedDict
22

33
from _pytest.runner import CollectReport
4-
from py.log import Producer
4+
from xdist._producer import Producer
55
from xdist.report import report_collection_diff
66
from xdist.workermanage import parse_spec_config
77

src/xdist/workermanage.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import execnet
1010

1111
import xdist.remote
12+
from xdist._producer import Producer
1213
from xdist.plugin import _sys_path
1314

1415

@@ -230,9 +231,7 @@ def __init__(self, nodemanager, gateway, config, putevent):
230231
}
231232
self._down = False
232233
self._shutdown_sent = False
233-
self.log = py.log.Producer("workerctl-%s" % gateway.id)
234-
if not self.config.option.debug:
235-
py.log.setconsumer(self.log._keywords, None)
234+
self.log = Producer(f"workerctl-{gateway.id}", enabled=config.option.debug)
236235

237236
def __repr__(self):
238237
return "<{} {}>".format(self.__class__.__name__, self.gateway.id)

0 commit comments

Comments
 (0)