aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Beregszaszi <alex@rtfs.hu>2017-11-23 01:05:01 +0800
committerGitHub <noreply@github.com>2017-11-23 01:05:01 +0800
commitea18bed319c67d509992572454e382d23493c1eb (patch)
tree1f542de561d4928c5c3cc4269b9e4dc785a013da
parent03b763aceeb0f2715ab024f2fb573574a7693f9c (diff)
parent4c50d3f8e12bbc158d0d8cc39d709b5f00382b66 (diff)
downloaddexon-solidity-ea18bed319c67d509992572454e382d23493c1eb.tar.gz
dexon-solidity-ea18bed319c67d509992572454e382d23493c1eb.tar.zst
dexon-solidity-ea18bed319c67d509992572454e382d23493c1eb.zip
Merge pull request #3235 from ethereum/explicit-tests
Improve expected test errors for some old cases
-rw-r--r--test/libsolidity/SolidityNameAndTypeResolution.cpp8
1 files changed, 4 insertions, 4 deletions
diff --git a/test/libsolidity/SolidityNameAndTypeResolution.cpp b/test/libsolidity/SolidityNameAndTypeResolution.cpp
index 4f576e51..97d359e8 100644
--- a/test/libsolidity/SolidityNameAndTypeResolution.cpp
+++ b/test/libsolidity/SolidityNameAndTypeResolution.cpp
@@ -1392,7 +1392,7 @@ BOOST_AUTO_TEST_CASE(events_with_same_name)
event A(uint i);
}
)";
- BOOST_CHECK(success(text));
+ CHECK_SUCCESS(text);
}
BOOST_AUTO_TEST_CASE(events_with_same_name_unnamed_arguments)
@@ -4033,7 +4033,7 @@ BOOST_AUTO_TEST_CASE(varM_disqualified_as_keyword)
}
}
)";
- BOOST_CHECK(!success(text));
+ CHECK_ERROR(text, DeclarationError, "Identifier not found or not unique.");
}
BOOST_AUTO_TEST_CASE(modifier_is_not_a_valid_typename)
@@ -4074,7 +4074,7 @@ BOOST_AUTO_TEST_CASE(long_uint_variable_fails)
}
}
)";
- BOOST_CHECK(!success(text));
+ CHECK_ERROR(text, DeclarationError, "Identifier not found or not unique.");
}
BOOST_AUTO_TEST_CASE(bytes10abc_is_identifier)
@@ -4113,7 +4113,7 @@ BOOST_AUTO_TEST_CASE(library_functions_do_not_have_value)
}
}
)";
- BOOST_CHECK(!success(text));
+ CHECK_ERROR(text, TypeError, "Member \"value\" not found or not visible after argument-dependent lookup in function ()");
}
BOOST_AUTO_TEST_CASE(invalid_fixed_types_0x7_mxn)