1 files changed, 34 insertions, 34 deletions
diff --git a/document/src/tests/tensor_fieldvalue/partial_modify/partial_modify_test.cpp b/document/src/tests/tensor_fieldvalue/partial_modify/partial_modify_test.cpp
index 71f35102d17..26f7a10468a 100644
--- a/document/src/tests/tensor_fieldvalue/partial_modify/partial_modify_test.cpp
+++ b/document/src/tests/tensor_fieldvalue/partial_modify/partial_modify_test.cpp
@@ -1,7 +1,7 @@
 // Copyright Verizon Media. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 
 #include <vespa/eval/eval/simple_value.h>
-#include <vespa/eval/eval/test/tensor_model.h>
+#include <vespa/eval/eval/test/gen_spec.h>
 #include <vespa/eval/eval/value_codec.h>
 #include <vespa/document/update/tensor_partial_update.h>
 #include <vespa/vespalib/util/stringfmt.h>
@@ -15,15 +15,13 @@ using namespace vespalib::eval::test;
 
 using vespalib::make_string_short::fmt;
 
-std::vector<Layout> modify_layouts = {
-    {x({"a"})},                           {x({"a"})},
-    {x({"a",""})},                        {x({"b","c","d","e"})},
-    {x(5)},                               {x({"1","2","foo","17"})},
-    {x({"a","b","c"}),y({"d","e"})},      {x({"b"}),y({"d"})},             
-    {x({"a","b","c"})},                   {x({"b","c","d"})},
-    {x(4),y({"a","b","c","d"}),z(5)},     {x({"1","2"}),y({"b","d"}),z({"1","3"})},
-    {x(3),y(2)},                          {x({"0","1"}),y({"0","1"})},
-    {x({"a","","b"})},                    {x({""})}
+std::vector<std::pair<vespalib::string,vespalib::string>> modify_layouts = {
+    {       "x4_1",         "x4_1" },
+    {       "x4_1",         "x4_2" },
+    {         "x4",         "x4_2" },
+    {   "x4_1y4_2",     "x4_2y4_1" },
+    {   "x4y4_1z4", "x4_2y4_2z4_2" },
+    {       "x3y2",     "x2_1y2_1" }
 };
 
 TensorSpec::Address sparsify(const TensorSpec::Address &input) {
@@ -52,7 +50,7 @@ TensorSpec reference_modify(const TensorSpec &a, const TensorSpec &b, join_fun_t
             result.add(cell.first, fun(v, iter->second));
         }
     }
-    return result;
+    return result.normalize();
 }
 
 Value::UP try_partial_modify(const TensorSpec &a, const TensorSpec &b, join_fun_t fun) {
@@ -69,36 +67,38 @@ TensorSpec perform_partial_modify(const TensorSpec &a, const TensorSpec &b, join
 }
 
 TEST(PartialModifyTest, partial_modify_works_for_simple_values) {
-    ASSERT_TRUE((modify_layouts.size() % 2) == 0);
-    for (size_t i = 0; i < modify_layouts.size(); i += 2) {
-        TensorSpec lhs = spec(modify_layouts[i], N());
-        TensorSpec rhs = spec(modify_layouts[i + 1], Div16(N()));
-        SCOPED_TRACE(fmt("\n===\nLHS: %s\nRHS: %s\n===\n", lhs.to_string().c_str(), rhs.to_string().c_str()));
-        for (auto fun: {operation::Add::f, operation::Mul::f, operation::Sub::f}) {
-            auto expect = reference_modify(lhs, rhs, fun);
-            auto actual = perform_partial_modify(lhs, rhs, fun);
-            EXPECT_EQ(actual, expect);
+    for (const auto &layouts: modify_layouts) {
+        for (auto lhs_ct: CellTypeUtils::list_types()) {
+            for (auto rhs_ct: CellTypeUtils::list_types()) {
+                TensorSpec lhs = GenSpec::from_desc(layouts.first).cells(lhs_ct).seq(N());
+                TensorSpec rhs = GenSpec::from_desc(layouts.second).cells(rhs_ct).seq(Div16(N()));
+                SCOPED_TRACE(fmt("\n===\nLHS: %s\nRHS: %s\n===\n", lhs.to_string().c_str(), rhs.to_string().c_str()));
+                for (auto fun: {operation::Add::f, operation::Mul::f, operation::Sub::f}) {
+                    auto expect = reference_modify(lhs, rhs, fun);
+                    auto actual = perform_partial_modify(lhs, rhs, fun);
+                    EXPECT_EQ(actual, expect);
+                }
+                auto fun = [](double, double keep) { return keep; };
+                auto expect = reference_modify(lhs, rhs, fun);
+                auto actual = perform_partial_modify(lhs, rhs, fun);
+                EXPECT_EQ(actual, expect);
+            }
         }
-        auto fun = [](double, double keep) { return keep; };
-        auto expect = reference_modify(lhs, rhs, fun);
-        auto actual = perform_partial_modify(lhs, rhs, fun);
-        EXPECT_EQ(actual, expect);
     }
 }
 
-std::vector<Layout> bad_layouts = {
-    {x(3)},                               {x(3)},
-    {x(3),y({"a"})},                      {x(3),y({"a"})},
-    {x({"a"})},                           {x({"a"}),y({"b"})},
-    {x({"a"}),y({"b"})},                  {x({"a"})},
-    {x({"a"})},                           {x({"a"}),y(1)}
+std::vector<std::pair<vespalib::string,vespalib::string>> bad_layouts = {
+    {       "x3",       "x3" },
+    {   "x3y4_1",   "x3y4_1" },
+    {     "x4_1", "x4_1y4_1" },
+    { "x4_1y4_1",     "x4_1" },
+    {     "x4_1",   "x4_1y1" }
 };
 
 TEST(PartialModifyTest, partial_modify_returns_nullptr_on_invalid_inputs) {
-    ASSERT_TRUE((bad_layouts.size() % 2) == 0);
-    for (size_t i = 0; i < bad_layouts.size(); i += 2) {
-        TensorSpec lhs = spec(bad_layouts[i], N());
-        TensorSpec rhs = spec(bad_layouts[i + 1], Div16(N()));
+    for (const auto &layouts: bad_layouts) {
+        TensorSpec lhs = GenSpec::from_desc(layouts.first).seq(N());
+        TensorSpec rhs = GenSpec::from_desc(layouts.second).seq(Div16(N()));
         SCOPED_TRACE(fmt("\n===\nLHS: %s\nRHS: %s\n===\n", lhs.to_string().c_str(), rhs.to_string().c_str()));
         for (auto fun: {operation::Add::f}) {
             auto actual = try_partial_modify(lhs, rhs, fun);