Skip to content

Commit b9e9054

Browse files
committed
Fix tests
1 parent 7ecf10e commit b9e9054

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

src/test/java/com/yannbriancon/interceptor/NPlusOneQueriesExceptionTest.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package com.yannbriancon.interceptor;
22

33
import com.yannbriancon.exception.NPlusOneQueriesException;
4-
import com.yannbriancon.utils.entity.Avatar;
54
import com.yannbriancon.utils.entity.Message;
65
import com.yannbriancon.utils.entity.User;
76
import com.yannbriancon.utils.repository.AvatarRepository;
@@ -78,13 +77,13 @@ void nPlusOneQueriesDetection_throwsExceptionWhenMissingEagerFetchingOnManyToOne
7877
try {
7978
// Test a method that should return a N+1 query
8079
// The query triggers N+1 queries to eager fetch the user field
81-
List<Avatar> avatars = avatarRepository.findAll();
80+
avatarRepository.findAll();
8281
assert false;
8382
} catch (NPlusOneQueriesException exception) {
8483
assertThat(exception.getMessage())
85-
.contains("N+1 queries detected with eager fetching on the query\n" +
84+
.contains("N+1 queries detected with eager fetching on the entity com.yannbriancon.utils.entity.User\n" +
8685
" at com.yannbriancon.interceptor.NPlusOneQueriesExceptionTest" +
87-
".hibernateQueryInterceptor_isDetectingNPlusOneQueriesWhenMissingEagerFetchingOnManyToOne");
86+
".nPlusOneQueriesDetection_throwsExceptionWhenMissingEagerFetchingOnManyToOne");
8887
}
8988
}
9089

src/test/java/com/yannbriancon/interceptor/NPlusOneQueriesLoggingTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ void hibernateQueryInterceptor_isDetectingNPlusOneQueriesWhenMissingLazyFetching
8080

8181
LoggingEvent loggingEvent = loggingEventCaptor.getAllValues().get(0);
8282
assertThat(loggingEvent.getMessage())
83-
.contains("N+1 queries detected with eager fetching on the query\n" +
83+
.contains("N+1 queries detected with eager fetching on the entity com.yannbriancon.utils.entity.Message\n" +
8484
" at com.yannbriancon.interceptor.NPlusOneQueriesLoggingTest." +
8585
"hibernateQueryInterceptor_isDetectingNPlusOneQueriesWhenMissingLazyFetchingOnEntityField");
8686
assertThat(Level.ERROR).isEqualTo(loggingEvent.getLevel());

0 commit comments

Comments
 (0)