Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Conversation

@WonderCsabo
Copy link
Member

Implements #172.

No tests for now, those will be in a separate commit as noted in #973.

This commit adds RoboGuice 2 support, RoboGuice 1 support is dropped.
WonderCsabo added a commit to WonderCsabo/androidannotations that referenced this pull request May 2, 2014
Initial implementation for upgrading to Robolectric 2.2.
SaveInstanceStateActivityParameterizedTest is failing, which is due to
a Robolectric bug described in androidannotations#973.
This commit dependes on RoboGuice 2 which is implemented in androidannotations#983.

All other tests should be OK. This involves lots of code change and
simplification thanks to Robolectric 2.2.
WonderCsabo added a commit to WonderCsabo/androidannotations that referenced this pull request May 2, 2014
Initial implementation for upgrading to Robolectric 2.2.
SaveInstanceStateActivityParameterizedTest is failing, which is due to
a Robolectric bug described in androidannotations#973.
This commit dependes on RoboGuice 2 which is implemented in androidannotations#983.

All other tests should be OK. This involves lots of code change and
simplification thanks to Robolectric 2.2.
@DayS
Copy link
Contributor

DayS commented May 10, 2014

Unit test doesn't seems to work with this PR.

@WonderCsabo
Copy link
Member Author

The unit tests for this are implemented in #984, as i noted before.

@DayS DayS added the Task label May 11, 2014
@DayS DayS added this to the Someday milestone May 11, 2014
@DayS DayS removed the Task label May 11, 2014
@DayS DayS removed this from the Someday milestone May 11, 2014
@DayS DayS merged commit 137353b into androidannotations:develop May 31, 2014
@WonderCsabo WonderCsabo deleted the 172_RoboGuice2 branch May 31, 2014 19:28
@DayS DayS added this to the 3.1 milestone Sep 7, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants