Mobile/Fennec/Android/Static analysis: Difference between revisions

Revision as of 01:29, 9 April 2016

Catches various errors & optimizations specifically related to Android code and resources

./mach gradle app:lint

example checks? `lint --show`
in automation? Treeherder tier 2 "Unit" job
config? mobile/android/app/lint.xml
open issues? Run locally for a complete list.
meta bug? meta bug
If you fix all issues for a warning, modify lint.xml to change your warning into an error!

Currently detects NullPointerExceptions and resource leaks in Android applications.

It's currently unknown how to run locally.

example checks? Infer docs
in automation? updated daily – planned move to Task Cluster. Use infer-filter to get the issues we care about from the daily update link.
config? N/A
open issues? See the automation link
meta bug? meta bug

Checks code style

./mach gradle app:checkstyle

Cloud-based static analysis that catches many miscellaneous Java errors, e.g. null dereference & dead code

example checks? N/A
in automation? See above link
config? Configured online
open issues? See online
meta bug? meta bug
You must be added as a member of the Coverity project in order to interact with the dashboard – talk to :Sylvestre
You can sign up for email notifications (via the online interface) when new Coverity defects are found

Checks for Javascript errors (e.g. syntax errors, almost like a compiled language! :D)

./mach eslint --setup  # run once, or if the command breaks for some reason
./mach eslint mobile/android

example checks? http://eslint.org/docs/rules/
in automation? Treeherder tier 1 "ES" job
config? mobile/*.eslintrc & root .eslintignore
open issues? Open a .eslintrc and enable checks.
meta bug? meta bug
If you fix all issues for a warning, modify the appropriate .eslintrc to change your warning into an error!

@@ Line 24: / Line 24: @@
 Checks code style
   ./mach gradle app:checkstyle
+Or [[Mobile/Fennec/Android/Static_analysis/checkstyle-idea|run it in Intellij/Android Studio]].
 * '''example checks?''' [http://checkstyle.sourceforge.net/google_style.html Google's style guide]
 * '''in automation?''' Treeherder tier 2 "checkstyle" job