Skip to content

test: Change spelling to "TangerineWhistle" and "SpuriousDragon"#1576

Merged
chfast merged 1 commit into
masterfrom
test/fork_names
Jun 29, 2026
Merged

test: Change spelling to "TangerineWhistle" and "SpuriousDragon"#1576
chfast merged 1 commit into
masterfrom
test/fork_names

Conversation

@chfast

@chfast chfast commented Jun 29, 2026

Copy link
Copy Markdown
Member

Spell fork names without space to match execution-specs:

  • "Tangerine Whistle" → "TangerineWhistle",
  • "Spurious Dragon" → "SpuriousDragon".

This allows us to execute new test suite for these forks in next release of tests in execution-specs. Also handy for evmone t8n.

We decided not to support old spelling as this fork name is old and not very user-facing.

Spell fork names without space to match execution-specs:
- "Tangerine Whistle" → "TangerineWhistle",
- "Spurious Dragon" → "SpuriousDragon".

This allows us to execute new test suite for these forks in next
release of tests in execution-specs. Also handy for `evmone t8n`.

We decided not to support old spelling as this fork name is old and
not very user-facing.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 231 to +234
case EVMC_TANGERINE_WHISTLE:
return "Tangerine Whistle";
return "TangerineWhistle";
case EVMC_SPURIOUS_DRAGON:
return "Spurious Dragon";
return "SpuriousDragon";
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.39%. Comparing base (a2a993c) to head (b160627).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
test/utils/utils.cpp 0.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1576   +/-   ##
=======================================
  Coverage   97.39%   97.39%           
=======================================
  Files         163      163           
  Lines       14680    14680           
  Branches     3407     3407           
=======================================
  Hits        14298    14298           
  Misses        280      280           
  Partials      102      102           
Flag Coverage Δ
eest-develop 89.31% <0.00%> (ø)
eest-develop-gmp 26.06% <0.00%> (ø)
eest-legacy 17.49% <0.00%> (ø)
eest-libsecp256k1 27.69% <0.00%> (ø)
eest-stable 89.40% <0.00%> (ø)
evmone-unittests 92.62% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
core 95.97% <ø> (ø)
tooling 90.21% <0.00%> (ø)
tests 99.80% <ø> (ø)
Files with missing lines Coverage Δ
test/utils/utils.cpp 79.66% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chfast chfast merged commit 4656551 into master Jun 29, 2026
23 of 24 checks passed
@chfast chfast deleted the test/fork_names branch June 29, 2026 14:06
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.

2 participants