ARROW-12950: [C++] Add more tests for count_substring

lidavidm · lidavidm · commit 883f27df8ad9 · 2021-06-07T12:26:36.000-04:00
diff --git a/cpp/src/arrow/compute/kernels/scalar_string_test.cc b/cpp/src/arrow/compute/kernels/scalar_string_test.cc
@@ -109,6 +109,10 @@ TYPED_TEST(TestBinaryKernels, CountSubstring) {
   this->CheckUnary("count_substring", R"(["", null, "abc"])", this->offset_type(),
                    "[1, null, 4]", &options_empty);
 
+  MatchSubstringOptions options_repeated{"aaa"};
+  this->CheckUnary("count_substring", R"(["", "aaaa", "aaaaa", "aaaaaa", "aaá"])",
+                   this->offset_type(), "[0, 1, 1, 2, 0]", &options_repeated);
+
   // TODO: case-insensitive
 }
 

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,10 @@ TYPED_TEST(TestBinaryKernels, CountSubstring) {`
`109`	`109`	`this->CheckUnary("count_substring", R"(["", null, "abc"])", this->offset_type(),`
`110`	`110`	`"[1, null, 4]", &options_empty);`
`111`	`111`
	`112`	`+ MatchSubstringOptions options_repeated{"aaa"};`
	`113`	`+ this->CheckUnary("count_substring", R"(["", "aaaa", "aaaaa", "aaaaaa", "aaá"])",`
	`114`	`+ this->offset_type(), "[0, 1, 1, 2, 0]", &options_repeated);`
	`115`	`+`
`112`	`116`	`// TODO: case-insensitive`
`113`	`117`	`}`
`114`	`118`