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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

znerol’s picture

znerol CreditAttribution: znerol commented
Status: Active » Needs review
FileSize
behavior_of-2540538-1.patch1.54 KB
Fabianx’s picture

Fabianx CreditAttribution: Fabianx for Acquia 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 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three 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.