Skip to content

Commit b98dd2b

Browse files
committed
fixed up test_pickle
1 parent 8f1b031 commit b98dd2b

File tree

2 files changed

+35
-3
lines changed

2 files changed

+35
-3
lines changed

pydra/workers/cf.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@ class Worker(base.Worker):
4646
"""A worker to execute in parallel using Python's concurrent futures."""
4747

4848
n_procs: int = attrs.field(factory=get_available_cpus)
49-
pool: cf.ProcessPoolExecutor = attrs.field()
49+
pool: cf.ProcessPoolExecutor = attrs.field(
50+
eq=False, init=False, hash=False, repr=False
51+
)
5052

5153
@pool.default
5254
def _pool_default(self) -> cf.ProcessPoolExecutor:

pydra/workers/tests/test_pickle.py

Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,43 @@
11
import pickle as pkl
22

3-
from pydra.workers import debug, cf, sge, slurm, dask, base
3+
from pydra.workers import debug, cf, sge, slurm
44

55

66
def test_pickle_debug_worker():
77
"""
8-
Test pickling of DebugWorker
8+
Test pickling of debug.Worker
99
"""
1010
worker = debug.Worker()
1111
worker2 = pkl.loads(pkl.dumps(worker))
1212
assert worker2.loop is None
1313
assert worker == worker2
14+
15+
16+
def test_pickle_cf_worker():
17+
"""
18+
Test pickling of cf.Worker
19+
"""
20+
worker = cf.Worker()
21+
worker2 = pkl.loads(pkl.dumps(worker))
22+
assert worker2.loop is None
23+
assert worker == worker2
24+
25+
26+
def test_pickle_sge_worker():
27+
"""
28+
Test pickling of sge.Worker
29+
"""
30+
worker = sge.Worker()
31+
worker2 = pkl.loads(pkl.dumps(worker))
32+
assert worker2.loop is None
33+
assert worker == worker2
34+
35+
36+
def test_pickle_slurm_worker():
37+
"""
38+
Test pickling of DebugWorker
39+
"""
40+
worker = slurm.Worker()
41+
worker2 = pkl.loads(pkl.dumps(worker))
42+
assert worker2.loop is None
43+
assert worker == worker2

0 commit comments

Comments
 (0)