Refactor constants and add private constructor in JsonConf.

Changed API_EMAIL and API_KEY to static final for proper constant usage. Added a private constructor in JsonConf to prevent instantiation, ensuring adherence to utility class design principles.
This commit is contained in:
2025-04-21 14:16:24 +02:00
parent c1675d0fba
commit e0f9043389
2 changed files with 11 additions and 5 deletions
@@ -12,6 +12,9 @@ import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
*/ */
class JsonConf { class JsonConf {
private JsonConf() {
}
static ObjectMapper initObjectMapper() { static ObjectMapper initObjectMapper() {
ObjectMapper mapper = new ObjectMapper(); ObjectMapper mapper = new ObjectMapper();
mapper.registerModule(new JavaTimeModule()); mapper.registerModule(new JavaTimeModule());
@@ -1,14 +1,17 @@
package codes.thischwa.cf; package codes.thischwa.cf;
import static org.junit.jupiter.api.Assertions.*;
import codes.thischwa.cf.model.RecordEntity; import codes.thischwa.cf.model.RecordEntity;
import codes.thischwa.cf.model.RecordType; import codes.thischwa.cf.model.RecordType;
import codes.thischwa.cf.model.ZoneEntity; import codes.thischwa.cf.model.ZoneEntity;
import java.time.LocalDate; import java.time.LocalDate;
import java.util.List; import java.util.List;
import lombok.extern.slf4j.Slf4j; import lombok.extern.slf4j.Slf4j;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertThrows;
import static org.junit.jupiter.api.Assertions.assertTrue;
import org.junit.jupiter.api.Test; import org.junit.jupiter.api.Test;
@Slf4j @Slf4j
@@ -18,8 +21,8 @@ public class CfClientTest {
private static final String SLD_STR = "devsld"; private static final String SLD_STR = "devsld";
private static final int TTL = 60; private static final int TTL = 60;
private final String API_EMAIL = System.getenv("API_EMAIL"); private static final String API_EMAIL = System.getenv("API_EMAIL");
private final String API_KEY = System.getenv("API_KEY"); private static final String API_KEY = System.getenv("API_KEY");
private final CfDnsClient client = new CfDnsClient(API_EMAIL, API_KEY); private final CfDnsClient client = new CfDnsClient(API_EMAIL, API_KEY);