op-node/p2p: Fix deleteRecord to return nil if store.Delete succeeds

[sebastianst]

Description

If deletion in the underlying store returns nil, deleteRecord should also return nil.

Tests

None (yet)

[codecov]

Codecov Report

Merging #8211 (7ffa3f7) into develop (2e35a8f) will increase coverage by 5.95%.
Report is 11 commits behind head on develop.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #8211      +/-   ##
===========================================
+ Coverage    53.44%   59.39%   +5.95%     
===========================================
  Files          162       86      -76     
  Lines         6040     1931    -4109     
  Branches       964      438     -526     
===========================================
- Hits          3228     1147    -2081     
+ Misses        2690      698    -1992     
+ Partials       122       86      -36     

Flag	Coverage Δ
cannon-go-tests	?
chain-mon-tests	?
common-ts-tests	?
contracts-bedrock-tests	59.39% <ø> (-1.87%)	⬇️
contracts-ts-tests	?
core-utils-tests	?
sdk-next-tests	?
sdk-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 82 files with indirect coverage changes

[mslipper]

CI is fixed in #8213.