Activate tests for Petersburg fork

Author: cburgdorf

.circleci/config.yml

@@ -84,6 +84,12 @@ jobs:
       - image: circleci/python:3.5
         environment:
           TOXENV: py35-native-blockchain-homestead
+  py35-native-blockchain-petersburg:
+    <<: *common
+    docker:
+      - image: circleci/python:3.5
+        environment:
+          TOXENV: py35-native-blockchain-petersburg
   py35-native-blockchain-tangerine_whistle:
     <<: *common
     docker:
@@ -157,6 +163,12 @@ jobs:
       - image: circleci/python:3.6
         environment:
           TOXENV: py36-native-blockchain-homestead
+  py36-native-blockchain-petersburg:
+    <<: *common
+    docker:
+      - image: circleci/python:3.6
+        environment:
+          TOXENV: py36-native-blockchain-petersburg
   py36-native-blockchain-tangerine_whistle:
     <<: *common
     docker:
@@ -219,6 +231,7 @@ workflows:
       - py36-native-blockchain-constantinople
       - py36-native-blockchain-frontier
       - py36-native-blockchain-homestead
+      - py36-native-blockchain-petersburg
       - py36-native-blockchain-tangerine_whistle
       - py36-native-blockchain-spurious_dragon
       - py36-native-blockchain-transition
@@ -232,6 +245,7 @@ workflows:
       - py35-native-blockchain-constantinople
       - py35-native-blockchain-frontier
       - py35-native-blockchain-homestead
+      - py35-native-blockchain-petersburg
       - py35-native-blockchain-tangerine_whistle
       - py35-native-blockchain-spurious_dragon
       - py35-native-blockchain-transition

eth/tools/builder/chain/builders.py

@@ -70,6 +70,7 @@
     SpuriousDragonVM,
     ByzantiumVM,
     ConstantinopleVM,
+    PetersburgVM,
 )
 
 
@@ -225,7 +226,7 @@ def dao_fork_at(dao_fork_block_number: BlockNumber,
 spurious_dragon_at = fork_at(SpuriousDragonVM)
 byzantium_at = fork_at(ByzantiumVM)
 constantinople_at = fork_at(ConstantinopleVM)
-
+petersburg_at = fork_at(PetersburgVM)
 
 GENESIS_DEFAULTS = (
     ('difficulty', 1),

eth/tools/fixtures/helpers.py

@@ -43,6 +43,7 @@
     BaseVM,
 )
 from eth.vm.forks import (
+    PetersburgVM,
     ConstantinopleVM,
     ByzantiumVM,
     TangerineWhistleVM,
@@ -131,6 +132,10 @@ def chain_vm_configuration(fixture: Dict[str, Any]) -> Iterable[Tuple[int, Type[
         return (
             (0, ConstantinopleVM),
         )
+    elif network == 'ConstantinopleFix':
+        return (
+            (0, PetersburgVM),
+        )
     elif network == 'FrontierToHomesteadAt5':
         HomesteadVM = BaseHomesteadVM.configure(support_dao_fork=False)
         return (

tests/conftest.py

@@ -20,6 +20,7 @@
     SpuriousDragonVM,
     ByzantiumVM,
     ConstantinopleVM,
+    PetersburgVM,
 )
 
 
@@ -71,6 +72,7 @@ def _file_logging(request):
                 SpuriousDragonVM,
                 ByzantiumVM,
                 ConstantinopleVM,
+                PetersburgVM,
                 ])
 def VM(request):
     return request.param

tests/core/chain-object/test_contract_call.py

@@ -26,6 +26,7 @@
     SpuriousDragonVM,
     ByzantiumVM,
     ConstantinopleVM,
+    PetersburgVM,
 )
 
 
@@ -202,6 +203,16 @@ def test_get_transaction_result(
             'useLotsOfGas()',
             OutOfGas,
         ),
+        (
+            PetersburgVM,
+            'doRevert()',
+            Revert,
+        ),
+        (
+            PetersburgVM,
+            'useLotsOfGas()',
+            OutOfGas,
+        ),
     ),
 )
 def test_get_transaction_result_revert(

tests/json-fixtures/test_transactions.py

@@ -31,6 +31,9 @@
 from eth.vm.forks.constantinople.transactions import (
     ConstantinopleTransaction
 )
+from eth.vm.forks.petersburg.transactions import (
+    PetersburgTransaction
+)
 
 from eth_typing.enums import (
     ForkName
@@ -96,6 +99,8 @@ def fixture_transaction_class(fixture_data):
         return ByzantiumTransaction
     elif fork_name == ForkName.Constantinople:
         return ConstantinopleTransaction
+    elif fork_name == "Petersburg":
+        return PetersburgTransaction
     elif fork_name == ForkName.Metropolis:
         pytest.skip("Metropolis Transaction class has not been implemented")
     else:

tox.ini

@@ -2,7 +2,7 @@
 envlist=
     py{35,36}-{core,database,transactions,vm}
     py{36}-{benchmark}
-    py{35,36}-native-blockchain-{frontier,homestead,tangerine_whistle,spurious_dragon,byzantium,constantinople,metropolis,transition}
+    py{35,36}-native-blockchain-{frontier,homestead,tangerine_whistle,spurious_dragon,byzantium,constantinople,petersburg,metropolis,transition}
     py37-{core}
     py{35,36}-lint
     py36-docs
@@ -28,6 +28,7 @@ commands=
     native-blockchain-spurious_dragon: pytest {posargs:tests/json-fixtures/test_blockchain.py --fork EIP158}
     native-blockchain-byzantium: pytest {posargs:tests/json-fixtures/test_blockchain.py --fork Byzantium}
     native-blockchain-constantinople: pytest {posargs:tests/json-fixtures/test_blockchain.py --fork Constantinople}
+    native-blockchain-petersburg: pytest {posargs:tests/json-fixtures/test_blockchain.py --fork ConstantinopleFix}
     native-blockchain-metropolis: pytest {posargs:tests/json-fixtures/test_blockchain.py --fork Metropolis}
     native-blockchain-transition: pytest {posargs:tests/json-fixtures/test_blockchain.py -k BlockchainTests/TransitionTests}