From 5c26d8e52fd48833ed22e4773915cc9b2a3b248b Mon Sep 17 00:00:00 2001 From: protolambda Date: Thu, 12 Dec 2019 16:29:30 +0100 Subject: [PATCH] fix normal signed case; only sign for 1 of the duplicate indices --- .../test_process_attester_slashing.py | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/test_libs/pyspec/eth2spec/test/phase_0/block_processing/test_process_attester_slashing.py b/test_libs/pyspec/eth2spec/test/phase_0/block_processing/test_process_attester_slashing.py index 3fc2f8a38..98a6e25e5 100644 --- a/test_libs/pyspec/eth2spec/test/phase_0/block_processing/test_process_attester_slashing.py +++ b/test_libs/pyspec/eth2spec/test/phase_0/block_processing/test_process_attester_slashing.py @@ -260,9 +260,14 @@ def test_att1_duplicate_index_normal_signed(spec, state): indices = attester_slashing.attestation_1.attesting_indices indices.pop(1) # remove an index, make room for the additional duplicate index. + attester_slashing.attestation_1.attesting_indices = sorted(indices) + + # sign it, the signature will be valid for a single occurence. If the transition accidentally ignores the duplicate. + sign_indexed_attestation(spec, state, attester_slashing.attestation_1) + indices.append(indices[0]) # add one of the indices a second time attester_slashing.attestation_1.attesting_indices = sorted(indices) - sign_indexed_attestation(spec, state, attester_slashing.attestation_1) + # it will just appear normal, unless the double index is spotted yield from run_attester_slashing_processing(spec, state, attester_slashing, False) @@ -275,9 +280,14 @@ def test_att2_duplicate_index_normal_signed(spec, state): indices = attester_slashing.attestation_2.attesting_indices indices.pop(2) # remove an index, make room for the additional duplicate index. + attester_slashing.attestation_2.attesting_indices = sorted(indices) + + # sign it, the signature will be valid for a single occurence. If the transition accidentally ignores the duplicate. + sign_indexed_attestation(spec, state, attester_slashing.attestation_2) + indices.append(indices[1]) # add one of the indices a second time attester_slashing.attestation_2.attesting_indices = sorted(indices) - sign_indexed_attestation(spec, state, attester_slashing.attestation_2) + # it will just appear normal, unless the double index is spotted yield from run_attester_slashing_processing(spec, state, attester_slashing, False)