Skip to content

Commit 8dce405

Browse files
committed
Provide clearer example
1 parent f61dc8f commit 8dce405

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

assertj/src/main/java/com/geekyhacker/assertj/UserRepository.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,17 @@
11
package com.geekyhacker.assertj;
22

3+
import java.util.Optional;
4+
35
public class UserRepository {
46

7+
public Optional<User> findBy(String username) {
8+
try {
9+
return Optional.of(findByUsername(username));
10+
} catch (UserNotFoundException userNotFoundException) {
11+
return Optional.empty();
12+
}
13+
}
14+
515
public User findByUsername(String username) throws UserNotFoundException {
616
if (!"test".equalsIgnoreCase(username)) {
717
throw new UserNotFoundException("Cannot find username: %s".formatted(username));

assertj/src/test/java/com/geekyhacker/assertj/UserRepositoryTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ void shouldReturnUserByUsername() throws UserNotFoundException {
2727
}
2828

2929
@Test
30-
void shouldNotThrowExceptionWhenGettingUserByUsername() {
31-
assertDoesNotThrow(() -> userRepository.findByUsername("test"));
32-
assertThatCode(() -> userRepository.findByUsername("test")).doesNotThrowAnyException();
30+
void shouldNotThrowExceptionWhenRetrievingUser() {
31+
assertDoesNotThrow(() -> userRepository.findBy("test"));
32+
assertThatCode(() -> userRepository.findBy("test")).doesNotThrowAnyException();
3333
}
3434

3535
@Test

0 commit comments

Comments
 (0)