Skip to content

[BI-2814] Create Sub Entity Dataset: Update OUs to supply levelNameDbIds#506

Open
jloux-brapi wants to merge 2 commits intodevelopfrom
feature/BI-2814
Open

[BI-2814] Create Sub Entity Dataset: Update OUs to supply levelNameDbIds#506
jloux-brapi wants to merge 2 commits intodevelopfrom
feature/BI-2814

Conversation

@jloux-brapi
Copy link
Copy Markdown
Collaborator

@jloux-brapi jloux-brapi commented Apr 1, 2026

Description

Story: BI-2814

  • Consolidated relevant code into DatasetService for use in creation experiment use case
  • Updated to BrAPI Java Client 2.2
  • Added methods to get data from the updated client in BrAPIObservationLevelDAO

Dependencies

BI-2826 MR

Testing

  • Create a program then do the following:

    • Create a new experiment with any level name
    • Create a sub-entity dataset with any level name (except top level name of this experiment, this will always result in an error as intended)
    • Verify no errors occur on creating the sub entity dataset, and that the data can be viewed for the sub entity dataset in the experiment, and the sub entity dataset data can also be exported without issue
  • In a program where there already exists an experiment with both a top level dataset and a sub entity dataset:

    • Create a sub-entity dataset that has the same name as an existing top level dataset in a different experiment in the program.
    • Verify no errors occur on creating the sub entity dataset, and that the data can be viewed for the sub entity dataset in the experiment, and the sub entity dataset data can also be exported without issue
  • In a program where there already exists an experiment with both a top level dataset and a sub entity dataset:

    • Create a sub-entity dataset that has the same name as an existing top level dataset in a different experiment in the program.
    • Verify no errors occur on creating the sub entity dataset, and that the data can be viewed for the sub entity dataset in the experiment, and the sub entity dataset data can also be exported without issue

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 create/modified unit and/or integration tests to cover this change or tests are not applicable
  • 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

…eation

- Consolidated relevant code into DatasetService for use in creation experiment use case
- Updated to BrAPI Java Client 2.2
- Added methods to get data from the updated client in BrAPIObservationLevelDAO
@jloux-brapi jloux-brapi requested review from a team and HMS17 and removed request for a team April 1, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant