commit 1e4f9329fb6257a8a1f6f17db0966b2dc91de674
author Cici Ruan <cuicuiruan@google.com> Wed Aug 29 15:14:15 2018 +0800
committer Cuicui Ruan <cuicuiruan@google.com> Mon Sep 10 17:59:30 2018 -0700
tree 8d57e698c8a1a548b11fa4fdd0c2bd00636615f6
parent de03fbdf280a563cb67a5be3c89913a1978d7605

Skip test for Chromebook

For Chromebook clamshell mode, the activity may not cover the whole
screen for portrait activity and landscape physical screen (It shows the black background on both sides). The test gets the smallest
screenlayout size as expected size from different orientations. Then it
compares the expected size with each screenlayout size in different
orientations. In conclusion, clamshell mode doesn't return same screenlayout size in different orientations, which fails the test. So skipping the test for Chromebook here.

Bug: 78248059
Test: atest android.dpi.cts.ConfigurationScreenLayoutTest#testScreenLayout on Kevin.
Change-Id: Ib39f22e634f2081010f468d484749e73a0f81560

tests/tests/dpi/src/android/dpi/cts/ConfigurationScreenLayoutTest.java

diff --git a/tests/tests/dpi/src/android/dpi/cts/ConfigurationScreenLayoutTest.java b/tests/tests/dpi/src/android/dpi/cts/ConfigurationScreenLayoutTest.java
index d7c9366..a7b2692 100644
--- a/tests/tests/dpi/src/android/dpi/cts/ConfigurationScreenLayoutTest.java
+++ b/tests/tests/dpi/src/android/dpi/cts/ConfigurationScreenLayoutTest.java
@@ -49,6 +49,14 @@
             tearDown();
             return;
         }
+        if (isPC()) {
+            // The test skips mainly for Chromebook clamshell mode. For Chromebook clamshell mode
+            // with non-rotated landscape physical screen, the portrait window/activity has special
+            // behavior with black background on both sides to make the window/activity look
+            // portrait, which returns smaller screen layout size.
+            tearDown();
+            return;
+        }
         int expectedScreenLayout = computeScreenLayout();
         int expectedSize = expectedScreenLayout & Configuration.SCREENLAYOUT_SIZE_MASK;
         int expectedLong = expectedScreenLayout & Configuration.SCREENLAYOUT_LONG_MASK;
@@ -163,4 +171,9 @@
         return (supportsLandscape && supportsPortrait)
                 || (!supportsLandscape && !supportsPortrait);
     }
+
+    // Check if it is a PC device
+    private boolean isPC() {
+        return hasDeviceFeature(PackageManager.FEATURE_PC);
+    }
 }