refactor: warp e2e tests no longer in single function

[JonathanOppenheimer]

Why this should be merged

This refactors the warp e2e tests in both subnet-evm and coreth to use a more standard ginkgo test structure (close to the one suggested by @michaelkaplan13 ). This provides better failure isolation by breaking each test functionality into its own It block within ordered Describe blocks for each test combination.

Closes #4699

How this works

Explicitly defined test combinations.

How this was tested

CI

Need to be documented in RELEASES.md?

No

[Elvis339]

No action required: the context created in BeforeAll is used for newWarpTest initialization, including IssueTxsToActivateProposerVMFork which waits for transactions. If BeforeAll has a shorter timeout than It blocks, this could potentially cause timeout issues. Maybe consider having newWarpTest create its own context like the other methods do. Not blocking if this hasn't caused issues in practice just my two cents.

[JonathanOppenheimer]

No action required: the context created in BeforeAll is used for newWarpTest initialization, including IssueTxsToActivateProposerVMFork which waits for transactions. If BeforeAll has a shorter timeout than It blocks, this could potentially cause timeout issues. Maybe consider having newWarpTest create its own context like the other methods do. Not blocking if this hasn't caused issues in practice just my two cents.

Nah this is a great catch! I will make this change