isActiveStorageContext() fails in 9.5.x

The attached patch adds a basic test that fails because of this issue.

This patch refactors isActiveStorageContext() to make use of Drupal\Component\DependencyInjection\ReverseContainer for reverse lookup of the storage service. Since this branch is currently cites compatibility with D9 in general, I have attempted to make the change backwards compatible, with the following approaches:

• In 9.5.1+, ReverseContainer will be used
• In 9.5.0, as ReverseContainer does not exist, getServiceIdMapping() will be used
• in 9.4 and below, it will fall back on the previous approach of the _serviceId property

At the point when only 9.5.1+ compatibility is needed, I think these deprecated fallbacks should probably be removed.

The patch includes the new test from @joegraduate thanks!

In my testing with 9.4.15, 9.5.0, and 9.5.1+ this seems to fix the original broken config_sync → test ( 📌 Address 3.0.x-dev test failures since Drupal 9.5.x Fixed ) that revealed this issue.

#4 looks great to me. I think we should commit this and create a new release that includes this change. Thanks @tadean for coming up with such a thorough backwards-compatible fix for this issue!

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