refactor(suite-native): conditional jest test condition negated

trezor · Dec 17, 2024 · 9055e1e · 9055e1e
1 parent cfb7ac2
commit 9055e1e
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 12 deletions.
diff --git a/suite-common/test-utils/src/conditionalDescribe.ts b/suite-common/test-utils/src/conditionalDescribe.ts
@@ -1,11 +1,7 @@
-export const conditionalDescribe = (
-    skipCondition: boolean,
-    title: string,
-    fn: jest.EmptyFunction,
-) => {
-    if (skipCondition) {
-        describe.skip(title, fn);
-    } else {
+export const conditionalDescribe = (condition: boolean, title: string, fn: jest.EmptyFunction) => {
+    if (condition) {
         describe(title, fn);
+    } else {
+        describe.skip(title, fn);
     }
 };
diff --git a/suite-native/app/e2e/tests/deeplinkPopup.test.ts b/suite-native/app/e2e/tests/deeplinkPopup.test.ts
@@ -38,7 +38,7 @@ const openUriScheme = (url: string, platformToOpen: 'android') => {
     });
 };
 
-conditionalDescribe(device.getPlatform() !== 'android', 'Deeplink connect popup.', () => {
+conditionalDescribe(device.getPlatform() === 'android', 'Deeplink connect popup.', () => {
     beforeAll(async () => {
         await new Promise(resolve => {
             server = http.createServer((req, res) => {

diff --git a/suite-native/app/e2e/tests/deviceSettings.test.ts b/suite-native/app/e2e/tests/deviceSettings.test.ts
@@ -16,7 +16,7 @@ import {
     restartApp,
 } from '../utils';
 
-conditionalDescribe(device.getPlatform() !== 'android', 'Device settings', () => {
+conditionalDescribe(device.getPlatform() === 'android', 'Device settings', () => {
     beforeAll(async () => {
         await prepareTrezorEmulator();
         await openApp({ newInstance: true });

diff --git a/suite-native/app/e2e/tests/onboardAndConnect.test.ts b/suite-native/app/e2e/tests/onboardAndConnect.test.ts
@@ -11,7 +11,7 @@ import { onCoinEnablingInit } from '../pageObjects/coinEnablingActions';
 const platform = device.getPlatform();
 
 conditionalDescribe(
-    device.getPlatform() !== 'android',
+    device.getPlatform() === 'android',
     'Go through onboarding and connect Trezor.',
     () => {
         beforeAll(async () => {

diff --git a/suite-native/app/e2e/tests/send.test.ts b/suite-native/app/e2e/tests/send.test.ts
@@ -66,7 +66,7 @@ const signTransactionAndSendIt = async () => {
     await onSendOutputsReview.clickSendTransaction();
 };
 
-conditionalDescribe(device.getPlatform() !== 'android', 'Send transaction flow.', () => {
+conditionalDescribe(device.getPlatform() === 'android', 'Send transaction flow.', () => {
     beforeAll(async () => {
         await prepareTrezorEmulator();
         await openApp({ newInstance: true });