Skip to content

[UI Angular] Fix resource tab#2689

Open
anishraiepam wants to merge 6 commits intomasterfrom
feature_anishraiepam
Open

[UI Angular] Fix resource tab#2689
anishraiepam wants to merge 6 commits intomasterfrom
feature_anishraiepam

Conversation

@anishraiepam
Copy link
Copy Markdown
Collaborator

@anishraiepam anishraiepam commented Apr 3, 2025

##2674

Fix resource tab

@anishraiepam anishraiepam requested a review from ElenaSpb as a code owner April 3, 2025 08:41
Comment thread src/main/kotlin/com/epam/brn/controller/ResourceController.kt Outdated
Comment thread src/main/kotlin/com/epam/brn/controller/ResourceController.kt Outdated
…_anishraiepam

# Conflicts:
#	src/main/kotlin/com/epam/brn/controller/ResourceController.kt
@ElenaSpb ElenaSpb linked an issue Apr 3, 2025 that may be closed by this pull request
@ElenaSpb ElenaSpb changed the title Feature anishraiepam to master merge [UI Angular] Fix resource tab Apr 3, 2025
@ElenaSpb ElenaSpb requested a review from Mikleprog April 3, 2025 09:38
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 3, 2025

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 3, 2025

Gradle Unit and Integration Test Results

497 tests  +1   495 ✔️ +1   38s ⏱️ ±0s
113 suites  - 1       2 💤 ±0 
113 files    - 1       0 ±0 

Results for commit 35bc07f. ± Comparison against base commit 3db59cc.

This pull request removes 7 and adds 8 tests. Note that renamed tests count towards both.
com.epam.brn.auth.filter.FirebaseTokenAuthenticationFilterTest ‑ should set authentication by UsernamePasswordAuthenticationToken when user exist in local DB and firebase and not create new user in local database()
com.epam.brn.auth.filter.FirebaseTokenAuthenticationFilterTest ‑ should set authentication by UsernamePasswordAuthenticationToken when user not exist in local DB and exist in firebase DB and create new user in local database()
com.epam.brn.controller.UserDetailControllerTest ‑ should delete doctor from patient$epam_brn()
com.epam.brn.controller.UserDetailControllerTest ‑ should get all headphones for user$epam_brn()
com.epam.brn.controller.UserDetailControllerTest ‑ should get doctor assigned to patient$epam_brn()
com.epam.brn.service.statistics.progress.status.impl.StudyHistoriesProgressStatusManagerTest ‑ getStatus should call only retrievers which support DAY period and return GOOD status()
com.epam.brn.service.statistics.progress.status.impl.StudyHistoriesProgressStatusManagerTest ‑ getStatus should call only retrievers which support WEEK period and return GOOD status()
com.epam.brn.auth.filter.FirebaseTokenAuthenticationFilterTest ‑ should set authentication when user exist in local DB()
com.epam.brn.auth.filter.FirebaseTokenAuthenticationFilterTest ‑ should set authentication when user not exist in local DB()
com.epam.brn.controller.UserDetailControllerTest ‑ should delete doctor from patient$epam_brn_test()
com.epam.brn.controller.UserDetailControllerTest ‑ should get all headphones for user$epam_brn_test()
com.epam.brn.controller.UserDetailControllerTest ‑ should get doctor assigned to patient$epam_brn_test()
com.epam.brn.service.cloud.AwsCloudServiceTest ‑ should check file is not exist()
com.epam.brn.service.cloud.AwsCloudServiceTest ‑ should check is file exist()
com.epam.brn.service.cloud.AwsCloudServiceTest ‑ should return fileNames for main folder()

♻️ This comment has been updated with latest results.

@anishraiepam anishraiepam requested a review from lifeart as a code owner June 10, 2025 06:07
@sonarqubecloud
Copy link
Copy Markdown

@lifeart
Copy link
Copy Markdown
Collaborator

lifeart commented Jun 16, 2025

Hi @anishraiepam! Thank you for contribution! Do you mind updating branch from master and check lint and test errors:
https://github.com/Brain-up/brn/actions/runs/15551883961/job/43783919728?pr=2689
https://github.com/Brain-up/brn/actions/runs/15551883961/job/43783919687?pr=2689

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] Admin: fix Resources tab

3 participants