Apache/struts
1
0
Fork 0
mirror of https://github.com/apache/struts.git synced 2026-08-05 14:47:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Reverse merges master

Browse Source
This commit is contained in:
Lukasz Lenart
2020-04-21 20:42:15 +02:00
parent 2f27e45ef4 230a300685
commit f071c9d0ed
13 changed files with 472 additions and 88 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Jenkinsfile
Vendored
+42 -21
View File
@@ -170,14 +170,21 @@ pipeline {
failure {
script {
emailext(
subject: "[BUILD-FAILURE]: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]'",
body: """
BUILD-FAILURE: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]':
Check console output at ${env.BUILD_URL}
""".stripMargin(),
to: "dev@struts.apache.org",
recipientProviders: [[$class: 'DevelopersRecipientProvider']]
recipientProviders: [[$class: 'DevelopersRecipientProvider']],
from: "Mr. Jenkins <jenkins@builds.apache.org>",
subject: "Jenkins job ${env.JOB_NAME}#${env.BUILD_NUMBER} failed",
body: """
There is a build failure in ${env.JOB_NAME}.
Build: ${env.BUILD_URL}
Logs: ${env.BUILD_URL}console
Changes: ${env.BUILD_URL}changes
--
Mr. Jenkins
Director of Continuous Integration
"""
)
}
}
@@ -186,14 +193,21 @@ pipeline {
unstable {
script {
emailext(
subject: "[BUILD-UNSTABLE]: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]'",
body: """
BUILD-UNSTABLE: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]':
Check console output at ${env.BUILD_URL}
""".stripMargin(),
to: "dev@struts.apache.org",
recipientProviders: [[$class: 'DevelopersRecipientProvider']]
recipientProviders: [[$class: 'DevelopersRecipientProvider']],
from: "Mr. Jenkins <jenkins@builds.apache.org>",
subject: "Jenkins job ${env.JOB_NAME}#${env.BUILD_NUMBER} unstable",
body: """
Some tests have failed in ${env.JOB_NAME}.
Build: ${env.BUILD_URL}
Logs: ${env.BUILD_URL}console
Changes: ${env.BUILD_URL}changes
--
Mr. Jenkins
Director of Continuous Integration
"""
)
}
}
@@ -202,14 +216,21 @@ pipeline {
fixed {
script {
emailext(
subject: "[BUILD-STABLE]: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]'",
body: """
BUILD-STABLE: Job '${env.JOB_NAME} [${env.BRANCH_NAME}] [${env.BUILD_NUMBER}]':
Is back to normal.
""".stripMargin(),
to: "dev@struts.apache.org",
recipientProviders: [[$class: 'DevelopersRecipientProvider']]
recipientProviders: [[$class: 'DevelopersRecipientProvider']],
from: 'Mr. Jenkins <jenkins@builds.apache.org>',
subject: "Jenkins job ${env.JOB_NAME}#${env.BUILD_NUMBER} back to normal",
body: """
The build for ${env.JOB_NAME} completed successfully and is back to normal.
Build: ${env.BUILD_URL}
Logs: ${env.BUILD_URL}console
Changes: ${env.BUILD_URL}changes
--
Mr. Jenkins
Director of Continuous Integration
"""
)
}
}
apps/rest-showcase/pom.xml
+1 -1
View File
@@ -80,7 +80,7 @@
<dependency>
<groupId>net.sourceforge.htmlunit</groupId>
<artifactId>htmlunit</artifactId>
<version>2.27</version>
<version>2.39.0</version>
<scope>test</scope>
</dependency>
apps/showcase/pom.xml
+3 -3
View File
@@ -144,7 +144,7 @@
<dependency>
<groupId>net.sourceforge.htmlunit</groupId>
<artifactId>htmlunit</artifactId>
<version>2.37.0</version>
<version>2.39.0</version>
<scope>test</scope>
</dependency>
@@ -152,7 +152,7 @@
<dependency>
<groupId>org.hibernate</groupId>
<artifactId>hibernate-validator</artifactId>
<version>5.4.3.Final</version>
<version>6.1.2.Final</version>
</dependency>
<!-- The Servlet API mocks in Spring Framework 4.x only supports Servlet 3.0 and higher.
@@ -171,7 +171,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-failsafe-plugin</artifactId>
<version>2.22.2</version>
<version>3.0.0-M4</version>
<configuration>
<includes>
<include>it.org.apache.struts2.showcase.*Test</include>
core/src/main/java/com/opensymphony/xwork2/conversion/impl/DateConverter.java
+3 -3
View File
@@ -18,7 +18,7 @@
*/
package com.opensymphony.xwork2.conversion.impl;
import org.apache.struts2.StrutsException;
import org.apache.struts2.conversion.TypeConversionException;
import java.lang.reflect.Constructor;
import java.lang.reflect.Member;
@@ -90,11 +90,11 @@ public class DateConverter extends DefaultTypeConverter {
Constructor constructor = toType.getConstructor(new Class[]{long.class});
return constructor.newInstance(new Object[]{Long.valueOf(result.getTime())});
} catch (Exception e) {
throw new StrutsException("Couldn't create class " + toType + " using default (long) constructor", e);
throw new TypeConversionException("Couldn't create class " + toType + " using default (long) constructor", e);
}
}
} catch (ParseException e) {
throw new StrutsException("Could not parse date", e);
throw new TypeConversionException("Could not parse date", e);
}
} else if (Date.class.isAssignableFrom(value.getClass())) {
result = (Date) value;
core/src/main/java/com/opensymphony/xwork2/conversion/impl/NumberConverter.java
+9 -9
View File
@@ -18,9 +18,9 @@
*/
package com.opensymphony.xwork2.conversion.impl;
import org.apache.struts2.conversion.TypeConversionException;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.apache.struts2.StrutsException;
import java.lang.reflect.Member;
import java.math.BigDecimal;
@@ -51,7 +51,7 @@ public class NumberConverter extends DefaultTypeConverter {
Object convertedValue = super.convertValue(context, value, toType);
if (!isInRange((Number) convertedValue, stringValue, toType))
throw new StrutsException("Overflow or underflow casting: \"" + stringValue + "\" into class " + convertedValue.getClass().getName());
throw new TypeConversionException("Overflow or underflow casting: \"" + stringValue + "\" into class " + convertedValue.getClass().getName());
return convertedValue;
} else {
@@ -67,11 +67,11 @@ public class NumberConverter extends DefaultTypeConverter {
Number number = numFormat.parse(stringValue, parsePos);
if (parsePos.getIndex() != stringValue.length()) {
throw new StrutsException("Unparseable number: \"" + stringValue + "\" at position "
throw new TypeConversionException("Unparseable number: \"" + stringValue + "\" at position "
+ parsePos.getIndex());
} else {
if (!isInRange(number, stringValue, toType))
throw new StrutsException("Overflow or underflow casting: \"" + stringValue + "\" into class " + number.getClass().getName());
throw new TypeConversionException("Overflow or underflow casting: \"" + stringValue + "\" into class " + number.getClass().getName());
value = super.convertValue(context, number, toType);
}
@@ -103,7 +103,7 @@ public class NumberConverter extends DefaultTypeConverter {
Number number = format.parse(stringValue, parsePosition);
if (parsePosition.getIndex() != stringValue.length()) {
throw new StrutsException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
throw new TypeConversionException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
}
return number;
@@ -123,11 +123,11 @@ public class NumberConverter extends DefaultTypeConverter {
Number number = format.parse(stringValue, parsePosition);
if (parsePosition.getIndex() != stringValue.length()) {
throw new StrutsException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
throw new TypeConversionException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
}
if (!isInRange(number, stringValue, Double.class)) {
throw new StrutsException("Overflow or underflow converting: \"" + stringValue + "\" into class " + number.getClass().getName());
throw new TypeConversionException("Overflow or underflow converting: \"" + stringValue + "\" into class " + number.getClass().getName());
}
if (number != null) {
@@ -151,11 +151,11 @@ public class NumberConverter extends DefaultTypeConverter {
Number number = format.parse(stringValue, parsePosition);
if (parsePosition.getIndex() != stringValue.length()) {
throw new StrutsException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
throw new TypeConversionException("Unparseable number: \"" + stringValue + "\" at position " + parsePosition.getIndex());
}
if (!isInRange(number, stringValue, Float.class)) {
throw new StrutsException("Overflow or underflow converting: \"" + stringValue + "\" into class " + number.getClass().getName());
throw new TypeConversionException("Overflow or underflow converting: \"" + stringValue + "\" into class " + number.getClass().getName());
}
if (number != null) {
core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkBasicConverter.java
+8 -8
View File
@@ -18,11 +18,11 @@
*/
package com.opensymphony.xwork2.conversion.impl;
import org.apache.struts2.conversion.TypeConversionException;
import com.opensymphony.xwork2.conversion.TypeConverter;
import com.opensymphony.xwork2.inject.Container;
import com.opensymphony.xwork2.inject.Inject;
import org.apache.struts2.StrutsConstants;
import org.apache.struts2.StrutsException;
import java.lang.reflect.Member;
import java.util.Calendar;
@@ -130,7 +130,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
}
if (result == null && value != null && !"".equals(value)) {
throw new StrutsException("Cannot create type " + toType + " from value " + value);
throw new TypeConversionException("Cannot create type " + toType + " from value " + value);
}
}
@@ -170,7 +170,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
try {
clazz = Class.forName((String) value);
} catch (ClassNotFoundException e) {
throw new StrutsException(e.getLocalizedMessage(), e);
throw new TypeConversionException(e.getLocalizedMessage(), e);
}
}
return clazz;
@@ -179,7 +179,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
private Object doConvertToCollection(Map<String, Object> context, Object o, Member member, String prop, Object value, Class toType) {
TypeConverter converter = container.getInstance(CollectionConverter.class);
if (converter == null) {
throw new StrutsException("TypeConverter with name [#0] must be registered first!", StrutsConstants.STRUTS_CONVERTER_COLLECTION);
throw new TypeConversionException("TypeConverter with name [#0] must be registered first! Converter: "+ StrutsConstants.STRUTS_CONVERTER_COLLECTION);
}
return converter.convertValue(context, o, member, prop, value, toType);
}
@@ -187,7 +187,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
private Object doConvertToArray(Map<String, Object> context, Object o, Member member, String prop, Object value, Class toType) {
TypeConverter converter = container.getInstance(ArrayConverter.class);
if (converter == null) {
throw new StrutsException("TypeConverter with name [#0] must be registered first!", StrutsConstants.STRUTS_CONVERTER_ARRAY);
throw new TypeConversionException("TypeConverter with name [#0] must be registered first! Converter: "+ StrutsConstants.STRUTS_CONVERTER_ARRAY);
}
return converter.convertValue(context, o, member, prop, value, toType);
}
@@ -195,7 +195,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
private Object doConvertToDate(Map<String, Object> context, Object value, Class toType) {
TypeConverter converter = container.getInstance(DateConverter.class);
if (converter == null) {
throw new StrutsException("TypeConverter with name [#0] must be registered first!", StrutsConstants.STRUTS_CONVERTER_DATE);
throw new TypeConversionException("TypeConverter with name [#0] must be registered first! Converter: "+ StrutsConstants.STRUTS_CONVERTER_DATE);
}
return converter.convertValue(context, null, null, null, value, toType);
}
@@ -203,7 +203,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
private Object doConvertToNumber(Map<String, Object> context, Object value, Class toType) {
TypeConverter converter = container.getInstance(NumberConverter.class);
if (converter == null) {
throw new StrutsException("TypeConverter with name [#0] must be registered first!", StrutsConstants.STRUTS_CONVERTER_NUMBER);
throw new TypeConversionException("TypeConverter with name [#0] must be registered first! Converter: "+ StrutsConstants.STRUTS_CONVERTER_NUMBER);
}
return converter.convertValue(context, null, null, null, value, toType);
}
@@ -211,7 +211,7 @@ public class XWorkBasicConverter extends DefaultTypeConverter {
private Object doConvertToString(Map<String, Object> context, Object value) {
TypeConverter converter = container.getInstance(StringConverter.class);
if (converter == null) {
throw new StrutsException("TypeConverter with name [#0] must be registered first!", StrutsConstants.STRUTS_CONVERTER_STRING);
throw new TypeConversionException("TypeConverter with name [#0] must be registered first! Converter: "+ StrutsConstants.STRUTS_CONVERTER_STRING);
}
return converter.convertValue(context, null, null, null, value, null);
}
core/src/main/java/com/opensymphony/xwork2/conversion/TypeConversionException.java → core/src/main/java/org/apache/struts2/conversion/TypeConversionException.java
+1 -1
View File
@@ -16,7 +16,7 @@
* specific language governing permissions and limitations
* under the License.
*/
package com.opensymphony.xwork2.conversion;
package org.apache.struts2.conversion;
import org.apache.struts2.StrutsException;
core/src/test/java/com/opensymphony/xwork2/conversion/impl/DateConverterTest.java
+118
View File
@@ -0,0 +1,118 @@
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you 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.opensymphony.xwork2.conversion.impl;
import com.opensymphony.xwork2.ActionContext;
import org.apache.struts2.conversion.TypeConversionException;
import org.apache.struts2.StrutsInternalTestCase;
import java.sql.Time;
import java.sql.Timestamp;
import java.text.DateFormat;
import java.util.Date;
import java.util.HashMap;
import java.util.Locale;
import java.util.Map;
public class DateConverterTest extends StrutsInternalTestCase {
private String INPUT_TIME_STAMP_STR;
private String INPUT_WHEN_LONG_CONSTRUCTOR_STR;
private Locale mxLocale = new Locale("es_MX", "MX");
private final static String RES_TIME_STAMP_STR = "2020-03-20 00:00:00.0";
private final static String TIME_01_59_10 = "01:59:10 AM";
private final static String DATE_STR = "2020-03-20";
private final static String DATE_CONVERTED = "Fri Mar 20 00:00:00";
private final static String INVALID_DATE = "99/99/2010";
private final static String MESSAGE_PARSE_ERROR = "Could not parse date";
private final static String MESSAGE_DEFAULT_CONSTRUCTOR_ERROR = "Couldn't create class null using default (long) constructor";
public void testSqlTimeType() {
DateConverter converter = new DateConverter();
Map<String, Object> context = new HashMap<>();
context.put(ActionContext.LOCALE, new Locale("es_MX", "MX"));
Object value = converter.convertValue(context, null, null, null, TIME_01_59_10, Time.class);
assertEquals("01:59:10", value.toString());
}
public void testSqlTimestampType() {
DateConverter converter = new DateConverter();
Map<String, Object> context = new HashMap<>();
context.put(ActionContext.LOCALE, mxLocale);
Object value = converter.convertValue(context, null, null, null, INPUT_TIME_STAMP_STR, Timestamp.class);
assertEquals(RES_TIME_STAMP_STR, value.toString());
}
public void testDateType() {
DateConverter converter = new DateConverter();
Map<String, Object> context = new HashMap<>();
context.put(ActionContext.LOCALE, new Locale("es_MX", "MX"));
Object value = converter.convertValue(context, null, null, null, DATE_STR, Date.class);
assertTrue(((Date) value).toString().startsWith(DATE_CONVERTED));
}
public void testTypeConversionExceptionWhenParseError() {
DateConverter converter = new DateConverter();
Map<String, Object> context = new HashMap<>();
context.put(ActionContext.LOCALE, new Locale("es_MX", "MX"));
try {
Object value = converter.convertValue(context, null, null, null, INVALID_DATE, Date.class);
fail("TypeConversionException expected - Conversion error occurred");
} catch (Exception ex) {
assertEquals(TypeConversionException.class, ex.getClass());
assertEquals(MESSAGE_PARSE_ERROR, ex.getMessage());
}
}
public void testTypeConversionExceptionWhenUsingLongConstructor() {
DateConverter converter = new DateConverter();
Map<String, Object> context = new HashMap<>();
context.put(ActionContext.LOCALE, new Locale("es_MX", "MX"));
try {
Object value = converter.convertValue(context, null, null, null, INPUT_WHEN_LONG_CONSTRUCTOR_STR, null);
fail("TypeConversionException expected - Error using default (long) constructor");
} catch (Exception ex) {
assertEquals(TypeConversionException.class, ex.getClass());
assertEquals(MESSAGE_DEFAULT_CONSTRUCTOR_ERROR, ex.getMessage());
}
}
@Override
protected void setUp() {
//Due to JEP 252: Use CLDR Locale Data by Default
DateFormat dFormat = DateFormat.getDateInstance(DateFormat.SHORT, mxLocale);
if(dFormat.format(new Date()).contains("-")){ // Format when Java 9 or greater
INPUT_TIME_STAMP_STR = "2020-03-20 00:00:00.000";
INPUT_WHEN_LONG_CONSTRUCTOR_STR = "2020-03-20";
}else{ // Format when Java 8 or lower
INPUT_TIME_STAMP_STR = "03/20/2020 00:00:00.000";
INPUT_WHEN_LONG_CONSTRUCTOR_STR = "03/31/20";
}
}
}
core/src/test/java/com/opensymphony/xwork2/conversion/impl/NumberConverterTest.java
+143
View File
@@ -20,6 +20,7 @@ package com.opensymphony.xwork2.conversion.impl;
import com.opensymphony.xwork2.SimpleFooAction;
import com.opensymphony.xwork2.XWorkTestCase;
import org.apache.struts2.conversion.TypeConversionException;
import org.apache.commons.lang3.StringUtils;
import java.math.BigDecimal;
@@ -28,6 +29,16 @@ import java.util.Map;
public class NumberConverterTest extends XWorkTestCase {
private final static String FLOAT_OUT_OF_RANGE = "3.5028235E38";
private final static String DOUBLE_OUT_OF_RANGE = "1.7976931348623157E309";
private final static String INTEGER_OUT_OF_RANGE = "2147483648";
private final static String MSG_OUT_OF_RANGE_CASTING = "Overflow or underflow casting";
private final static String MSG_OUT_OF_RANGE_CONVERTING = "Overflow or underflow converting";
private final static String MSG_UNPARSEABLE_NUMBER = "Unparseable number";
private final static String MSG_TEST_FAILS_OUT_OF_RANGE = "TypeConversionException expected when OUT OF RANGE";
private final static String MSG_TEST_FAILS_UNPARSEABLE_NUMBER = "TypeConversionException expected when UNPARSEABLE NUMBER";
private final static Locale LOCALE_MEXICO = new Locale("es_MX", "MX");
public void testStringToNumberConversionPL() throws Exception {
// given
NumberConverter converter = new NumberConverter();
@@ -158,5 +169,137 @@ public class NumberConverterTest extends XWorkTestCase {
assertEquals(1234.4F, value);
}
public void testExceptionWhenPrimitiveIsOutOfRange() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
// when
try {
Object value = converter.convertValue(context, null, null, null, INTEGER_OUT_OF_RANGE, int.class);
fail(MSG_TEST_FAILS_OUT_OF_RANGE);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_OUT_OF_RANGE_CASTING));
}
}
public void testExceptionWhenANotPrimitiveIsUnparsable() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
String strValue = "1.2";
// when
try {
Object value = converter.convertValue(context, null, null, null, strValue, Byte.class);
fail(MSG_TEST_FAILS_UNPARSEABLE_NUMBER);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_UNPARSEABLE_NUMBER));
}
}
public void testExceptionWhenANotPrimitiveIsOutOfRange() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
String strValue = "129";
// when
try {
Object value = converter.convertValue(context, null, null, null, strValue, Byte.class);
fail(MSG_TEST_FAILS_OUT_OF_RANGE);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_OUT_OF_RANGE_CASTING));
}
}
public void testExceptionWhenUnparseableInConvertToBigDecimal() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
String strValue = "1-23";
// when
try {
Object value = converter.convertValue(context, null, null, null, strValue, BigDecimal.class);
fail(MSG_TEST_FAILS_UNPARSEABLE_NUMBER);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_UNPARSEABLE_NUMBER));
}
}
public void testExceptionWhenUnparseableInConvertToDouble() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
String strValue = "1-23";
// when
try {
Object value = converter.convertValue(context, null, null, null, strValue, Double.class);
fail(MSG_TEST_FAILS_UNPARSEABLE_NUMBER);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_UNPARSEABLE_NUMBER));
}
}
public void testExceptionWhenOutOfRangeInConvertToDouble() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
// when
try {
Object value = converter.convertValue(context, null, null, null, DOUBLE_OUT_OF_RANGE, Double.class);
fail(MSG_TEST_FAILS_OUT_OF_RANGE);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_OUT_OF_RANGE_CONVERTING));
}
}
public void testExceptionWhenOutOfRangeInConvertToFloat() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
// when
try {
Object value = converter.convertValue(context, null, null, null, FLOAT_OUT_OF_RANGE, Float.class);
fail(MSG_TEST_FAILS_OUT_OF_RANGE);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_OUT_OF_RANGE_CONVERTING));
}
}
public void testExceptionWhenUnparseableInConvertToFloat() {
// given
NumberConverter converter = new NumberConverter();
Map<String, Object> context = createContextWithLocale(LOCALE_MEXICO);
String strValue = "1-23";
// when
try {
Object value = converter.convertValue(context, null, null, null, strValue, Float.class);
fail(MSG_TEST_FAILS_UNPARSEABLE_NUMBER);
} catch (Exception ex) {
// then
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_UNPARSEABLE_NUMBER));
}
}
}
core/src/test/java/com/opensymphony/xwork2/conversion/impl/XWorkBasicConverterTest.java
+88 -2
View File
@@ -20,8 +20,11 @@ package com.opensymphony.xwork2.conversion.impl;
import com.opensymphony.xwork2.ActionContext;
import com.opensymphony.xwork2.XWorkTestCase;
import org.apache.struts2.conversion.TypeConversionException;
import com.opensymphony.xwork2.inject.Container;
import com.opensymphony.xwork2.test.annotations.Person;
import org.apache.struts2.StrutsException;
import org.mockito.Mockito;
import java.math.BigDecimal;
import java.math.BigInteger;
@@ -37,6 +40,9 @@ import java.util.*;
public class XWorkBasicConverterTest extends XWorkTestCase {
private XWorkBasicConverter basicConverter;
private Container mockedContainer;
private final static String MSG_EXCEPTION_EXPECTED = "TypeConversionException expected";
private final static String MSG_TYPE_CONVERTER_EXCEPTION = "TypeConverter with name";
// TODO: test for every possible conversion
// take into account of empty string
@@ -285,11 +291,92 @@ public class XWorkBasicConverterTest extends XWorkTestCase {
Class toType = String.class;
basicConverter.convertValue(context, value, null, s, value, toType);
}
public void testExceptionWhenCantCreateTypeFromValue() {
try{
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, 4, Date.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith("Cannot create type"));
}
}
public void testExceptionInDoConvertToClass() {
try{
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, "Foo", Class.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
}
}
public void testExceptionInDoConvertToCollection() {
try{
Mockito.when(mockedContainer.getInstanceNames(CollectionConverter.class)).thenReturn(null);
basicConverter.setContainer(mockedContainer);
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, "Foo", ArrayList.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_TYPE_CONVERTER_EXCEPTION));
}
}
public void testExceptionInDoConvertToArray() {
try{
int[] arrayInt = new int[1];
Mockito.when(mockedContainer.getInstanceNames(ArrayConverter.class)).thenReturn(null);
basicConverter.setContainer(mockedContainer);
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, "Foo", arrayInt.getClass());
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_TYPE_CONVERTER_EXCEPTION));
}
}
public void testExceptionInDoConvertToDate() {
try{
Mockito.when(mockedContainer.getInstanceNames(DateConverter.class)).thenReturn(null);
basicConverter.setContainer(mockedContainer);
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, "Foo", Date.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_TYPE_CONVERTER_EXCEPTION));
}
}
public void testExceptionInDoConvertToNumber() {
try{
Mockito.when(mockedContainer.getInstanceNames(NumberConverter.class)).thenReturn(null);
basicConverter.setContainer(mockedContainer);
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, "Foo", int.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_TYPE_CONVERTER_EXCEPTION));
}
}
public void testExceptionInDoConvertToString() {
try{
Mockito.when(mockedContainer.getInstanceNames(StringConverter.class)).thenReturn(null);
basicConverter.setContainer(mockedContainer);
Object convertedObject = basicConverter.convertValue(new HashMap<String, Object>(), null, null, null, 1, String.class);
fail(MSG_EXCEPTION_EXPECTED);
}catch(Exception ex){
assertEquals(TypeConversionException.class, ex.getClass());
assertTrue(ex.getMessage().startsWith(MSG_TYPE_CONVERTER_EXCEPTION));
}
}
@Override
protected void setUp() throws Exception {
super.setUp();
basicConverter = container.getInstance(XWorkBasicConverter.class);
mockedContainer = Mockito.mock(Container.class);
}
@Override
@@ -297,6 +384,5 @@ public class XWorkBasicConverterTest extends XWorkTestCase {
super.tearDown();
ActionContext.clear();
}
}
plugins/bean-validation/pom.xml
+3 -3
View File
@@ -41,7 +41,7 @@
<dependency>
<groupId>javax.validation</groupId>
<artifactId>validation-api</artifactId>
<version>1.1.0.Final</version>
<version>2.0.1.Final</version>
</dependency>
<dependency>
@@ -52,7 +52,7 @@
<dependency>
<groupId>org.hibernate</groupId>
<artifactId>hibernate-validator</artifactId>
<version>5.4.3.Final</version>
<version>6.1.2.Final</version>
<scope>test</scope>
</dependency>
<dependency>
@@ -82,7 +82,7 @@
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-impl</artifactId>
<version>2.3.1</version>
<version>2.3.2</version>
<scope>test</scope>
</dependency>
<dependency>
plugins/cdi/pom.xml
+1 -1
View File
@@ -47,7 +47,7 @@
</dependency>
<dependency>
<groupId>org.jboss.weld</groupId>
<groupId>org.jboss.weld.se</groupId>
<artifactId>weld-se</artifactId>
<scope>test</scope>
</dependency>
pom.xml
+52 -36
View File
@@ -72,6 +72,14 @@
</notifiers>
</ciManagement>
<distributionManagement>
<site>
<id>struts-site</id>
<name>Apache Struts</name>
<url>https://struts.apache.org/maven/</url>
</site>
</distributionManagement>
<modules>
<module>bom</module>
<module>core</module>
@@ -99,21 +107,21 @@
<java.version>1.8</java.version>
<!-- dependency versions in alphanumeric order -->
<asm.version>7.2</asm.version>
<jackson.version>2.10.1</jackson.version>
<log4j2.version>2.12.1</log4j2.version>
<ognl.version>3.2.12</ognl.version>
<slf4j.version>1.7.29</slf4j.version>
<spring.platformVersion>4.3.25.RELEASE</spring.platformVersion>
<asm.version>7.3.1</asm.version>
<jackson.version>2.10.3</jackson.version>
<log4j2.version>2.13.1</log4j2.version>
<ognl.version>3.2.14</ognl.version>
<slf4j.version>1.7.30</slf4j.version>
<spring.platformVersion>4.3.26.RELEASE</spring.platformVersion>
<tiles.version>3.0.8</tiles.version>
<tiles-request.version>1.0.7</tiles-request.version>
<!-- Site generation -->
<fluido-skin.version>1.8</fluido-skin.version>
<fluido-skin.version>1.9</fluido-skin.version>
<!-- Sonar -->
<sonar.host.url>https://builds.apache.org/analysis/</sonar.host.url>
<maven-surefire-plugin.version>2.22.2</maven-surefire-plugin.version>
<maven-surefire-plugin.version>3.0.0-M4</maven-surefire-plugin.version>
</properties>
<profiles>
@@ -223,7 +231,7 @@
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.8.4</version>
<version>0.8.5</version>
<executions>
<execution>
<id>prepare-agent</id>
@@ -287,17 +295,17 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-war-plugin</artifactId>
<version>3.2.2</version>
<version>3.2.3</version>
</plugin>
<plugin>
<groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId>
<version>3.5.0</version>
<version>4.2.1</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
<version>3.1.1</version>
<version>3.1.2</version>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
@@ -353,7 +361,7 @@
<plugin>
<groupId>org.owasp</groupId>
<artifactId>dependency-check-maven</artifactId>
<version>5.2.4</version>
<version>5.3.2</version>
<configuration>
<suppressionFiles>
<suppressionFile>src/etc/project-suppression.xml</suppressionFile>
@@ -366,7 +374,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<version>3.0.0-M2</version>
<version>3.0.0-M3</version>
<executions>
<execution>
<id>enforce</id>
@@ -425,7 +433,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-site-plugin</artifactId>
<version>3.8.2</version>
<version>3.9.0</version>
<configuration>
<relativizeDecorationLinks>false</relativizeDecorationLinks>
</configuration>
@@ -433,12 +441,12 @@
<dependency>
<groupId>org.apache.maven.doxia</groupId>
<artifactId>doxia-core</artifactId>
<version>1.9</version>
<version>1.9.1</version>
</dependency>
<dependency>
<groupId>org.apache.maven.doxia</groupId>
<artifactId>doxia-module-markdown</artifactId>
<version>1.9</version>
<version>1.9.1</version>
</dependency>
</dependencies>
</plugin>
@@ -678,13 +686,13 @@
<dependency>
<groupId>org.freemarker</groupId>
<artifactId>freemarker</artifactId>
<version>2.3.28</version>
<version>2.3.30</version>
</dependency>
<dependency>
<groupId>org.apache.felix</groupId>
<artifactId>org.apache.felix.main</artifactId>
<version>4.6.1</version>
<version>6.0.3</version>
</dependency>
<dependency>
<groupId>org.apache.felix</groupId>
@@ -713,7 +721,7 @@
<dependency>
<groupId>org.apache.velocity</groupId>
<artifactId>velocity-engine-core</artifactId>
<version>2.1</version>
<version>2.2</version>
</dependency>
<dependency>
@@ -746,13 +754,13 @@
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
<version>4.12</version>
<version>4.13</version>
</dependency>
<dependency>
<groupId>org.easymock</groupId>
<artifactId>easymock</artifactId>
<version>3.5.1</version>
<version>4.2</version>
<scope>test</scope>
</dependency>
@@ -766,7 +774,7 @@
<dependency>
<groupId>org.glassfish</groupId>
<artifactId>javax.el</artifactId>
<version>3.0.1-b10</version>
<version>3.0.1-b11</version>
</dependency>
<dependency>
@@ -779,21 +787,21 @@
<dependency>
<groupId>javax.servlet</groupId>
<artifactId>jstl</artifactId>
<version>1.1.2</version>
<version>1.2</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-jasper</artifactId>
<version>8.5.37</version>
<version>8.5.53</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-api</artifactId>
<version>8.5.37</version>
<version>8.5.53</version>
<scope>provided</scope>
</dependency>
@@ -870,7 +878,7 @@
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-juli</artifactId>
<version>8.5.37</version>
<version>8.5.53</version>
</dependency>
<!-- Commons -->
@@ -903,7 +911,7 @@
<dependency>
<groupId>org.apache.commons</groupId>
<artifactId>commons-lang3</artifactId>
<version>3.9</version>
<version>3.10</version>
</dependency>
<dependency>
<groupId>org.apache.commons</groupId>
@@ -1025,14 +1033,22 @@
<dependency>
<groupId>org.assertj</groupId>
<artifactId>assertj-core</artifactId>
<version>2.9.1</version>
<version>3.15.0</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
<version>2.23.0</version>
<version>3.3.3</version>
<exclusions>
<!-- The mockito-core artifact and easymock artifact use different versions of objenesis (2.6 vs 3.1).
Excluding the older version here to pass enforcer. When next upgrading mockito-core, confirm whether this exclusion is still required. -->
<exclusion>
<groupId>org.objenesis</groupId>
<artifactId>objenesis</artifactId>
</exclusion>
</exclusions>
<scope>test</scope>
</dependency>
@@ -1063,7 +1079,7 @@
<dependency>
<groupId>org.testng</groupId>
<artifactId>testng</artifactId>
<version>5.14.10</version>
<version>7.1.0</version>
<scope>compile</scope>
<optional>true</optional>
</dependency>
@@ -1131,26 +1147,26 @@
<dependency>
<groupId>org.apache.juneau</groupId>
<artifactId>juneau-marshall</artifactId>
<version>7.2.2</version>
<version>8.1.3</version>
</dependency>
<!-- CDI & Weld -->
<dependency>
<groupId>javax.enterprise</groupId>
<artifactId>cdi-api</artifactId>
<version>1.0-SP4</version>
<version>1.2</version>
</dependency>
<dependency>
<groupId>org.jboss.weld</groupId>
<artifactId>weld-core</artifactId>
<version>1.0.1-SP4</version>
<version>2.2.16.SP1</version>
</dependency>
<dependency>
<groupId>org.jboss.weld</groupId>
<groupId>org.jboss.weld.se</groupId>
<artifactId>weld-se</artifactId>
<version>1.0.1-Final</version>
<version>2.2.16.SP1</version>
</dependency>
<dependency>