Skip to content

Commit a4d7066

Browse files
committed
Log res.error() in GetWithRange test assertions to diagnose flaky CI failures
1 parent bd455f8 commit a4d7066

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

test/test.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5753,7 +5753,7 @@ TEST_F(ServerTest, GetWithRange1) {
57535753
make_range_header({{3, 5}}),
57545754
{"Accept-Encoding", ""},
57555755
});
5756-
ASSERT_TRUE(res);
5756+
ASSERT_TRUE(res) << "Error: " << to_string(res.error());
57575757
EXPECT_EQ(StatusCode::PartialContent_206, res->status);
57585758
EXPECT_EQ("3", res->get_header_value("Content-Length"));
57595759
EXPECT_EQ(true, res->has_header("Content-Range"));
@@ -5766,7 +5766,7 @@ TEST_F(ServerTest, GetWithRange2) {
57665766
make_range_header({{1, -1}}),
57675767
{"Accept-Encoding", ""},
57685768
});
5769-
ASSERT_TRUE(res);
5769+
ASSERT_TRUE(res) << "Error: " << to_string(res.error());
57705770
EXPECT_EQ(StatusCode::PartialContent_206, res->status);
57715771
EXPECT_EQ("6", res->get_header_value("Content-Length"));
57725772
EXPECT_EQ(true, res->has_header("Content-Range"));
@@ -5779,7 +5779,7 @@ TEST_F(ServerTest, GetWithRange3) {
57795779
make_range_header({{0, 0}}),
57805780
{"Accept-Encoding", ""},
57815781
});
5782-
ASSERT_TRUE(res);
5782+
ASSERT_TRUE(res) << "Error: " << to_string(res.error());
57835783
EXPECT_EQ(StatusCode::PartialContent_206, res->status);
57845784
EXPECT_EQ("1", res->get_header_value("Content-Length"));
57855785
EXPECT_EQ(true, res->has_header("Content-Range"));
@@ -5792,7 +5792,7 @@ TEST_F(ServerTest, GetWithRange4) {
57925792
make_range_header({{-1, 2}}),
57935793
{"Accept-Encoding", ""},
57945794
});
5795-
ASSERT_TRUE(res);
5795+
ASSERT_TRUE(res) << "Error: " << to_string(res.error());
57965796
EXPECT_EQ(StatusCode::PartialContent_206, res->status);
57975797
EXPECT_EQ("2", res->get_header_value("Content-Length"));
57985798
EXPECT_EQ(true, res->has_header("Content-Range"));
@@ -5805,7 +5805,7 @@ TEST_F(ServerTest, GetWithRange5) {
58055805
make_range_header({{0, 5}}),
58065806
{"Accept-Encoding", ""},
58075807
});
5808-
ASSERT_TRUE(res);
5808+
ASSERT_TRUE(res) << "Error: " << to_string(res.error());
58095809
EXPECT_EQ(StatusCode::PartialContent_206, res->status);
58105810
EXPECT_EQ("6", res->get_header_value("Content-Length"));
58115811
EXPECT_EQ(true, res->has_header("Content-Range"));

0 commit comments

Comments
 (0)