| commit | 16c7157867e612168144a98de37abd208f0c48a0 | [log] [tgz] |
|---|---|---|
| author | Changyeon Jo <changyeon@google.com> | Fri Jun 18 10:08:53 2021 -0700 |
| committer | Changyeon Jo <changyeon@google.com> | Fri Jun 18 10:08:53 2021 -0700 |
| tree | 7956257b81663982fa723a1440ae4d359e5b98e0 | |
| parent | 63d41410c02ee9aa046a31548c46f4634d4d6d3c [diff] |
Add EvsManager null check to tearDown()
If CarEvsService is not enable on the device under test, it breaks an
assumption in CarEvsManagerTest::setUp() and therefore the test executor
calls tearDown() method with a null EvsManager reference.
CarEvsManagerTest::tearDown() requests EvsManager to stop a video stream
and fails to invoke a method because of null EvsManager reference.
To fix this issue, this CL add a null check before invoking a
CarEvsManager method in tearDown().
Fix: 191192986
Test: Run 'atest CarEvsManagerTest -c' and confirm all test results are
assumption failed on Seahawk.
Change-Id: Ib21e13ebfd646a65e8e207d3f809674a6f96b6c2
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.