Skip to content

fix: complete CI coverage#169

Open
ameya-deshmukh wants to merge 6 commits intoseismicfrom
ameya/complete-ci-coverage
Open

fix: complete CI coverage#169
ameya-deshmukh wants to merge 6 commits intoseismicfrom
ameya/complete-ci-coverage

Conversation

@ameya-deshmukh
Copy link
Contributor

No description provided.

Fix and re-enable tests that were incorrectly excluded from CI:

- anvil_api::can_get_node_info — expect MERCURY instead of PRAGUE
- api::can_call_with_undersized_max_fee_per_gas — decode last_sender
  from seismic_call result instead of tx request
- revert::test_solc_revert_example — remove unused sender/with_from
- api::can_call_on_pending_block — account for Mercury ms timestamps
- transaction::get_blocktimestamp_works — account for Mercury ms
  timestamps and adjust mock timestamp to match
Update the test audit doc from 25 → 15 excluded tests. Move the 5
fixed tests into a "Previously Excluded, Now Fixed" section with
root causes and fixes applied. Clean up stray blank lines in CI yml.
Regenerate hardcoded Merkle proof bytes in test_account_proof to
account for the 3 system contracts (AES_LIB, DIRECTORY, INTELLIGENCE)
injected at genesis, which add extra nodes to the account trie.
Seismic intentionally strips calldata from the top-level CALL and
return data from STATICCALL in transaction traces for privacy. Update
expected trace entries to match shielded output (empty input/output).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant