Skip to content

Commit

Permalink
[pinpoint-apm#11290] logback-it, add test for full-pattern replacement
Browse files Browse the repository at this point in the history
  • Loading branch information
yjqg6666 committed Nov 4, 2024
1 parent 6702c59 commit 0e8e4ca
Show file tree
Hide file tree
Showing 2 changed files with 46 additions and 1 deletion.
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
/*
* Copyright 2024 NAVER Corp.
*
* 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.navercorp.pinpoint.it.plugin.logback;

import com.navercorp.pinpoint.it.plugin.utils.AgentPath;
import com.navercorp.pinpoint.it.plugin.utils.PluginITConstants;
import com.navercorp.pinpoint.test.plugin.*;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;

@PluginForkedTest
@PinpointAgent(AgentPath.PATH)
@Dependency({"ch.qos.logback:logback-classic:[1.0.13],[1.1.0,1.1.11],[1.2.0,1.2.6]", "org.slf4j:slf4j-api:1.7.12", PluginITConstants.VERSION})
@ImportPlugin({"com.navercorp.pinpoint:pinpoint-logback-plugin"})
@PinpointConfig("pinpoint-spring-bean-test.config")
@JvmArgument({"-DtestLoggerEnable=false", "-Dprofiler.logback.logging.pattern.full_replace.with=%d{yyyy-MM-dd HH:mm:ss} [%p] [%t] %c [TxId:%X{PtxId} ReqId:%X{PreqId}] -LogbackIT- %m%n"})
@TransformInclude("org.slf4j.")
public class LogbackPatternFullReplaceIT extends LogbackTestBase {

@Test
public void test() {
checkMDC();
}

@Test
public void patternUpdate() {
String log = checkPatternUpdate();
Assertions.assertTrue(log.contains("-LogbackIT-"), "contains full-replace string LogbackIT");
}

}
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ protected void checkMDC() {

private Logger logger;

protected void checkPatternUpdate() {
protected String checkPatternUpdate() {

final String msg = "pattern";
StdoutRecorder stdoutRecorder = new StdoutRecorder();
Expand All @@ -58,6 +58,7 @@ public void run() {

Assertions.assertNotNull(logger, "logger null");
checkVersion(logger);
return log;
}

private void checkVersion(Logger logger) {
Expand Down

0 comments on commit 0e8e4ca

Please sign in to comment.