Speed up build time (#701)

Speed up build time by removing some of the most time consuming tests

* CALLBlake2f_MaxRounds - runs Blake2f 4 billion rounds, something no
  one ever does and there is never enough gas to do. We have a unit test
  for this already.
* The "do this 50k times" tests.  Each has a smaller "5k times" or less
  variant. This drifts into performance testing instead of conformance
  testing

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
pull/703/head
Danno Ferrin 5 years ago committed by GitHub
parent dcf8b12c8a
commit 7b410b3ba8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      ethereum/core/src/test/java/org/hyperledger/besu/ethereum/vm/BlockchainReferenceTestTools.java
  2. 7
      ethereum/core/src/test/java/org/hyperledger/besu/ethereum/vm/GeneralStateReferenceTestTools.java

@ -71,6 +71,10 @@ public class BlockchainReferenceTestTools {
// Absurd amount of gas, doesn't run in parallel // Absurd amount of gas, doesn't run in parallel
params.blacklist("randomStatetest94_\\w+"); params.blacklist("randomStatetest94_\\w+");
// Don't do time consuming tests
params.blacklist("CALLBlake2f_MaxRounds.*");
params.blacklist(".*\\w50000[-_].*");
} }
public static Collection<Object[]> generateTestParametersForConfig(final String[] filePath) { public static Collection<Object[]> generateTestParametersForConfig(final String[] filePath) {

@ -82,13 +82,20 @@ public class GeneralStateReferenceTestTools {
if (EIPS_TO_RUN.isEmpty()) { if (EIPS_TO_RUN.isEmpty()) {
params.blacklistAll(); params.blacklistAll();
} }
// Known incorrect test. // Known incorrect test.
params.blacklist( params.blacklist(
"RevertPrecompiledTouch(_storage)?-(EIP158|Byzantium|Constantinople|ConstantinopleFix)"); "RevertPrecompiledTouch(_storage)?-(EIP158|Byzantium|Constantinople|ConstantinopleFix)");
// Gas integer value is too large to construct a valid transaction. // Gas integer value is too large to construct a valid transaction.
params.blacklist("OverflowGasRequire"); params.blacklist("OverflowGasRequire");
// Consumes a huge amount of memory // Consumes a huge amount of memory
params.blacklist("static_Call1MB1024Calldepth-\\w"); params.blacklist("static_Call1MB1024Calldepth-\\w");
// Don't do time consuming tests
params.blacklist("CALLBlake2f_MaxRounds.*");
params.blacklist(".*\\w50000[-_].*");
} }
public static Collection<Object[]> generateTestParametersForConfig(final String[] filePath) { public static Collection<Object[]> generateTestParametersForConfig(final String[] filePath) {

Loading…
Cancel
Save