Skip to content

fix for failing test DateTimeCollapse2. Should be a single interval, …#89

Closed
bryantaustin13 wants to merge 1 commit intocqframework:mainfrom
bryantaustin13:DateTimeCollapse2_CqlIntervalOperatorsTest_fix
Closed

fix for failing test DateTimeCollapse2. Should be a single interval, …#89
bryantaustin13 wants to merge 1 commit intocqframework:mainfrom
bryantaustin13:DateTimeCollapse2_CqlIntervalOperatorsTest_fix

Conversation

@bryantaustin13
Copy link
Contributor

…not an array of intervals

<capability code="interval-operators"/>
<expression>collapse { Interval[DateTime(2012, 1, 1), DateTime(2012, 1, 15)], Interval[DateTime(2012, 1, 16), DateTime(2012, 5, 25)] }</expression>
<output>{Interval [ @2012-01-01T, @2012-05-25T ]}</output>
<output>Interval [ @2012-01-01T, @2012-05-25T ]</output>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So is the issue the HAPI server doesn't return a list?

Isn't it expected that CQL collapse will return a list? https://cql.hl7.org/09-b-cqlreference.html#collapse

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants