Skip to content

BI-2921 - Study Cache 4 Remove rest of CRUD operations from BrAPIStudyDAO - #555

Merged
jloux-brapi merged 2 commits into
epic/BI-2862from
feature/BI-2921
Sep 14, 2026
Merged

jloux-brapi merged 2 commits into
epic/BI-2862from
feature/BI-2921

Conversation

@humsika

@humsika humsika commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Description

Story: BI-2921

This PR removes the remaining ProgramCache usage from BrAPIStudyDAO.

No API contract or user-facing behavior is intentionally changed. Study create, update, and delete operations now rely directly on BrAPI instead of updating or refreshing Redis study-cache entries.

Dependencies

bi-web: epic/BI-2862
bi-api: feature/BI-2921
brapi-Java-ProdServer: epic/BI-2862

Testing

  1. Open Experiments & Observations for a test program.
  2. Import a new experiment containing one or more new environments.
  3. Confirm the import succeeds and the environments are displayed correctly in the experiment.
  4. Refresh the page or navigate away and return. Confirm the environments still display correctly without cache-related errors.
  5. Delete the test experiment and refresh the page.
  6. Confirm the deleted experiment and its associated data do not reappear.

This confirms that BrAPI studies can be created, retrieved, displayed, and handled during deletion after removing the study cache from BrAPIStudyDAO.

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 created/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

@humsika
humsika requested a review from jloux-brapi September 14, 2026 14:58
@jloux-brapi
jloux-brapi changed the base branch from develop to epic/BI-2862 September 14, 2026 16:47
@jloux-brapi
jloux-brapi merged commit 67b7b79 into epic/BI-2862 Sep 14, 2026
6 checks passed
@jloux-brapi
jloux-brapi deleted the feature/BI-2921 branch September 14, 2026 18:24
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.

2 participants