Roman Stratiienko | 13cc366 | 2020-08-29 21:35:39 +0300 | [diff] [blame] | 1 | #include "utils/Worker.h" |
Roman Stratiienko | aa3cd54 | 2020-08-29 11:26:16 +0300 | [diff] [blame] | 2 | |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 3 | #include <gtest/gtest.h> |
| 4 | #include <hardware/hardware.h> |
| 5 | |
| 6 | #include <chrono> |
| 7 | |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 8 | using android::Worker; |
| 9 | |
| 10 | struct TestWorker : public Worker { |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 11 | TestWorker() : Worker("test-worker", HAL_PRIORITY_URGENT_DISPLAY){}; |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 12 | |
| 13 | int Init() { |
| 14 | return InitWorker(); |
| 15 | } |
| 16 | |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 17 | void Routine() override { |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 18 | Lock(); |
| 19 | if (!enabled_) { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 20 | auto ret = WaitForSignalOrExitLocked(); |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 21 | if (ret == -EINTR) { |
| 22 | Unlock(); |
| 23 | return; |
| 24 | } |
| 25 | // should only reached here if it was enabled |
| 26 | if (!enabled_) |
| 27 | printf("Shouldn't reach here while disabled %d %d\n", value, ret); |
| 28 | } |
| 29 | value++; |
| 30 | Unlock(); |
| 31 | } |
| 32 | |
| 33 | void Control(bool enable) { |
| 34 | bool changed = false; |
| 35 | Lock(); |
| 36 | if (enabled_ != enable) { |
| 37 | enabled_ = enable; |
| 38 | changed = true; |
| 39 | } |
| 40 | Unlock(); |
| 41 | |
| 42 | if (enable && changed) |
| 43 | Signal(); |
| 44 | } |
| 45 | |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 46 | // NOLINTNEXTLINE: should not be public |
| 47 | int value{}; |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 48 | |
| 49 | private: |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 50 | bool enabled_{}; |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 51 | }; |
| 52 | |
| 53 | struct WorkerTest : public testing::Test { |
| 54 | TestWorker worker; |
| 55 | |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 56 | void SetUp() override { |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 57 | worker.Init(); |
| 58 | } |
| 59 | |
| 60 | void small_delay() { |
| 61 | std::this_thread::sleep_for(std::chrono::milliseconds(20)); |
| 62 | } |
| 63 | }; |
| 64 | |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 65 | // NOLINTNEXTLINE: required by gtest macros |
| 66 | TEST_F(WorkerTest, TestWorker) { |
Rhys Kidd | c7a6d27 | 2018-02-26 01:17:42 -0500 | [diff] [blame] | 67 | // already isInitialized so should succeed |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 68 | ASSERT_TRUE(worker.initialized()); |
| 69 | |
| 70 | int val = worker.value; |
| 71 | small_delay(); |
| 72 | |
| 73 | // value shouldn't change when isInitialized |
| 74 | ASSERT_EQ(val, worker.value); |
| 75 | |
| 76 | worker.Control(true); |
| 77 | small_delay(); |
| 78 | |
| 79 | // while locked, value shouldn't be changing |
| 80 | worker.Lock(); |
| 81 | val = worker.value; |
| 82 | small_delay(); |
| 83 | ASSERT_EQ(val, worker.value); |
| 84 | worker.Unlock(); |
| 85 | |
| 86 | small_delay(); |
| 87 | // value should be different now |
| 88 | ASSERT_NE(val, worker.value); |
| 89 | |
| 90 | worker.Control(false); |
| 91 | worker.Lock(); |
| 92 | val = worker.value; |
| 93 | worker.Unlock(); |
| 94 | small_delay(); |
| 95 | |
| 96 | // value should be same |
| 97 | ASSERT_EQ(val, worker.value); |
| 98 | |
| 99 | worker.Exit(); |
| 100 | ASSERT_FALSE(worker.initialized()); |
| 101 | } |
| 102 | |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 103 | // NOLINTNEXTLINE: required by gtest macros |
| 104 | TEST_F(WorkerTest, ExitWhileRunning) { |
Adrian Salido | fa37f67 | 2017-02-16 10:29:46 -0800 | [diff] [blame] | 105 | worker.Control(true); |
| 106 | |
| 107 | std::this_thread::sleep_for(std::chrono::milliseconds(50)); |
| 108 | worker.Exit(); |
| 109 | } |