OpenAssetIO / OpenAssetIO-Manager-BAL

A contrived "asset management system" for OpenAssetIO integration test cases.
Apache License 2.0
9 stars 6 forks source link

Add relationship-write support #62

Closed elliotcmorris closed 9 months ago

elliotcmorris commented 1 year ago

https://github.com/OpenAssetIO/OpenAssetIO/issues/1016#issuecomment-1669903074

Currently BAL when registering dosen't do the check where it sees if the Traitsets are the same, and then branches to either do a new version if they are, or a new entity if they are not.

This is behaviour allows us to demonstrate the canonical behaviour defined by https://github.com/OpenAssetIO/OpenAssetIO/issues/1016.

To do this, we'll need some sort of relationship trait to create a relationship when a new entity is made. We've been planning to integrate mediacreation with bal anyway (See https://github.com/OpenAssetIO/OpenAssetIO-Manager-BAL/issues/49), so we can use the mediacreation Relationship trait for this.

ACs

When register is called against an entityRef that already exists:

foundrytom commented 1 year ago

This is behaviour is required to test https://github.com/OpenAssetIO/OpenAssetIO/issues/1016, and is wanted in general for BAL.

Can we park this issue for now. This behaviour is only required to test the concept, not the implementation in the core API (a test for that is that the enums exist).

foundrytom commented 1 year ago

Adding a little more context:

elliotcmorris commented 9 months ago

Probably part of #88

Closing however as this isn't neccesarily the way we want BAL, or any manager, to behave. It's certain a way it could behave, but not the canonical way, so we don't need to prioritise explicit support.