Jody Garnett ( 
https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=557058%3Ae422bb7f-4808-47e3-837f-13db0e6681e3
 ) *created* an issue

GeoServer ( 
https://osgeo-org.atlassian.net/browse/GEOS?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 ) / Task ( 
https://osgeo-org.atlassian.net/browse/GEOS-10238?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 ) GEOS-10238 ( 
https://osgeo-org.atlassian.net/browse/GEOS-10238?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 ) Test logging WARNING: Extension lookup, but ApplicationContext is unset ( 
https://osgeo-org.atlassian.net/browse/GEOS-10238?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 )

Issue Type: Task Assignee: Unassigned Components: Main Created: 18/Sep/21 1:12 
AM Environment:

maven

Priority: Medium Reporter: Jody Garnett ( 
https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?accountId=557058%3Ae422bb7f-4808-47e3-837f-13db0e6681e3
 )

GeoServer Extensions provides the ability to lookup functionality in the 
runtime environment (spring application context), and this class can be mocked 
during testing to try out functionality without a full integration test when an 
application context is not available.

In specific instances extension lookup is stubbed with a mock object allowing 
the extensions themselves to be tested. Over time the codebase changes, 
introducing more "expected" extensions - introducing warnings in tests that 
have already been written and were not updated to mock these new interfaces.

Ideas:

* Quick: Update the GeoServer Extension lookup so warnings are only produced in 
production, and null is returned during test cases.
* Safe: Update our procedures so that adding an extension requires also adding 
a mock object to the test environment to avoid these warnings.

Note the warning itself (reporting back when an extension is not available as 
application context is not provided) and produces two valid warnings on startup 
as of 2.20-RC where code is executing "too soon".

( 
https://osgeo-org.atlassian.net/browse/GEOS-10238#add-comment?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 ) Add Comment ( 
https://osgeo-org.atlassian.net/browse/GEOS-10238#add-comment?atlOrigin=eyJpIjoiMTUxMjhjODFiZDA0NDk0OGE0OWQxNWRmZjM1ZDllNzQiLCJwIjoiaiJ9
 )

Get Jira notifications on your phone! Download the Jira Cloud app for Android ( 
https://play.google.com/store/apps/details?id=com.atlassian.android.jira.core&referrer=utm_source%3DNotificationLink%26utm_medium%3DEmail
 ) or iOS ( 
https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=EmailNotificationLink&mt=8
 ) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100177- 
sha1:149c709 )
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to