hemasrishalini opened a new issue, #4639:
URL: https://github.com/apache/polaris/issues/4639

   ### Describe the bug
   
   DefaultConfigurationStoreTest currently covers feature configuration 
defaults and valid configuration behavior.
   
   However, it does not test invalid feature JSON scenarios in 
BehaviorChangesConfiguration / FeatureConfiguration handling.
   
   Missing cases:
   - malformed feature JSON
   - invalid or unexpected JSON structure
   - error path when configuration parsing fails
   
   Impact:
   Without this test coverage, invalid configuration input may not be properly 
validated or may fail silently.
   
   Expected behavior:
   Invalid feature JSON should be rejected with a clear error/exception path.
   
   Suggested fix:
   Add a unit/integration test in DefaultConfigurationStoreTest to validate 
invalid feature JSON handling and ensure proper error response.
   
   ### To Reproduce
   
   Not applicable — this is a missing test coverage issue in existing codebase.
   
   ### Actual Behavior
   
   Invalid feature JSON in BehaviorChangesConfiguration is not explicitly 
tested in DefaultConfigurationStoreTest.
   
   ### Expected Behavior
   
   Invalid feature JSON should be validated and rejected with a proper 
error/exception path.
   
   ### Additional context
   
   This issue is based on observed missing test coverage in 
DefaultConfigurationStoreTest for BehaviorChangesConfiguration handling of 
invalid feature JSON.
   
   ### System information
   
   OS: Linux
   Polaris Catalog Version: latest (main branch)
   Object storage & setup: local/test environment


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to