#3471 closed defect (fixed)

The eventual-send in LocalWrapper complicates testing

Reported by: exarkun Owned by: GitHub <noreply@…>
Priority: normal Milestone: undecided
Component: unknown Version: n/a
Keywords: review Cc:
Launchpad Bug:

Description

LocalWrapper?.callRemote forces asynchronous completion using eventually. This introduces a global reactor dependency which complicates any testing done using this class. Since the class only exists to facilitate testing, this is particularly counter-productive.

Change History (2)

comment:2 Changed at 2020-10-15T17:15:40Z by GitHub <noreply@…>

  • Owner set to GitHub <noreply@…>
  • Resolution set to fixed
  • Status changed from new to closed

In 4d56b5f/trunk:

Merge pull request #858 from tahoe-lafs/3471.immediate-localwrapper

Allow LocalWrapper? to be immediate

Fixes: ticket:3471

Note: See TracTickets for help on using tickets.