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;
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;
dri-devel@lists.freedesktop.org