New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adjusted summary report #553
adjusted summary report #553
Conversation
dcb906d
to
c60b57e
Compare
c60b57e
to
3ab0974
Compare
assert ( | ||
response["resourceModel"] == current_resource_model | ||
), "All properties specified in the request MUST be present in the model \ | ||
returned, and they MUST match exactly, with the exception of properties\ | ||
defined as writeOnlyProperties in the resource schema" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
think this error message might be misleading because there should be more exceptions:
readOnlyProperties, writeOnlyProperties, conditionallyReadOnlyProperties, defaults, insertionOrder, uniqueItems, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this been taken from the handler contract agreement
assert response["resourceModel"] == { | ||
**current_resource_model, | ||
**update_resource_model, | ||
} | ||
}, "All properties specified in the update request MUST be present in the \ | ||
model returned, and they MUST match exactly, with the exception of \ | ||
properties defined as writeOnlyProperties in the resource schema" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
think this error message might be misleading because there should be more exceptions:
readOnlyProperties, writeOnlyProperties, conditionallyReadOnlyProperties, defaults, insertionOrder, uniqueItems, etc.
Issue #, if available:
Description of changes:
added messages to all assertions so it will be clear why certain test failed;
removed the full stack trace to a
short
option as it outputs the assertion message and method it comes from;By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.