`General`: Add server tests for interview endpoints #1586

Abi107717 · 2025-12-21T13:53:03Z

Checklist

General

I tested all changes and their related features with all corresponding user types.
Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
I chose a title conforming to the naming conventions for pull requests.

Description

This PR adds comprehensive server tests for the interview management endpoints.

Review Progress

Code Review

Code Review 1

Test Coverage

Warning: Server tests failed. Coverage could not be fully measured. Please check the workflow logs.

Last updated: 2026-01-18 21:00:57 UTC

github-actions · 2025-12-21T13:56:02Z

📊 Test Coverage Report Client
Please review the client coverage report using one of the following methods, and consider adding tests to improve coverage in your modified files:
🔍 View coverage locally

Please run following command in your terminal:
npm run test:ci

and then run:
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub

Download the HTML coverage report from the GitHub Actions artifacts found at the bottom of the workflow run (under the section called Upload Client Test Coverage Report).
Open the downloaded index.html file in your browser to inspect the coverage details.

github-actions · 2025-12-21T14:06:30Z

📊 Test Coverage Report Client
Please review the client coverage report using one of the following methods, and consider adding tests to improve coverage in your modified files:
🔍 View coverage locally

Please run following command in your terminal:
npm run test:ci

and then run:
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub

Download the HTML coverage report from the GitHub Actions artifacts found at the bottom of the workflow run (under the section called Upload Client Test Coverage Report).
Open the downloaded index.html file in your browser to inspect the coverage details.

Bofan-Zhu

Thanks for adding these tests 😊. I just left 2 comments

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java

…ints

…gement-endpoints' into 1585-server-tests-interview-management-endpoints

github-actions · 2025-12-22T13:55:23Z

🤖 No OpenAPI or client changes needed.

…ints

github-actions · 2025-12-22T14:34:32Z

🤖 No OpenAPI or client changes needed.

Bofan-Zhu

Thanks for adding the tests. I just left some questions on the queries

src/main/java/de/tum/cit/aet/application/repository/ApplicationRepository.java

src/main/java/de/tum/cit/aet/interview/repository/InterviewProcessRepository.java

github-actions · 2025-12-28T15:30:30Z

🤖 No OpenAPI or client changes needed.

sachmii

Looks good, thank you.

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java

…ints

github-actions · 2025-12-29T18:37:55Z

🤖 No OpenAPI or client changes needed.

…gement-endpoints' into 1585-server-tests-interview-management-endpoints

github-actions · 2025-12-29T18:45:04Z

🤖 No OpenAPI or client changes needed.

github-actions · 2025-12-29T18:48:33Z

🤖 No OpenAPI or client changes needed.

github-actions · 2026-01-07T12:05:43Z

There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions.

…ints

github-actions · 2026-01-11T20:48:49Z

📊 Server Test Coverage Too Low

🔍 View coverage locally:

./gradlew test jacocoTestReport
open build/reports/jacoco/test/html/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-server" artifact from this workflow run.

github-actions · 2026-01-11T20:49:57Z

🤖 No OpenAPI or client changes needed.

github-actions · 2026-01-11T20:50:24Z

📊 Client Test Coverage Too Low

🔍 View coverage locally:

npm run test:ci
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-client" artifact from this workflow run.

…ints

github-actions · 2026-01-15T17:57:46Z

📊 Server Test Coverage Too Low

🔍 View coverage locally:

./gradlew test jacocoTestReport
open build/reports/jacoco/test/html/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-server" artifact from this workflow run.

github-actions · 2026-01-15T17:57:47Z

🤖 No OpenAPI or client changes needed.

github-actions · 2026-01-15T17:58:38Z

📊 Client Test Coverage Too Low

🔍 View coverage locally:

npm run test:ci
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-client" artifact from this workflow run.

…ints

github-actions · 2026-01-18T13:28:32Z

📊 Server Test Coverage Too Low

🔍 View coverage locally:

./gradlew test jacocoTestReport
open build/reports/jacoco/test/html/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-server" artifact from this workflow run.

github-actions · 2026-01-18T13:29:28Z

🤖 No OpenAPI or client changes needed.

github-actions · 2026-01-18T13:30:18Z

📊 Client Test Coverage Too Low

🔍 View coverage locally:

npm run test:ci
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-client" artifact from this workflow run.

…ints

github-actions · 2026-01-18T21:00:58Z

@Abi107717 Test coverage could not be fully measured because some tests failed. Please check the workflow logs for details.

github-actions · 2026-01-18T21:01:10Z

🤖 No OpenAPI or client changes needed.

github-actions · 2026-01-18T21:01:48Z

📊 Client Test Coverage Too Low

🔍 View coverage locally:

npm run test:ci
open build/test-results/lcov-report/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-client" artifact from this workflow run.

github-actions · 2026-01-18T21:02:10Z

📊 Server Test Coverage Too Low

🔍 View coverage locally:

./gradlew test jacocoTestReport
open build/reports/jacoco/test/html/index.html

🌐 View coverage from GitHub:
Download the "coverage-report-server" artifact from this workflow run.

Add tests

17a012f

Abi107717 self-assigned this Dec 21, 2025

Abi107717 requested a review from az108 as a code owner December 21, 2025 13:53

Abi107717 linked an issue Dec 21, 2025 that may be closed by this pull request

Server Tests: Interview Management Endpoints #1585

Open

16 tasks

github-actions bot added client Pull requests that update TypeScript code. (Added Automatically!) tests labels Dec 21, 2025

fix codacy

b13157e

Bofan-Zhu reviewed Dec 21, 2025

View reviewed changes

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java Show resolved Hide resolved

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java Outdated Show resolved Hide resolved

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

8472569

…ints

github-actions bot removed the client Pull requests that update TypeScript code. (Added Automatically!) label Dec 21, 2025

Abi107717 added 2 commits December 22, 2025 14:51

remove "unit Test" and add employee tests

fccf122

Merge remote-tracking branch 'origin/1585-server-tests-interview-mana…

b0c493f

…gement-endpoints' into 1585-server-tests-interview-management-endpoints

github-actions bot added server Pull requests that update Java code. (Added Automatically!) application Code changes in application module. labels Dec 22, 2025

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

8618ee9

…ints

Abi107717 requested a review from Bofan-Zhu December 22, 2025 14:31

Abi107717 added the ready for review label Dec 25, 2025

Bofan-Zhu requested changes Dec 27, 2025

View reviewed changes

src/main/java/de/tum/cit/aet/application/repository/ApplicationRepository.java Show resolved Hide resolved

src/main/java/de/tum/cit/aet/interview/repository/InterviewProcessRepository.java Outdated Show resolved Hide resolved

improve queries

8668578

Abi107717 requested a review from Bofan-Zhu December 28, 2025 15:34

sachmii requested changes Dec 29, 2025

View reviewed changes

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java Show resolved Hide resolved

src/test/java/de/tum/cit/aet/interview/web/rest/InterviewResourceTest.java Outdated Show resolved Hide resolved

Abi107717 and others added 2 commits December 29, 2025 19:33

group tests and add more checks

4070494

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

761b503

…ints

fix codacy and client style

761eb2e

Merge remote-tracking branch 'origin/1585-server-tests-interview-mana…

69169dc

…gement-endpoints' into 1585-server-tests-interview-management-endpoints

fix client style

d31ba80

Abi107717 requested a review from sachmii December 29, 2025 18:51

sachmii approved these changes Dec 30, 2025

View reviewed changes

github-actions bot added stale and removed ready for review labels Jan 7, 2026

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

b5d80d3

…ints

github-actions bot removed the stale label Jan 12, 2026

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

737ea5c

…ints

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

7298446

…ints

Merge branch 'main' into 1585-server-tests-interview-management-endpo…

e73b126

…ints

General: Add server tests for interview endpoints #1586

Are you sure you want to change the base?

General: Add server tests for interview endpoints #1586

Conversation

Abi107717 commented Dec 21, 2025 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Checklist

General

Description

Review Progress

Code Review

Test Coverage

Uh oh!

github-actions bot commented Dec 21, 2025

Uh oh!

github-actions bot commented Dec 21, 2025

Uh oh!

Bofan-Zhu left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Dec 22, 2025

Uh oh!

github-actions bot commented Dec 22, 2025

Uh oh!

Bofan-Zhu left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Dec 28, 2025

Uh oh!

sachmii left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Dec 29, 2025

Uh oh!

github-actions bot commented Dec 29, 2025

Uh oh!

github-actions bot commented Dec 29, 2025

Uh oh!

github-actions bot commented Jan 7, 2026

Uh oh!

github-actions bot commented Jan 11, 2026

Uh oh!

github-actions bot commented Jan 11, 2026

Uh oh!

github-actions bot commented Jan 11, 2026

Uh oh!

github-actions bot commented Jan 15, 2026

Uh oh!

github-actions bot commented Jan 15, 2026

Uh oh!

github-actions bot commented Jan 15, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

github-actions bot commented Jan 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

`General`: Add server tests for interview endpoints #1586

`General`: Add server tests for interview endpoints #1586

Abi107717 commented Dec 21, 2025 •

edited by github-actions bot

Loading