diff --git a/src/main/java/com/kousenit/dataorientedprogramming/UserService.java b/src/main/java/com/kousenit/dataorientedprogramming/UserService.java index 91fbf3b..f883bb3 100644 --- a/src/main/java/com/kousenit/dataorientedprogramming/UserService.java +++ b/src/main/java/com/kousenit/dataorientedprogramming/UserService.java @@ -4,10 +4,10 @@ public class UserService { public static String getPersonInfo(Person person) { + // Exhaustive switch expression because Person is sealed return switch (person) { case User user -> "User: %s (%s)".formatted(user.name(), user.email()); - case Admin admin -> - "Admin: %s (%s) with permissions: %s" + case Admin admin -> "Admin: %s (%s) with permissions: %s" .formatted(admin.name(), admin.email(), admin.permissions()); }; } diff --git a/src/test/java/com/kousenit/dataorientedprogramming/UserServiceTest.java b/src/test/java/com/kousenit/dataorientedprogramming/UserServiceTest.java index 8060cc1..08fae04 100644 --- a/src/test/java/com/kousenit/dataorientedprogramming/UserServiceTest.java +++ b/src/test/java/com/kousenit/dataorientedprogramming/UserServiceTest.java @@ -11,8 +11,7 @@ class UserServiceTest { @Test void testGetPersonInfo() { Person user = new User(1, "John Doe", "john@example.com"); - Person admin = new Admin(2, "Jane Smith", - "jane@example.com", "ALL"); + Person admin = new Admin(2, "Jane Smith", "jane@example.com", "ALL"); assertAll( () -> assertThat(UserService.getPersonInfo(user))