Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

We'd like to have a consistent and safe way for study designers (almost always the implementers, so people with the DEVELOPER role) to delete and remove things as they set up a study. Often out deletes actually our delete APIs remove things from the database, and so those calls are only accessible to administrators for test clean-up.

...

  • by default, should not include deleted apples
  • because deleted apples may be referenced elsewhere, UIs may still need them for labels, etc., so include a flag to return deleted apples, too

Tests

  • Can retrieve items (getAppleWorks()
  • With a logically deleted item, the includeDeleted flag shows/hides these logically deleted items (getApplesIncludes/ExcludesDeleted)
  • Physically deleted item does not appear regardless of the includeDeleted flag (this is a good integration test) (getApplesExcludesPhysicallyDeleted)

GET /apples/id

  • always return this item regardless of deletion status so references do not break
  • Note: if developers try and delete things retrieve objects, to verify they are not referencing older models in their apps, this aspect of our API could be misleading. Their apps will continue to work, but the returned items will be marked deleted=true. Just something to be aware of.

...

  • Can retrieve item regardless of whether or not it is marked deleted (getApplesIncludesLogicallyDeleted)

POST /apples

  • should not be able to create an apple in a deleted state (deleted always = false) (createAppleNeverDeleted)

POST /apples/id

  • should be able change the deleted flag if you can update the object

Tests

  • If item is deleted and update is deleted, then throw an EntityNotFoundException (updateLogicallyDeletedAppleThrows)
  • If item is deleted and update is not deleted, this is allowed and the resulting item is not deleted (updateCanUndeleteApple)
  • If item is not deleted and update is not deleted, this is allowed and the resulting item is not deleted (updateApple)
  • If item is not deleted and update is deleted, this is allowed and resulting item is deleted (no need to prevent this)
  • If item is deleted and update is deleted, then throw an EntityNotFoundException
  • (updateCanDeleteApple)

DELETE /apples/id or DELETE /apples?id=x

...

  • If item is already logically deleted, throw an EntityNotFoundException (deletingLogicallyDeletedAppleThrows)
  • If item is not marked deleted, mark it as deleted (logicallyDeletingAppleWorks)

DELETE /applies/id?physical=true or DELETE /apples?id=x&physical=true

...

  • If item is not in database, throw an EntityNotFoundException

...

  • (physicallyDeletingMissingAppleThrows)
  • If item is in database but logically deleted, this should physically delete the item (physicallyDeletingLogicallyDeletedAppleWorks)

Dependent Objects

Objects that cannot be deleted because they would leave other objects with referential integrity problems:

  • If the referencing object is logically deleted, the dependent object can be logically deleted;
  • If the referencing object is logically deleted, the dependent object cannot be physically deleted;
  • If the referencing object is physically deleted, then the there isn't a problem anymore. The dependent object can be logically or physically deleted.

Tests

NameWhat we should test
getAppleIncludesLogicallyDeletedThe get apple call returns logically deleted apples
getApplesIncludes/ExcludesLogicallyDeletedThe apple collection calls include or exclude logically deleted apples based on the flag
createAppleCannotBeDeletedCreating an apple with deleted=true just creates the apple with deleted

...

=false
updateLogicallyDeletedAppleThrowsCannot update if saved and updated apple are both deleted=true
updateCanUndeleteApplesaved apple deleted=true, updated apple deleted=false works
updateCanDeleteApplesaved apple deleted=false, updated apple deleted=true works (no reason to prevent it)
deleteLogicallyDeletedAppleThrowsCannot delete an apple that is already deleted
deleteAppleCan logically delete an apple
deleteAppleMissingAppleThrowsTrying to logically delete a logically deleted apple throws
deleteApplePermanentlyCan permanently delete an apple
deletePermanentlyLogicallyDeletedAppleCan physically delete an apple that is logically deleted (rather than getting a 404)
deletePermanentlyMissingAppleThrows
includeDeletedInRequestParamsRequests that take this parameter should return the value in the request params
physicalInRequestParamsRequests that take this parameter should return the value in the request params