Spring/spring-boot
Spring/spring-boot
1
0
Fork 0
mirror of https://github.com/spring-projects/spring-boot.git synced 2024-09-03 04:26:12 +08:00
Code Issues Packages Projects Releases Wiki Activity

Unwrap InvocationTargetException in isLogConfigurationMessage

Browse Source 
See gh-12958
This commit is contained in:
Henri Tremblay 2018-05-11 11:23:09 +01:00 committed by Andy Wilkinson
parent 34af02380c
commit 2953ed1dca
2 changed files with 85 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
spring-boot/src/main/java/org/springframework/boot/SpringBootExceptionHandler.java
View File

@ -86,6 +86,10 @@ class SpringBootExceptionHandler implements UncaughtExceptionHandler {
* @return {@code true} if the exception contains a log configuration message
*/
private boolean isLogConfigurationMessage(Throwable ex) {
if (ex instanceof InvocationTargetException) {
return isLogConfigurationMessage(ex.getCause());
}
String message = ex.getMessage();
if (message != null) {
for (String candidate : LOG_CONFIGURATION_MESSAGES) {

81
spring-boot/src/test/java/org/springframework/boot/SpringBootExceptionHandlerTest.java Normal file
View File

@ -0,0 +1,81 @@
/*
* Copyright 2012-2018 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.boot;
import java.lang.reflect.InvocationTargetException;
import org.junit.Rule;
import org.junit.Test;
import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.junit.MockitoJUnit;
import org.mockito.junit.MockitoRule;
import static org.mockito.ArgumentMatchers.same;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.verifyZeroInteractions;
/**
* Tests for {@link SpringBootExceptionHandler}.
*
* @author Henri Tremblay
*/
public class SpringBootExceptionHandlerTest {
@Rule
public MockitoRule rule = MockitoJUnit.rule();
@Mock
private Thread.UncaughtExceptionHandler parent;
@InjectMocks
private SpringBootExceptionHandler handler;
@Test
public void uncaughtException_shouldNotForwardLoggedErrorToParent() {
Thread thread = Thread.currentThread();
Exception ex = new Exception();
this.handler.registerLoggedException(ex);
this.handler.uncaughtException(thread, ex);
verifyZeroInteractions(this.parent);
}
@Test
public void uncaughtException_shouldForwardLogConfigurationErrorToParent() {
Thread thread = Thread.currentThread();
Exception ex = new Exception("[stuff] Logback configuration error detected [stuff]");
this.handler.registerLoggedException(ex);
this.handler.uncaughtException(thread, ex);
verify(this.parent).uncaughtException(same(thread), same(ex));
}
@Test
public void uncaughtException_shouldForwardLogConfigurationErrorToParentEvenWhenWrapped() {
Thread thread = Thread.currentThread();
Exception ex = new InvocationTargetException(new Exception("[stuff] Logback configuration error detected [stuff]", new Exception()));
this.handler.registerLoggedException(ex);
this.handler.uncaughtException(thread, ex);
verify(this.parent).uncaughtException(same(thread), same(ex));
}
}