-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(Effects): Introduce new Effects testing module #70
Conversation
|
||
export function provideMockActions(source: Observable<any>): Provider; | ||
export function provideMockActions(factory: () => Observable<any>): Provider; | ||
export function provideMockActions(factoryOrSource: (() => Observable<any>) | Observable<any>): Provider { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs an integration test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add the integration test in the example app after this gets merged
Review is done. Test in the store spec needs to be fixed since the behavior has changed in Rx |
f7b2d94
to
3986200
Compare
3986200
to
c84425e
Compare
Introduces a new testing module for @ngrx/effects that is designed to work well with marble testing.
Example usage: