Skip to content

Added additional tests for CqlStringOperatorsTest as suggested by Claude - #136

Merged
brynrhodes merged 4 commits into
cqframework:mainfrom
bryantaustin13:claudeStringOperators
Oct 19, 2026
Merged

brynrhodes merged 4 commits into
cqframework:mainfrom
bryantaustin13:claudeStringOperators

Conversation

@bryantaustin13

Copy link
Copy Markdown
Contributor

No description provided.

… operator, empty-affix/pattern, Combine null handling) for connectathon testing

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@brynrhodes brynrhodes added the confirmed Indicates that an independent engine implementer has confirmed the behavior label Sep 14, 2026
Comment thread tests/cql/CqlStringOperatorsTest.xml Resolved
The empty-pattern behavior of LastPositionOf was first documented in
2.0, as with PositionOfEmpty.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bryantaustin13

Copy link
Copy Markdown
Contributor Author

LastPositionOfEmpty now uses version="2.0" l

@bryantaustin13

Copy link
Copy Markdown
Contributor Author

@brynrhodes I applied the same change to LastPositionOfEmpty, which now uses version="2.0" like PositionOfEmpty. The empty-pattern sentence was added to LastPositionOf in 2.0 as well; it isn't in 1.5.3. (881e179)

@brynrhodes

Copy link
Copy Markdown
Member

The text in 2.0 is a clarification of the behavior, it is still expected to pass as a 1.0 test.

@brynrhodes
brynrhodes merged commit d6e6956 into cqframework:main Oct 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

confirmed Indicates that an independent engine implementer has confirmed the behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants