op-e2e: Delete TestCrossLayerUser

[mslipper]

Deletes TestCrossLayerUser. This is an action test that deposits, makes a transaction, then withdraws. It is our most flaky test by far:

I'm advocating for removing it because we have these tests covered in multiple other places:

• The acceptance tests validate the full deposit/withdrawal flow
• Deposits are validated implicitly by numerous E2E tests
• Fault proofs have extensive unit tests that cover the proposal/game flow, as well as a dedicated E2E test

In sum, this test provides little value in return for failing often and slowing down our CI pipeline.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.37%. Comparing base (aa4db40) to head (2e78849).
⚠️ Report is 6 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #17106      +/-   ##
===========================================
- Coverage    45.75%   42.37%   -3.38%     
===========================================
  Files         1478     1317     -161     
  Lines       119949   110709    -9240     
===========================================
- Hits         54877    46917    -7960     
+ Misses       61226    60108    -1118     
+ Partials      3846     3684     -162     

Flag	Coverage Δ
cannon-go-tests-64	?
contracts-bedrock-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 175 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pcw109550]

Approving with mentioning that we may delete more codes like ActCompleteWithdrawal, ActCheckStartWithdrawal, ActStartWithdrawal, ActProveWithdrawal methods