Skip to content

[BI-2267]Trim trailing white space on breeding method file uploads (FAILED QA)#399

Merged
davedrp merged 1 commit intodevelopfrom
bug/BI-2267_2
Sep 9, 2024
Merged

[BI-2267]Trim trailing white space on breeding method file uploads (FAILED QA)#399
davedrp merged 1 commit intodevelopfrom
bug/BI-2267_2

Conversation

@davedrp
Copy link
Contributor

@davedrp davedrp commented Sep 5, 2024

Description

BI-2267 Trim trailing white space on breeding method file uploads (FAILED QA)

THE BUG caused by the new code for BI-2267

IF the he Breeding Method field in the germplasm import file has the value "Bulk or population " (with a trailing blank),
THEN the import fails with an erroneous "Invalid Breeding Method" error.

BACKGROUND

  • The Breeding Method field in the germplasm import file must match the name column in the breeding_method table exactly.
  • One entry in the breeding_method table has a trailing blank in the name column; "Bulk or population ".
  • The new code for BI-2267 trims the white space off of the value for the Breeding Method field in the germplasm import file.
  • this causes an erroneous "Invalid Breeding Method" error (because "Bulk or population" <> "Bulk or population ").

Solution

Two fixes were made. Either one would fix the immediate problem, but both solutions are present for completeness and to prevent future problems.

  1. A migration was added to remove the trailing space in the breeding_method table.
  2. The lookup code for the name column in the breeding_method table, now includes DSL.trim().

Dependencies

bi-web: develop

Testing

Expected Result
It should import without errors.

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 tested that my code works with both the brapi-java-server and BreedBase
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <please include a link to TAF run>

@davedrp davedrp requested review from a team, dmeidlin and nickpalladino and removed request for a team September 5, 2024 14:17
@github-actions github-actions bot added the bug Something isn't working label Sep 5, 2024
@davedrp davedrp merged commit e9e4213 into develop Sep 9, 2024
@davedrp davedrp deleted the bug/BI-2267_2 branch September 9, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants