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
meejah commented 2018-09-04 20:50:02 +00:00
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.
tahoe-lafs added the
code
normal
enhancement
1.12.1
labels 2018-09-04 20:50:02 +00:00
tahoe-lafs added this to the undecided milestone 2018-09-04 20:50:02 +00:00
meejah commented 2018-09-06 18:37:34 +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.
tahoe-lafs 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
exarkun commented 2020-01-21 20:41:06 +00:00
Author
Owner

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 project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

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