akka/akka-core

StubbedActorContext.scheduleOnce returns canceled Cancellable

Open

#28.596 aperta il 11 feb 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Scala (3547 fork)batch import
1 - triagedhelp wantedt:testingt:typed

Metriche repository

Star
 (13.277 star)
Metriche merge PR
 (Merge medio 8g 19h) (10 PR mergiate in 30 g)

Descrizione

The current implementation of StubbedActorContext.scheduleOnce always returns an already canceled Cancellable. This makes it impossible to test any logic that relies on the state of the scheduled event (keeping an existing one vs. creating a new one based on its state).

Since one would expect to succeed in scheduling a new event with StubbedActorContext.scheduleOnce, shouldn't the stub returned be one that is uncancelle and transitions to cancelled if .cancel()` is called? If that change in behavior is acceptable, I'd gladly provide a PR for it.

Guida contributor

Direzione di ricerca
Modificare StubbedActorContext.scheduleOnce per restituire un Cancellable non inizialmente cancellato e implementare cancel() per farlo passare allo stato cancellato. Verificare che i test esistenti passino e aggiungere un test per il nuovo comportamento.
Tech stack
scala
Dominio
backenddistributed systems
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ScalaAkka
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60