openactive / OpenActive.Server.NET

.NET server library, including an OpenActive Reference Implementation
MIT License
0 stars 7 forks source link

orderItemStatus is missing from OrderItem at P #132

Closed lukehesluke closed 3 years ago

lukehesluke commented 3 years ago

access-channel-update-notifications_access-channel-update-notifications_OpenBookingApprovalFlow_ScheduledSession.md

## ScheduledSession >> P >> validation of P
### Specs
* ❌️ passes validation checks
    - Error: FAILURE: $.orderedItem[0].orderItemStatus: Required property `orderItemStatus` is missing from `OrderItem`.

This error is currently happening in master but was not previously. Not sure when it started. Maybe a data model upgrade with more stringent checks or recent change to .NET?

lukehesluke commented 3 years ago

nvm this is actually a test-suite issue. Just need to upgrade data-models. Doing that now

lukehesluke commented 3 years ago

Fixed here: https://github.com/openactive/openactive-test-suite/pull/394