(openmeetings) branch master updated: [OPENMEETINGS-2783] sonar should be more happy

[solomax]

This is an automated email from the ASF dual-hosted git repository.

solomax pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openmeetings.git


The following commit(s) were added to refs/heads/master by this push:
     new 662c2df69 [OPENMEETINGS-2783] sonar should be more happy
662c2df69 is described below

commit 662c2df6978cb451b93284f188ebc2ffdd89e598
Author: Maxim Solodovnik <solo...@apache.org>
AuthorDate: Tue Jul 9 10:19:32 2024 +0700

    [OPENMEETINGS-2783] sonar should be more happy
---
 .../mediaserver/TestRecordingFlowMocked.java             | 16 ++++++++--------
 .../java/org/apache/openmeetings/config/TestConfig.java  |  8 +++-----
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git 
a/openmeetings-mediaserver/src/test/java/org/apache/openmeetings/mediaserver/TestRecordingFlowMocked.java
 
b/openmeetings-mediaserver/src/test/java/org/apache/openmeetings/mediaserver/TestRecordingFlowMocked.java
index 15348eb78..44813898c 100644
--- 
a/openmeetings-mediaserver/src/test/java/org/apache/openmeetings/mediaserver/TestRecordingFlowMocked.java
+++ 
b/openmeetings-mediaserver/src/test/java/org/apache/openmeetings/mediaserver/TestRecordingFlowMocked.java
@@ -103,17 +103,17 @@ class TestRecordingFlowMocked extends BaseMockedTest {
        }
 
        private Client getClientWithRoom() {
-               Client c = getClient();
-               c.setRoom(new Room());
-               c.getRoom().setId(ROOM_ID);
-               return c;
+               Client client = getClient();
+               client.setRoom(new Room());
+               client.getRoom().setId(ROOM_ID);
+               return client;
        }
 
        private Client getClientFull() {
-               Client c = getClientWithRoom();
-               c.getRoom().setAllowRecording(true);
-               c.allow(Room.Right.MODERATOR);
-               return c;
+               Client client = getClientWithRoom();
+               client.getRoom().setAllowRecording(true);
+               client.allow(Room.Right.MODERATOR);
+               return client;
        }
 
        @Test
diff --git 
a/openmeetings-web/src/test/java/org/apache/openmeetings/config/TestConfig.java 
b/openmeetings-web/src/test/java/org/apache/openmeetings/config/TestConfig.java
index 033898db9..899d4f800 100644
--- 
a/openmeetings-web/src/test/java/org/apache/openmeetings/config/TestConfig.java
+++ 
b/openmeetings-web/src/test/java/org/apache/openmeetings/config/TestConfig.java
@@ -35,13 +35,11 @@ class TestConfig extends AbstractOmServerTest {
 
        @Test
        void getConfigKey() {
-               System.err.println("THIS");
+               Configuration smtpServer = cfgDao.get(CONFIG_SMTP_SERVER);
 
-               Configuration smtp_server = cfgDao.get(CONFIG_SMTP_SERVER);
+               log.error("smtpServer.getUser(): {}", smtpServer.getUser());
 
-               System.err.println("smtp_server " + smtp_server.getUser());
-
-               assertNull(smtp_server.getUser());
+               assertNull(smtpServer.getUser());
        }
 
        @Test