diff mbox

[hwc,v1] drm_hwcomposer: Fix comment on worker_test

Message ID 20180226061742.19999-1-rhyskidd@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Rhys Kidd Feb. 26, 2018, 6:17 a.m. UTC 
Reflect the boolean logic of the accompanying code, which is run after
Worker.InitWorker() has completed in test setup.

Change-Id: If5187bdaa944c5d74a70c55b5b5e58e1baa20511
Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>
---
 tests/worker_test.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Robert Foss Feb. 28, 2018, 11 a.m. UTC | #1 
Thanks Rhys! Pushed.

I removed the Change-Id field though.


Rob.

On 02/26/2018 07:17 AM, Rhys Kidd wrote:
> Reflect the boolean logic of the accompanying code, which is run after
> Worker.InitWorker() has completed in test setup.
> 
> Change-Id: If5187bdaa944c5d74a70c55b5b5e58e1baa20511
> Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>
> ---
>   tests/worker_test.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/worker_test.cpp b/tests/worker_test.cpp
> index 38f91db..82523f0 100644
> --- a/tests/worker_test.cpp
> +++ b/tests/worker_test.cpp
> @@ -66,7 +66,7 @@ struct WorkerTest : public testing::Test {
>   };
>   
>   TEST_F(WorkerTest, test_worker) {
> -  // already isInitialized so should fail
> +  // already isInitialized so should succeed
>     ASSERT_TRUE(worker.initialized());
>   
>     int val = worker.value;
>
diff mbox

Patch

diff --git a/tests/worker_test.cpp b/tests/worker_test.cpp
index 38f91db..82523f0 100644
--- a/tests/worker_test.cpp
+++ b/tests/worker_test.cpp
@@ -66,7 +66,7 @@  struct WorkerTest : public testing::Test {
 };
 
 TEST_F(WorkerTest, test_worker) {
-  // already isInitialized so should fail
+  // already isInitialized so should succeed
   ASSERT_TRUE(worker.initialized());
 
   int val = worker.value;