Description of changes:
Prior to these changes, the following contract tests were failing:
contract_delete_read (AssertionError: The deletion handler's response object MUST NOT contain a model)
contract_delete_update (AssertionError: The deletion handler's response object MUST NOT contain a model)
contract_delete_delete (AssertionError: The deletion handler's response object MUST NOT contain a model)
contract_delete_create (AssertionError: The deletion handler's response object MUST NOT contain a model)
contract_read_without_create (AssertionError: A read handler MUST return FAILED with a NotFound error code if the resource does not exist)
contract_update_non_existent_resource (AssertionError: An update handler MUST return FAILED with a NotFound error code if the resource did not exist prior to the update request)
These changes resolve those failing tests by changing the following:
Modify the memory size for the Function resources as recommended by CFN, as this was causing the tests to fail without completing.
Remove resource model from delete handler response
Add ResourceNotFound and InvalidArn error codes to a set of exceptions which should be mapped to NotFound handler error code.
Use describeActivity call to check if an activity exists before attempting to delete it.
Add check to see if ARN is null before making API call
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Description of changes: Prior to these changes, the following contract tests were failing:
AssertionError: The deletion handler's response object MUST NOT contain a model)AssertionError: The deletion handler's response object MUST NOT contain a model)AssertionError: The deletion handler's response object MUST NOT contain a model)AssertionError: The deletion handler's response object MUST NOT contain a model)AssertionError: A read handler MUST return FAILED with a NotFound error code if the resource does not exist)AssertionError: An update handler MUST return FAILED with a NotFound error code if the resource did not exist prior to the update request)These changes resolve those failing tests by changing the following:
ResourceNotFoundandInvalidArnerror codes to a set of exceptions which should be mapped toNotFoundhandler error code.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.