Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

SpringTestBase attaches multiple log appenders for log message verification #978

Closed
mariusoe opened this issue Jan 20, 2021 · 0 comments · Fixed by #979
Closed

SpringTestBase attaches multiple log appenders for log message verification #978

mariusoe opened this issue Jan 20, 2021 · 0 comments · Fixed by #979
Assignees
@mariusoe
Labels
area/general bug Something isn't working

Comments

@mariusoe
Copy link
Member

In the SpringTestBase class of the agent's unit tests, a mock appender is attached to the logger to verify log messages. Unfortuently, the affected logic initializes multiple appender instances and attaches for each test invocation a new one, leading to a large amount of log appenders. This leads in the CI tests to OutOfMemory errors.
@mariusoe mariusoe added bug Something isn't working area/general labels Jan 20, 2021
@mariusoe mariusoe self-assigned this Jan 20, 2021
mariusoe added a commit to mariusoe/inspectit-oce that referenced this issue Jan 20, 2021
@mariusoe
Closes inspectIT#978 - SpringTestBase attaches multiple log appenders…
1338c14 
… for log message verification
@mariusoe mariusoe mentioned this issue Jan 20, 2021
Merged
mariusoe added a commit that referenced this issue Jan 21, 2021
@mariusoe
Closes #978 - SpringTestBase attaches multiple log appenders for log …
6fb0b5b 
…message verification (#979)
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@mariusoe mariusoe

Labels
area/general bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@mariusoe