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());
