build: unit test for output and exception packages

Author: jknack

jooby/src/main/java/io/jooby/exception/MissingValueException.java

@@ -36,14 +36,6 @@ public String getName() {
     return name;
   }
 
-  /**
-   * Check if the given value is null and throw a {@link MissingValueException} exception.
-   *
-   * @param name Attribute's name.
-   * @param value Value to check.
-   * @param <T> Value type.
-   * @return Input value
-   */
   public static <T> T requireNonNull(String name, @Nullable T value) {
     if (value == null) {
       throw new MissingValueException(name);

jooby/src/main/java/io/jooby/internal/output/CompositeOutput.java

@@ -50,6 +50,7 @@ public BufferedOutput write(byte[] source, int offset, int length) {
   public BufferedOutput clear() {
     chunks.forEach(ByteBuffer::clear);
     chunks.clear();
+    size = 0;
     return this;
   }
 

jooby/src/main/java/io/jooby/output/OutputOptions.java

@@ -18,7 +18,10 @@ public class OutputOptions {
 
   /** Creates a default options. */
   public OutputOptions() {
-    long maxMemory = Runtime.getRuntime().maxMemory();
+    this(Runtime.getRuntime().maxMemory());
+  }
+
+  protected OutputOptions(long maxMemory) {
     // smaller than 64mb of ram we use 512b buffers
     if (maxMemory < 64 * 1024 * 1024) {
       // use 512b buffers

jooby/src/test/java/io/jooby/SessionStoreTest.java

@@ -5,11 +5,7 @@
  */
 package io.jooby;
 
-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.assertNull;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
@@ -241,4 +237,18 @@ void testSignedFactories() {
       // the lambdas instantiated by the `SessionStore.signed()` factory.
     }
   }
+
+  @Test
+  @DisplayName("Verify unsupported session store")
+  void testUnsupportedStore() {
+    assertThrows(Usage.class, () -> SessionStore.UNSUPPORTED.newSession(ctx));
+    assertThrows(Usage.class, () -> SessionStore.UNSUPPORTED.findSession(ctx));
+    assertThrows(
+        Usage.class, () -> SessionStore.UNSUPPORTED.deleteSession(ctx, mock(Session.class)));
+    assertThrows(
+        Usage.class, () -> SessionStore.UNSUPPORTED.touchSession(ctx, mock(Session.class)));
+    assertThrows(Usage.class, () -> SessionStore.UNSUPPORTED.saveSession(ctx, mock(Session.class)));
+    assertThrows(
+        Usage.class, () -> SessionStore.UNSUPPORTED.renewSessionId(ctx, mock(Session.class)));
+  }
 }

jooby/src/test/java/io/jooby/exception/MissingValueExceptionTest.java

@@ -0,0 +1,48 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.exception;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+class MissingValueExceptionTest {
+
+  @Test
+  @DisplayName("Verify constructor and getName accurately store and return the parameter name")
+  void testConstructorAndGetter() {
+    String paramName = "userId";
+    MissingValueException exception = new MissingValueException(paramName);
+
+    assertEquals(paramName, exception.getName());
+    assertEquals("Missing value: 'userId'", exception.getMessage());
+  }
+
+  @Test
+  @DisplayName("requireNonNull should return the value if it is not null")
+  void testRequireNonNullSuccess() {
+    String name = "username";
+    String value = "edgar";
+
+    String result = MissingValueException.requireNonNull(name, value);
+
+    assertEquals(value, result);
+  }
+
+  @Test
+  @DisplayName("requireNonNull should throw MissingValueException if the value is null")
+  void testRequireNonNullFailure() {
+    String name = "apiKey";
+
+    MissingValueException ex =
+        assertThrows(
+            MissingValueException.class, () -> MissingValueException.requireNonNull(name, null));
+
+    assertEquals(name, ex.getName());
+    assertEquals("Missing value: 'apiKey'", ex.getMessage());
+  }
+}

jooby/src/test/java/io/jooby/exception/RegistryExceptionTest.java

@@ -0,0 +1,40 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.exception;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+import io.jooby.StatusCode;
+
+class RegistryExceptionTest {
+
+  @Test
+  @DisplayName("Verify constructor with message sets status to 500 and stores message")
+  void testConstructorWithMessage() {
+    String message = "Service not found: MyService";
+    RegistryException exception = new RegistryException(message);
+
+    assertEquals(StatusCode.SERVER_ERROR, exception.getStatusCode());
+    assertEquals(message, exception.getMessage());
+    assertNull(exception.getCause());
+  }
+
+  @Test
+  @DisplayName("Verify constructor with message and cause sets status to 500 and stores both")
+  void testConstructorWithMessageAndCause() {
+    String message = "Dependency injection failed";
+    Throwable cause = new RuntimeException("Root cause error");
+    RegistryException exception = new RegistryException(message, cause);
+
+    assertEquals(StatusCode.SERVER_ERROR, exception.getStatusCode());
+    assertEquals(message, exception.getMessage());
+    assertEquals(cause, exception.getCause());
+  }
+}

jooby/src/test/java/io/jooby/exception/TypeMismatchExceptionTest.java

@@ -0,0 +1,62 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package io.jooby.exception;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+
+import java.lang.reflect.Type;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+import io.jooby.StatusCode;
+import io.jooby.problem.HttpProblem;
+
+class TypeMismatchExceptionTest {
+
+  @Test
+  @DisplayName("Verify constructor with cause stores name, message, and cause correctly")
+  void testConstructorWithCause() {
+    String name = "age";
+    Type type = int.class;
+    Throwable cause = new NumberFormatException("For input string: \"abc\"");
+
+    TypeMismatchException exception = new TypeMismatchException(name, type, cause);
+
+    assertEquals(name, exception.getName());
+    assertEquals("Cannot convert value: 'age', to: 'int'", exception.getMessage());
+    assertEquals(cause, exception.getCause());
+  }
+
+  @Test
+  @DisplayName("Verify constructor without cause sets cause to null")
+  void testConstructorWithoutCause() {
+    String name = "active";
+    Type type = boolean.class;
+
+    TypeMismatchException exception = new TypeMismatchException(name, type);
+
+    assertEquals(name, exception.getName());
+    assertEquals("Cannot convert value: 'active', to: 'boolean'", exception.getMessage());
+    assertNull(exception.getCause());
+  }
+
+  @Test
+  @DisplayName("Verify toHttpProblem returns a correctly populated HttpProblem instance")
+  void testToHttpProblem() {
+    String name = "price";
+    Type type = double.class;
+    TypeMismatchException exception = new TypeMismatchException(name, type);
+
+    HttpProblem problem = exception.toHttpProblem();
+
+    // BadRequestException usually defaults to 400
+    assertEquals(StatusCode.BAD_REQUEST.value(), problem.getStatus());
+    assertEquals("Type Mismatch", problem.getTitle());
+    assertEquals("Cannot convert value: 'price', to: 'double'", problem.getDetail());
+  }
+}