test(e2e): cover GMP packet timeout

[gjermundgaraba]

Summary

• Adds GMP timeout coverage that sends a short-lived call, advances destination time, waits for the relayer-reported timeout, verifies the matching TimeoutPacket event, and confirms the destination Counter remains unchanged.
• Adds configurable GMP packet timeouts and reuses the shared receipt-based timeout verifier introduced earlier in the stack.
• Renames the shared timeout constants and unchanged-counter assertion so they apply across Transfer, IFT, and GMP tests.

Testing

• make check
• make test-e2e E2E_MODE=complete

_{Stack created with GitHub Stacks CLI • Give Feedback 💬}

Closes FOU-1103

[linear-code]

FOU-1103

[greptile-apps]

Greptile Summary

Adds end-to-end GMP packet-timeout coverage and configurable GMP timeout durations, while generalizing shared timeout constants and unchanged-Counter verification.

• Adds a controlled-mining GMP timeout scenario and records it in the E2E matrix.
• Reuses receipt-based TimeoutPacket verification for GMP calls.
• Renames timeout constants shared by Transfer, IFT, and GMP tests.

Confidence Score: 4/5

The PR should not merge until the GMP timeout test mines enough destination blocks for the advanced timestamp to become finalized and provable.

The new test restarts the relayer after mining only the first post-timeout block, while the configured one-block finality offset makes timeout processing inspect the preceding block whose timestamp has not expired.

Files Needing Attention: e2e/gmp_test.go

Important Files Changed

Filename	Overview
e2e/gmp_test.go	Adds the GMP timeout scenario, but does not mine the additional block required by the harness finality offset.
e2e/internal/e2etest/traffic_gmp.go	Adds configurable packet lifetimes and correctly delegates GMP timeout receipt verification to the shared helper.
e2e/ift_test.go	Renames shared timeout constants without changing behavior and provides the established extra-mining pattern for finalized timeout tests.
e2e/transfer_test.go	Generalizes timeout constant names across packet types without changing their values.
e2e/test-matrix.md	Documents the new controlled-mining GMP timeout coverage.

Sequence Diagram

sequenceDiagram
    participant Test
    participant Destination
    participant Relayer
    Test->>Relayer: Stop
    Test->>Destination: Send GMP packet (5s timeout)
    Test->>Destination: AdvanceTime(25s) and mine block N+1
    Test->>Relayer: Restart
    Relayer->>Destination: Check finalized height N
    Destination-->>Relayer: Timestamp still before timeout
    Relayer-->>Test: No TIMED_OUT state
    Test-->>Test: AwaitState fails

Loading

_{Reviews (1): Last reviewed commit: "test(e2e): cover GMP packet timeout" | Re-trigger Greptile}

[greptile-apps]

Timeout height remains unfinalized

When controlled mining advances time, AdvanceTime mines only the first post-timeout block, while the configured one-block finality offset makes the relayer inspect the preceding block. That block still predates the timeout, so the relayer cannot submit it and AwaitState fails to reach PACKET_STATE_TIMED_OUT.

Suggested change

	require.NoError(t, mining.AdvanceTime(ctx, packetTimeoutAdvance))
	relayer = e2etest.StartRelayer(t, driver, env)
	require.NoError(t, mining.AdvanceTime(ctx, packetTimeoutAdvance))
	require.NoError(t, mining.Mine(ctx, 1))
	relayer = e2etest.StartRelayer(t, driver, env)

[srdtrk]

lgtm

The merge-base changed after approval.