Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BI-2355] - Non-informative error message: regression #422

Closed
wants to merge 12 commits into from

Conversation

HMS17
Copy link
Contributor

@HMS17 HMS17 commented Oct 31, 2024

Closed due to messy commit list

Description

Story: BI-2355 - Non-informative error message: regression

This error was due to tablesaw processing brackets weirdly, resulting in the column name stored in data not matching the column name stored in dynamic column names. This was fixed in two parts:

  1. Removing brackets from ontology term names on upload

  2. Preventing ontology terms with brackets in the name from being created by extending the functionality created in BI-2328

Note that this means for the error message re ontology terms not found, periods and brackets will be removed from the invalid ontology term names

Dependencies

n/a

Testing

Test that on upload of an experiment file with invalid ontology term columns headings that contain brackets and/or periods that the unknown error message no longer shows and instead the invalid ontology term message shows

Test that upload of experiment file with valid ontology terms proceeds as expected

Test that the appropriate error is thrown when an ontology term contains ".", "[" and/or "]" and that that error is not thrown when the ontology term does not contain those characters for:

  • Ontology Term Creation in UI
  • Ontology Term Edit
  • Ontology Term File Upload

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>

@github-actions github-actions bot added the bug Something isn't working label Oct 31, 2024
@HMS17 HMS17 closed this Nov 1, 2024
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.

2 participants