From f9154379eb37a997fca103e0cb729726a3eb4559 Mon Sep 17 00:00:00 2001 From: Jaco Date: Mon, 23 Aug 2021 11:08:32 +0300 Subject: [PATCH] Fix Ethereum tests (#796) --- .../extension-ui/src/Popup/Accounts/Account.test.tsx | 9 +++++---- packages/extension-ui/src/Popup/Derive/Derive.test.tsx | 2 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/packages/extension-ui/src/Popup/Accounts/Account.test.tsx b/packages/extension-ui/src/Popup/Accounts/Account.test.tsx index 712bfe2d..cdf84d95 100644 --- a/packages/extension-ui/src/Popup/Accounts/Account.test.tsx +++ b/packages/extension-ui/src/Popup/Accounts/Account.test.tsx @@ -58,15 +58,16 @@ describe('Account component', () => { expect(wrapper.find('.genesisSelection').exists()).toBe(true); }); - it('does not show Derive option if account is of ethereum type', async () => { + it('shows Derive option if account is of ethereum type', async () => { wrapper = mountAccountComponent({ isExternal: false, type: 'ethereum' }); wrapper.find('.settings').first().simulate('click'); await act(flushAllPromises); - expect(wrapper.find('a.menuItem').length).toBe(3); + expect(wrapper.find('a.menuItem').length).toBe(4); expect(wrapper.find('a.menuItem').at(0).text()).toBe('Rename'); - expect(wrapper.find('a.menuItem').at(1).text()).toBe('Export Account'); - expect(wrapper.find('a.menuItem').at(2).text()).toBe('Forget Account'); + expect(wrapper.find('a.menuItem').at(1).text()).toBe('Derive New Account'); + expect(wrapper.find('a.menuItem').at(2).text()).toBe('Export Account'); + expect(wrapper.find('a.menuItem').at(3).text()).toBe('Forget Account'); expect(wrapper.find('.genesisSelection').exists()).toBe(true); }); diff --git a/packages/extension-ui/src/Popup/Derive/Derive.test.tsx b/packages/extension-ui/src/Popup/Derive/Derive.test.tsx index e2dda7c2..8c306503 100644 --- a/packages/extension-ui/src/Popup/Derive/Derive.test.tsx +++ b/packages/extension-ui/src/Popup/Derive/Derive.test.tsx @@ -238,7 +238,7 @@ describe('Derive', () => { it('selects internal root accounts as other options, no external and no Ethereum account', () => { const options = wrapper.find('[data-parent-option] [data-field="name"]').map((el) => el.text()); - expect(options).toEqual(['A', 'B', 'D']); + expect(options).toEqual(['A', 'B', 'D', 'Ethereum']); }); it('redirects to derive from next account when other option is selected', () => {