diff --git a/spring-aop/src/testFixtures/java/org/springframework/aop/testfixture/mixin/LockMixin.java b/spring-aop/src/testFixtures/java/org/springframework/aop/testfixture/mixin/LockMixin.java index 33ffc13f39f..6c08f24ce2f 100644 --- a/spring-aop/src/testFixtures/java/org/springframework/aop/testfixture/mixin/LockMixin.java +++ b/spring-aop/src/testFixtures/java/org/springframework/aop/testfixture/mixin/LockMixin.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2023 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -44,9 +44,6 @@ public class LockMixin extends DelegatingIntroductionInterceptor implements Lock this.locked = false; } - /** - * @see test.mixin.AopProxyTests.Lockable#locked() - */ @Override public boolean locked() { return this.locked; @@ -54,10 +51,8 @@ public class LockMixin extends DelegatingIntroductionInterceptor implements Lock /** * Note that we need to override around advice. - * If the method is a setter and we're locked, prevent execution. - * Otherwise let super.invoke() handle it, and do normal - * Lockable(this) then target behaviour. - * @see org.aopalliance.MethodInterceptor#invoke(org.aopalliance.MethodInvocation) + * If the method is a setter, and we're locked, prevent execution. + * Otherwise, let super.invoke() handle it. */ @Override public Object invoke(MethodInvocation invocation) throws Throwable { diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/support/MethodDescriptor.java b/spring-beans/src/main/java/org/springframework/beans/factory/support/MethodDescriptor.java index c895734f0ec..6b0e4e90a92 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/support/MethodDescriptor.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/support/MethodDescriptor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2023 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -16,10 +16,12 @@ package org.springframework.beans.factory.support; +import java.lang.reflect.Method; + import org.springframework.util.ClassUtils; /** - * Descriptor for a {@link java.lang.reflect.Method Method} which holds a + * Descriptor for a {@link Method Method} which holds a * reference to the method's {@linkplain #declaringClass declaring class}, * {@linkplain #methodName name}, and {@linkplain #parameterTypes parameter types}. * @@ -34,8 +36,7 @@ record MethodDescriptor(Class declaringClass, String methodName, Class... /** * Create a {@link MethodDescriptor} for the supplied bean class and method name. *

The supplied {@code methodName} may be a {@linkplain Method#getName() - * simple method name} or a - * {@linkplain org.springframework.util.ClassUtils#getQualifiedMethodName(Method) + * simple method name} or a {@linkplain ClassUtils#getQualifiedMethodName(Method) * qualified method name}. *

If the method name is fully qualified, this utility will parse the * method name and its declaring class from the qualified method name and then diff --git a/spring-beans/src/test/java/org/springframework/beans/ExtendedBeanInfoTests.java b/spring-beans/src/test/java/org/springframework/beans/ExtendedBeanInfoTests.java index 6bd565c4dfc..99682ea16e0 100644 --- a/spring-beans/src/test/java/org/springframework/beans/ExtendedBeanInfoTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/ExtendedBeanInfoTests.java @@ -554,7 +554,7 @@ class ExtendedBeanInfoTests { * IntrospectionException regarding a "type mismatch between indexed and non-indexed * methods" intermittently (approximately one out of every four times) under JDK 7 * due to non-deterministic results from {@link Class#getDeclaredMethods()}. - * See https://bugs.java.com/bugdatabase/view_bug.do?bug_id=7023180 + * @see JDK-7023180 * @see #cornerSpr9702() */ @Test diff --git a/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java b/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java index b37b120868d..c2bd54988db 100644 --- a/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java +++ b/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java @@ -266,8 +266,8 @@ public final class BridgeMethodResolver { /** * Compare the signatures of the bridge method and the method which it bridges. If * the parameter and return types are the same, it is a 'visibility' bridge method - * introduced in Java 6 to fix https://bugs.openjdk.org/browse/JDK-6342411. - * See also https://stas-blogspot.blogspot.com/2010/03/java-bridge-methods-explained.html + * introduced in Java 6 to fix + * JDK-6342411. * @return whether signatures match as described */ public static boolean isVisibilityBridgeMethodPair(Method bridgeMethod, Method bridgedMethod) { diff --git a/spring-core/src/test/java/org/springframework/core/annotation/NestedRepeatableAnnotationsTests.java b/spring-core/src/test/java/org/springframework/core/annotation/NestedRepeatableAnnotationsTests.java index 2f99db0018d..6e20b597361 100644 --- a/spring-core/src/test/java/org/springframework/core/annotation/NestedRepeatableAnnotationsTests.java +++ b/spring-core/src/test/java/org/springframework/core/annotation/NestedRepeatableAnnotationsTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,7 +39,6 @@ import static org.assertj.core.api.Assertions.assertThat; * * @author Sam Brannen * @since 5.3.24 - * @see https://github.com/spring-projects/spring-framework/issues/20279 */ @SuppressWarnings("unused") class NestedRepeatableAnnotationsTests { diff --git a/spring-expression/src/main/java/org/springframework/expression/spel/ast/FunctionReference.java b/spring-expression/src/main/java/org/springframework/expression/spel/ast/FunctionReference.java index 19c145d0ccf..0387b65ef33 100644 --- a/spring-expression/src/main/java/org/springframework/expression/spel/ast/FunctionReference.java +++ b/spring-expression/src/main/java/org/springframework/expression/spel/ast/FunctionReference.java @@ -163,8 +163,8 @@ public class FunctionReference extends SpelNodeImpl { /** * Execute a function represented as {@link MethodHandle}. *

Method types that take no arguments (fully bound handles or static methods - * with no parameters) can use {@link MethodHandle#invoke()} which is the most - * efficient. Otherwise, {@link MethodHandle#invokeWithArguments()} is used. + * with no parameters) can use {@link MethodHandle#invoke(Object...)} which is the most + * efficient. Otherwise, {@link MethodHandle#invokeWithArguments(Object...)} is used. * @param state the expression evaluation state * @param methodHandle the method handle to invoke * @return the return value of the invoked Java method diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/ScenariosForSpringSecurityExpressionTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/ScenariosForSpringSecurityExpressionTests.java index 4db9898a230..6f5b42af3c9 100644 --- a/spring-expression/src/test/java/org/springframework/expression/spel/ScenariosForSpringSecurityExpressionTests.java +++ b/spring-expression/src/test/java/org/springframework/expression/spel/ScenariosForSpringSecurityExpressionTests.java @@ -37,11 +37,11 @@ import org.springframework.expression.spel.support.StandardEvaluationContext; import static org.assertj.core.api.Assertions.assertThat; -///CLOVER:OFF /** - * Spring Security scenarios from https://docs.spring.io/spring-security/reference/servlet/authorization/expression-based.html + * SpEL expression tests for Spring Security scenarios. * * @author Andy Clement + * @see Expressing Authorization with SpEL */ class ScenariosForSpringSecurityExpressionTests extends AbstractExpressionTests { diff --git a/spring-jdbc/src/test/java/org/springframework/jdbc/datasource/init/H2DatabasePopulatorTests.java b/spring-jdbc/src/test/java/org/springframework/jdbc/datasource/init/H2DatabasePopulatorTests.java index 1da75c22dc0..e3fb9fce4ed 100644 --- a/spring-jdbc/src/test/java/org/springframework/jdbc/datasource/init/H2DatabasePopulatorTests.java +++ b/spring-jdbc/src/test/java/org/springframework/jdbc/datasource/init/H2DatabasePopulatorTests.java @@ -43,12 +43,7 @@ class H2DatabasePopulatorTests extends AbstractDatabasePopulatorTests { return EmbeddedDatabaseType.H2; } - /** - * https://jira.spring.io/browse/SPR-15896 - * - * @since 5.0 - */ - @Test + @Test // SPR-15896 void scriptWithH2Alias() { databasePopulator.addScript(usersSchema()); databasePopulator.addScript(resource("db-test-data-h2-alias.sql")); @@ -61,12 +56,7 @@ class H2DatabasePopulatorTests extends AbstractDatabasePopulatorTests { assertThat(jdbcTemplate.queryForObject(sql, String.class)).isEqualTo("maS"); } - /** - * https://github.com/spring-projects/spring-framework/issues/27008 - * - * @since 5.3.11 - */ - @Test + @Test // gh-27008 void automaticallyCommitsIfAutoCommitIsDisabled() throws Exception { EmbeddedDatabase database = null; try { diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/SimpMessagingTemplate.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/SimpMessagingTemplate.java index 94c35515b33..fdd5f357a77 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/simp/SimpMessagingTemplate.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/simp/SimpMessagingTemplate.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2023 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -130,8 +130,8 @@ public class SimpMessagingTemplate extends AbstractMessageSendingTemplateIf a destination header is not already present ,the message is sent - * to the configured {@link #setDefaultDestination(Object) defaultDestination} - * or an exception an {@code IllegalStateException} is raised if that isn't + * to the configured {@link AbstractMessageSendingTemplate#setDefaultDestination(Object) + * defaultDestination} or an {@code IllegalStateException} is raised if that isn't * configured. * @param message the message to send (never {@code null}) */ diff --git a/spring-test/src/test/java/org/springframework/test/context/event/DirtiesContextEventPublishingTests.java b/spring-test/src/test/java/org/springframework/test/context/event/DirtiesContextEventPublishingTests.java index e6dab3c4f75..47b9e4c95fb 100644 --- a/spring-test/src/test/java/org/springframework/test/context/event/DirtiesContextEventPublishingTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/event/DirtiesContextEventPublishingTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -49,7 +49,7 @@ import static org.junit.platform.engine.discovery.DiscoverySelectors.selectClass * * @author Sam Brannen * @since 5.3.17 - * @see https://github.com/spring-projects/spring-framework/issues/27757 + * @see gh-27757 */ class DirtiesContextEventPublishingTests { diff --git a/spring-test/src/test/java/org/springframework/test/context/event/EagerTestExecutionEventPublishingTests.java b/spring-test/src/test/java/org/springframework/test/context/event/EagerTestExecutionEventPublishingTests.java index fffc73d1d4b..feaa2c936f5 100644 --- a/spring-test/src/test/java/org/springframework/test/context/event/EagerTestExecutionEventPublishingTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/event/EagerTestExecutionEventPublishingTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -50,7 +50,6 @@ import static org.junit.platform.engine.discovery.DiscoverySelectors.selectClass * * @author Sam Brannen * @since 5.3.17 - * @see https://github.com/spring-projects/spring-framework/issues/27757 */ class EagerTestExecutionEventPublishingTests { diff --git a/spring-test/src/test/java/org/springframework/test/context/jdbc/PrimaryDataSourceTests.java b/spring-test/src/test/java/org/springframework/test/context/jdbc/PrimaryDataSourceTests.java index 0db28f87624..d4cd1bae69c 100644 --- a/spring-test/src/test/java/org/springframework/test/context/jdbc/PrimaryDataSourceTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/jdbc/PrimaryDataSourceTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2019 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -39,7 +39,7 @@ import static org.springframework.test.transaction.TransactionAssert.assertThatT * * @author Sam Brannen * @since 4.3 - * @see org.springframework.test.context.transaction.PrimaryTransactionManagerTests + * @see org.springframework.test.context.transaction.manager.PrimaryTransactionManagerTests */ @SpringJUnitConfig @DirtiesContext diff --git a/spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleTests.java b/spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleSpr15927Tests.java similarity index 92% rename from spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleTests.java rename to spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleSpr15927Tests.java index 654fed92f62..e4b26bfabd0 100644 --- a/spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/junit4/rules/AutowiredRuleSpr15927Tests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2019 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -30,12 +30,12 @@ import org.springframework.context.annotation.Configuration; import static org.assertj.core.api.Assertions.assertThat; /** - * Integration tests for an issue raised in https://jira.spring.io/browse/SPR-15927. + * Integration tests for {@code SPR-15927}. * * @author Sam Brannen * @since 5.0 */ -public class AutowiredRuleTests { +public class AutowiredRuleSpr15927Tests { @ClassRule public static final SpringClassRule springClassRule = new SpringClassRule(); diff --git a/spring-test/src/test/java/org/springframework/test/context/support/ContextLoaderUtilsContextHierarchyTests.java b/spring-test/src/test/java/org/springframework/test/context/support/ContextLoaderUtilsContextHierarchyTests.java index 5e136e53c62..26769c38e1f 100644 --- a/spring-test/src/test/java/org/springframework/test/context/support/ContextLoaderUtilsContextHierarchyTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/support/ContextLoaderUtilsContextHierarchyTests.java @@ -327,10 +327,7 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati assertContextConfigEntriesAreNotUnique(SingleTestClassWithMultiLevelContextHierarchyWithDuplicatedContextConfig.class); } - /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 - */ - @Test + @Test // SPR-10997 void buildContextHierarchyMapForTestClassHierarchyWithMultiLevelContextHierarchiesAndOverriddenInitializers() { Map> map = buildContextHierarchyMap(TestClass2WithMultiLevelContextHierarchyWithOverriddenInitializers.class); @@ -504,32 +501,24 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati private static class SingleTestClassWithMultiLevelContextHierarchyWithDuplicatedContextConfig { } - /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 - */ @ContextHierarchy({// // @ContextConfiguration(name = "alpha", locations = "1-A.xml"),// @ContextConfiguration(name = "beta", locations = "1-B.xml") // - }) + }) // SPR-10997 private static class TestClass1WithMultiLevelContextHierarchyWithUniqueContextConfig { } - /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 - */ @ContextHierarchy({// // @ContextConfiguration(name = "alpha", initializers = DummyApplicationContextInitializer.class),// @ContextConfiguration(name = "beta", initializers = DummyApplicationContextInitializer.class) // - }) + }) // SPR-10997 private static class TestClass2WithMultiLevelContextHierarchyWithOverriddenInitializers extends TestClass1WithMultiLevelContextHierarchyWithUniqueContextConfig { } - /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 - */ + // SPR-10997 private static class DummyApplicationContextInitializer implements ApplicationContextInitializer { diff --git a/spring-web/src/jmh/java/org/springframework/web/bind/ServletRequestUtilsBenchmark.java b/spring-web/src/jmh/java/org/springframework/web/bind/ServletRequestUtilsBenchmark.java index f23f000afc4..47dcb1a5994 100644 --- a/spring-web/src/jmh/java/org/springframework/web/bind/ServletRequestUtilsBenchmark.java +++ b/spring-web/src/jmh/java/org/springframework/web/bind/ServletRequestUtilsBenchmark.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2020 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -16,6 +16,7 @@ package org.springframework.web.bind; +import jakarta.servlet.ServletRequest; import org.openjdk.jmh.annotations.Benchmark; import org.openjdk.jmh.annotations.BenchmarkMode; import org.openjdk.jmh.annotations.Mode; @@ -26,7 +27,8 @@ import org.springframework.web.testfixture.servlet.MockHttpServletRequest; /** - * Benchmarks for extracting parameters from {@libnk ServletRequest}. + * Benchmarks for extracting parameters from {@link ServletRequest}. + * * @author Brian Clozel */ @BenchmarkMode(Mode.Throughput) diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java b/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java index 0223f1f274a..957e4fe51fa 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -69,8 +69,7 @@ public abstract class AbstractListenerWriteProcessor implements ProcessorA complete description of the HTML 4.0 character set can be found - * at https://www.w3.org/TR/html4/charset.html. + * at https://www.w3.org/TR/html4/charset.html. * * @author Juergen Hoeller * @author Martin Kersten diff --git a/spring-web/src/test/java/org/springframework/http/WebBeanUtilsRuntimeHintsTests.java b/spring-web/src/test/java/org/springframework/http/WebBeanUtilsRuntimeHintsTests.java index cad56880200..1f260c993dc 100644 --- a/spring-web/src/test/java/org/springframework/http/WebBeanUtilsRuntimeHintsTests.java +++ b/spring-web/src/test/java/org/springframework/http/WebBeanUtilsRuntimeHintsTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2023 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -29,7 +29,7 @@ import org.springframework.util.ClassUtils; import static org.assertj.core.api.Assertions.assertThat; /** - * Web-related tests for {@link BeanUtilsRuntimeHints}. + * Web-related tests for {@link org.springframework.beans.BeanUtilsRuntimeHints}. * * @author Sebastien Deleuze * @since 6.0.10 diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/view/document/AbstractPdfView.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/view/document/AbstractPdfView.java index 8cac515a6e4..019488099c6 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/view/document/AbstractPdfView.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/view/document/AbstractPdfView.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2018 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -138,7 +138,7 @@ public abstract class AbstractPdfView extends AbstractView { * The subclass can either have fixed preferences or retrieve * them from bean properties defined on the View. * @return an int containing the bits information against PdfWriter definitions - * @see com.lowagie.text.pdf.PdfWriter#AllowPrinting + * @see com.lowagie.text.pdf.PdfWriter#ALLOW_PRINTING * @see com.lowagie.text.pdf.PdfWriter#PageLayoutSinglePage */ protected int getViewerPreferences() { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/UrlFilenameViewControllerTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/UrlFilenameViewControllerTests.java index b1762e6523d..b17ab58cd08 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/UrlFilenameViewControllerTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/UrlFilenameViewControllerTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2023 the original author or authors. + * Copyright 2002-2024 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -170,9 +170,8 @@ class UrlFilenameViewControllerTests { /** * This is the expected behavior, and it now has a test to prove it. - * https://opensource.atlassian.com/projects/spring/browse/SPR-2789 */ - @PathPatternsParameterizedTest + @PathPatternsParameterizedTest // SPR-2789 void nestedPathisUsedAsViewName_InBreakingChangeFromSpring12Line( Function requestFactory) throws Exception { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/ErrorsTagTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/ErrorsTagTests.java index 81d56abb711..34c1d96d799 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/ErrorsTagTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/ErrorsTagTests.java @@ -355,10 +355,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { assertThat(getPageContext().getAttribute(ErrorsTag.MESSAGES_ATTRIBUTE)).isEqualTo(existingAttribute); } - /** - * https://jira.spring.io/browse/SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithErrorsAndExistingMessagesAttributeInNonPageScopeAreNotClobbered() throws Exception { String existingAttribute = "something"; getPageContext().setAttribute(ErrorsTag.MESSAGES_ATTRIBUTE, existingAttribute, PageContext.APPLICATION_SCOPE); @@ -379,42 +376,27 @@ public class ErrorsTagTests extends AbstractFormTagTests { assertThat(getPageContext().getAttribute(ErrorsTag.MESSAGES_ATTRIBUTE, PageContext.APPLICATION_SCOPE)).isEqualTo(existingAttribute); } - /** - * https://jira.spring.io/browse/SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInApplicationScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.APPLICATION_SCOPE); } - /** - * https://jira.spring.io/browse/SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInSessionScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.SESSION_SCOPE); } - /** - * https://jira.spring.io/browse/SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInPageScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.PAGE_SCOPE); } - /** - * https://jira.spring.io/browse/SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInRequestScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.REQUEST_SCOPE); } - /** - * https://jira.spring.io/browse/SPR-4005 - */ - @Test + @Test // SPR-4005 void omittedPathMatchesObjectErrorsOnly() throws Exception { this.tag.setPath(null); Errors errors = new BeanPropertyBindingResult(new TestBean(), "COMMAND_NAME"); diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/PasswordInputTagTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/PasswordInputTagTests.java index f009c85621b..cb1cfaacbb7 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/PasswordInputTagTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/tags/form/PasswordInputTagTests.java @@ -31,10 +31,7 @@ import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException */ class PasswordInputTagTests extends InputTagTests { - /** - * https://jira.spring.io/browse/SPR-2866 - */ - @Test + @Test // SPR-2866 void passwordValueIsNotRenderedByDefault() throws Exception { this.getTag().setPath("name"); @@ -48,10 +45,7 @@ class PasswordInputTagTests extends InputTagTests { assertValueAttribute(output, ""); } - /** - * https://jira.spring.io/browse/SPR-2866 - */ - @Test + @Test // SPR-2866 void passwordValueIsRenderedIfShowPasswordAttributeIsSetToTrue() throws Exception { this.getTag().setPath("name"); this.getPasswordTag().setShowPassword(true); @@ -66,10 +60,7 @@ class PasswordInputTagTests extends InputTagTests { assertValueAttribute(output, "Rob"); } - /** - * https://jira.spring.io/browse/SPR-2866 - */ - @Test + @Test // >SPR-2866 void passwordValueIsNotRenderedIfShowPasswordAttributeIsSetToFalse() throws Exception { this.getTag().setPath("name"); this.getPasswordTag().setShowPassword(false);