Consider adding option in isoltest to check for secondary error messages
Open
low effort
low impact
should have
testing :hammer:
- Dominant language
- C++
- Stars
- 25.7k
- Forks
- 6.2k
- Avg merge
- 2d 19h
- Merged PRs (30d)
- 29
Description
Currently, secondary error messages do not show up in `isoltest` and it is necessary to create a command line test to properly check those.
Suggested in https://github.com/ethereum/solidity/pull/15463#discussion_r1947067163.
Contributor guide
Research direction
Start with the isoltest entry point and read the discussion linked from PR 15463, focusing on how secondary error messages are currently handled. Determine the expected option behavior from that discussion, then add coverage showing that isoltest can check secondary errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, solidity
- Domain
- compilers, testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100