Fix temporary directory leaks in the component updater unit tests.
Bug: 773049
Change-Id: Ib49946e9831e737e7ab65d5566b4601006a4bedf
Reviewed-on: https://chromium-review.googlesource.com/707593
Reviewed-by: Joshua Pawlicki <[email protected]>
Commit-Queue: Sorin Jianu <[email protected]>
Cr-Commit-Position: refs/heads/master@{#507527}diff --git a/components/update_client/update_client_unittest.cc b/components/update_client/update_client_unittest.cc
index 11c10d8..65b6f71 100644
--- a/components/update_client/update_client_unittest.cc
+++ b/components/update_client/update_client_unittest.cc
@@ -949,11 +949,8 @@
download_metrics.total_bytes = 0;
download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("jebgalgnebhfojomionfpkfelancnnkf.crx"), &path));
-
+ // The result must not include a file path in the case of errors.
result.error = -118;
- result.response = path;
result.downloaded_bytes = 0;
result.total_bytes = 0;
} else if (url.path() ==
@@ -1776,11 +1773,8 @@
download_metrics.total_bytes = 2105;
download_metrics.download_time_ms = 1000;
- EXPECT_TRUE(MakeTestFile(
- TestFilePath("ihfokbkgjpifnbbojhneepfflplebdkc_1to2.crx"), &path));
-
+ // The response must not include a file path in the case of errors.
result.error = -1;
- result.response = path;
result.downloaded_bytes = 0;
result.total_bytes = 2105;
} else if (url.path() ==
