Unverified Commit 189a8018 authored by Nikita Titov's avatar Nikita Titov Committed by GitHub
Browse files

fix compiler warning about types conversion in cpp tests (#4418)

parent b918b5b2
......@@ -18,7 +18,7 @@ TEST(Partition, JustWorks) {
std::vector<score_t> gradients({0.5f, 5.0f, 1.0f, 2.0f, 2.0f});
data_size_t middle_begin, middle_end;
ArrayArgs<score_t>::Partition(&gradients, 0, gradients.size(), &middle_begin, &middle_end);
ArrayArgs<score_t>::Partition(&gradients, 0, static_cast<int>(gradients.size()), &middle_begin, &middle_end);
EXPECT_EQ(gradients[middle_begin + 1], gradients[middle_end - 1]);
EXPECT_GT(gradients[0], gradients[middle_begin + 1]);
......@@ -28,14 +28,14 @@ TEST(Partition, JustWorks) {
TEST(Partition, PartitionOneElement) {
std::vector<score_t> gradients({0.5f});
data_size_t middle_begin, middle_end;
ArrayArgs<score_t>::Partition(&gradients, 0, gradients.size(), &middle_begin, &middle_end);
ArrayArgs<score_t>::Partition(&gradients, 0, static_cast<int>(gradients.size()), &middle_begin, &middle_end);
EXPECT_EQ(gradients[middle_begin + 1], gradients[middle_end - 1]);
}
TEST(Partition, Empty) {
std::vector<score_t> gradients;
data_size_t middle_begin, middle_end;
ArrayArgs<score_t>::Partition(&gradients, 0, gradients.size(), &middle_begin, &middle_end);
ArrayArgs<score_t>::Partition(&gradients, 0, static_cast<int>(gradients.size()), &middle_begin, &middle_end);
EXPECT_EQ(middle_begin, -1);
EXPECT_EQ(middle_end, 0);
......@@ -44,7 +44,7 @@ TEST(Partition, Empty) {
TEST(Partition, AllEqual) {
std::vector<score_t> gradients({0.5f, 0.5f, 0.5f});
data_size_t middle_begin, middle_end;
ArrayArgs<score_t>::Partition(&gradients, 0, gradients.size(), &middle_begin, &middle_end);
ArrayArgs<score_t>::Partition(&gradients, 0, static_cast<int>(gradients.size()), &middle_begin, &middle_end);
EXPECT_EQ(gradients[middle_begin + 1], gradients[middle_end - 1]);
EXPECT_EQ(middle_begin, -1);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment