fix: verify allowlist tx receipt status in e2e validator helper

[vikions]

Summary

addValidatorToAllowlist in the E2E validator helper was awaiting
waitForTransactionReceipt but discarding the result entirely.
This means a reverted transaction would still be treated as a
successful allowlist operation — the test would continue without
any indication that the on-chain call failed.

This patch captures the receipt and explicitly checks its status,
throwing a descriptive error if the transaction did not succeed.
This prevents false-positive test results when the allowlist tx reverts.

Changes

• test/e2e/framework/validators.ts: capture receipt from
  waitForTransactionReceipt and assert status === "success"

Test plan

• Existing E2E validator test suites exercise this code path
• A reverted allowlist tx will now surface as an explicit failure
  rather than passing silently

[stiiifff]

Thanks for your contribution @vikions !

[vikions]

Thanks for your contribution @vikions !

The docker CI fails because this PR comes from a fork, and GitHub doesn’t allow pushing images to the org’s GHCR from fork PRs.
Maybe we can skip --push on pull_request and only push on main (or tags)?