chore: remove unnecessary async from tests

apps/trading/client-pages/transfer/transfer.spec.tsx

@@ -29,7 +29,7 @@ describe('Transfer page', () => {
     expect(screen.getByTestId('assetId')).toBeEmptyDOMElement();
   });
 
-  it('assetId should be passed down', async () => {
+  it('assetId should be passed down', () => {
     const assetId = 'foo';
     const route = '/transfer?assetId=' + assetId;
     renderJsx(route);

apps/trading/client-pages/transfer/transfer.tsx

@@ -1,7 +1,7 @@
-import { GetStarted } from '../../components/welcome-dialog';
+import { useSearchParams } from 'react-router-dom';
 import { t } from '@vegaprotocol/i18n';
 import { TransferContainer } from '@vegaprotocol/accounts';
-import { useSearchParams } from 'react-router-dom';
+import { GetStarted } from '../../components/welcome-dialog';
 
 export const Transfer = () => {
   const [searchParams] = useSearchParams();

apps/trading/client-pages/withdraw/withdraw.spec.tsx

@@ -21,7 +21,7 @@ const renderJsx = (route = '/withdraw') => {
 };
 
 describe('Withdraw page', () => {
-  it('should be properly rendered', async () => {
+  it('should be properly rendered', () => {
     renderJsx();
     expect(
       screen.getByRole('heading', { level: 1, name: 'Withdraw' })
@@ -29,7 +29,7 @@ describe('Withdraw page', () => {
     expect(screen.getByTestId('assetId')).toBeEmptyDOMElement();
   });
 
-  it('assetId should be passed down', async () => {
+  it('assetId should be passed down', () => {
     const assetId = 'foo';
     const route = '/withdraw?assetId=' + assetId;
     renderJsx(route);

apps/trading/client-pages/withdraw/withdraw.tsx

@@ -1,6 +1,6 @@
 import { useSearchParams } from 'react-router-dom';
-import { GetStarted } from '../../components/welcome-dialog';
 import { t } from '@vegaprotocol/i18n';
+import { GetStarted } from '../../components/welcome-dialog';
 import { WithdrawContainer } from '../../components/withdraw-container';
 
 export const Withdraw = () => {