Injecting inner classes doesn't give an informative error message. about jukito

[olafleur]

Do you still have this error in Jukito 1.4?

[durron597]

Yes this error is still present in Jukito 1.4

[olafleur]

@christiangoudreau @PhilBeaudoin Is it possible to precise this error message in Jukito?

[durron597]

@olafleur It's harder than it seems, because Jukito creates mocks for objects that aren't properly handled by Guice. So for Guice, this is easy, they just dump the appropriate error message. But Jukito works by finding Guice errors and creating mocks. Somehow you'd have to special case this scenario and not mock.

[christiangoudreau]

Do we keep that open? Is it something useful. You clearly defined the problem durron and knowing if the error is right or wrong can be tricky.

[durron597]

I fixed this, but it's failing buildhive for some reason. I thought I had matched your style this time - I never did manage to get the checkstyle plugin to work on my computer. Anyway, I thought if I mentioned the case number in the commit message it would attach the pull request according to this StackOverflow post: http://stackoverflow.com/a/7189981/1768232 But it didn't actually work.

Anyway, I don't want to spam your pull request history further, but here's the fix: #56

[christiangoudreau]

Closed through : #56