Commit 8b43cc42 authored by ssid's avatar ssid Committed by Commit Bot

Fix ProcessMemoryMetricsEmitterTest

Do not check for uptime to be less than 10. Process could be running
longer in the test. This does not fix the failure due to missing uptime
value.

BUG=731466

Change-Id: I0b176da810dd8f3bc90ad4b94386bc46e68fa1e7
Reviewed-on: https://chromium-review.googlesource.com/c/1338954Reviewed-by: default avatarErik Chen <erikchen@chromium.org>
Reviewed-by: default avatarIlya Sherman <isherman@chromium.org>
Commit-Queue: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630104}
parent 16f86086
...@@ -329,7 +329,7 @@ class ProcessMemoryMetricsEmitterTest ...@@ -329,7 +329,7 @@ class ProcessMemoryMetricsEmitterTest
const int64_t* value = ukm::TestUkmRecorder::GetEntryMetric(entry, name); const int64_t* value = ukm::TestUkmRecorder::GetEntryMetric(entry, name);
ASSERT_TRUE(value) << name; ASSERT_TRUE(value) << name;
EXPECT_GE(*value, 0) << name; EXPECT_GE(*value, 0) << name;
EXPECT_LE(*value, 10) << name; EXPECT_LE(*value, 300) << name;
} }
void CheckAllUkmEntries(size_t entry_count = 1u) { void CheckAllUkmEntries(size_t entry_count = 1u) {
......
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