Skip to content

Conversation

@anvacaru
Copy link
Contributor

@anvacaru anvacaru commented Dec 5, 2025

  1. Fixing a bug in Makefile introduced Disable unsupported conformance test-suites #2817. In pytest, the -k flags are additive. I replaced them with a single -k flag that uses and to filter the test suites that get executed.
  2. Some tests in test/failing.llvm were failing because they needed a chainId of 1 instead of 0.
  3. Added a fallback rule for load "transaction" that captures blocks that are incorrectly encoded.
  4. Made adjustments to the _test function by removing redundant rewrites and lookups.
  5. Refactored the _skipped_tests function to use frozenset instead of list.

@anvacaru anvacaru changed the title Test harness tweaks Minor tweaks to the test harness Dec 5, 2025
@anvacaru anvacaru marked this pull request as ready for review December 5, 2025 14:34
Copy link
Contributor

@juliankuners juliankuners left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, though I left a small comment

@anvacaru anvacaru merged commit 306ef67 into master Dec 8, 2025
23 of 24 checks passed
@anvacaru anvacaru deleted the test-harness-tweaks branch December 8, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants