This is an automated email from the ASF dual-hosted git repository.
isapego pushed a commit to branch ignite-17607
in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/ignite-17607 by this push:
new cec1bcd204 IGNITE-17607 Fix tests
cec1bcd204 is described below
commit cec1bcd20448e41c064a93710fbfc55733c95bd9
Author: Igor Sapego <[email protected]>
AuthorDate: Mon Mar 20 19:45:50 2023 +0300
IGNITE-17607 Fix tests
---
modules/platforms/cpp/tests/client-test/compute_test.cpp | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/modules/platforms/cpp/tests/client-test/compute_test.cpp
b/modules/platforms/cpp/tests/client-test/compute_test.cpp
index 9734c4d7f6..91d14489e0 100644
--- a/modules/platforms/cpp/tests/client-test/compute_test.cpp
+++ b/modules/platforms/cpp/tests/client-test/compute_test.cpp
@@ -102,7 +102,7 @@ TEST_F(compute_test, get_cluster_nodes) {
std::sort(cluster_nodes.begin(), cluster_nodes.end(),
[](const auto &n1, const auto &n2) { return n1.get_name() <
n2.get_name(); });
- ASSERT_EQ(2, cluster_nodes.size());
+ ASSERT_EQ(4, cluster_nodes.size());
EXPECT_FALSE(cluster_nodes[0].get_id().empty());
EXPECT_FALSE(cluster_nodes[1].get_id().empty());
@@ -150,10 +150,12 @@ TEST_F(compute_test, execute_broadcast_one_node) {
TEST_F(compute_test, execute_broadcast_all_nodes) {
auto res = m_client.get_compute().broadcast(get_node_set(), NODE_NAME_JOB,
{"42"});
- ASSERT_EQ(res.size(), 2);
+ ASSERT_EQ(res.size(), 4);
EXPECT_EQ(res[get_node(0)].value(), get_node(0).get_name() + "42");
EXPECT_EQ(res[get_node(1)].value(), get_node(1).get_name() + "42");
+ EXPECT_EQ(res[get_node(2)].value(), get_node(2).get_name() + "42");
+ EXPECT_EQ(res[get_node(3)].value(), get_node(3).get_name() + "42");
}
TEST_F(compute_test, execute_with_args) {
@@ -240,7 +242,7 @@ TEST_F(compute_test, all_arg_types) {
}
TEST_F(compute_test, execute_colocated) {
- std::map<std::int32_t, std::string> nodes_for_values = {{1, ""}, {2,
"_2"}, {3, ""}, {5, "_2"}};
+ std::map<std::int32_t, std::string> nodes_for_values = {{1, "_4"}, {5,
"_2"}, {9, "_3"}, {10, ""}, {11, "_2"}};
for (const auto &var : nodes_for_values) {
SCOPED_TRACE("key=" + std::to_string(var.first) + ", node=" +
var.second);