| commit | ddcae249c56d889064f5692f4713f3299ca14c0a | [log] [tgz] |
|---|---|---|
| author | felipeal <felipeal@google.com> | Fri May 01 08:10:40 2020 -0700 |
| committer | Felipe Leme <felipeal@google.com> | Fri May 01 13:43:30 2020 -0700 |
| tree | e54485528e9ab2f8aef33f18df288512d16fe340 | |
| parent | fc95ea8f9f04302a46051d38a9ba92636cf353ae [diff] |
Improvements on AndroidCarApiTest package. - Use JUnit4 instead of deprecated JUnit3 Android classe. - Move common code (like assertMainThread()) to common class. - Use assertThrows() to check for exceptions. - Throw exception on tests instead of catching and failing. - Added missing @Test annotations. - Change some tests to extend CarApiTestBase - Removed @RunWith(AndroidJUnit4.class) - these tests can run with JUnit.class - Other cosmetics changes like using lambda instead of annonymous classes. - Disabled CarNavigationManagerTest.testSendEvent() as it's not passing Test: atest AndroidCarApiTest Fixes: 155477313 Bug: 155447204 Bug: 155343605 Bug: 142554800 Bug: 30839080 Change-Id: I4273ed92c37eae3cd6ad16fa3ae88ea982fc7db8
Native (C++) code format is required to be compatible with .clang-format file. Run
git clang-format --style=file --extension='h,cpp,cc' HEAD~
Note that clang-format is not desirable for Android java files. Therefore the command line above is limited to specific extensions.