Commit Graph

4 Commits

Author SHA1 Message Date
Ryan Nowak 55e0dba8dd
Make Actors unit testable (#576)
Fixes:  #574

Changes ActorHost constructor to be public again. Now the state manager
and http interactor are set via properties. This means that code in unit
tests won't be able to cover the methods that interact with timers or
reminders - however this was already the case. Testing state management
is covered by replacing the `IActorStateManager` with a mock.

Logged an issue to improve this.

Also added validation. If you try to use something that's not fully
initialized it will result in an exception with a meaningful message
instead of a null reference.

Also skips tests that are failing due to #573. We know the cause of why
these are failing, and these tests have not been stable since they were
introduced. Failing additional CI builds is not giving us new
information.
2021-02-01 11:47:02 -08:00
vinayada1 a4f6f041bc
Merge into release (#565)
*Merge master into release branch

Co-authored-by: Ryan Nowak <nowakra@gmail.com>

Co-authored-by: Ryan Nowak <nowakra@gmail.com>
Co-authored-by: Carlos Mendible <cmendible@gmail.com>
Co-authored-by: Arghya Sadhu <arghya88@gmail.com>
Co-authored-by: Per Ökvist <perokvist@users.noreply.github.com>
Co-authored-by: Sander Molenkamp <a.molenkamp@gmail.com>
Co-authored-by: LukePammant <Luke.pammant@gmail.com>
2021-01-27 14:46:35 -08:00
vinayada1 6a5d8ea09d
remove binaryformatter (#456)
* remove binaryformatter
2020-11-30 16:24:10 -08:00
Aman Bhardwaj bb3d7326cb
adding tests for DaprHttpInteractor in Actors, removing unused code. (#364) 2020-08-07 12:43:59 -07:00