Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Upgrade to RoboGuice 2#983

Merged
DayS merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:172_RoboGuice2WonderCsabo/androidannotations:172_RoboGuice2Copy head branch name to clipboard
May 31, 2014
Merged

Upgrade to RoboGuice 2#983
DayS merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:172_RoboGuice2WonderCsabo/androidannotations:172_RoboGuice2Copy head branch name to clipboard

Conversation

@WonderCsabo

Copy link
Copy Markdown
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

DayS commented May 10, 2014

Copy link
Copy Markdown
Contributor

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

@WonderCsabo

Copy link
Copy Markdown
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.