summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Lee Shombert <shombert@google.com> 2024-10-03 12:36:44 -0700
committer Lee Shombert <shombert@google.com> 2024-10-03 12:36:44 -0700
commitaa381d3b4800b5e004afb3d7dfbb5819ca175d6c (patch)
tree4253135eefaef26239e5d9aa966fef84cd933c93
parentf5df21d070e1101d0fcbc0937d39e0f2c816ad3a (diff)
Correct an ApplicationSharedMemory unit test
The test `ApplicationSharedMemory.canRead()` verifies that the network time can be read from shared memory. The read will throw if the time is not initialized, either because there is no time source or because the feature that writes to shared memory is disabled. This change accepts the particular `DateTimeException`. Note that an exception is thrown only if the memory was actually read, which means the test passed. Flag: EXEMPT test-only Bug: 365575551 Test: atest * ApplicationSharedMemoryTest32 Change-Id: I455cb5b6e8b8a04557929495dfda8979f9da8e89
-rw-r--r--tests/Internal/src/com/android/internal/os/ApplicationSharedMemoryTest.java9
1 files changed, 7 insertions, 2 deletions
diff --git a/tests/Internal/src/com/android/internal/os/ApplicationSharedMemoryTest.java b/tests/Internal/src/com/android/internal/os/ApplicationSharedMemoryTest.java
index e3a129fb1059..d03ad5cb2877 100644
--- a/tests/Internal/src/com/android/internal/os/ApplicationSharedMemoryTest.java
+++ b/tests/Internal/src/com/android/internal/os/ApplicationSharedMemoryTest.java
@@ -61,8 +61,13 @@ public class ApplicationSharedMemoryTest {
@Test
public void canRead() {
ApplicationSharedMemory instance = ApplicationSharedMemory.getInstance();
- instance.getLatestNetworkTimeUnixEpochMillisAtZeroElapsedRealtimeMillis();
- // Don't actually care about the value of the above.
+ try {
+ instance.getLatestNetworkTimeUnixEpochMillisAtZeroElapsedRealtimeMillis();
+ // Don't actually care about the value of the above.
+ } catch (java.time.DateTimeException e) {
+ // This exception is okay during testing. It means there was no time source, which
+ // could be because of network problems or a feature being flagged off.
+ }
}
/** Application processes should not have mutable access. */