Skip to content

Conversation

@alxkm
Copy link
Contributor

@alxkm alxkm commented Jul 31, 2025

Added additional test cases for OpenSearch filter expression converter:

  • testEmptyList() - Handles IN operations with empty lists gracefully
  • testSingleItemList() - Validates IN operations with single-item lists
  • testNullValue() - Tests equality checks against null values
  • testNestedJsonPath() - Supports nested field paths like entity.profile.name
  • testNumericStringValue() - Properly handles numeric values as strings
  • testZeroValue() - Edge case testing for zero values
  • testComplexNestedGroups() - Complex boolean logic with multiple AND/OR groupings
  • testMixedDataTypes() - Real-world scenarios mixing booleans, numbers, and strings
  • testNinWithMixedTypes() - NOT IN operations with various data types
  • testEmptyStringValue() - Not-equal comparisons with empty strings
  • testArrayIndexAccess() - Direct array element access like tags[0]

@alxkm alxkm changed the title test: Add additional test coverage for OpenSearchAiSearchFilterExpressionConverter test: Add additional test coverage for OpenSearchAiSearchFilterExpressionConverter Aug 4, 2025
@sobychacko sobychacko added this to the 1.1.0.M1 milestone Aug 6, 2025
@sobychacko sobychacko merged commit 5d20ea7 into spring-projects:main Aug 6, 2025
2 checks passed
scionaltera pushed a commit to scionaltera/spring-ai that referenced this pull request Sep 3, 2025
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
Willam2004 pushed a commit to Willam2004/spring-ai that referenced this pull request Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants