Skip to content

Added some tests for making up the test coverage #6754

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Merged
sima-zhu merged 5 commits into from
Dec 11, 2019
Merged

Added some tests for making up the test coverage #6754

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
db08f58
Added some tests for make up the logging coverage
sima-zhu Dec 11, 2019
521fa5e
Removed the extra one
sima-zhu Dec 11, 2019
55c3a10
Added tests for fluxUtils
sima-zhu Dec 11, 2019
98480be
Added tests for FluxUtils
sima-zhu Dec 11, 2019
ab674fd
Fixed compilation and linting
sima-zhu Dec 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
52 changes: 52 additions & 0 deletions sdk/core/azure-core/src/test/java/com/azure/core/util/logging/ClientLoggerTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,58 @@ public void logExceptionStackTrace() {
assertTrue(logValues.contains(runtimeException.getStackTrace()[0].toString()));
}

/**
* Tests that logging an exception when the log level is ERROR the stack trace is logged.
*/
@Test
public void logExceptionStackTraceWithErrorLevel() {
String logMessage = "This is an exception";
String exceptionMessage = "An exception message";
RuntimeException runtimeException = createRuntimeException(exceptionMessage);

String originalLogLevel = setupLogLevel(1);
logMessage(new ClientLogger(ClientLoggerTests.class), 4, logMessage, runtimeException);
setPropertyToOriginalOrClear(Configuration.PROPERTY_AZURE_LOG_LEVEL, originalLogLevel);

String logValues = new String(logCaptureStream.toByteArray(), StandardCharsets.UTF_8);
assertTrue(logValues.contains(logMessage + System.lineSeparator() + runtimeException.getMessage()));
assertTrue(logValues.contains(runtimeException.getStackTrace()[0].toString()));
}


/**
* Tests that logging an exception when the log level is ERROR the stack trace is logged.
*/
@Test
public void logExceptionStackTraceWithNoLogLevel() {
String logMessage = "This is an exception";
String exceptionMessage = "An exception message";
RuntimeException runtimeException = createRuntimeException(exceptionMessage);

String originalLogLevel = setupLogLevel(1);
logMessage(new ClientLogger(ClientLoggerTests.class), 5, logMessage, runtimeException);
setPropertyToOriginalOrClear(Configuration.PROPERTY_AZURE_LOG_LEVEL, originalLogLevel);

String logValues = new String(logCaptureStream.toByteArray(), StandardCharsets.UTF_8);
assertTrue(logValues.isEmpty());
}

/**
* Tests that logging an exception when the log level is ERROR the stack trace is logged.
*/
@Test
public void logExceptionWithInvalidLogLevel() {
String logMessage = "This is an exception";
Object runtimeException = new Object();

String originalLogLevel = setupLogLevel(1);
logMessage(new ClientLogger(ClientLoggerTests.class), 3, logMessage, runtimeException);
setPropertyToOriginalOrClear(Configuration.PROPERTY_AZURE_LOG_LEVEL, originalLogLevel);

String logValues = new String(logCaptureStream.toByteArray(), StandardCharsets.UTF_8);
assertTrue(logValues.contains(logMessage));
}

/**
* Tests that logging an exception as warning won't include the stack trace when the environment log level isn't
* VERBOSE. Additionally, this tests that the exception message isn't logged twice as logging an exception uses
Expand Down