Code cleanup: in the Entity class. Remove code handling a 'annotations' key in the entity 'properties'
This seems like it may be quite a lot of changes on top of cleanup I'm already doing for Annotations and EntityBundles so I've created a separate issue for it.
There is a relic from when we used to include URIs as a part of the Entity's properties (e.g. file_entity.properties.annotations had a value of '/repo/v1/entity/syn1234/annotations', which was returned from the backend).
Since we also include user-specified Entity annotations(as a dict) in the constructor, we have snippets of code in the Entity class that handle cases where the annotations dict got erroneously was passed into the properties object.
None of this is necessary anymore since we don't return URIs from the backend anymore
some examples of code that needs to be removed/refactored: