diff options
author | Alex Beregszaszi <alex@rtfs.hu> | 2018-02-20 23:42:02 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-02-20 23:42:02 +0800 |
commit | f5f00b4ee9e6e9038e3b84986ab3ff78504331fd (patch) | |
tree | fc43837724bdd5f2c1e66e15353a446b2f806088 /test/libsolidity | |
parent | a6b52fdc34650c74597c1bcdc5075b6375c62650 (diff) | |
parent | 58bbb04276f27d3ecfa96dda54fbad80f8bbdbea (diff) | |
download | dexon-solidity-f5f00b4ee9e6e9038e3b84986ab3ff78504331fd.tar.gz dexon-solidity-f5f00b4ee9e6e9038e3b84986ab3ff78504331fd.tar.zst dexon-solidity-f5f00b4ee9e6e9038e3b84986ab3ff78504331fd.zip |
Merge pull request #3535 from ethereum/fixShadowing
Fix shadowing detection for aliases
Diffstat (limited to 'test/libsolidity')
-rw-r--r-- | test/libsolidity/Imports.cpp | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/test/libsolidity/Imports.cpp b/test/libsolidity/Imports.cpp index 03287b28..dc1174f4 100644 --- a/test/libsolidity/Imports.cpp +++ b/test/libsolidity/Imports.cpp @@ -266,7 +266,28 @@ BOOST_AUTO_TEST_CASE(shadowing_builtins_with_multiple_imports) } } - +BOOST_AUTO_TEST_CASE(shadowing_builtins_with_alias) +{ + CompilerStack c; + c.addSource("B.sol", "contract C {} pragma solidity >=0.0;"); + c.addSource("b", R"( + pragma solidity >=0.0; + import {C as msg} from "B.sol"; + )"); + BOOST_CHECK(c.compile()); + auto numErrors = c.errors().size(); + // Sometimes we get the prerelease warning, sometimes not. + BOOST_CHECK(1 <= numErrors && numErrors <= 2); + for (auto const& e: c.errors()) + { + string const* msg = e->comment(); + BOOST_REQUIRE(msg); + BOOST_CHECK( + msg->find("pre-release") != string::npos || + msg->find("shadows a builtin symbol") != string::npos + ); + } +} BOOST_AUTO_TEST_SUITE_END() |