Skip to content

Commit

Permalink
test(e2e): remove use_tab_navigator override (#5327)
Browse files Browse the repository at this point in the history
### Description

Follow up from #5318. Remove override that is no longer required

### Test plan

CI

### Related issues

- Part of ACT-1133

### Backwards compatibility

Yes

### Network scalability

N/A
  • Loading branch information
satish-ravi authored Apr 26, 2024
1 parent 52fb14d commit 0db5dd2
Show file tree
Hide file tree
Showing 17 changed files with 6 additions and 81 deletions.
5 changes: 0 additions & 5 deletions e2e/src/CeloPage.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,6 @@ import { launchApp } from './utils/retries'

describe('Celo page', () => {
beforeAll(async () => {
// TODO(ACT-1133): remove launchApp once drawer nav is removed
await launchApp({
newInstance: true,
launchArgs: { statsigGateOverrides: 'use_tab_navigator=true' },
})
await quickOnboarding()
})

Expand Down
3 changes: 0 additions & 3 deletions e2e/src/Discover.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,6 @@ describe('Discover tab', () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES', camera: 'YES' },
launchArgs: {
statsigGateOverrides: `use_tab_navigator=true`,
},
})
await waitForElementByIdAndTap('Tab/Discover')
})
Expand Down
9 changes: 0 additions & 9 deletions e2e/src/HomeFeed.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,6 @@ beforeAll(async () => {

// iOS only as getAttributes on multiple elements is not supported on Android
describe(':ios: Home Feed', () => {
// TODO(ACT-1133): remove this beforeAll when drawer nav is removed
beforeAll(async () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES', camera: 'YES' },
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
})

it('should show correct information on tap of feed item', async () => {
// Load Wallet Home
await waitForElementId('WalletHome')
Expand Down
2 changes: 0 additions & 2 deletions e2e/src/Pin.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@ describe('Given PIN', () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES' },
// TODO(ACT-1133): remove launchArgs
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
await quickOnboarding()
})
Expand Down
8 changes: 0 additions & 8 deletions e2e/src/QRScanner.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,14 +40,6 @@ describe(':ios: Given QR Scanner', () => {
})

describe("When 'scanning' QR", () => {
beforeAll(async () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES', camera: 'YES' },
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
})

beforeEach(async () => {
await reloadReactNative()
await waitForElementId('HomeAction-Receive')
Expand Down
3 changes: 0 additions & 3 deletions e2e/src/usecases/Assets.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,6 @@ export default Assets = () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES', camera: 'YES' },
launchArgs: {
statsigGateOverrides: `use_tab_navigator=true`,
},
})
let mnemonic = SAMPLE_BACKUP_KEY
if (balance === 'zero') {
Expand Down
2 changes: 1 addition & 1 deletion e2e/src/usecases/ChooseYourAdventure.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export default ChooseYourAdventure = () => {
await launchApp({
newInstance: true,
launchArgs: {
statsigGateOverrides: `use_tab_navigator=true,show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
statsigGateOverrides: `show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
},
})
await quickOnboarding({ stopOnCYA: true, cloudBackupEnabled: true })
Expand Down
5 changes: 0 additions & 5 deletions e2e/src/usecases/FiatConnectTransferOut.js
Original file line number Diff line number Diff line change
Expand Up @@ -189,11 +189,6 @@ export const fiatConnectNonKycTransferOut = () => {

export const fiatConnectKycTransferOut = () => {
it('FiatConnect cash out', async () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES', camera: 'YES' },
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
// ******** First time experience ************
const cashOutAmount = 0.01
const gasAmount = 0.015
Expand Down
4 changes: 2 additions & 2 deletions e2e/src/usecases/NewAccountOnboarding.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ export default NewAccountOnboarding = () => {
delete: true,
permissions: { notifications: 'YES', contacts: 'YES' },
launchArgs: {
statsigGateOverrides: `use_tab_navigator=true,show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
statsigGateOverrides: `show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
},
})
await sleep(5000)
Expand Down Expand Up @@ -159,7 +159,7 @@ export default NewAccountOnboarding = () => {
await launchApp({
newInstance: true,
launchArgs: {
statsigGateOverrides: `use_tab_navigator=true,show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
statsigGateOverrides: `show_cloud_account_backup_setup=true,show_cloud_account_backup_restore=true`,
},
})
await quickOnboarding({ mnemonic: testRecoveryPhrase, cloudBackupEnabled: true })
Expand Down
2 changes: 0 additions & 2 deletions e2e/src/usecases/NewAccountPhoneVerification.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,6 @@ export default NewAccountPhoneVerification = () => {
await launchApp({
delete: true,
permissions: { notifications: 'YES', contacts: 'YES' },
// TODO(ACT-1133): remove launchArgs
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})

// Create new account
Expand Down
2 changes: 0 additions & 2 deletions e2e/src/usecases/OffRamps.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ export default offRamps = () => {
beforeAll(async () => {
await launchApp({
newInstance: true,
// TODO(ACT-1133): remove launchArgs
launchArgs: { statsigGateOverrides: 'use_tab_navigator=true' },
})
})
beforeEach(async () => {
Expand Down
2 changes: 0 additions & 2 deletions e2e/src/usecases/OnRamps.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ export default onRamps = () => {
beforeAll(async () => {
await launchApp({
newInstance: true,
// TODO(AC-1133): remove launchArgs
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
})
beforeEach(async () => {
Expand Down
12 changes: 1 addition & 11 deletions e2e/src/usecases/ResetAccount.js
Original file line number Diff line number Diff line change
@@ -1,18 +1,8 @@
import { SAMPLE_BACKUP_KEY } from '../utils/consts'
import { reloadReactNative, launchApp } from '../utils/retries'
import { reloadReactNative } from '../utils/retries'
import { enterPinUiIfNecessary, navigateToSettings, waitForElementId } from '../utils/utils'

export default ResetAccount = () => {
// TODO(ACT-1133): remove this launchApp as it is only needed to update
// statsig gate overrides
beforeAll(async () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES' },
launchArgs: { statsigGateOverrides: 'use_tab_navigator=true' },
})
})

beforeEach(async () => {
await reloadReactNative()
})
Expand Down
2 changes: 0 additions & 2 deletions e2e/src/usecases/RestoreAccountOnboarding.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@ export default RestoreAccountOnboarding = () => {
await device.installApp()
await launchApp({
permissions: { notifications: 'YES', contacts: 'YES' },
// TODO(ACT-1133): remove launchArgs
launchArgs: { statsigGateOverrides: 'use_tab_navigator=true' },
})
})

Expand Down
10 changes: 2 additions & 8 deletions e2e/src/usecases/Send.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,7 @@ export default Send = () => {
let commentText
beforeAll(async () => {
commentText = `${new Date().getTime()}-${parseInt(Math.random() * 100000)}`
await launchApp({
newInstance: true,
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
await launchApp({ newInstance: true })
})

it('Then should navigate to send search input from home action', async () => {
Expand Down Expand Up @@ -113,10 +110,7 @@ export default Send = () => {
let commentText
beforeAll(async () => {
commentText = `${new Date().getTime()}`
await launchApp({
newInstance: true,
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
await launchApp({ newInstance: true })
})

it('Then should navigate to send search input from home action', async () => {
Expand Down
10 changes: 0 additions & 10 deletions e2e/src/usecases/Settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,6 @@ import { navigateToSettings, scrollIntoView, sleep, waitForElementByIdAndTap } f
const faker = require('@faker-js/faker')

export default Settings = () => {
// TODO(ACT-1133): remove this launchApp as it is only needed to update
// statsig gate overrides
beforeAll(async () => {
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES' },
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
})

beforeEach(async () => {
await reloadReactNative()
await navigateToSettings()
Expand Down
6 changes: 0 additions & 6 deletions e2e/src/usecases/Support.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,6 @@ export default Support = () => {
}

it('Send Message to Support', async () => {
// TODO(ACT-1133): remove launchApp
await launchApp({
newInstance: true,
permissions: { notifications: 'YES', contacts: 'YES' },
launchArgs: { statsigGateOverrides: `use_tab_navigator=true` },
})
await waitForElementByIdAndTap('WalletHome/AccountCircle')
await waitForElementByIdAndTap('ProfileMenu/Help')
await waitFor(element(by.id('SupportContactLink')))
Expand Down

0 comments on commit 0db5dd2

Please sign in to comment.