[jloux-brapi]

Description

Story: BI-3082

Supporting changes:

New Lists API endpoint /search/lists/{listDbId}/items added to allow list type entity search requests. In particular, this search request will be used by bi-api to support the requirement of sorting, filtering, and paginating germplasm data that belongs in lists. Previously issues were found with the cache-removal implementation of germplasm lists, and this body should work should address all of these issues. To support a generalized approach to this endpoint, the following features were implemented:

• Generalized interface ListItemSearchRequestHandler for all ListTypeSearchRequests to implement. Only Germplasm implements this right now, if BrAPI wants to support all list types we (or them) can implement the rest at that time
• Models to support ListItemWithFullItem<T> responses added. In particular, this new model object provides ListItem information, as well as the full BrAPI model object that is represented by . In the case of DeltaBreed, this is germplasm, but if other list types are implemented, this generalized approach will allow for different models to use this object.
• Validation of generalized search requests that come in for a particular list type. If requests come in with attributes/properties not found in the expected search request type, a 400 error is thrown with a descriptive message of what happened.
• Renamed EntityColumnNameAndType to SortAndFilterEntityColumnInfo after new support requirements caused more attributes to be added to this model. These new attributes, supplied on creation, tell the SearchQueryBuilder about special use cases that were previously being overwritten or ignored with existing features. These new attributes make this more explicit.
• BrAPIRepositry/Impl support to search on multiple entities which is required for this new endpoint's use case. This code has been generalized to support any such use case.
• SearchQueryBuilder support for multiple entity search requests
• Changes to BrAPIGermplasmService.buildGermplasmSearchQuery() to prevent fetching the synonyms collection in the initial query. This was seen as a problem for using on the list query in general because it would have required the usage of BrAPIRespository.findAllBySearchPaginatingWithFetches(), which utilizes a difficult to edit subquery. It was found through these changes in fact that this method can likely be removed in favor of simply not fetching the first collection, then iterating through all the collections you need to fetch to populate before conversion. It's the subject of this new ticket

Dependencies

• bi-api feature/BI-3082
• brapi feature/BI-3082

Testing

Testing for this bi-api PR is sufficient for the testing of this ticket.

Supplemental testing can be done on the new /search/lists/{listDbId}/items endpoint as well to test corner cases as well, but not required.

Checklist:

• I have performed a self-review of my own code
• I have tested my code and ensured it meets the acceptance criteria of the story
• I have commented my code, particularly in hard-to-understand areas
• I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth