move emission of EngineResetConfirmedEvent to happen only when TryUpdateEngineEvent handler succeeds

[geoknee]

Closes #16351

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.31%. Comparing base (0f866d9) to head (ad89fe0).

❗ There is a different number of reports uploaded between BASE (0f866d9) and HEAD (ad89fe0). Click for more details.

HEAD has 3 uploads less than BASE

Flag	BASE (0f866d9)	HEAD (ad89fe0)
cannon-go-tests-64	2	0
contracts-bedrock-tests	2	1

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #16414       +/-   ##
============================================
- Coverage    82.14%   45.31%   -36.83%     
============================================
  Files          161     1335     +1174     
  Lines         8883   108247    +99364     
============================================
+ Hits          7297    49055    +41758     
- Misses        1448    55679    +54231     
- Partials       138     3513     +3375     

Flag	Coverage Δ
cannon-go-tests-64	?
contracts-bedrock-tests	96.24% <ø> (ø)

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

Files with missing lines	Coverage Δ
op-node/rollup/engine/events.go	76.83% <100.00%> (ø)

... and 1283 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.