You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@turbine.apache.org by gk...@apache.org on 2021/01/21 14:48:20 UTC
svn commit: r1885759 - in /turbine/core/trunk: conf/test/log4j2-test.xml
conf/test/log4j2.xml pom.xml
src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java
src/test/org/apache/turbine/util/uri/TurbineURITest.java
Author: gk
Date: Thu Jan 21 14:48:19 2021
New Revision: 1885759
URL: http://svn.apache.org/viewvc?rev=1885759&view=rev
Log:
- fix log encoding
- fix null test
- update to released version for fulcrum testcontainer/security snapshots
Modified:
turbine/core/trunk/conf/test/log4j2-test.xml
turbine/core/trunk/conf/test/log4j2.xml
turbine/core/trunk/pom.xml
turbine/core/trunk/src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java
turbine/core/trunk/src/test/org/apache/turbine/util/uri/TurbineURITest.java
Modified: turbine/core/trunk/conf/test/log4j2-test.xml
URL: http://svn.apache.org/viewvc/turbine/core/trunk/conf/test/log4j2-test.xml?rev=1885759&r1=1885758&r2=1885759&view=diff
==============================================================================
--- turbine/core/trunk/conf/test/log4j2-test.xml (original)
+++ turbine/core/trunk/conf/test/log4j2-test.xml Thu Jan 21 14:48:19 2021
@@ -23,7 +23,7 @@
<PatternLayout pattern="%d [%t] %-5p %c - %m%n"/>
</Console>
<File name="logfile" fileName="target/turbine-test.log">
- <PatternLayout pattern="%d [%t] %-5p %c - %m%n"/>
+ <PatternLayout pattern="%d [%t] %-5p %c - %m%n" charset="UTF-8"/>
</File>
</Appenders>
<Loggers>
Modified: turbine/core/trunk/conf/test/log4j2.xml
URL: http://svn.apache.org/viewvc/turbine/core/trunk/conf/test/log4j2.xml?rev=1885759&r1=1885758&r2=1885759&view=diff
==============================================================================
--- turbine/core/trunk/conf/test/log4j2.xml (original)
+++ turbine/core/trunk/conf/test/log4j2.xml Thu Jan 21 14:48:19 2021
@@ -24,7 +24,7 @@
<PatternLayout pattern="%d [%t] %-5p %c - %m%n"/>
</Console>
<File name="logfile" fileName="target/turbine-test.log">
- <PatternLayout pattern="%d [%t] %-5p %c - %m%n"/>
+ <PatternLayout pattern="%d [%t] %-5p %c - %m%n" charset="UTF-8"/>
</File>
</Appenders>
<Loggers>
Modified: turbine/core/trunk/pom.xml
URL: http://svn.apache.org/viewvc/turbine/core/trunk/pom.xml?rev=1885759&r1=1885758&r2=1885759&view=diff
==============================================================================
--- turbine/core/trunk/pom.xml (original)
+++ turbine/core/trunk/pom.xml Thu Jan 21 14:48:19 2021
@@ -1081,7 +1081,7 @@
<dependency>
<groupId>org.apache.fulcrum</groupId>
<artifactId>fulcrum-testcontainer</artifactId>
- <version>1.0.9-SNAPSHOT</version>
+ <version>1.0.9</version>
<scope>test</scope>
</dependency>
<dependency>
@@ -1098,6 +1098,10 @@
<scope>test</scope>
<optional>true</optional>
</dependency>
+ <!-- do not add junit 4 support e.g. with org.junit.platform runner or launcher,
+ as it is not compatible with jupiter tags, will throw
+ [WARNING] Couldn't load group class 'docker' in Surefire|Failsafe plugin
+ -->
</dependencies>
<profiles>
@@ -1182,15 +1186,19 @@
</execution>
</executions>
</plugin>
+ <plugin>
+ <artifactId>maven-failsafe-plugin</artifactId>
+ <version>3.0.0-M5</version>
+ </plugin>
</plugins>
</build>
<dependencies>
- <!-- docker testcontainer deps start here, until torque 4.1 is releases start it like:
+ <!-- docker testcontainer requires at least torque v5.0, start it like
inside IDE (add dependencies):
- mvn eclipse:eclispe -Pdocker-testcontainer
+ mvn eclipse:eclipse -Pdocker-testcontainer
- outside IDE
- mvn test -Pdocker-testcontainer -Dtorque.version=4.1-SNAPSHOT
+ from console:
+ mvn test -Pdocker-testcontainer
-->
<dependency>
@@ -1237,10 +1245,11 @@
<turbine.site.path>turbine/development/turbine-5.1</turbine.site.path>
<fulcrum.intake>2.0.1-SNAPSHOT</fulcrum.intake>
<fulcrum.parser>2.0.1</fulcrum.parser>
- <fulcrum.security>2.0.0-SNAPSHOT</fulcrum.security>
+ <fulcrum.security>2.0.0</fulcrum.security>
<torque.version>5.0</torque.version>
+ <junit.platform>1.7.0</junit.platform>
<doclint>none</doclint>
- <docker.testcontainers.version>1.15.0-rc2</docker.testcontainers.version>
+ <docker.testcontainers.version>1.15.1</docker.testcontainers.version>
<jacoco.skip>true</jacoco.skip>
<argLine></argLine>
</properties>
Modified: turbine/core/trunk/src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java
URL: http://svn.apache.org/viewvc/turbine/core/trunk/src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java?rev=1885759&r1=1885758&r2=1885759&view=diff
==============================================================================
--- turbine/core/trunk/src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java (original)
+++ turbine/core/trunk/src/test/org/apache/turbine/testcontainer/BuildContainerWithDockerfileTest.java Thu Jan 21 14:48:19 2021
@@ -182,7 +182,7 @@ class BuildContainerWithDockerfileTest {
if (!MY_SQL_CONTAINER.isRunning()) {
MY_SQL_CONTAINER.start();
}
-
+
String serviceHost = MY_SQL_CONTAINER.getContainerIpAddress();
Integer mappedPort = MY_SQL_CONTAINER.getMappedPort(SERVICE_PORT);// e.g. 32811
log.info("generate jdbc url from {}, mapped Port: {}, bounded port: {}", serviceHost, mappedPort, MY_SQL_CONTAINER.getBoundPortNumbers());
@@ -191,9 +191,11 @@ class BuildContainerWithDockerfileTest {
// String genJDBC = ((MySQLContainer)MY_SQL_CONTAINER).getJdbcUrl();
// log.info( "generated connect url: {}", genJDBC);
// }
- String targetJDBC = //genJDBC;
+ String targetJDBC =
String.format("jdbc:mysql://%s:%d/%s?loggerLevel=OFF", serviceHost,
mappedPort, DATABASE_NAME);
+ // changing the jdbc string prefix to jdbc:tc:mysql does handle the test database setup,
+ // https://www.testcontainers.org/modules/databases/jdbc/
log.info( "used connect url: {}", targetJDBC);
return targetJDBC;
}
Modified: turbine/core/trunk/src/test/org/apache/turbine/util/uri/TurbineURITest.java
URL: http://svn.apache.org/viewvc/turbine/core/trunk/src/test/org/apache/turbine/util/uri/TurbineURITest.java?rev=1885759&r1=1885758&r2=1885759&view=diff
==============================================================================
--- turbine/core/trunk/src/test/org/apache/turbine/util/uri/TurbineURITest.java (original)
+++ turbine/core/trunk/src/test/org/apache/turbine/util/uri/TurbineURITest.java Thu Jan 21 14:48:19 2021
@@ -164,12 +164,14 @@ public class TurbineURITest extends Base
parserService.putParser(pp);
pp = parserService.getParser(DefaultParameterParser.class);
- pp.add("test", (String) null);
+ pp.add("test", "null");
+// pp.add("test", (String) null); // isnotnull guarded
turi.add(1, pp); // 1 = query data
// Should make the following work so as to be consistent with directly
// added values.
- // assertEquals("/context/servlet/turbine?test=null",
- // turi.getRelativeLink());
+ assertEquals("/context/servlet/turbine?test=null",
+ turi.getRelativeLink());
+
turi.removeQueryData("test");
assertEquals("/context/servlet/turbine", turi.getRelativeLink());