diff options
author | 2015-08-27 12:57:25 +0000 | |
---|---|---|
committer | 2015-08-27 14:45:32 +0000 | |
commit | a37e86c720dcb4a18368b5594676faceb5050e2e (patch) | |
tree | 507de187e6d824892251a76a5aeb266ae39d112a | |
parent | eb4f2ad9bc2aa6efe788f8bf63f53c942a193a4b (diff) |
EvalExceptionWithStackTrace does no longer remove non-EvalException exceptions from its cause.
--
MOS_MIGRATED_REVID=101673097
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java | 14 | ||||
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/syntax/ExceptionTest.java | 50 |
2 files changed, 62 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java b/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java index d1baabc5fd..686ccab083 100644 --- a/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java +++ b/src/main/java/com/google/devtools/build/lib/syntax/EvalExceptionWithStackTrace.java @@ -23,14 +23,24 @@ import java.util.Deque; import java.util.LinkedList; /** - * EvalException with a stack trace + * EvalException with a stack trace. */ public class EvalExceptionWithStackTrace extends EvalException { private StackTraceElement mostRecentElement; public EvalExceptionWithStackTrace(Exception original, Location callLocation) { - super(callLocation, getNonEmptyMessage(original), original.getCause()); + super(callLocation, getNonEmptyMessage(original), getCause(original)); + } + + /** + * Returns the "real" cause of this exception. + * + * <p>If the original exception is an EvalException, its cause is returned. + * Otherwise, the original exception itself is seen as the cause for this exception. + */ + private static Throwable getCause(Exception ex) { + return (ex instanceof EvalException) ? ex.getCause() : ex; } /** diff --git a/src/test/java/com/google/devtools/build/lib/syntax/ExceptionTest.java b/src/test/java/com/google/devtools/build/lib/syntax/ExceptionTest.java new file mode 100644 index 0000000000..b737106c39 --- /dev/null +++ b/src/test/java/com/google/devtools/build/lib/syntax/ExceptionTest.java @@ -0,0 +1,50 @@ +// Copyright 2015 Google Inc. All rights reserved. +// +// 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 com.google.devtools.build.lib.syntax; + +import static com.google.common.truth.Truth.assertThat; + +import com.google.devtools.build.lib.events.Location; + +import org.junit.Test; +import org.junit.runner.RunWith; +import org.junit.runners.JUnit4; + +/** + * A test class for exceptions. + */ +@RunWith(JUnit4.class) +public class ExceptionTest { + + @Test + public void testEmptyMessage() throws Exception { + EvalExceptionWithStackTrace ex = + new EvalExceptionWithStackTrace(new NullPointerException(), Location.BUILTIN); + assertThat(ex).hasMessage("Null Pointer"); + } + + @Test + public void testExceptionCause() throws Exception { + IllegalArgumentException iae = new IllegalArgumentException("foo"); + EvalException ee = new EvalException(Location.BUILTIN, iae); + + runExceptionTest(iae, iae); + runExceptionTest(ee, iae); + } + + private void runExceptionTest(Exception toThrow, Exception expectedCause) { + EvalExceptionWithStackTrace ex = new EvalExceptionWithStackTrace(toThrow, Location.BUILTIN); + assertThat(ex.getCause()).isEqualTo(expectedCause); + } +} |