Problem/Motivation

Follow-up to #2509898: Additional uncaught exception thrown while handling exception after service changes, the ErrorContainer::get() and ExceptionContainer::get() blows up for any service which is requested. As a result the behavior of the tests depends on unrelated factors like the sequence of service instantiation during bootstrap. This can lead to bogus test failures when attempting to change code in the early bootstrap or the error handler.

Proposed resolution

Restrict the failure condition to one service (e.g. http_kernel).

Remaining tasks

User interface changes

API changes

Data model changes

Beta evaluation: Unfrozen as test only changes.

CommentFileSizeAuthor
#1 behavior_of-2540538-1.patch1.54 KBznerol

Comments

znerol’s picture

znerol commented
Status: Active » Needs review
StatusFileSize
new behavior_of-2540538-1.patch1.54 KB
fabianx’s picture

fabianx commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

- RTBC, makes sense to me.
- Added short beta evaluation.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Make sense to me to. Committed e536cb0 and pushed to 8.0.x. Thanks!

  • alexpott committed e536cb0 on 8.0.x 
    Issue #2540538 by znerol: Behavior of testErrorContainer() and...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.