- PermissionsDispatcher now have a custom lint
- Adding custom lint is not that hard
- LintDetectorTest…? Forget it
PermissionsDispatcher now have a custom lint
First of all, I am one of collaborators of PermissionsDispatcher which makes developers handle runtime permissions very easily.
PermissionsDispatcher now have a custom lint for its library. Initial issue came from this comment.
The library requires the developer to define a method with
@NeedsPermission. Like this
1 2 3
However, this method should not directly access. Instead, the developers must use a method that PermissionsDispatcher generated.
1 2 3 4 5 6 7
In order to make sure the developers did not call this method, we decide to provide custom lint for this library.
How to add custom lint for library
To create custom lint for Android application, check links below
- Writing Custom Lint Rules
- Writing a Lint Check
- @hotchime’s blog post at qiita (For Japanese developer)
Custom lint for library needs extra settings to inject
lint.jar to its .aar file
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27
Basically, if there is
compileLint task, then make the task depends on
copyLintJar task which copies
The user of this library doesn’t need to put lint.jar to
.android/lint/. Just sync library and then run
Check this code
1 2 3 4
This is the code that reads from
I believe most developers doesn’t want to put project in the specific directory in order to pass the test cases.
Actually, current version of
LintDetectorTest has a lots of problems with loading resources, java source and “class” file. (Yes! you need to supply .class file to run the test)
In order to use the
LintDetectorTest, the developer must override
protected InputStream getTestResource(String relativePath, boolean expectExists) like what AOSP did
LintDetectorTest is still Beta. I assumed there are a lot of work to make it production release.
If it comes to production, I hope it does not require me to pass .java/.class file like this way.
Now how to test???
I don’t know.
The only way I came up is manual testing.
So, someone who know better way to test lint, please let me know!