-
Notifications
You must be signed in to change notification settings - Fork 36
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
[feature] App level alias override #1071
Comments
Nice idea. |
This was largely complete when I picked up it. I've added tests for groups and scenarios. @thecapdan can you do your thing and do some exploratory testing? |
I think we need to update the docs to demonstrate how to use this new feature, maybe here: http://bladerunnerjs.org/docs/use/ioc/ |
The spec tests for this are a little unclear - so we should tidy those up + I'll add an example in the it-app |
"With the option to further override (as you can now) on workbench/test level." I assume this means we have app level aliases that we would like to optionally override at the test level. did we decide how this was going to be done, if so, can we document it |
Moving back to dev.
herre's the error i get when trying to use an app level alias:
|
While looking at whether we had a test case for the issue @thecapdan described above I noticed This needs a lot more work. |
retested. working now. + overriding the app level alias in the default aspect works as expected. needs release note, but will merge anyway because im a soft touch |
I would like to be able to define an alias override that would apply to everything in the app (aspects, workbenches & tests). With the option to further override (as you can now) on workbench/test level.
Right now I have to copy the same alias override (in
aliases.xml
) to a lot of different places in my app.The text was updated successfully, but these errors were encountered: