Skip to content

Commit

Permalink
build: fix build break in #2790. (#2806)
Browse files Browse the repository at this point in the history
Some absl::optional changes missed due to not being merged with #2688.

Signed-off-by: Harvey Tuch <[email protected]>
  • Loading branch information
htuch authored Mar 14, 2018
1 parent abe1b24 commit 7d03b23
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/common/http/conn_manager_impl_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -929,7 +929,7 @@ TEST_F(HttpConnectionManagerImplTest, TestAccessLogWithInvalidRequest) {
EXPECT_CALL(*handler, log(_, _, _))
.WillOnce(Invoke(
[](const HeaderMap*, const HeaderMap*, const RequestInfo::RequestInfo& request_info) {
EXPECT_TRUE(request_info.responseCode().valid());
EXPECT_TRUE(request_info.responseCode());
EXPECT_EQ(request_info.responseCode().value(), uint32_t(400));
EXPECT_NE(nullptr, request_info.downstreamLocalAddress());
EXPECT_NE(nullptr, request_info.downstreamRemoteAddress());
Expand Down

0 comments on commit 7d03b23

Please sign in to comment.