Removed act warnings in the unit tests console

[DaniSomoza]

What it solves

Removed act warnings in the unit tests console

How this PR fixes it

How to test it

Screenshots

[github-actions]

CLA Assistant Lite All Contributors have signed the CLA.

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	1	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[github-actions]

• Safe Multisig app rinkeby

[github-actions]

• Safe Multisig app polygon

[github-actions]

• Safe Multisig app mainnet

[github-actions]

E2E Tests Passed
Check the results here: https://github.com/gnosis/safe-react-e2e-tests/actions/runs/1195120722

[iamacook]

Love the test! I have a few other suggestions but can't seem to comment on some areas of Load.test.tsx:

• networkId, customState, validSafeAddress and validSafeENSNameDomain seem to be the same in all tests so I would move them outside so as to not repeat yourself.
• Lines 126, 164, 208 should be inside act.
• waitFor on line 128 could possibly be act if you are not waiting on something to load.
• Depending on if the mocked function needs to be called legitimately elsewhere, you could move the mocks on lines 156 and 190 out of the tests into beforeEach. You can then clear them afterEach instead of repeating yourself on lines 166 and 215.