fix: removing exception trigger test

synonymdev · Jun 13, 2024 · 817ffbc · 817ffbc
1 parent b15db0f
commit 817ffbc
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 37 deletions.
diff --git a/e2e/settings.e2e.js b/e2e/settings.e2e.js
@@ -613,31 +613,31 @@ d('Settings', () => {
 		});
 	});
 
-	d('Dev Settings', () => {
-		it('Shows the crash error screen when triggering render error', async () => {
-			if (checkComplete('settings-dev')) {
-				return;
-			}
-
-			await element(by.id('Settings')).tap();
-			if (!__DEV__) {
-				await element(by.id('DevOptions')).multiTap(5); // enable dev mode
-			}
-			await element(by.id('DevSettings')).tap();
-
-			// Error screen will not be rendered in development mode
-			if (__DEV__) {
-				markComplete('settings-dev');
-				return;
-			}
-
-			await element(by.id('TriggerRenderError')).tap();
-			await expect(element(by.id('ErrorClose'))).toBeVisible();
-			await expect(element(by.id('ErrorReport'))).toBeVisible();
-
-			markComplete('settings-dev');
-		});
-	});
+	// d('Dev Settings', () => {
+	// 	it('Shows the crash error screen when triggering render error', async () => {
+	// 		if (checkComplete('settings-dev')) {
+	// 			return;
+	// 		}
+
+	// 		await element(by.id('Settings')).tap();
+	// 		if (!__DEV__) {
+	// 			await element(by.id('DevOptions')).multiTap(5); // enable dev mode
+	// 		}
+	// 		await element(by.id('DevSettings')).tap();
+
+	// 		// Error screen will not be rendered in development mode
+	// 		if (__DEV__) {
+	// 			markComplete('settings-dev');
+	// 			return;
+	// 		}
+
+	// 		await element(by.id('TriggerRenderError')).tap();
+	// 		await expect(element(by.id('ErrorClose'))).toBeVisible();
+	// 		await expect(element(by.id('ErrorReport'))).toBeVisible();
+
+	// 		markComplete('settings-dev');
+	// 	});
+	// });
 
 	d('Support', () => {
 		it('Can see app status', async () => {

diff --git a/src/screens/Settings/DevSettings/index.tsx b/src/screens/Settings/DevSettings/index.tsx
@@ -49,7 +49,7 @@ const DevSettings = ({
 	const dispatch = useAppDispatch();
 	const { t } = useTranslation('lightning');
 	const [showDialog, setShowDialog] = useState(false);
-	const [throwError, setThrowError] = useState(false);
+	// const [throwError, setThrowError] = useState(false);
 	const selectedWallet = useAppSelector(selectedWalletSelector);
 	const selectedNetwork = useAppSelector(selectedNetworkSelector);
 	const addressType = useAppSelector(addressTypeSelector);
@@ -162,14 +162,14 @@ const DevSettings = ({
 						refreshWallet({ selectedWallet, selectedNetwork }).then();
 					},
 				},
-				{
-					title: 'Trigger exception in React render',
-					type: EItemType.button,
-					testID: 'TriggerRenderError',
-					onPress: (): void => {
-						setThrowError(true);
-					},
-				},
+				// {
+				// 	title: 'Trigger exception in React render',
+				// 	type: EItemType.button,
+				// 	testID: 'TriggerRenderError',
+				// 	onPress: (): void => {
+				// 		setThrowError(true);
+				// 	},
+				// },
 				{
 					title: 'Trigger exception in action handler',
 					type: EItemType.button,
@@ -331,9 +331,9 @@ const DevSettings = ({
 		},
 	];
 
-	if (throwError) {
-		throw new Error('test render error');
-	}
+	// if (throwError) {
+	// 	throw new Error('test render error');
+	// }
 
 	return (
 		<>