Use SingleAttestation for Fulu in p2p attestation map (#14809)

rkapka · nalepae · web-flow · commit e473d7cc4d37 · 2025-01-20T10:35:52.000Z
* Use `SingleAttestation` for Fulu in p2p attestation map.

* Fix `TestExtractDataType`.

---------

Co-authored-by: Manu NALEPA &lt;enalepa@offchainlabs.com&gt;
diff --git a/beacon-chain/p2p/types/object_mapping.go b/beacon-chain/p2p/types/object_mapping.go
@@ -123,7 +123,7 @@ func InitializeDataMaps() {
 			return &ethpb.SingleAttestation{}, nil
 		},
 		bytesutil.ToBytes4(params.BeaconConfig().FuluForkVersion): func() (ethpb.Att, error) {
-			return &ethpb.AttestationElectra{}, nil
+			return &ethpb.SingleAttestation{}, nil
 		},
 	}
 
diff --git a/beacon-chain/sync/decode_pubsub_test.go b/beacon-chain/sync/decode_pubsub_test.go
@@ -292,7 +292,7 @@ func TestExtractDataType(t *testing.T) {
 				return wsb
 			}(),
 			wantMd:        wrapper.WrappedMetadataV1(&ethpb.MetaDataV1{}),
-			wantAtt:       &ethpb.AttestationElectra{},
+			wantAtt:       &ethpb.SingleAttestation{},
 			wantAggregate: &ethpb.SignedAggregateAttestationAndProofElectra{},
 			wantErr:       false,
 		},
diff --git a/changelog/radek_fulu-object-mapping.md b/changelog/radek_fulu-object-mapping.md
@@ -0,0 +1,3 @@
+### Fixed
+
+- Use `SingleAttestation` for Fulu in p2p attestation map.

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ func InitializeDataMaps() {`
`123`	`123`	`return &ethpb.SingleAttestation{}, nil`
`124`	`124`	`},`
`125`	`125`	`bytesutil.ToBytes4(params.BeaconConfig().FuluForkVersion): func() (ethpb.Att, error) {`
`126`		`- return &ethpb.AttestationElectra{}, nil`
	`126`	`+ return &ethpb.SingleAttestation{}, nil`
`127`	`127`	`},`
`128`	`128`	`}`
`129`	`129`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+### Fixed`
	`2`	`+`
	`3`	+- Use `SingleAttestation` for Fulu in p2p attestation map.