Update exclusions to match the API documentation generator's

Author: szemate

Makefile

@@ -25,7 +25,7 @@ all: $(OUTDIR)/accountability.py \
 	 $(OUTDIR)/upgrade_manager.py
 
 $(OUTDIR)/accountability.py: $(call gentargets,Accountability)
-	$(call abigen,$^) --exclude distributeRewards,finalize,setEpochPeriod >$@
+	$(call abigen,$^) --exclude distributeRewards,finalize,handleEvent,setEpochPeriod >$@
 
 $(OUTDIR)/acu.py: $(call gentargets,ACU)
 	$(call abigen,$^) --exclude setOperator,setOracle,update >$@
@@ -46,13 +46,13 @@ $(OUTDIR)/non_stakable_vesting.py: $(call gentargets,NonStakableVesting)
 	$(call abigen,$^) --exclude unlockTokens >$@
 
 $(OUTDIR)/oracle.py: $(call gentargets,Oracle)
-	$(call abigen,$^) --exclude finalize,setOperator,setVoters >$@
+	$(call abigen,$^) --exclude finalize,setOperator,setVoters,vote >$@
 
 $(OUTDIR)/stabilization.py: $(call gentargets,Stabilization)
 	$(call abigen,$^) --exclude setOperator,setOracle >$@
 
 $(OUTDIR)/supply_control.py: $(call gentargets,SupplyControl)
-	$(call abigen,$^) --exclude setOperator >$@
+	$(call abigen,$^) --exclude burn,mint,setOperator >$@
 
 $(OUTDIR)/upgrade_manager.py: $(call gentargets,UpgradeManager)
 	$(call abigen,$^) --exclude setOperator >$@

autonity/contracts/accountability.py

@@ -329,43 +329,6 @@ def get_validator_faults(
             for elem in return_value
         ]
 
-    def handle_event(
-        self,
-        _event: Event,
-    ) -> contract.ContractFunction:
-        """Binding for `handleEvent` on the Accountability contract.
-
-        Handle an accountability event. Need to be called by a registered validator
-        account as the treasury-linked account will be used in case of a successful
-        slashing event. todo(youssef): rethink modifiers here, consider splitting this
-        into multiple functions.
-
-        Parameters
-        ----------
-        _event : Event
-
-        Returns
-        -------
-        web3.contract.contract.ContractFunction
-            A contract function instance to be sent in a transaction.
-        """
-        return self._contract.functions.handleEvent(
-            (
-                _event.chunks,
-                _event.chunk_id,
-                int(_event.event_type),
-                int(_event.rule),
-                _event.reporter,
-                _event.offender,
-                _event.raw_proof,
-                _event.id,
-                _event.block,
-                _event.epoch,
-                _event.reporting_block,
-                _event.message_hash,
-            ),
-        )
-
     def slashing_history(
         self,
         key0: eth_typing.ChecksumAddress,

autonity/contracts/oracle.py

@@ -335,39 +335,6 @@ def symbols(
         ).call()
         return str(return_value)
 
-    def vote(
-        self,
-        _commit: int,
-        _reports: typing.List[int],
-        _salt: int,
-    ) -> contract.ContractFunction:
-        """Binding for `vote` on the Oracle contract.
-
-        Vote for the current period. In order to save gas, if (reports[i] ==
-        INVALID_PRICE)g the symbols. if the validator leave consensus committee then his
-        vote is discarded. if a validator joins the consensus committee then his first
-        vote is not taken into account. Only allowed to vote once per round.
-
-        Parameters
-        ----------
-        _commit : int
-            hash of the new reports        _reports reveal of the reports for the
-            previous cycle.        _salt  slat value which was used to generate last
-            round commitment
-        _reports : typing.List[int]
-        _salt : int
-
-        Returns
-        -------
-        web3.contract.contract.ContractFunction
-            A contract function instance to be sent in a transaction.
-        """
-        return self._contract.functions.vote(
-            _commit,
-            _reports,
-            _salt,
-        )
-
     def vote_period(
         self,
     ) -> int:

autonity/contracts/supply_control.py

@@ -66,48 +66,6 @@ def available_supply(
         return_value = self._contract.functions.availableSupply().call()
         return int(return_value)
 
-    def burn(
-        self,
-    ) -> contract.ContractFunction:
-        """Binding for `burn` on the SupplyControl contract.
-
-        Burn Auton by taking it out of circulation. Only the stabilizer is authorized to
-        burn Auton.
-
-        Returns
-        -------
-        web3.contract.contract.ContractFunction
-            A contract function instance to be sent in a transaction.
-        """
-        return self._contract.functions.burn()
-
-    def mint(
-        self,
-        recipient: eth_typing.ChecksumAddress,
-        amount: int,
-    ) -> contract.ContractFunction:
-        """Binding for `mint` on the SupplyControl contract.
-
-        Mint Auton and send it to the recipient. Only the stabilizer is authorized to
-        mint Auton. The recipient cannot be the stabilizer or the zero address.
-
-        Parameters
-        ----------
-        recipient : eth_typing.ChecksumAddress
-            Recipient of the Auton
-        amount : int
-            Amount of Auton to mint (non-zero)
-
-        Returns
-        -------
-        web3.contract.contract.ContractFunction
-            A contract function instance to be sent in a transaction.
-        """
-        return self._contract.functions.mint(
-            recipient,
-            amount,
-        )
-
     def set_stabilizer(
         self,
         stabilizer_: eth_typing.ChecksumAddress,

tests/test_sanity.py

@@ -38,8 +38,6 @@
     List[ChecksumAddress]: ["0x0123456789abcDEF0123456789abCDef01234567"],
 }
 
-SKIPPED = [("Accountability", "handle_event")]
-
 
 def pytest_generate_tests(metafunc):
     if "contract_function" not in metafunc.fixturenames:
@@ -63,11 +61,6 @@ def pytest_generate_tests(metafunc):
         for attr_name in dir(contract):
             if attr_name.startswith("_"):
                 continue
-            if any(
-                binding.__name__ == skipped_binding and attr_name == skipped_attr
-                for skipped_binding, skipped_attr in SKIPPED
-            ):
-                continue
             attr = getattr(contract, attr_name)
             if isinstance(attr, Callable):
                 functions.append(attr)