Commit fd32fb67 authored by Josh Nohle's avatar Josh Nohle Committed by Commit Bot

[Nearby] Fix early return in device data updater tests

Bug: b/157685298
Change-Id: I502eb7923a4620e0dfbc4edde178e4cc3f2d043f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332869
Auto-Submit: Josh Nohle <nohle@chromium.org>
Reviewed-by: default avatarJames Vecore <vecore@google.com>
Commit-Queue: Josh Nohle <nohle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795032}
parent f3c95fae
...@@ -176,14 +176,14 @@ class NearbyShareDeviceDataUpdaterImplTest : public ::testing::Test { ...@@ -176,14 +176,14 @@ class NearbyShareDeviceDataUpdaterImplTest : public ::testing::Test {
case UpdateDeviceRequestResult::kSuccess: case UpdateDeviceRequestResult::kSuccess:
std::move(client->update_device_requests()[0].callback) std::move(client->update_device_requests()[0].callback)
.Run(TestResponse()); .Run(TestResponse());
return; break;
case UpdateDeviceRequestResult::kHttpFailure: case UpdateDeviceRequestResult::kHttpFailure:
std::move(client->update_device_requests()[0].error_callback) std::move(client->update_device_requests()[0].error_callback)
.Run(NearbyShareRequestError::kBadRequest); .Run(NearbyShareRequestError::kBadRequest);
return; break;
case UpdateDeviceRequestResult::kTimeout: case UpdateDeviceRequestResult::kTimeout:
FastForward(kTestTimeout); FastForward(kTestTimeout);
return; break;
} }
EXPECT_EQ(num_responses + 1, responses_.size()); EXPECT_EQ(num_responses + 1, responses_.size());
......
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