search

specify / specify7

Specify 7
https://www.specifysoftware.org/products/specify-7/
GNU General Public License v2.0
64 stars 36 forks source link

Expand TreeDefItem backend test coverage #5035

Open melton-jason opened 3 months ago

melton-jason commented 3 months ago

As of #4257, a lot of behavior was added relating to TreeDefitems. Tests should be added to both catch any existing bugs in the code as well as to ensure maintainability and prevent future changes to the code from causing unwanted/unexpected behavior.

There has already been multiple issue which would have been caught if tests were in place and had complete coverage of the changes in #4257:

melton-jason commented 3 months ago

Nevermind, I have found there are already tests for this (focused on using the API): https://github.com/specify/specify7/blob/1f028e7f9b2eb8b60808bfb2f4f146752addd034/specifyweb/specify/tests/test_trees.py#L214-L407.

Can we then ensure there is coverage for all of the changes from #4257?