test: send to contract address

app/components/Views/confirmations/SendFlow/Amount/__snapshots__/index.test.tsx.snap

@@ -316,14 +316,14 @@ exports[`Amount should convert ERC-20 token value to USD 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -1034,14 +1034,14 @@ exports[`Amount should convert ETH to USD 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -1752,14 +1752,14 @@ exports[`Amount should convert USD to ERC-20 token value 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -2487,14 +2487,14 @@ exports[`Amount should convert USD to ETH 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -3222,14 +3222,14 @@ exports[`Amount should display correct balance 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -3939,14 +3939,14 @@ exports[`Amount should not show a warning when conversion rate is available 1`]
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -4656,14 +4656,14 @@ exports[`Amount should not show a warning when transfering collectibles 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -5358,14 +5358,14 @@ exports[`Amount should proceed if balance is sufficient while on Native primary
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -6071,14 +6071,14 @@ exports[`Amount should render correctly 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -6783,14 +6783,14 @@ exports[`Amount should show a warning when conversion rate is not available 1`]
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View
@@ -7495,14 +7495,14 @@ exports[`Amount should show an error message if balance is insufficient 1`] = `
                           "flex": 1,
                         }
                       }
-                      testID="amount-screen"
                     >
                       <RCTScrollView
                         style={
                           {
                             "marginBottom": 60,
                           }
                         }
+                        testID="amount-screen"
                       >
                         <View>
                           <View

app/components/Views/confirmations/SendFlow/Amount/index.js

@@ -1403,12 +1403,11 @@ class Amount extends PureComponent {
     const styles = createStyles(colors);
 
     return (
-      <SafeAreaView
-        edges={['bottom']}
-        style={styles.wrapper}
-        testID={AmountViewSelectorsIDs.CONTAINER}
-      >
-        <ScrollView style={styles.scrollWrapper}>
+      <SafeAreaView edges={['bottom']} style={styles.wrapper}>
+        <ScrollView
+          style={styles.scrollWrapper}
+          testID={AmountViewSelectorsIDs.CONTAINER}
+        >
           {!hasExchangeRate && !selectedAsset.tokenId ? (
             <Alert
               small

app/components/Views/confirmations/SendFlow/ErrorMessage/index.js

@@ -5,6 +5,7 @@ import { strings } from '../../../../../../locales/i18n';
 import Alert, { AlertType } from '../../../../Base/Alert';
 import Text from '../../../../Base/Text';
 import { CommonSelectorsIDs } from '../../../../../../e2e/selectors/Common.selectors';
+import { SendViewSelectorsIDs } from '../../../../../../e2e/selectors/SendView.selectors';
 
 const styles = StyleSheet.create({
   button: {
@@ -30,7 +31,11 @@ export default function ErrorMessage(props) {
             {errorMessage}
           </Text>
           {errorContinue && (
-            <TouchableOpacity onPress={onContinue} style={styles.button}>
+            <TouchableOpacity
+              onPress={onContinue}
+              style={styles.button}
+              testID={SendViewSelectorsIDs.I_UNDERSTAND_RISK_ID}
+            >
               <Text small link centered>
                 {strings('transaction.continueError')}
               </Text>

e2e/pages/Send/AmountView.js

@@ -6,6 +6,9 @@ class AmountView {
   get currencySwitch() {
     return Matchers.getElementByID(AmountViewSelectorsIDs.CURRENCY_SWITCH);
   }
+  get container() {
+    return Matchers.getElementByID(AmountViewSelectorsIDs.CONTAINER);
+  }
 
   get nextButton() {
     return device.getPlatform() === 'ios'
@@ -22,9 +25,7 @@ class AmountView {
   }
 
   async typeInTransactionAmount(amount) {
-    device.getPlatform() === 'android'
-      ? await Gestures.typeTextAndHideKeyboard(this.amountInputField, amount)
-      : await Gestures.replaceTextInField(this.amountInputField, amount);
+    await Gestures.typeTextAndHideKeyboard(this.amountInputField, amount);
   }
 
   async tapCurrencySwitch() {

e2e/pages/Send/SendView.js

@@ -38,6 +38,14 @@ class SendView {
     return Matchers.getElementByID(SendViewSelectorsIDs.NO_ETH_MESSAGE);
   }
 
+  get iUnderstandWarningButton() {
+    // Warning appears when sending to a contract address
+    return Matchers.getElementByID(SendViewSelectorsIDs.I_UNDERSTAND_RISK_ID);
+  }
+  async tapIUnderstandButton() {
+    await Gestures.waitAndTap(this.iUnderstandWarningButton);
+  }
+
   async tapCancelButton() {
     await Gestures.waitAndTap(this.cancelButton);
   }

e2e/selectors/SendView.selectors.js

@@ -10,4 +10,5 @@ export const SendViewSelectorsIDs = {
   SEND_CANCEL_BUTTON: 'send-cancel-button',
   ADD_ADDRESS_BUTTON: 'add-address-button-a',
   CONTAINER_ID: 'send-screen',
+  I_UNDERSTAND_RISK_ID: 'understand-risk-id',
 };

e2e/specs/confirmations/approve-custom-erc20.spec.js

@@ -17,7 +17,7 @@ import { ActivitiesViewSelectorsText } from '../../selectors/ActivitiesView.sele
 
 const HST_CONTRACT = SMART_CONTRACTS.HST;
 
-describe(SmokeConfirmations('ERC20 tokens'), () => {
+describe(SmokeConfirmations('Approve Custom ERC20 tokens'), () => {
   beforeAll(async () => {
     jest.setTimeout(170000);
     if (device.getPlatform() === 'android') {

e2e/specs/confirmations/approve-default-erc20.spec.js

@@ -20,7 +20,7 @@ import TestDApp from '../../pages/Browser/TestDApp';
 const HST_CONTRACT = SMART_CONTRACTS.HST;
 const EXPECTED_TOKEN_AMOUNT = '7';
 
-describe(SmokeConfirmations('ERC20 tokens'), () => {
+describe(SmokeConfirmations('Approve ERC20 tokens'), () => {
   beforeAll(async () => {
     jest.setTimeout(170000);
     if (device.getPlatform() === 'android') {

e2e/specs/confirmations/send-erc20-with-dapp.spec.js

@@ -19,7 +19,7 @@ import Assertions from '../../utils/Assertions';
 
 const HST_CONTRACT = SMART_CONTRACTS.HST;
 
-describe(SmokeConfirmations('ERC20 tokens'), () => {
+describe(SmokeConfirmations('Send ERC20 tokens'), () => {
   beforeAll(async () => {
     jest.setTimeout(170000);
     await TestHelpers.reverseServerPort();

e2e/specs/confirmations/send-to-contact.spec.js

@@ -23,7 +23,7 @@ import Assertions from '../../utils/Assertions';
 
 const fixtureServer = new FixtureServer();
 
-describe(SmokeConfirmations('Send ETH'), () => {
+describe(SmokeConfirmations('Send ETH to Contact'), () => {
   const TOKEN_NAME = enContent.unit.eth;
   const AMOUNT = '0.12345';
 
@@ -67,6 +67,7 @@ describe(SmokeConfirmations('Send ETH'), () => {
 
     await SendView.tapNextButton();
 
+    await Assertions.checkIfVisible(await AmountView.container);
     await AmountView.typeInTransactionAmount(AMOUNT);
     await AmountView.tapNextButton();
     await Assertions.checkIfTextIsDisplayed('Test Name 1');

e2e/specs/confirmations/send-to-contract-address.spec.js

@@ -0,0 +1,62 @@
+'use strict';
+
+import { SmokeConfirmations } from '../../tags';
+import AmountView from '../../pages/Send/AmountView';
+import SendView from '../../pages/Send/SendView';
+import TransactionConfirmationView from '../../pages/Send/TransactionConfirmView';
+import { loginToApp } from '../../viewHelper';
+import TabBarComponent from '../../pages/TabBarComponent';
+import WalletActionsModal from '../../pages/modals/WalletActionsModal';
+import FixtureBuilder from '../../fixtures/fixture-builder';
+import {
+  loadFixture,
+  startFixtureServer,
+  stopFixtureServer,
+} from '../../fixtures/fixture-helper';
+import { CustomNetworks } from '../../resources/networks.e2e';
+import TestHelpers from '../../helpers';
+import FixtureServer from '../../fixtures/fixture-server';
+import { getFixturesServerPort } from '../../fixtures/utils';
+import Assertions from '../../utils/Assertions';
+
+const fixtureServer = new FixtureServer();
+
+describe(SmokeConfirmations('Send ETH to Contract'), () => {
+  const AMOUNT = '1';
+
+  beforeEach(async () => {
+    await TestHelpers.reverseServerPort();
+    const fixture = new FixtureBuilder()
+      .withNetworkController(CustomNetworks.Tenderly)
+      .build();
+    await startFixtureServer(fixtureServer);
+    await loadFixture(fixtureServer, { fixture });
+    await device.launchApp({
+      permissions: { notifications: 'YES' },
+      launchArgs: { fixtureServerPort: `${getFixturesServerPort()}` },
+    });
+    await loginToApp();
+  });
+
+  afterAll(async () => {
+    await stopFixtureServer(fixtureServer);
+  });
+
+  it('should send ETH to a contract from inside the wallet', async () => {
+    const DAI = '0x6B175474E89094C44Da98b954EedeAC495271d0F';
+
+    await TabBarComponent.tapActions();
+    await WalletActionsModal.tapSendButton();
+
+    await SendView.inputAddress(DAI);
+    await Assertions.checkIfVisible(await SendView.iUnderstandWarningButton);
+    await SendView.tapIUnderstandButton();
+    await Assertions.checkIfVisible(await AmountView.container);
+
+    await AmountView.typeInTransactionAmount(AMOUNT);
+    await AmountView.tapNextButton();
+
+    await TransactionConfirmationView.tapConfirmButton();
+    await TabBarComponent.tapActivity();
+  });
+});