Fix lint errors

Author: adiasg

tests/helpers/consensus.py

@@ -33,6 +33,7 @@ def consensus_on_attestation(slashing_db: SlashingDB, attestation_duty: Attestat
     The consensus protocol must use `consensus_is_valid_attestation_data` to determine
     validity of the proposed attestation value.
     """
+    # TODO: Use this method in tests instead of dvspec.consensus.consensus_on_attestation
     attestation_data = bn_produce_attestation_data(attestation_duty.slot, attestation_duty.committee_index)
     assert consensus_is_valid_attestation_data(slashing_db, attestation_data)
     return attestation_data
@@ -45,4 +46,5 @@ def consensus_on_block(slashing_db: SlashingDB, proposer_duty: ProposerDuty) ->
     The consensus protocol must use `consensus_is_valid_block` to determine
     validity of the proposed block value.
     """
+    # TODO: Implement & use this method in tests instead of dvspec.consensus.consensus_on_block
     pass

tests/helpers/eth_node_interface.py

@@ -84,7 +84,8 @@ def bn_produce_block(slot: Slot, randao_reveal: BLSSignature, graffiti: Bytes32)
 """
 
 
-def fill_attestation_duties_with_val_index(state: State, attestation_duties: List[AttestationDuty]) -> List[AttestationDuty]:
+def fill_attestation_duties_with_val_index(state: State,
+                                           attestation_duties: List[AttestationDuty]) -> List[AttestationDuty]:
     val_index_to_pubkey = {}
     for dv in state.distributed_validators:
         val_index_to_pubkey[dv.validator_identity.index] = dv.validator_identity.pubkey
@@ -93,7 +94,6 @@ def fill_attestation_duties_with_val_index(state: State, attestation_duties: Lis
     return attestation_duties
 
 
-
 def fill_proposer_duties_with_val_index(state: State, proposer_duties: List[ProposerDuty]) -> List[ProposerDuty]:
     val_index_to_pubkey = {}
     for dv in state.distributed_validators:

tests/helpers/state.py

@@ -23,11 +23,11 @@ def build_distributed_validator(validator_identity: ValidatorIdentity, num_coval
     for i in range(1, num_covalidators):
         co_validators.append(CoValidator(validator_identity=validator_identity, pubkey=BLSPubkey(0x00), index=i))
     slashing_db = SlashingDB(interchange_format_version=5,
-                            genesis_validators_root=Root(),
-                            data=[])
+                             genesis_validators_root=Root(),
+                             data=[])
     distributed_validator = DistributedValidator(validator_identity=validator_identity,
-                                                co_validators=co_validators,
-                                                slashing_db=slashing_db)
+                                                 co_validators=co_validators,
+                                                 slashing_db=slashing_db)
     return distributed_validator
 
 

tests/helpers/time.py

@@ -17,6 +17,7 @@
 Time Methods
 """
 
+
 def time_generator() -> Iterator[int]:
     time = GENESIS_TIME
     while True:

tests/test.py

@@ -20,7 +20,7 @@
 def test_basic_attestation():
     state = build_state(5)
     time = get_current_time()
-    
+
     current_epoch = compute_epoch_at_time(time)
     validator_indices = get_validator_indices(state)
     attestation_duties = bn_get_attestation_duties_for_epoch(validator_indices, current_epoch+1)
@@ -32,4 +32,4 @@ def test_basic_attestation():
 
     # TODO: Need to replace dvspec.consensus.consensus_on_attestation with
     # tests.helpers.consensus.consensus_on_attestation
-    serve_attestation_duty(slashing_db, attestation_duty)
+    serve_attestation_duty(slashing_db, attestation_duty)