-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Open
Labels
bug 🐛low effortThere is not much implementation work to be done. The task is very easy or tiny.There is not much implementation work to be done. The task is very easy or tiny.low impactChanges are not very noticeable or potential benefits are limited.Changes are not very noticeable or potential benefits are limited.should compile without errorError is reported even though it shouldn't. Source is fine.Error is reported even though it shouldn't. Source is fine.should haveWe like the idea but it’s not important enough to be a part of the roadmap.We like the idea but it’s not important enough to be a part of the roadmap.testing 🔨
Description
Steps to reproduce
Run isoltest on a specific test (given below), trying to update it interactively to add gas usage to expected output:
./test/tools/isoltest --no-smt -t "semanticTests/deployedCodeExclusion/module_function_deployed*" --enforce-gas-cost --enforce-gas-cost-min-value 0First, isoltest asks if we want to update expectations. On update, the test is re-run with updated expectations and isoltest crashes:
Re-running test case...
semanticTests/deployedCodeExclusion/module_function_deployed.sol: Unhandled exception during test: /solidity/libsolidity/interface/CompilerStack.cpp(1703): Throw in function const CompilerStack::Contract &solidity::frontend::CompilerStack::contract(const std::string &) const
Dynamic exception type: boost::wrapexcept<solidity::langutil::InternalCompilerError>
std::exception::what: Contract "" not found.
[solidity::util::tag_comment*] = Contract "" not found.
Metadata
Metadata
Assignees
Labels
bug 🐛low effortThere is not much implementation work to be done. The task is very easy or tiny.There is not much implementation work to be done. The task is very easy or tiny.low impactChanges are not very noticeable or potential benefits are limited.Changes are not very noticeable or potential benefits are limited.should compile without errorError is reported even though it shouldn't. Source is fine.Error is reported even though it shouldn't. Source is fine.should haveWe like the idea but it’s not important enough to be a part of the roadmap.We like the idea but it’s not important enough to be a part of the roadmap.testing 🔨