diff --git a/spring-beans/src/test/java/org/springframework/beans/BeanUtilsTests.java b/spring-beans/src/test/java/org/springframework/beans/BeanUtilsTests.java index 0909ad36400cbd87af82f2552663051ecdb36ebb..bea3883b3fd11f089aed12e916db864103783b2f 100644 --- a/spring-beans/src/test/java/org/springframework/beans/BeanUtilsTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/BeanUtilsTests.java @@ -193,7 +193,7 @@ public class BeanUtilsTests { source.setFlag2(true); InvalidProperty target = new InvalidProperty(); BeanUtils.copyProperties(source, target); - assertEquals(target.getName(), "name"); + assertEquals("name", target.getName()); assertTrue(target.getFlag1()); assertTrue(target.getFlag2()); } diff --git a/spring-beans/src/test/java/org/springframework/beans/BeanWrapperGenericsTests.java b/spring-beans/src/test/java/org/springframework/beans/BeanWrapperGenericsTests.java index a429fe656b3a20042df7e3b23a27196a62c9c08c..3260b0bf84aebcbfcb2de59fb79a5ac7e8b09f17 100644 --- a/spring-beans/src/test/java/org/springframework/beans/BeanWrapperGenericsTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/BeanWrapperGenericsTests.java @@ -518,7 +518,7 @@ public class BeanWrapperGenericsTests { assertEquals("y", bw.getPropertyValue("data['x']")); bw.setPropertyValue("data['message']", "it works!"); - assertEquals(data.get("message"), "it works!"); + assertEquals("it works!", data.get("message")); } diff --git a/spring-beans/src/test/java/org/springframework/beans/factory/ConcurrentBeanFactoryTests.java b/spring-beans/src/test/java/org/springframework/beans/factory/ConcurrentBeanFactoryTests.java index b24b5db907fa66ef225a60174e6d88ccf2c5efff..c21ab35c9f5c91bea752194036845d0c45757998 100644 --- a/spring-beans/src/test/java/org/springframework/beans/factory/ConcurrentBeanFactoryTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/factory/ConcurrentBeanFactoryTests.java @@ -126,8 +126,8 @@ public class ConcurrentBeanFactoryTests { ConcurrentBean b1 = (ConcurrentBean) factory.getBean("bean1"); ConcurrentBean b2 = (ConcurrentBean) factory.getBean("bean2"); - assertEquals(b1.getDate(), DATE_1); - assertEquals(b2.getDate(), DATE_2); + assertEquals(DATE_1, b1.getDate()); + assertEquals(DATE_2, b2.getDate()); } diff --git a/spring-beans/src/test/java/org/springframework/beans/factory/xml/CollectionMergingTests.java b/spring-beans/src/test/java/org/springframework/beans/factory/xml/CollectionMergingTests.java index 1c6ce78bbdf65f99b3a26d0a91c71184920bf932..b0cdb5f023ae6337be0c112095ac879ba0b12ae1 100644 --- a/spring-beans/src/test/java/org/springframework/beans/factory/xml/CollectionMergingTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/factory/xml/CollectionMergingTests.java @@ -55,9 +55,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithList"); List list = bean.getSomeList(); assertEquals("Incorrect size", 3, list.size()); - assertEquals(list.get(0), "Rob Harrop"); - assertEquals(list.get(1), "Rod Johnson"); - assertEquals(list.get(2), "Juergen Hoeller"); + assertEquals("Rob Harrop", list.get(0)); + assertEquals("Rod Johnson", list.get(1)); + assertEquals("Juergen Hoeller", list.get(2)); } @Test @@ -98,9 +98,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithMap"); Map map = bean.getSomeMap(); assertEquals("Incorrect size", 3, map.size()); - assertEquals(map.get("Rob"), "Sally"); - assertEquals(map.get("Rod"), "Kerry"); - assertEquals(map.get("Juergen"), "Eva"); + assertEquals("Sally", map.get("Rob")); + assertEquals("Kerry", map.get("Rod")); + assertEquals("Eva", map.get("Juergen")); } @Test @@ -119,9 +119,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithProps"); Properties props = bean.getSomeProperties(); assertEquals("Incorrect size", 3, props.size()); - assertEquals(props.getProperty("Rob"), "Sally"); - assertEquals(props.getProperty("Rod"), "Kerry"); - assertEquals(props.getProperty("Juergen"), "Eva"); + assertEquals("Sally", props.getProperty("Rob")); + assertEquals("Kerry",props.getProperty("Rod")); + assertEquals("Eva", props.getProperty("Juergen")); } @Test @@ -129,9 +129,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithListInConstructor"); List list = bean.getSomeList(); assertEquals("Incorrect size", 3, list.size()); - assertEquals(list.get(0), "Rob Harrop"); - assertEquals(list.get(1), "Rod Johnson"); - assertEquals(list.get(2), "Juergen Hoeller"); + assertEquals("Rob Harrop", list.get(0)); + assertEquals("Rod Johnson", list.get(1)); + assertEquals("Juergen Hoeller", list.get(2)); } @Test @@ -172,9 +172,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithMapInConstructor"); Map map = bean.getSomeMap(); assertEquals("Incorrect size", 3, map.size()); - assertEquals(map.get("Rob"), "Sally"); - assertEquals(map.get("Rod"), "Kerry"); - assertEquals(map.get("Juergen"), "Eva"); + assertEquals("Sally", map.get("Rob")); + assertEquals("Kerry", map.get("Rod")); + assertEquals("Eva", map.get("Juergen")); } @Test @@ -193,9 +193,9 @@ public class CollectionMergingTests { TestBean bean = (TestBean) this.beanFactory.getBean("childWithPropsInConstructor"); Properties props = bean.getSomeProperties(); assertEquals("Incorrect size", 3, props.size()); - assertEquals(props.getProperty("Rob"), "Sally"); - assertEquals(props.getProperty("Rod"), "Kerry"); - assertEquals(props.getProperty("Juergen"), "Eva"); + assertEquals("Sally", props.getProperty("Rob")); + assertEquals("Kerry", props.getProperty("Rod")); + assertEquals("Eva", props.getProperty("Juergen")); } } diff --git a/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java b/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java index 94ff6c5c8ad388a65c1091f3442ef1cd30a1e2a2..7d8049df2d4b204c9d05c14199d963ff5a45e3b0 100644 --- a/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java +++ b/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java @@ -1064,12 +1064,12 @@ public abstract class AbstractAopProxyTests { pc.addAdvisor(dp); pc.setTarget(tb); ITestBean it = (ITestBean) createProxy(pc); - assertEquals(dp.count, 0); + assertEquals(0, dp.count); it.getAge(); - assertEquals(dp.count, 1); + assertEquals(1, dp.count); it.setAge(11); - assertEquals(it.getAge(), 11); - assertEquals(dp.count, 2); + assertEquals(11, it.getAge()); + assertEquals(2, dp.count); } @Test @@ -1083,16 +1083,16 @@ public abstract class AbstractAopProxyTests { this.mockTargetSource.setTarget(tb); pc.setTargetSource(mockTargetSource); ITestBean it = (ITestBean) createProxy(pc); - assertEquals(dp.count, 0); + assertEquals(0, dp.count); it.getAge(); // Statically vetoed assertEquals(0, dp.count); it.setAge(11); - assertEquals(it.getAge(), 11); - assertEquals(dp.count, 1); + assertEquals(11, it.getAge()); + assertEquals(1, dp.count); // Applies statically but not dynamically it.setName("joe"); - assertEquals(dp.count, 1); + assertEquals(1, dp.count); } @Test diff --git a/spring-context/src/test/java/org/springframework/context/annotation/configuration/ConfigurationClassProcessingTests.java b/spring-context/src/test/java/org/springframework/context/annotation/configuration/ConfigurationClassProcessingTests.java index e77f1fd3d782727426b678172030a6e877f9e2dd..6ac6bdea58cca0763a8961c29bd44b85483b789e 100644 --- a/spring-context/src/test/java/org/springframework/context/annotation/configuration/ConfigurationClassProcessingTests.java +++ b/spring-context/src/test/java/org/springframework/context/annotation/configuration/ConfigurationClassProcessingTests.java @@ -154,7 +154,7 @@ public class ConfigurationClassProcessingTests { public void simplestPossibleConfig() { BeanFactory factory = initBeanFactory(SimplestPossibleConfig.class); String stringBean = factory.getBean("stringBean", String.class); - assertEquals(stringBean, "foo"); + assertEquals("foo", stringBean); } @Test @@ -163,7 +163,7 @@ public class ConfigurationClassProcessingTests { assertEquals(Object.class, factory.getType("stringBean")); assertFalse(factory.isTypeMatch("stringBean", String.class)); String stringBean = factory.getBean("stringBean", String.class); - assertEquals(stringBean, "foo"); + assertEquals("foo", stringBean); } @Test diff --git a/spring-context/src/test/java/org/springframework/remoting/rmi/RmiSupportTests.java b/spring-context/src/test/java/org/springframework/remoting/rmi/RmiSupportTests.java index 2317a114b9f439947ed16f6da462174e02d5dad4..6ccabd6daf6da50ce653831a85252fc5a7dd169c 100644 --- a/spring-context/src/test/java/org/springframework/remoting/rmi/RmiSupportTests.java +++ b/spring-context/src/test/java/org/springframework/remoting/rmi/RmiSupportTests.java @@ -55,7 +55,7 @@ public class RmiSupportTests { assertTrue(factory.getObject() instanceof IRemoteBean); IRemoteBean proxy = (IRemoteBean) factory.getObject(); proxy.setName("myName"); - assertEquals(RemoteBean.name, "myName"); + assertEquals("myName", RemoteBean.name); assertEquals(1, factory.counter); } @@ -179,7 +179,7 @@ public class RmiSupportTests { IBusinessBean proxy = (IBusinessBean) factory.getObject(); assertFalse(proxy instanceof IRemoteBean); proxy.setName("myName"); - assertEquals(RemoteBean.name, "myName"); + assertEquals("myName", RemoteBean.name); assertEquals(1, factory.counter); } diff --git a/spring-core/src/test/java/org/springframework/core/ConstantsTests.java b/spring-core/src/test/java/org/springframework/core/ConstantsTests.java index 3f795060149ad397f4f12eb63e705bd126c6f9bd..3c5c34c70a52673a495be2aebb3293afbd85520f 100644 --- a/spring-core/src/test/java/org/springframework/core/ConstantsTests.java +++ b/spring-core/src/test/java/org/springframework/core/ConstantsTests.java @@ -37,9 +37,9 @@ public class ConstantsTests { assertEquals(A.class.getName(), c.getClassName()); assertEquals(9, c.getSize()); - assertEquals(c.asNumber("DOG").intValue(), A.DOG); - assertEquals(c.asNumber("dog").intValue(), A.DOG); - assertEquals(c.asNumber("cat").intValue(), A.CAT); + assertEquals(A.DOG, c.asNumber("DOG").intValue()); + assertEquals(A.DOG, c.asNumber("dog").intValue()); + assertEquals(A.CAT, c.asNumber("cat").intValue()); try { c.asNumber("bogus"); @@ -152,20 +152,20 @@ public class ConstantsTests { public void toCode() { Constants c = new Constants(A.class); - assertEquals(c.toCode(new Integer(0), ""), "DOG"); - assertEquals(c.toCode(new Integer(0), "D"), "DOG"); - assertEquals(c.toCode(new Integer(0), "DO"), "DOG"); - assertEquals(c.toCode(new Integer(0), "DoG"), "DOG"); - assertEquals(c.toCode(new Integer(0), null), "DOG"); - assertEquals(c.toCode(new Integer(66), ""), "CAT"); - assertEquals(c.toCode(new Integer(66), "C"), "CAT"); - assertEquals(c.toCode(new Integer(66), "ca"), "CAT"); - assertEquals(c.toCode(new Integer(66), "cAt"), "CAT"); - assertEquals(c.toCode(new Integer(66), null), "CAT"); - assertEquals(c.toCode("", ""), "S1"); - assertEquals(c.toCode("", "s"), "S1"); - assertEquals(c.toCode("", "s1"), "S1"); - assertEquals(c.toCode("", null), "S1"); + assertEquals("DOG", c.toCode(new Integer(0), "")); + assertEquals("DOG", c.toCode(new Integer(0), "D")); + assertEquals("DOG", c.toCode(new Integer(0), "DO")); + assertEquals("DOG", c.toCode(new Integer(0), "DoG")); + assertEquals("DOG", c.toCode(new Integer(0), null)); + assertEquals("CAT", c.toCode(new Integer(66), "")); + assertEquals("CAT", c.toCode(new Integer(66), "C")); + assertEquals("CAT", c.toCode(new Integer(66), "ca")); + assertEquals("CAT", c.toCode(new Integer(66), "cAt")); + assertEquals("CAT", c.toCode(new Integer(66), null)); + assertEquals("S1", c.toCode("", "")); + assertEquals("S1", c.toCode("", "s")); + assertEquals("S1", c.toCode("", "s1")); + assertEquals("S1", c.toCode("", null)); try { c.toCode("bogus", "bogus"); fail("Should have thrown ConstantException"); @@ -179,8 +179,8 @@ public class ConstantsTests { catch (Constants.ConstantException expected) { } - assertEquals(c.toCodeForProperty(new Integer(1), "myProperty"), "MY_PROPERTY_NO"); - assertEquals(c.toCodeForProperty(new Integer(2), "myProperty"), "MY_PROPERTY_YES"); + assertEquals("MY_PROPERTY_NO", c.toCodeForProperty(new Integer(1), "myProperty")); + assertEquals("MY_PROPERTY_YES", c.toCodeForProperty(new Integer(2), "myProperty")); try { c.toCodeForProperty("bogus", "bogus"); fail("Should have thrown ConstantException"); @@ -188,20 +188,20 @@ public class ConstantsTests { catch (Constants.ConstantException expected) { } - assertEquals(c.toCodeForSuffix(new Integer(0), ""), "DOG"); - assertEquals(c.toCodeForSuffix(new Integer(0), "G"), "DOG"); - assertEquals(c.toCodeForSuffix(new Integer(0), "OG"), "DOG"); - assertEquals(c.toCodeForSuffix(new Integer(0), "DoG"), "DOG"); - assertEquals(c.toCodeForSuffix(new Integer(0), null), "DOG"); - assertEquals(c.toCodeForSuffix(new Integer(66), ""), "CAT"); - assertEquals(c.toCodeForSuffix(new Integer(66), "T"), "CAT"); - assertEquals(c.toCodeForSuffix(new Integer(66), "at"), "CAT"); - assertEquals(c.toCodeForSuffix(new Integer(66), "cAt"), "CAT"); - assertEquals(c.toCodeForSuffix(new Integer(66), null), "CAT"); - assertEquals(c.toCodeForSuffix("", ""), "S1"); - assertEquals(c.toCodeForSuffix("", "1"), "S1"); - assertEquals(c.toCodeForSuffix("", "s1"), "S1"); - assertEquals(c.toCodeForSuffix("", null), "S1"); + assertEquals("DOG", c.toCodeForSuffix(new Integer(0), "")); + assertEquals("DOG", c.toCodeForSuffix(new Integer(0), "G")); + assertEquals("DOG", c.toCodeForSuffix(new Integer(0), "OG")); + assertEquals("DOG", c.toCodeForSuffix(new Integer(0), "DoG")); + assertEquals("DOG", c.toCodeForSuffix(new Integer(0), null)); + assertEquals("CAT", c.toCodeForSuffix(new Integer(66), "")); + assertEquals("CAT", c.toCodeForSuffix(new Integer(66), "T")); + assertEquals("CAT", c.toCodeForSuffix(new Integer(66), "at")); + assertEquals("CAT", c.toCodeForSuffix(new Integer(66), "cAt")); + assertEquals("CAT", c.toCodeForSuffix(new Integer(66), null)); + assertEquals("S1", c.toCodeForSuffix("", "")); + assertEquals("S1", c.toCodeForSuffix("", "1")); + assertEquals("S1", c.toCodeForSuffix("", "s1")); + assertEquals("S1", c.toCodeForSuffix("", null)); try { c.toCodeForSuffix("bogus", "bogus"); fail("Should have thrown ConstantException"); diff --git a/spring-core/src/test/java/org/springframework/core/env/MutablePropertySourcesTests.java b/spring-core/src/test/java/org/springframework/core/env/MutablePropertySourcesTests.java index 6b4246256da866e0d94e1b981faa2343b5c031ec..7ec6d966d9089a7244888eb32f8ae175073f43bc 100644 --- a/spring-core/src/test/java/org/springframework/core/env/MutablePropertySourcesTests.java +++ b/spring-core/src/test/java/org/springframework/core/env/MutablePropertySourcesTests.java @@ -96,7 +96,7 @@ public class MutablePropertySourcesTests { assertThat(sources.size(), equalTo(6)); assertThat(sources.contains("a"), is(false)); - assertEquals(sources.remove("a"), null); + assertNull(sources.remove("a")); assertThat(sources.size(), equalTo(6)); String bogusPS = "bogus"; diff --git a/spring-core/src/test/java/org/springframework/core/type/AnnotationMetadataTests.java b/spring-core/src/test/java/org/springframework/core/type/AnnotationMetadataTests.java index 11d8839b5ee7e92d76ffb402642489571419a6c8..31f80520b7590a887766e3c67625e2bd1d076d1a 100644 --- a/spring-core/src/test/java/org/springframework/core/type/AnnotationMetadataTests.java +++ b/spring-core/src/test/java/org/springframework/core/type/AnnotationMetadataTests.java @@ -365,7 +365,7 @@ public class AnnotationMetadataTests { assertArrayEquals(new String[] { Void.class.getName() }, (String[]) optionalArray[0].get("classArray")); assertArrayEquals(new String[] { Void.class.getName() }, optionalArray[0].getStringArray("classArray")); - assertEquals(metadata.getAnnotationAttributes(DirectAnnotation.class.getName()).get("value"), "direct"); + assertEquals("direct", metadata.getAnnotationAttributes(DirectAnnotation.class.getName()).get("value")); allMeta = metadata.getAllAnnotationAttributes(DirectAnnotation.class.getName()).get("value"); assertThat(new HashSet<>(allMeta), is(equalTo(new HashSet(Arrays.asList("direct", "meta"))))); } diff --git a/spring-core/src/test/java/org/springframework/util/FastByteArrayOutputStreamTests.java b/spring-core/src/test/java/org/springframework/util/FastByteArrayOutputStreamTests.java index 15217c87b3e29ce1f4b9c4dc130d2ed116baa39c..074e80cda7648bdd4c3e8d420d1fc9fb7d7bcdb8 100644 --- a/spring-core/src/test/java/org/springframework/util/FastByteArrayOutputStreamTests.java +++ b/spring-core/src/test/java/org/springframework/util/FastByteArrayOutputStreamTests.java @@ -167,7 +167,7 @@ public class FastByteArrayOutputStreamTests { this.os.write(this.helloBytes); InputStream inputStream = this.os.getInputStream(); assertEquals(inputStream.read(), this.helloBytes[0]); - assertEquals(inputStream.skip(1), 1); + assertEquals(1, inputStream.skip(1)); assertEquals(inputStream.read(), this.helloBytes[2]); assertEquals(this.helloBytes.length - 3, inputStream.available()); } diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/AbstractExpressionTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/AbstractExpressionTests.java index 0ff6458b54466178c910acb59f0c0fdc551feec1..b54bd54138e7a24bf8115d9aa482fcc26a7d23bd 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/AbstractExpressionTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/AbstractExpressionTests.java @@ -70,7 +70,7 @@ public abstract class AbstractExpressionTests { if (expectedValue == null) { return; // no point doing other checks } - assertEquals("Expression returned null value, but expected '" + expectedValue + "'", expectedValue, null); + assertNull("Expression returned null value, but expected '" + expectedValue + "'", expectedValue); } Class resultType = value.getClass(); @@ -100,7 +100,7 @@ public abstract class AbstractExpressionTests { if (expectedValue == null) { return; // no point doing other checks } - assertEquals("Expression returned null value, but expected '" + expectedValue + "'", expectedValue, null); + assertNull("Expression returned null value, but expected '" + expectedValue + "'", expectedValue); } Class resultType = value.getClass(); @@ -132,7 +132,7 @@ public abstract class AbstractExpressionTests { if (expectedValue == null) { return; // no point doing other checks } - assertEquals("Expression returned null value, but expected '" + expectedValue + "'", expectedValue, null); + assertNull("Expression returned null value, but expected '" + expectedValue + "'", expectedValue); } Class resultType = value.getClass(); if (expectedValue instanceof String) { @@ -142,8 +142,8 @@ public abstract class AbstractExpressionTests { else { assertEquals("Did not get expected value for expression '" + expression + "'.", expectedValue, value); } - assertEquals("Type of the result was not as expected. Expected '" + expectedClassOfResult + - "' but result was of type '" + resultType + "'", expectedClassOfResult.equals(resultType), true); + assertNull("Type of the result was not as expected. Expected '" + expectedClassOfResult + + "' but result was of type '" + resultType + "'", expectedClassOfResult.equals(resultType)); boolean isWritable = expr.isWritable(eContext); if (isWritable != shouldBeWritable) { diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/ConstructorInvocationTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/ConstructorInvocationTests.java index b0083e1c906dbc96877ef2e86ae3624e3be0aa1b..8f1da6801a7160a4031f80550bbffbb64afbc6cb 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/ConstructorInvocationTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/ConstructorInvocationTests.java @@ -103,7 +103,7 @@ public class ConstructorInvocationTests extends AbstractExpressionTests { eContext.setRootObject(new Tester()); eContext.setVariable("bar", 3); Object o = expr.getValue(eContext); - assertEquals(o, 3); + assertEquals(3, o); assertEquals(1, parser.parseExpression("counter").getValue(eContext)); // Now the expression has cached that throwException(int) is the right thing to diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/EvaluationTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/EvaluationTests.java index a338868f288f43a478e9ffe8361e4c838027796f..bc45bdd823fbeb4d298358fca268c26e97dde159 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/EvaluationTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/EvaluationTests.java @@ -535,7 +535,7 @@ public class EvaluationTests extends AbstractExpressionTests { int twentyFour = parser.parseExpression("2.0 * 3e0 * 4").getValue(Integer.class); assertEquals(24, twentyFour); double one = parser.parseExpression("8.0 / 5e0 % 2").getValue(Double.class); - assertEquals(1.6d, one, 0); + assertEquals(1.6d, one, 0d); int o = parser.parseExpression("8.0 / 5e0 % 2").getValue(Integer.class); assertEquals(1, o); int sixteen = parser.parseExpression("-2 ^ 4").getValue(Integer.class); diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/ExpressionLanguageScenarioTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/ExpressionLanguageScenarioTests.java index 72ffad73dae85cbba24e0a638e3b156ab1d495da..0ef096eb8bd1a622fbb2150b612872eb95d1e5bc 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/ExpressionLanguageScenarioTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/ExpressionLanguageScenarioTests.java @@ -219,7 +219,7 @@ public class ExpressionLanguageScenarioTests extends AbstractExpressionTests { fail("Should not be allowed to set oranges to be blue !"); } catch (SpelEvaluationException ee) { - assertEquals(ee.getMessageCode(), SpelMessage.PROPERTY_OR_FIELD_NOT_WRITABLE_ON_NULL); + assertEquals(SpelMessage.PROPERTY_OR_FIELD_NOT_WRITABLE_ON_NULL, ee.getMessageCode()); } } @@ -240,7 +240,7 @@ public class ExpressionLanguageScenarioTests extends AbstractExpressionTests { fail("Should not be allowed to set peas to be blue !"); } catch (SpelEvaluationException ee) { - assertEquals(ee.getMessageCode(), SpelMessage.PROPERTY_OR_FIELD_NOT_WRITABLE_ON_NULL); + assertEquals(SpelMessage.PROPERTY_OR_FIELD_NOT_WRITABLE_ON_NULL, ee.getMessageCode()); } } diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/LiteralExpressionTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/LiteralExpressionTests.java index d7f8eacf04b05ac3a9f14c5c8de5602c5138b7d4..89e78d31a63b4f03f985247bdbf91c67ee1b6689 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/LiteralExpressionTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/LiteralExpressionTests.java @@ -59,7 +59,7 @@ public class LiteralExpressionTests { } catch (EvaluationException ee) { // success, not allowed - whilst here, check the expression value in the exception - assertEquals(ee.getExpressionString(), "somevalue"); + assertEquals("somevalue", ee.getExpressionString()); } try { LiteralExpression lEx = new LiteralExpression("somevalue"); @@ -68,7 +68,7 @@ public class LiteralExpressionTests { } catch (EvaluationException ee) { // success, not allowed - whilst here, check the expression value in the exception - assertEquals(ee.getExpressionString(), "somevalue"); + assertEquals("somevalue", ee.getExpressionString()); } try { LiteralExpression lEx = new LiteralExpression("somevalue"); @@ -77,7 +77,7 @@ public class LiteralExpressionTests { } catch (EvaluationException ee) { // success, not allowed - whilst here, check the expression value in the exception - assertEquals(ee.getExpressionString(), "somevalue"); + assertEquals("somevalue", ee.getExpressionString()); } } diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/MethodInvocationTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/MethodInvocationTests.java index 2b0f9c1e4f6810cfef4b65be075364f159d6ab1d..61b5a437183676e371ac2f72e6030934b195f096 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/MethodInvocationTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/MethodInvocationTests.java @@ -98,7 +98,7 @@ public class MethodInvocationTests extends AbstractExpressionTests { StandardEvaluationContext eContext = TestScenarioCreator.getTestEvaluationContext(); eContext.setVariable("bar", 3); Object o = expr.getValue(eContext); - assertEquals(o, 3); + assertEquals(3, o); assertEquals(1, parser.parseExpression("counter").getValue(eContext)); // Now the expression has cached that throwException(int) is the right thing to call @@ -294,7 +294,7 @@ public class MethodInvocationTests extends AbstractExpressionTests { public void testMethodOfClass() throws Exception { Expression expression = parser.parseExpression("getName()"); Object value = expression.getValue(new StandardEvaluationContext(String.class)); - assertEquals(value, "java.lang.String"); + assertEquals("java.lang.String", value); } @Test diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/PropertyAccessTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/PropertyAccessTests.java index ac06f7d36210ef6a89413e5eb69fb3eafbfa70ae..f25bf974e035d48d69c9f3e2fcb1d4895cc8e82d 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/PropertyAccessTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/PropertyAccessTests.java @@ -115,7 +115,7 @@ public class PropertyAccessTests extends AbstractExpressionTests { ctx.addPropertyAccessor(new StringyPropertyAccessor()); Expression expr = parser.parseRaw("new String('hello').flibbles"); Integer i = expr.getValue(ctx, Integer.class); - assertEquals((int) i, 7); + assertEquals(7, (int) i); // The reflection one will be used for other properties... expr = parser.parseRaw("new String('hello').CASE_INSENSITIVE_ORDER"); @@ -125,7 +125,7 @@ public class PropertyAccessTests extends AbstractExpressionTests { expr = parser.parseRaw("new String('hello').flibbles"); expr.setValue(ctx, 99); i = expr.getValue(ctx, Integer.class); - assertEquals((int) i, 99); + assertEquals(99, (int) i); // Cannot set it to a string value try { @@ -165,7 +165,7 @@ public class PropertyAccessTests extends AbstractExpressionTests { public void testAccessingPropertyOfClass() throws Exception { Expression expression = parser.parseExpression("name"); Object value = expression.getValue(new StandardEvaluationContext(String.class)); - assertEquals(value, "java.lang.String"); + assertEquals("java.lang.String", value); } @Test diff --git a/spring-jdbc/src/test/java/org/springframework/jdbc/core/support/JdbcDaoSupportTests.java b/spring-jdbc/src/test/java/org/springframework/jdbc/core/support/JdbcDaoSupportTests.java index f2d4522ad9439fd5dc0c51aabdfe1a19b16355fa..af6de84c23e0ba0cea098f7e6bf9703ffa9f68b1 100644 --- a/spring-jdbc/src/test/java/org/springframework/jdbc/core/support/JdbcDaoSupportTests.java +++ b/spring-jdbc/src/test/java/org/springframework/jdbc/core/support/JdbcDaoSupportTests.java @@ -47,7 +47,7 @@ public class JdbcDaoSupportTests { dao.afterPropertiesSet(); assertEquals("Correct DataSource", ds, dao.getDataSource()); assertEquals("Correct JdbcTemplate", ds, dao.getJdbcTemplate().getDataSource()); - assertEquals("initDao called", test.size(), 1); + assertEquals("initDao called", 1, test.size()); } @Test @@ -63,7 +63,7 @@ public class JdbcDaoSupportTests { dao.setJdbcTemplate(template); dao.afterPropertiesSet(); assertEquals("Correct JdbcTemplate", dao.getJdbcTemplate(), template); - assertEquals("initDao called", test.size(), 1); + assertEquals("initDao called", 1, test.size()); } } diff --git a/spring-jdbc/src/test/java/org/springframework/jdbc/object/SqlQueryTests.java b/spring-jdbc/src/test/java/org/springframework/jdbc/object/SqlQueryTests.java index dfcd8d62a2cee0e63ca2b0da7f818816d1f8b9d0..4382ce989a879e568268ccd3cf1bdfc465d9ba44 100644 --- a/spring-jdbc/src/test/java/org/springframework/jdbc/object/SqlQueryTests.java +++ b/spring-jdbc/src/test/java/org/springframework/jdbc/object/SqlQueryTests.java @@ -615,7 +615,7 @@ public class SqlQueryTests { ids.add(2); List cust = query.findCustomers(ids); - assertEquals("We got two customers back", cust.size(), 2); + assertEquals("We got two customers back", 2, cust.size()); assertEquals("First customer id was assigned correctly", cust.get(0).getId(), 1); assertEquals("First customer forename was assigned correctly", cust.get(0).getForename(), "rod"); assertEquals("Second customer id was assigned correctly", cust.get(1).getId(), 2); @@ -664,7 +664,7 @@ public class SqlQueryTests { CustomerQuery query = new CustomerQuery(dataSource); List cust = query.findCustomers(1); - assertEquals("We got two customers back", cust.size(), 2); + assertEquals("We got two customers back", 2, cust.size()); assertEquals("First customer id was assigned correctly", cust.get(0).getId(), 1); assertEquals("First customer forename was assigned correctly", cust.get(0).getForename(), "rod"); assertEquals("Second customer id was assigned correctly", cust.get(1).getId(), 2); diff --git a/spring-jdbc/src/test/java/org/springframework/jdbc/object/StoredProcedureTests.java b/spring-jdbc/src/test/java/org/springframework/jdbc/object/StoredProcedureTests.java index 7df5f2da7c0797d5c9a1f82c8e5e65e017b12354..24b045e8cc20d0abed8b899f380bf5e54cf61342 100644 --- a/spring-jdbc/src/test/java/org/springframework/jdbc/object/StoredProcedureTests.java +++ b/spring-jdbc/src/test/java/org/springframework/jdbc/object/StoredProcedureTests.java @@ -194,7 +194,7 @@ public class StoredProcedureTests { t.setExceptionTranslator(new SQLStateSQLExceptionTranslator()); StoredProcedureConfiguredViaJdbcTemplate sp = new StoredProcedureConfiguredViaJdbcTemplate(t); - assertEquals(sp.execute(11), 5); + assertEquals(5, sp.execute(11)); assertEquals(1, t.calls); verify(callableStatement).setObject(1, 11, Types.INTEGER); @@ -216,7 +216,7 @@ public class StoredProcedureTests { JdbcTemplate t = new JdbcTemplate(); t.setDataSource(dataSource); StoredProcedureConfiguredViaJdbcTemplate sp = new StoredProcedureConfiguredViaJdbcTemplate(t); - assertEquals(sp.execute(1106), 4); + assertEquals(4, sp.execute(1106)); verify(callableStatement).setObject(1, 1106, Types.INTEGER); verify(callableStatement).registerOutParameter(2, Types.INTEGER); } diff --git a/spring-jms/src/test/java/org/springframework/jms/core/support/JmsGatewaySupportTests.java b/spring-jms/src/test/java/org/springframework/jms/core/support/JmsGatewaySupportTests.java index e3a33b29884af9f4082152e664672be9156301e0..09675b2e4feef38e87c4e7b66ce9b4159ae48ee3 100644 --- a/spring-jms/src/test/java/org/springframework/jms/core/support/JmsGatewaySupportTests.java +++ b/spring-jms/src/test/java/org/springframework/jms/core/support/JmsGatewaySupportTests.java @@ -46,7 +46,7 @@ public class JmsGatewaySupportTests { gateway.afterPropertiesSet(); assertEquals("Correct ConnectionFactory", mockConnectionFactory, gateway.getConnectionFactory()); assertEquals("Correct JmsTemplate", mockConnectionFactory, gateway.getJmsTemplate().getConnectionFactory()); - assertEquals("initGatway called", test.size(), 1); + assertEquals("initGatway called", 1, test.size()); } @Test @@ -62,7 +62,7 @@ public class JmsGatewaySupportTests { gateway.setJmsTemplate(template); gateway.afterPropertiesSet(); assertEquals("Correct JmsTemplate", template, gateway.getJmsTemplate()); - assertEquals("initGateway called", test.size(), 1); + assertEquals("initGateway called", 1, test.size()); } } diff --git a/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java b/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java index ec9029c8f697e5eda8112361067aa45ade1f47fe..0b5ca7eaaf83262b55a74535a7e78f7da18843a7 100644 --- a/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java +++ b/spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java @@ -301,7 +301,7 @@ public class MockHttpServletResponseTests { public void modifyStatusAfterSendError() throws IOException { response.sendError(HttpServletResponse.SC_NOT_FOUND); response.setStatus(HttpServletResponse.SC_OK); - assertEquals(response.getStatus(),HttpServletResponse.SC_NOT_FOUND); + assertEquals(HttpServletResponse.SC_NOT_FOUND, response.getStatus()); } @Test // SPR-10414 @@ -309,7 +309,7 @@ public class MockHttpServletResponseTests { public void modifyStatusMessageAfterSendError() throws IOException { response.sendError(HttpServletResponse.SC_NOT_FOUND); response.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR,"Server Error"); - assertEquals(response.getStatus(),HttpServletResponse.SC_NOT_FOUND); + assertEquals(HttpServletResponse.SC_NOT_FOUND, response.getStatus()); } } diff --git a/spring-tx/src/test/java/org/springframework/transaction/interceptor/TransactionAttributeEditorTests.java b/spring-tx/src/test/java/org/springframework/transaction/interceptor/TransactionAttributeEditorTests.java index f0d7285e25d03aaa35c20b9085b3df046b2b8b7f..cbfcabd0d672f1ee96095cfa25764b2cea5e1405 100644 --- a/spring-tx/src/test/java/org/springframework/transaction/interceptor/TransactionAttributeEditorTests.java +++ b/spring-tx/src/test/java/org/springframework/transaction/interceptor/TransactionAttributeEditorTests.java @@ -92,9 +92,9 @@ public class TransactionAttributeEditorTests { pe.setAsText("PROPAGATION_MANDATORY,ISOLATION_REPEATABLE_READ,timeout_10,-IOException,+MyRuntimeException"); TransactionAttribute ta = (TransactionAttribute) pe.getValue(); assertNotNull(ta); - assertEquals(ta.getPropagationBehavior(), TransactionDefinition.PROPAGATION_MANDATORY); - assertEquals(ta.getIsolationLevel(), TransactionDefinition.ISOLATION_REPEATABLE_READ); - assertEquals(ta.getTimeout(), 10); + assertEquals(TransactionDefinition.PROPAGATION_MANDATORY, ta.getPropagationBehavior()); + assertEquals(TransactionDefinition.ISOLATION_REPEATABLE_READ, ta.getIsolationLevel()); + assertEquals(10, ta.getTimeout()); assertFalse(ta.isReadOnly()); assertTrue(ta.rollbackOn(new RuntimeException())); assertFalse(ta.rollbackOn(new Exception())); @@ -109,9 +109,9 @@ public class TransactionAttributeEditorTests { pe.setAsText("+IOException,readOnly,ISOLATION_READ_COMMITTED,-MyRuntimeException,PROPAGATION_SUPPORTS"); TransactionAttribute ta = (TransactionAttribute) pe.getValue(); assertNotNull(ta); - assertEquals(ta.getPropagationBehavior(), TransactionDefinition.PROPAGATION_SUPPORTS); - assertEquals(ta.getIsolationLevel(), TransactionDefinition.ISOLATION_READ_COMMITTED); - assertEquals(ta.getTimeout(), TransactionDefinition.TIMEOUT_DEFAULT); + assertEquals(TransactionDefinition.PROPAGATION_SUPPORTS, ta.getPropagationBehavior()); + assertEquals(TransactionDefinition.ISOLATION_READ_COMMITTED, ta.getIsolationLevel()); + assertEquals(TransactionDefinition.TIMEOUT_DEFAULT, ta.getTimeout()); assertTrue(ta.isReadOnly()); assertTrue(ta.rollbackOn(new RuntimeException())); assertFalse(ta.rollbackOn(new Exception())); @@ -132,9 +132,9 @@ public class TransactionAttributeEditorTests { pe.setAsText(source.toString()); TransactionAttribute ta = (TransactionAttribute) pe.getValue(); assertEquals(ta, source); - assertEquals(ta.getPropagationBehavior(), TransactionDefinition.PROPAGATION_SUPPORTS); - assertEquals(ta.getIsolationLevel(), TransactionDefinition.ISOLATION_REPEATABLE_READ); - assertEquals(ta.getTimeout(), 10); + assertEquals(TransactionDefinition.PROPAGATION_SUPPORTS, ta.getPropagationBehavior()); + assertEquals(TransactionDefinition.ISOLATION_REPEATABLE_READ, ta.getIsolationLevel()); + assertEquals(10, ta.getTimeout()); assertTrue(ta.isReadOnly()); assertTrue(ta.rollbackOn(new RuntimeException())); assertFalse(ta.rollbackOn(new Exception())); @@ -159,9 +159,9 @@ public class TransactionAttributeEditorTests { pe.setAsText(source.toString()); TransactionAttribute ta = (TransactionAttribute) pe.getValue(); assertEquals(ta, source); - assertEquals(ta.getPropagationBehavior(), TransactionDefinition.PROPAGATION_SUPPORTS); - assertEquals(ta.getIsolationLevel(), TransactionDefinition.ISOLATION_REPEATABLE_READ); - assertEquals(ta.getTimeout(), 10); + assertEquals(TransactionDefinition.PROPAGATION_SUPPORTS, ta.getPropagationBehavior()); + assertEquals(TransactionDefinition.ISOLATION_REPEATABLE_READ, ta.getIsolationLevel()); + assertEquals(10, ta.getTimeout()); assertTrue(ta.isReadOnly()); assertTrue(ta.rollbackOn(new IllegalArgumentException())); assertFalse(ta.rollbackOn(new IllegalStateException())); diff --git a/spring-web/src/test/java/org/springframework/http/converter/json/SpringHandlerInstantiatorTests.java b/spring-web/src/test/java/org/springframework/http/converter/json/SpringHandlerInstantiatorTests.java index db37c5734c23853270bf4f49f09f9550d967d42a..be48c1f30056bf6b2952f7d5736da2368deddcd2 100644 --- a/spring-web/src/test/java/org/springframework/http/converter/json/SpringHandlerInstantiatorTests.java +++ b/spring-web/src/test/java/org/springframework/http/converter/json/SpringHandlerInstantiatorTests.java @@ -92,7 +92,7 @@ public class SpringHandlerInstantiatorTests { public void autowiredDeserializer() throws IOException { String json = "{\"username\":\"bob\"}"; User user = this.objectMapper.readValue(json, User.class); - assertEquals(user.getUsername(), "BOB"); + assertEquals("BOB", user.getUsername()); } @Test diff --git a/spring-web/src/test/java/org/springframework/remoting/caucho/CauchoRemotingTests.java b/spring-web/src/test/java/org/springframework/remoting/caucho/CauchoRemotingTests.java index 6b0d6a9595052260954a6c9e6c7ddb90cbc6da28..d6b9fd71bc1ff3d5e5dca8649ce32f5652b7db6f 100644 --- a/spring-web/src/test/java/org/springframework/remoting/caucho/CauchoRemotingTests.java +++ b/spring-web/src/test/java/org/springframework/remoting/caucho/CauchoRemotingTests.java @@ -99,8 +99,8 @@ public class CauchoRemotingTests { assertTrue(factory.getObject() instanceof ITestBean); ITestBean bean = (ITestBean) factory.getObject(); - assertEquals(proxyFactory.user, "test"); - assertEquals(proxyFactory.password, "bean"); + assertEquals("test", proxyFactory.user); + assertEquals("bean", proxyFactory.password); assertTrue(proxyFactory.overloadEnabled); exception.expect(RemoteAccessException.class); diff --git a/spring-web/src/test/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolverTests.java b/spring-web/src/test/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolverTests.java index 3bf9087e7186d8312594820226b32ad8c84b4250..657877e0962262acb7b9277b18b6afb912b7bfdc 100644 --- a/spring-web/src/test/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolverTests.java +++ b/spring-web/src/test/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolverTests.java @@ -62,7 +62,7 @@ public class MappingMediaTypeFileExtensionResolverTests { MappingMediaTypeFileExtensionResolver resolver = new MappingMediaTypeFileExtensionResolver(mapping); MediaType mediaType = resolver.lookupMediaType("JSON"); - assertEquals(mediaType, MediaType.APPLICATION_JSON); + assertEquals(MediaType.APPLICATION_JSON, mediaType); } } diff --git a/spring-web/src/test/java/org/springframework/web/context/request/async/WebAsyncManagerTests.java b/spring-web/src/test/java/org/springframework/web/context/request/async/WebAsyncManagerTests.java index b02fb64615879587e1a55c60bdc3386e94584014..91f974193896645e29dbede54f969e6bec64f205 100644 --- a/spring-web/src/test/java/org/springframework/web/context/request/async/WebAsyncManagerTests.java +++ b/spring-web/src/test/java/org/springframework/web/context/request/async/WebAsyncManagerTests.java @@ -66,7 +66,7 @@ public class WebAsyncManagerTests { fail("Expected exception"); } catch (IllegalStateException ex) { - assertEquals(ex.getMessage(), "AsyncWebRequest must not be null"); + assertEquals("AsyncWebRequest must not be null", ex.getMessage()); } try { @@ -74,7 +74,7 @@ public class WebAsyncManagerTests { fail("Expected exception"); } catch (IllegalStateException ex) { - assertEquals(ex.getMessage(), "AsyncWebRequest must not be null"); + assertEquals("AsyncWebRequest must not be null", ex.getMessage()); } } @@ -256,7 +256,7 @@ public class WebAsyncManagerTests { fail("Expected exception"); } catch (IllegalArgumentException ex) { - assertEquals(ex.getMessage(), "Callable must not be null"); + assertEquals("Callable must not be null", ex.getMessage()); } } @@ -357,7 +357,7 @@ public class WebAsyncManagerTests { fail("Expected exception"); } catch (IllegalArgumentException ex) { - assertEquals(ex.getMessage(), "DeferredResult must not be null"); + assertEquals("DeferredResult must not be null", ex.getMessage()); } } diff --git a/spring-web/src/test/java/org/springframework/web/jsf/DelegatingPhaseListenerTests.java b/spring-web/src/test/java/org/springframework/web/jsf/DelegatingPhaseListenerTests.java index 230b2c3f42f94662bf20ebfd3708a093f491e424..6bfb58be803eecbb5f4aa0590e0eb9281f995c39 100644 --- a/spring-web/src/test/java/org/springframework/web/jsf/DelegatingPhaseListenerTests.java +++ b/spring-web/src/test/java/org/springframework/web/jsf/DelegatingPhaseListenerTests.java @@ -51,7 +51,7 @@ public class DelegatingPhaseListenerTests { TestListener target = new TestListener(); beanFactory.addBean("testListener", target); - assertEquals(delPhaseListener.getPhaseId(), PhaseId.ANY_PHASE); + assertEquals(PhaseId.ANY_PHASE, delPhaseListener.getPhaseId()); PhaseEvent event = new PhaseEvent(facesContext, PhaseId.INVOKE_APPLICATION, new MockLifecycle()); delPhaseListener.beforePhase(event); @@ -68,7 +68,7 @@ public class DelegatingPhaseListenerTests { beanFactory.addBean("testListener1", target1); beanFactory.addBean("testListener2", target2); - assertEquals(delPhaseListener.getPhaseId(), PhaseId.ANY_PHASE); + assertEquals(PhaseId.ANY_PHASE, delPhaseListener.getPhaseId()); PhaseEvent event = new PhaseEvent(facesContext, PhaseId.INVOKE_APPLICATION, new MockLifecycle()); delPhaseListener.beforePhase(event); diff --git a/spring-web/src/test/java/org/springframework/web/util/HtmlUtilsTests.java b/spring-web/src/test/java/org/springframework/web/util/HtmlUtilsTests.java index 8f7c0d6e96f6361221a94ff77629922c7d447c00..2a3aed4283161c4692f546b690eafaa6489d6174 100644 --- a/spring-web/src/test/java/org/springframework/web/util/HtmlUtilsTests.java +++ b/spring-web/src/test/java/org/springframework/web/util/HtmlUtilsTests.java @@ -42,7 +42,7 @@ public class HtmlUtilsTests { public void testHtmlUnescape() { String escaped = ""This is a quote'"; String unescaped = HtmlUtils.htmlUnescape(escaped); - assertEquals(unescaped, "\"This is a quote'"); + assertEquals("\"This is a quote'", unescaped); } @Test diff --git a/spring-web/src/test/java/org/springframework/web/util/ServletContextPropertyUtilsTests.java b/spring-web/src/test/java/org/springframework/web/util/ServletContextPropertyUtilsTests.java index 21082779a4cea770868b96e60a5c4c5d7e1c3070..6a17f33f64d5ce11baf7c92e0610d21dc992dfee 100644 --- a/spring-web/src/test/java/org/springframework/web/util/ServletContextPropertyUtilsTests.java +++ b/spring-web/src/test/java/org/springframework/web/util/ServletContextPropertyUtilsTests.java @@ -31,7 +31,7 @@ public class ServletContextPropertyUtilsTests { MockServletContext servletContext = new MockServletContext(); servletContext.setInitParameter("test.prop", "bar"); String resolved = ServletContextPropertyUtils.resolvePlaceholders("${test.prop:foo}", servletContext); - assertEquals(resolved, "bar"); + assertEquals("bar", resolved); } @Test @@ -40,7 +40,7 @@ public class ServletContextPropertyUtilsTests { System.setProperty("test.prop", "bar"); try { String resolved = ServletContextPropertyUtils.resolvePlaceholders("${test.prop:foo}", servletContext); - assertEquals(resolved, "bar"); + assertEquals("bar", resolved); } finally { System.clearProperty("test.prop"); diff --git a/spring-web/src/test/java/org/springframework/web/util/TagUtilsTests.java b/spring-web/src/test/java/org/springframework/web/util/TagUtilsTests.java index 2ae57f4f5a73b3a304ad682cbb1daaa927dcd3e6..85c185dfbbed4e2373dd1f34e620d5b6ee295793 100644 --- a/spring-web/src/test/java/org/springframework/web/util/TagUtilsTests.java +++ b/spring-web/src/test/java/org/springframework/web/util/TagUtilsTests.java @@ -34,10 +34,10 @@ public class TagUtilsTests { @Test public void getScopeSunnyDay() { - assertEquals(TagUtils.SCOPE_PAGE, "page"); - assertEquals(TagUtils.SCOPE_APPLICATION, "application"); - assertEquals(TagUtils.SCOPE_SESSION, "session"); - assertEquals(TagUtils.SCOPE_REQUEST, "request"); + assertEquals("page", TagUtils.SCOPE_PAGE); + assertEquals("application", TagUtils.SCOPE_APPLICATION); + assertEquals("session", TagUtils.SCOPE_SESSION); + assertEquals("request", TagUtils.SCOPE_REQUEST); assertEquals(PageContext.PAGE_SCOPE, TagUtils.getScope("page")); assertEquals(PageContext.REQUEST_SCOPE, TagUtils.getScope("request")); diff --git a/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/UrlBasedViewResolverTests.java b/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/UrlBasedViewResolverTests.java index 84296cfd803fa7cc8574d26c9688a592fb3aca1d..223f9aa6f4f7575e5d0022310d5c8ceff1f5fa55 100644 --- a/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/UrlBasedViewResolverTests.java +++ b/spring-webflux/src/test/java/org/springframework/web/reactive/result/view/UrlBasedViewResolverTests.java @@ -72,8 +72,8 @@ public class UrlBasedViewResolverTests { .consumeNextWith(view -> { assertEquals(RedirectView.class, view.getClass()); RedirectView redirectView = (RedirectView) view; - assertEquals(redirectView.getUrl(), "foo"); - assertEquals(redirectView.getStatusCode(), HttpStatus.SEE_OTHER); + assertEquals("foo", redirectView.getUrl()); + assertEquals(HttpStatus.SEE_OTHER, redirectView.getStatusCode()); }) .expectComplete() .verify(Duration.ZERO); @@ -88,8 +88,8 @@ public class UrlBasedViewResolverTests { .consumeNextWith(view -> { assertEquals(RedirectView.class, view.getClass()); RedirectView redirectView = (RedirectView) view; - assertEquals(redirectView.getUrl(), "foo"); - assertEquals(redirectView.getStatusCode(), HttpStatus.FOUND); + assertEquals("foo", redirectView.getUrl()); + assertEquals(HttpStatus.FOUND, redirectView.getStatusCode()); }) .expectComplete() .verify(Duration.ZERO); diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/CookieLocaleResolverTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/CookieLocaleResolverTests.java index 37f8c9b38a8001526f9cb63aa6ca6fc16f4edb07..a03c0dd8f449e4a958c5ddcfa3ca05e4a3f6f4be 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/CookieLocaleResolverTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/CookieLocaleResolverTests.java @@ -49,7 +49,7 @@ public class CookieLocaleResolverTests { CookieLocaleResolver resolver = new CookieLocaleResolver(); resolver.setCookieName("LanguageKoekje"); Locale loc = resolver.resolveLocale(request); - assertEquals(loc.getLanguage(), "nl"); + assertEquals("nl", loc.getLanguage()); } @Test diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/LocaleResolverTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/LocaleResolverTests.java index 99b790eca7376658151ba7d06a8827aa28a50ecf..74ad302ebad6f1d40ca0a0c87f9b5acd60472144 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/LocaleResolverTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/i18n/LocaleResolverTests.java @@ -68,7 +68,7 @@ public class LocaleResolverTests { // check original locale Locale locale = localeResolver.resolveLocale(request); - assertEquals(locale, Locale.UK); + assertEquals(Locale.UK, locale); // set new locale try { localeResolver.setLocale(request, response, Locale.GERMANY); @@ -76,7 +76,7 @@ public class LocaleResolverTests { fail("should not be able to set Locale"); // check new locale locale = localeResolver.resolveLocale(request); - assertEquals(locale, Locale.GERMANY); + assertEquals(Locale.GERMANY, locale); } catch (UnsupportedOperationException ex) { if (shouldSet) { @@ -89,10 +89,10 @@ public class LocaleResolverTests { LocaleContextResolver localeContextResolver = (LocaleContextResolver) localeResolver; LocaleContext localeContext = localeContextResolver.resolveLocaleContext(request); if (shouldSet) { - assertEquals(localeContext.getLocale(), Locale.GERMANY); + assertEquals(Locale.GERMANY, localeContext.getLocale()); } else { - assertEquals(localeContext.getLocale(), Locale.UK); + assertEquals(Locale.UK, localeContext.getLocale()); } assertTrue(localeContext instanceof TimeZoneAwareLocaleContext); assertNull(((TimeZoneAwareLocaleContext) localeContext).getTimeZone()); @@ -108,7 +108,7 @@ public class LocaleResolverTests { fail("should not be able to set Locale"); } localeContext = localeContextResolver.resolveLocaleContext(request); - assertEquals(localeContext.getLocale(), Locale.US); + assertEquals(Locale.US, localeContext.getLocale()); if (localeContextResolver instanceof AbstractLocaleContextResolver) { assertEquals(((TimeZoneAwareLocaleContext) localeContext).getTimeZone(), TimeZone.getTimeZone("GMT+1")); } @@ -119,20 +119,20 @@ public class LocaleResolverTests { localeContextResolver.setLocaleContext(request, response, new SimpleTimeZoneAwareLocaleContext(Locale.GERMANY, TimeZone.getTimeZone("GMT+2"))); localeContext = localeContextResolver.resolveLocaleContext(request); - assertEquals(localeContext.getLocale(), Locale.GERMANY); + assertEquals(Locale.GERMANY, localeContext.getLocale()); assertTrue(localeContext instanceof TimeZoneAwareLocaleContext); assertEquals(((TimeZoneAwareLocaleContext) localeContext).getTimeZone(), TimeZone.getTimeZone("GMT+2")); localeContextResolver.setLocaleContext(request, response, new SimpleTimeZoneAwareLocaleContext(null, TimeZone.getTimeZone("GMT+3"))); localeContext = localeContextResolver.resolveLocaleContext(request); - assertEquals(localeContext.getLocale(), Locale.UK); + assertEquals(Locale.UK, localeContext.getLocale()); assertTrue(localeContext instanceof TimeZoneAwareLocaleContext); assertEquals(((TimeZoneAwareLocaleContext) localeContext).getTimeZone(), TimeZone.getTimeZone("GMT+3")); if (localeContextResolver instanceof AbstractLocaleContextResolver) { ((AbstractLocaleContextResolver) localeContextResolver).setDefaultLocale(Locale.GERMANY); - assertEquals(localeContext.getLocale(), Locale.GERMANY); + assertEquals(Locale.GERMANY, localeContext.getLocale()); } } catch (UnsupportedOperationException ex) { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/BindTagTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/BindTagTests.java index 0ad26ff580ca7d6f7781cd60b75d652851639d95..87334a976bb71d435ab0bc1d8cf68b55f9369a2b 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/BindTagTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/BindTagTests.java @@ -195,7 +195,7 @@ public class BindTagTests extends AbstractTagTests { tag.doStartTag(); BindStatus status = (BindStatus) pc.getAttribute(BindTag.STATUS_VARIABLE_NAME, PageContext.REQUEST_SCOPE); assertEquals("Error messages String should be 'message1'", - status.getErrorMessagesAsString(","), "message1"); + "message1", status.getErrorMessagesAsString(",")); // two errors pc = createPageContext(); @@ -209,7 +209,7 @@ public class BindTagTests extends AbstractTagTests { tag.doStartTag(); status = (BindStatus) pc.getAttribute(BindTag.STATUS_VARIABLE_NAME, PageContext.REQUEST_SCOPE); assertEquals("Error messages String should be 'message1,message2'", - status.getErrorMessagesAsString(","), "message1,message2"); + "message1,message2", status.getErrorMessagesAsString(",")); // no errors pc = createPageContext(); @@ -220,7 +220,7 @@ public class BindTagTests extends AbstractTagTests { tag.setPath("tb"); tag.doStartTag(); status = (BindStatus) pc.getAttribute(BindTag.STATUS_VARIABLE_NAME, PageContext.REQUEST_SCOPE); - assertEquals("Error messages String should be ''", status.getErrorMessagesAsString(","), ""); + assertEquals("Error messages String should be ''", "", status.getErrorMessagesAsString(",")); } @Test