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

#374 - Extend api to save and load cohort and dataselection #459

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

michael-82
Copy link
Collaborator

@michael-82 michael-82 commented Feb 27, 2025

Remove Templates and Saved queries.
Implement Dataqueries as discussed in the corresponding issue ( #374 )

@michael-82 michael-82 added the enhancement New feature or request label Feb 27, 2025
@michael-82 michael-82 added this to the v7.0.0-alpha milestone Feb 27, 2025
@michael-82 michael-82 self-assigned this Feb 27, 2025
@michael-82 michael-82 linked an issue Feb 27, 2025 that may be closed by this pull request
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

Attention: Patch coverage is 94.73684% with 12 lines in your changes missing coverage. Please review.

Project coverage is 75.75%. Comparing base (0dac4a2) to head (acb10cc).

Files with missing lines Patch % Lines
...sibility_gui_backend/config/WebSecurityConfig.java 0.00% 5 Missing ⚠️
...ility_gui_backend/query/persistence/Dataquery.java 0.00% 4 Missing ⚠️
...sibility_gui_backend/query/api/DataExtraction.java 0.00% 1 Missing ⚠️
..._gui_backend/query/dataquery/DataqueryHandler.java 98.61% 0 Missing and 1 partial ⚠️
...ckend/query/v5/DataqueryHandlerRestController.java 99.19% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #459      +/-   ##
=============================================
+ Coverage      75.61%   75.75%   +0.13%     
+ Complexity       580      549      -31     
=============================================
  Files            154      150       -4     
  Lines           2633     2487     -146     
  Branches         229      214      -15     
=============================================
- Hits            1991     1884     -107     
+ Misses           576      540      -36     
+ Partials          66       63       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DisplayName("getDataquerySlotsJson() -> succeeds")
void getDataquerySlotsJson_succeeds() throws JsonProcessingException {
var dataqueryHandler = createDataqueryHandler();
var usedSlots = new Random().nextLong();

Check warning

Code scanning / CodeQL

Random used only once Warning test

Random object created and used only once.
@michael-82 michael-82 force-pushed the feature/374-extend-api-to-save-and-load-cohort-and-dataselection branch from efd88c5 to b4b7961 Compare February 28, 2025 08:53
- move api to v5
- add Dataquery
- remove query template and saved query
- add database migration script
- remove unused methods, exceptions and tests
- update ontology tag in github integration test
- modify swagger file
- convert saved queries and templates to dataqueries in db migration
@michael-82 michael-82 force-pushed the feature/374-extend-api-to-save-and-load-cohort-and-dataselection branch from f3d8e98 to 2428a6f Compare February 28, 2025 10:08
- remove some unused imports
- fix some minor potential errors
@michael-82 michael-82 marked this pull request as ready for review February 28, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend API to save and load cohort and dataselection
1 participant