Browse Source

test: disable CapturableScreen tests on Windows x64 (#41544)

* test: disable CapturableScreen tests on Windows x64

(cherry picked from commit 60a288a2ca699e4a9b7637d4b0360974c6be5aba)

* test: disable js-execute-iframe" case should not crash on win 32-bit

(cherry picked from commit d545ae049b41b6fe35f4ee519aa63e0a28858cca)
John Kleinschmidt 1 year ago
parent
commit
33e61a19ef
2 changed files with 8 additions and 4 deletions
  1. 6 2
      spec/crash-spec.ts
  2. 2 2
      spec/lib/screen-helpers.ts

+ 6 - 2
spec/crash-spec.ts

@@ -40,9 +40,13 @@ const shouldRunCase = (crashCase: string) => {
     case 'quit-on-crashed-event': {
       return (process.platform !== 'win32' || process.arch !== 'ia32');
     }
-    // TODO(jkleinsc) fix this test on Linux on arm/arm64
+    // TODO(jkleinsc) fix this test on Linux on arm/arm64 and 32bit windows
     case 'js-execute-iframe': {
-      return (process.platform !== 'linux' || (process.arch !== 'arm64' && process.arch !== 'arm'));
+      if (process.platform === 'win32') {
+        return process.arch !== 'ia32';
+      } else {
+        return (process.platform !== 'linux' || (process.arch !== 'arm64' && process.arch !== 'arm'));
+      }
     }
     default: {
       return true;

+ 2 - 2
spec/lib/screen-helpers.ts

@@ -99,8 +99,8 @@ export const areColorsSimilar = (
  * - Linux: virtual screen display is 0x0
  * - Win32 arm64 (WOA): virtual screen display is 0x0
  * - Win32 ia32: skipped
+ * - Win32 x64: virtual screen display is 0x0
  */
 export const hasCapturableScreen = () => {
-  return process.platform === 'darwin' ||
-    (process.platform === 'win32' && process.arch === 'x64');
+  return process.platform === 'darwin';
 };