pass reactor as an argument to allmydata.test.test_node.testing_tub #2946

Open
opened 2018-09-04 20:50:02 +00:00 by meejah · 2 comments
Owner

It is a better Twisted practice to pass in a "reactor" argument to any functions that need one (instead of importing it). Tahoe-LAFS should follow this practice.

It is a better Twisted practice to pass in a "reactor" argument to any functions that need one (instead of importing it). Tahoe-LAFS should follow this practice.
meejah added the
code
normal
enhancement
1.12.1
labels 2018-09-04 20:50:02 +00:00
meejah added this to the undecided milestone 2018-09-04 20:50:02 +00:00
Author
Owner

allmydata.test.test_node has testing_tub that should get a reactor arg.

`allmydata.test.test_node` has `testing_tub` that should get a reactor arg.
exarkun changed title from pass reactor as an argument to pass reactor as an argument to allmydata.test.test_node.testing_tub 2020-01-21 20:40:47 +00:00

Narrowed the focus of this issue to the one API mentioned. All should eventually be fixed but likely not at the same time.

Narrowed the focus of this issue to the one API mentioned. All should eventually be fixed but likely not at the same time.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Reference: tahoe-lafs/trac-2024-07-25#2946
No description provided.