diff options
author | Geir Storli <geirst@verizonmedia.com> | 2019-02-25 08:18:16 +0000 |
---|---|---|
committer | Geir Storli <geirst@verizonmedia.com> | 2019-02-25 08:18:16 +0000 |
commit | 531435ea9289334a543fae7cef4d56a4d4ef34fb (patch) | |
tree | a6c31228e7d931d709e7045653a74751e468f8b7 /eval/src/tests | |
parent | efe78d724c0d0eb7e49ef6f4ed42d4efbc61a3ac (diff) |
Migrate unit tests to gtest.
Diffstat (limited to 'eval/src/tests')
6 files changed, 48 insertions, 41 deletions
diff --git a/eval/src/tests/tensor/tensor_add_operation/CMakeLists.txt b/eval/src/tests/tensor/tensor_add_operation/CMakeLists.txt index 6c19b13db7e..e511b23c5ec 100644 --- a/eval/src/tests/tensor/tensor_add_operation/CMakeLists.txt +++ b/eval/src/tests/tensor/tensor_add_operation/CMakeLists.txt @@ -4,5 +4,6 @@ vespa_add_executable(eval_tensor_add_operation_test_app TEST tensor_add_operation_test.cpp DEPENDS vespaeval + gtest ) vespa_add_test(NAME eval_tensor_add_operation_test_app COMMAND eval_tensor_add_operation_test_app) diff --git a/eval/src/tests/tensor/tensor_add_operation/tensor_add_operation_test.cpp b/eval/src/tests/tensor/tensor_add_operation/tensor_add_operation_test.cpp index 5ac1c503b5d..2f36aba6e3a 100644 --- a/eval/src/tests/tensor/tensor_add_operation/tensor_add_operation_test.cpp +++ b/eval/src/tests/tensor/tensor_add_operation/tensor_add_operation_test.cpp @@ -4,7 +4,7 @@ #include <vespa/eval/tensor/default_tensor_engine.h> #include <vespa/eval/tensor/sparse/sparse_tensor.h> #include <vespa/eval/tensor/test/test_utils.h> -#include <vespa/vespalib/testkit/test_kit.h> +#include <vespa/vespalib/gtest/gtest.h> using vespalib::eval::Value; using vespalib::eval::TensorSpec; @@ -18,10 +18,10 @@ assertAdd(const TensorSpec &source, const TensorSpec &arg, const TensorSpec &exp auto argTensor = makeTensor<Tensor>(arg); auto resultTensor = sourceTensor->add(*argTensor); auto actual = resultTensor->toSpec(); - EXPECT_EQUAL(actual, expected); + EXPECT_EQ(actual, expected); } -TEST("require that cells can be added to a sparse tensor") +TEST(TensorAddTest, cells_can_be_added_to_a_sparse_tensor) { assertAdd(TensorSpec("tensor(x{},y{})") .add({{"x","a"},{"y","b"}}, 2) @@ -35,4 +35,4 @@ TEST("require that cells can be added to a sparse tensor") .add({{"x","e"},{"y","f"}}, 7)); } -TEST_MAIN() { TEST_RUN_ALL(); } +GTEST_MAIN_RUN_ALL_TESTS diff --git a/eval/src/tests/tensor/tensor_modify_operation/CMakeLists.txt b/eval/src/tests/tensor/tensor_modify_operation/CMakeLists.txt index b0ffa48eb3b..481a0aac7fb 100644 --- a/eval/src/tests/tensor/tensor_modify_operation/CMakeLists.txt +++ b/eval/src/tests/tensor/tensor_modify_operation/CMakeLists.txt @@ -4,5 +4,6 @@ vespa_add_executable(eval_tensor_modify_operation_test_app TEST tensor_modify_operation_test.cpp DEPENDS vespaeval + gtest ) vespa_add_test(NAME eval_tensor_modify_operation_test_app COMMAND eval_tensor_modify_operation_test_app) diff --git a/eval/src/tests/tensor/tensor_modify_operation/tensor_modify_operation_test.cpp b/eval/src/tests/tensor/tensor_modify_operation/tensor_modify_operation_test.cpp index 5cacff1881b..ff59b28d60c 100644 --- a/eval/src/tests/tensor/tensor_modify_operation/tensor_modify_operation_test.cpp +++ b/eval/src/tests/tensor/tensor_modify_operation/tensor_modify_operation_test.cpp @@ -1,12 +1,12 @@ // Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#include <vespa/vespalib/testkit/test_kit.h> -#include <vespa/vespalib/util/stringfmt.h> +#include <vespa/eval/eval/tensor_spec.h> #include <vespa/eval/tensor/cell_values.h> +#include <vespa/eval/tensor/default_tensor_engine.h> #include <vespa/eval/tensor/sparse/sparse_tensor.h> #include <vespa/eval/tensor/test/test_utils.h> -#include <vespa/eval/tensor/default_tensor_engine.h> -#include <vespa/eval/eval/tensor_spec.h> +#include <vespa/vespalib/gtest/gtest.h> +#include <vespa/vespalib/util/stringfmt.h> using vespalib::eval::Value; using vespalib::eval::TensorSpec; @@ -34,53 +34,57 @@ checkUpdate(const TensorSpec &source, const TensorSpec &update, const TensorSpec auto actual = actualTensor->toSpec(); auto expectTensor = makeTensor<Tensor>(expect); auto expectPadded = expectTensor->toSpec(); - EXPECT_EQUAL(actual, expectPadded); + EXPECT_EQ(actual, expectPadded); } -TEST("require that sparse tensors can be modified") { +TEST(TensorModifyTest, sparse_tensors_can_be_modified) +{ checkUpdate(TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 11) - .add({{"x","9"},{"y","9"}}, 11), + .add({{"x","8"},{"y","9"}}, 11) + .add({{"x","9"},{"y","9"}}, 11), TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 2), + .add({{"x","8"},{"y","9"}}, 2), TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 2) - .add({{"x","9"},{"y","9"}}, 11)); + .add({{"x","8"},{"y","9"}}, 2) + .add({{"x","9"},{"y","9"}}, 11)); } -TEST("require that dense tensors can be modified") { +TEST(TensorModifyTest, dense_tensors_can_be_modified) +{ checkUpdate(TensorSpec("tensor(x[10],y[10])") - .add({{"x",8},{"y",9}}, 11) - .add({{"x",9},{"y",9}}, 11), + .add({{"x",8},{"y",9}}, 11) + .add({{"x",9},{"y",9}}, 11), TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 2), + .add({{"x","8"},{"y","9"}}, 2), TensorSpec("tensor(x[10],y[10])") - .add({{"x",8},{"y",9}}, 2) - .add({{"x",9},{"y",9}}, 11)); + .add({{"x",8},{"y",9}}, 2) + .add({{"x",9},{"y",9}}, 11)); } -TEST("require that sparse tensors ignore updates to missing cells") { +TEST(TensorModifyTest, sparse_tensors_ignore_updates_to_missing_cells) +{ checkUpdate(TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 11) - .add({{"x","9"},{"y","9"}}, 11), + .add({{"x","8"},{"y","9"}}, 11) + .add({{"x","9"},{"y","9"}}, 11), TensorSpec("tensor(x{},y{})") - .add({{"x","7"},{"y","9"}}, 2) - .add({{"x","8"},{"y","9"}}, 2), + .add({{"x","7"},{"y","9"}}, 2) + .add({{"x","8"},{"y","9"}}, 2), TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 2) - .add({{"x","9"},{"y","9"}}, 11)); + .add({{"x","8"},{"y","9"}}, 2) + .add({{"x","9"},{"y","9"}}, 11)); } -TEST("require that dense tensors ignore updates to out of range cells") { +TEST(TensorModifyTest, dense_tensors_ignore_updates_to_out_of_range_cells) +{ checkUpdate(TensorSpec("tensor(x[10],y[10])") - .add({{"x",8},{"y",9}}, 11) - .add({{"x",9},{"y",9}}, 11), + .add({{"x",8},{"y",9}}, 11) + .add({{"x",9},{"y",9}}, 11), TensorSpec("tensor(x{},y{})") - .add({{"x","8"},{"y","9"}}, 2) - .add({{"x","10"},{"y","9"}}, 2), + .add({{"x","8"},{"y","9"}}, 2) + .add({{"x","10"},{"y","9"}}, 2), TensorSpec("tensor(x[10],y[10])") - .add({{"x",8},{"y",9}}, 2) - .add({{"x",9},{"y",9}}, 11)); + .add({{"x",8},{"y",9}}, 2) + .add({{"x",9},{"y",9}}, 11)); } -TEST_MAIN() { TEST_RUN_ALL(); } +GTEST_MAIN_RUN_ALL_TESTS diff --git a/eval/src/tests/tensor/tensor_remove_operation/CMakeLists.txt b/eval/src/tests/tensor/tensor_remove_operation/CMakeLists.txt index 8dfb8181f2b..ffc71563107 100644 --- a/eval/src/tests/tensor/tensor_remove_operation/CMakeLists.txt +++ b/eval/src/tests/tensor/tensor_remove_operation/CMakeLists.txt @@ -4,5 +4,6 @@ vespa_add_executable(eval_tensor_remove_operation_test_app TEST tensor_remove_operation_test.cpp DEPENDS vespaeval + gtest ) vespa_add_test(NAME eval_tensor_remove_operation_test_app COMMAND eval_tensor_remove_operation_test_app) diff --git a/eval/src/tests/tensor/tensor_remove_operation/tensor_remove_operation_test.cpp b/eval/src/tests/tensor/tensor_remove_operation/tensor_remove_operation_test.cpp index 8b0c44a6e06..cb28019c4ee 100644 --- a/eval/src/tests/tensor/tensor_remove_operation/tensor_remove_operation_test.cpp +++ b/eval/src/tests/tensor/tensor_remove_operation/tensor_remove_operation_test.cpp @@ -5,7 +5,7 @@ #include <vespa/eval/tensor/default_tensor_engine.h> #include <vespa/eval/tensor/sparse/sparse_tensor.h> #include <vespa/eval/tensor/test/test_utils.h> -#include <vespa/vespalib/testkit/test_kit.h> +#include <vespa/vespalib/gtest/gtest.h> using vespalib::eval::Value; using vespalib::eval::TensorSpec; @@ -19,10 +19,10 @@ assertRemove(const TensorSpec &source, const TensorSpec &arg, const TensorSpec & auto argTensor = makeTensor<SparseTensor>(arg); auto resultTensor = sourceTensor->remove(CellValues(*argTensor)); auto actual = resultTensor->toSpec(); - EXPECT_EQUAL(actual, expected); + EXPECT_EQ(actual, expected); } -TEST("require that cells can be removed from a sparse tensor") +TEST(TensorRemoveTest, cells_can_be_removed_from_a_sparse_tensor) { assertRemove(TensorSpec("tensor(x{},y{})") .add({{"x","a"},{"y","b"}}, 2) @@ -34,7 +34,7 @@ TEST("require that cells can be removed from a sparse tensor") .add({{"x","a"},{"y","b"}}, 2)); } -TEST("require that all cells can be removed from a sparse tensor") +TEST(TensorRemoveTest, all_cells_can_be_removed_from_a_sparse_tensor) { assertRemove(TensorSpec("tensor(x{},y{})") .add({{"x","a"},{"y","b"}}, 2), @@ -43,4 +43,4 @@ TEST("require that all cells can be removed from a sparse tensor") TensorSpec("tensor(x{},y{})")); } -TEST_MAIN() { TEST_RUN_ALL(); } +GTEST_MAIN_RUN_ALL_TESTS |