How to test: - No functional change in the REST API PUT /artifacts/{id}. You can still add, change or remove both forward and reverse links. You can still set a parent through the "parent" key and set forward links through the "links" key. Notes: - ArtifactUpdateHandler is replaced by ArtifactReverseLinksUpdater. It has only one method, and it detects by itself what to do. If there is an artifact link value and there are submitted reverse links, it will build the diff (by building the ChangeReverseLinksCommand) and build a new changeset for each reverse link. It will then add the new changeset for the current artifact itself, and try to save all new changesets. - There is no interface in front of ArtifactReverseLinksUpdater because it does not deal with DB directly, it is a "middle man". This lets us write a "sociable" test for PUTHandler: we input Representation objects, we stub only the DB Accesses (for fields and artifacts) and assert that we created the expected changesets. Our tests use as many "real" objects as possible. part of request #34374 Be able to explicitly add, change or remove forward and reverse links Change-Id: I6aba1ce6aeda2e966a0b51f1f1a0f494a03ca0b4

+697 −823

Author

Joris MASSON (jmasson) 2023-10-02 14:41

Committer

Joris MASSON (jmasson) 2023-10-04 14:51

Hash 7cf212c4605ebdec1d7b798b9173bc9846dc3189

Parent fd8048e86528885492b06af34e18bd99cca7e034

Reference git #tuleap/stable/7cf212c4605ebdec1d7b798b9173bc9846dc3189