Stakeholder review needed before the work gets merged
[ ] Engineering (needed in most cases)
[ ] Design
[ ] Product
[ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
Tests were originally disabled because they were flaky. Run tests a few times before merging.
To address:
remove duplicate tests
e.g. testing #maliciousTradeOrder in test/e2e/tests/ppom/ppom-blockaid-alert-trade-order-farming.spec.js and test/e2e/tests/ppom/ppom-blockaid-alert.spec.js
What is this about?
Re-enable all specs after cdn mocks have been implemented. If the task ends up being too big for 1 PR, it could be split in several PRs.
Once this is merged https://github.com/MetaMask/metamask-extension/pull/23062
Scenario
No response
Design
No response
Technical Details
No response
Threat Modeling Framework
No response
Acceptance Criteria
No response
Stakeholder review needed before the work gets merged
References
No response