Skip to content

Commit

Permalink
[#noissue] Fix testRecordNotChainedException test failure
Browse files Browse the repository at this point in the history
  • Loading branch information
intr3p1d committed Nov 20, 2024
1 parent 5598e76 commit ba7836f
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,9 @@ public TestExceptionStorage() {

@Override
public void store(List<ExceptionWrapper> wrappers) {
logger.error(wrappers);
logger.info(wrappers);
this.wrappers.addAll(wrappers);
logger.info(String.format("current wrappers: ", this.wrappers));
}

@Override
Expand Down Expand Up @@ -176,7 +177,6 @@ public void testRecordNotChainedException() {
notChainedException(throwableFunction);
} catch (Throwable e) {
expected1 = newExceptionWrappers(exceptions.get(exceptions.size() - 2), START_TIME, 1);
exceptionRecorder.recordException(e, START_TIME);
logger.warn(exceptionStorage.getWrappers());
actual1 = new ArrayList<>(exceptionStorage.getWrappers());
throwable = e;
Expand Down Expand Up @@ -215,7 +215,6 @@ public void testRecordRethrowGivenException() {
rethrowGivenException(throwableFunction);
} catch (Throwable e) {
expected = newExceptionWrappers(e, START_TIME, 1);
exceptionRecorder.recordException(e, START_TIME);
actual = exceptionStorage.getWrappers();
Assertions.assertTrue(actual.isEmpty());
}
Expand Down

0 comments on commit ba7836f

Please sign in to comment.