Iris: Fix server tests

[Hialus]

Checklist

General

• This is a small issue that I tested locally and was confirmed by another developer on a test server.
• I chose a title conforming to the naming conventions for pull requests.

Server

• Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
• I followed the coding and design guidelines.
• I added multiple integration tests (Spring) related to the features (with a high test coverage).
• I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
• I documented the Java code using JavaDoc style.

Motivation and Context

Currently on develop-iris the server tests are failing. Since it is a protected branch, the fixes are done here.

Description

Changes:

• Moved the admin settings endpoint according to new rules
• Removed unused methods in IrisMessageContentRepository
• Fixed the error handling in IrisConnectorService::getOfferedModels
• Added mocking of the /api/v1/models endpoint of Pyris
• Added unit tests for the IrisConnectorService
• Smaller fixes

Review Progress

Performance Review

• I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
• I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

• Code Review 1
• Code Review 2

Test Coverage

Class/File	Line Coverage	Confirmation (assert/expect)
IrisAuthorizationInterceptor.java	100%	✅
IrisConnectorService.java	91%	✅
IrisWebsocketService.java	88%	✅
AdminIrisSettingsResource.java	42%	✅
IrisSettingsResource.java	96%	✅

[github-actions]

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.

[DominikRemo]

Looked through the tests
Left a question and a suggestion

[laadvo]

I have two suggestions regarding the tests in IrisMessageIntegrationTest in general:

1. You could consider using less users (instead of 14, I think 2 should suffice for all tests in userUtilService.addUsers(TEST_PREFIX, 14,..))
2. There is a lot of duplicated code making the tests confuscated. You could consider extracting test setup into the @BeforeEach and other helper methods

[Hialus]

I have two suggestions regarding the tests in IrisMessageIntegrationTest in general:

1. You could consider using less users (instead of 14, I think 2 should suffice for all tests in `userUtilService.addUsers(TEST_PREFIX, 14,..)`)

2. There is a lot of duplicated code making the tests confuscated. You could consider extracting test setup into the `@BeforeEach` and other helper methods

1. True. This was older code and this is no longer needed. Thanks for noticing, I fixed it.
2. I tried to reduce it a bit, but while the code is similar, it is not always the same, so further reducing wouldn't really be worth it imo.