-
-
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
initialState on StoreModule.forFeature ignored #206
Comments
For know, i can work around this using:
|
Can you provide a small reproduction of this issue in a repo? |
The build issues should be fixed. You don't need to do a build to run the tests though |
I can confirm the build issues were fixed. Let me see if i can reproduce my problem with a spec... |
Okay, @brandonroberts , i have submitted a pull request with an integration test that demonstrates this issue. Not sure it fits with the flavor of the project, so you may not want to merge it, but it was the simplest way I could come up with... Thanks for looking into this. |
I'm submitting a...
What is the current behavior?
In my unit tests, I have a simple setup function:
This allows me to hydrate the store with a known initial state before each test. Using the
console.log
statements, I verified that the incoming state correct, however, the store only initializes thestore
half, and leaves theairport
half uninitialized (falls through to the default value defined in the reducer). Perhaps I am doing something wrong by including both aforRoot
and aforFeature
in the sameimports
, but not sure on that...Expected behavior:
The
Store
should be initialized with both sets ofinitialState
.Minimal reproduction of the problem with instructions:
I attempted to create a reproduction by creating another test in this project, but alas, I am unable to build:
Perhaps you don't support windows for builds?
Version of affected browser(s),operating system(s), npm, node and ngrx:
Windows: Microsoft Windows [Version 10.0.14393]
node: v8.2.1
npm: 5.0.3
ngrx/store: 4.0.0
ngrx/effects: 4.0.1
Other information:
💗 ngrx? Please consider supporting our collective: 👉 donate
The text was updated successfully, but these errors were encountered: