SCRUM-5829 collate both SGD spcies into a single species facet

[cmpich]

No description provided.

[claude]

Code Review

Changes reviewed: AlleleSearchResultConverter.java — adds normalizeSpeciesName() to map "Saccharomyces cerevisiae S288C" → "Saccharomyces cerevisiae" in allele search result documents.

Verdict: Looks good. The logic is correct and the null guard is appropriate. The normalization is applied consistently at all three places where taxon.getName() was used for display (species field, nameKey, and gene symbol).

One minor note: the second call to normalizeSpeciesName(taxon.getName()) at line ~64 is redundant since taxon hasn't changed since the first call at line ~42 within the same loop iteration. The speciesName local from the first block could be hoisted to the outer scope and reused. Not a bug, just a small inefficiency.

This is consistent with the existing reverse-mapping pattern already in DiseaseController and GeneController (which expand the short name back to S288C for ES queries). No correctness or performance concerns.

[oblodgett]

speciesName needs to be allele.getTaxon().getSpecies().getName() and we need to revert the SGD special handling of the name.

[oblodgett]

We are going to need to do this story: https://agr-jira.atlassian.net/browse/SCRUM-2758 to get rid of this issue.