Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Check for unsupported constraints #566

Merged
merged 8 commits into from
Aug 29, 2022
Merged

Conversation

gaurav
Copy link
Member

@gaurav gaurav commented Aug 18, 2022

This PR adds TRAPI Tests to ensure that CAM-KP-API responds correctly to:

Closes #565

@gaurav gaurav changed the base branch from master to permit-unlimited-recursion-attributes August 18, 2022 22:29
@gaurav gaurav mentioned this pull request Aug 18, 2022
15 tasks
Base automatically changed from permit-unlimited-recursion-attributes to master August 25, 2022 20:57
gaurav added 4 commits August 25, 2022 23:29
This test is now implemented in PR #560.
It previously returned a TRAPIMessage, but this prevented us from being
able to return log messages or other statuses.
@gaurav gaurav marked this pull request as ready for review August 26, 2022 04:03
@gaurav gaurav requested a review from balhoff August 26, 2022 04:08
Copy link
Contributor

@balhoff balhoff left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@gaurav gaurav merged commit 0956c32 into master Aug 29, 2022
@gaurav gaurav deleted the check-for-unsupported-constraints branch August 29, 2022 20:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure that CAM-KP-API responds correctly to unsupported constraints
2 participants