-
Notifications
You must be signed in to change notification settings - Fork 57
Parameterized Test Failure #170 #172
base: master
Are you sure you want to change the base?
Conversation
jrote1
commented
Aug 22, 2014
@vicb the pull request you requested |
ParameterizedClass<GenericArgument> _parameterizedClass; | ||
|
||
ClassWithParameterizedDependency(this._parameterizedClass); | ||
} |
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.
Please fix the style:
- space before "{"
- indentation is 2 white spaces
Thanks, that will help to get the problem fixed. |
Sorted all the styling issues. CLA already signed |
Awesome, thanks |
Any progress on this? |
I'm afraid there is no way to fix other than #106 but that implies to raise Dart req to 1.7+ (1.7 is not out yet, 1.6 has just been released). I would love to be proved wrong though. |
I have just started looking into it and have noticed that injector.get returns the correct but the test still fails which to me does not make any sense! |
Any help is appreciated here. Remember to test both the Dart & JS versions. |
#170 Tag Issue |