-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add missing system services #1151
Add missing system services #1151
Conversation
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.
This service shouldn't be commented out. We can still manually check that it is injected.
Only the test should be commented out.
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.
I comment this out because Robolectric prints messages to System.err about it. But if you think i can re-enable.
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.
I didn't notice. There is indeed a warning in the maven output.
I think we should enable it anyway because it can help us to see if the generated code at least compile.
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.
Yes, you are right.
|
Services not supported by Robolectric should be commented out only in the test class. |
3ef19c4 to
6d2c1a2
Compare
|
I updated the commit. Also i restored alphabetical order. |
Lot of system services were missing which are added in newer Android versions. Now AndroidAnnotations support those. Also added tests for all services, and commented out those which cannot be tested due to Android version or Robolectric issues.
5e8c128 to
1fee3bb
Compare
|
@yDelouis can we merge this? |
|
I think so. |
Add missing system services
Lot of system services were missing which are added in newer Android versions. Now AndroidAnnotations support those. Also added tests for all services, and commented out those which cannot be tested due to Android version or Robolectric issues.