From abdccffa39d11b3f4bbdbe3a1563b321d9d6509c Mon Sep 17 00:00:00 2001 From: Maksim Sasnouski Date: Fri, 8 Mar 2024 22:44:39 +0300 Subject: [PATCH 1/2] Fix Javadoc This commit fixes various Javadoc issues across the code base. See gh-32403 --- .../aop/testfixture/mixin/LockMixin.java | 6 +++--- .../beans/factory/support/MethodDescriptor.java | 6 +++--- .../springframework/beans/ExtendedBeanInfoTests.java | 2 +- .../springframework/core/BridgeMethodResolver.java | 4 ++-- .../annotation/NestedRepeatableAnnotationsTests.java | 4 ++-- .../expression/spel/ast/FunctionReference.java | 4 ++-- .../ScenariosForSpringSecurityExpressionTests.java | 2 +- .../datasource/init/H2DatabasePopulatorTests.java | 4 ++-- .../messaging/simp/SimpMessagingTemplate.java | 4 ++-- .../event/DirtiesContextEventPublishingTests.java | 4 ++-- .../EagerTestExecutionEventPublishingTests.java | 4 ++-- .../test/context/jdbc/PrimaryDataSourceTests.java | 4 ++-- .../context/junit4/rules/AutowiredRuleTests.java | 4 ++-- .../ContextLoaderUtilsContextHierarchyTests.java | 8 ++++---- .../web/bind/ServletRequestUtilsBenchmark.java | 4 ++-- .../reactive/AbstractListenerWriteProcessor.java | 4 ++-- .../web/util/HtmlCharacterEntityReferences.java | 4 ++-- .../http/WebBeanUtilsRuntimeHintsTests.java | 4 ++-- .../web/servlet/view/document/AbstractPdfView.java | 4 ++-- .../servlet/mvc/UrlFilenameViewControllerTests.java | 4 ++-- .../web/servlet/tags/form/ErrorsTagTests.java | 12 ++++++------ .../web/servlet/tags/form/PasswordInputTagTests.java | 6 +++--- 22 files changed, 51 insertions(+), 51 deletions(-) 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 33ffc13f39..7d4ec1a6be 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. @@ -45,7 +45,7 @@ public class LockMixin extends DelegatingIntroductionInterceptor implements Lock } /** - * @see test.mixin.AopProxyTests.Lockable#locked() + * @see org.springframework.aop.testfixture.mixin.Lockable#locked() */ @Override public boolean locked() { @@ -57,7 +57,7 @@ public class LockMixin extends DelegatingIntroductionInterceptor implements Lock * 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) + * @see org.aopalliance.intercept.MethodInterceptor#invoke(MethodInvocation) */ @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 c895734f0e..03ddf5c224 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. @@ -33,9 +33,9 @@ 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() + *

The supplied {@code methodName} may be a {@linkplain java.lang.reflect.Method#getName() * simple method name} or a - * {@linkplain org.springframework.util.ClassUtils#getQualifiedMethodName(Method) + * {@linkplain org.springframework.util.ClassUtils#getQualifiedMethodName(java.lang.reflect.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 6bd565c4df..a648bf23cf 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 : Change in specified-to-be-unspecified ordering of getDeclaredMethods causes application problems * @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 b37b120868..d4ad950520 100644 --- a/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java +++ b/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java @@ -266,9 +266,9 @@ 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 + * @see Java bridge methods explained */ public static boolean isVisibilityBridgeMethodPair(Method bridgeMethod, Method bridgedMethod) { if (bridgeMethod == 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 2f99db0018..b6e88e7c13 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,7 @@ 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 + * @see gh-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 19c145d0cc..1bcc196f96 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... var1)} which is the most + * efficient. Otherwise, {@link MethodHandle#invokeWithArguments(Object... arguments)} 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 4db9898a23..8d57a86491 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 @@ -39,7 +39,7 @@ 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 + * Spring Security scenarios from Expressing Authorization with SpEL * * @author Andy Clement */ 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 1da75c22dc..1f897a8571 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 @@ -44,7 +44,7 @@ class H2DatabasePopulatorTests extends AbstractDatabasePopulatorTests { } /** - * https://jira.spring.io/browse/SPR-15896 + * SPR-15896 * * @since 5.0 */ @@ -62,7 +62,7 @@ class H2DatabasePopulatorTests extends AbstractDatabasePopulatorTests { } /** - * https://github.com/spring-projects/spring-framework/issues/27008 + * gh-27008 * * @since 5.3.11 */ 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 94c35515b3..5df97e571f 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,7 +130,7 @@ public class SimpMessagingTemplate extends AbstractMessageSendingTemplateIf a destination header is not already present ,the message is sent - * to the configured {@link #setDefaultDestination(Object) defaultDestination} + * to the configured {@link AbstractMessageSendingTemplate#setDefaultDestination(Object) defaultDestination} * or an exception 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 e6dab3c4f7..47b9e4c95f 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 fffc73d1d4..6d6dc4b444 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,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 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 0db28f8762..d4cd1bae69 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/AutowiredRuleTests.java index 654fed92f6..70e651968e 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/AutowiredRuleTests.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,7 +30,7 @@ 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 an issue raised in SPR-15927. * * @author Sam Brannen * @since 5.0 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 5e136e53c6..ce715283d3 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 @@ -328,7 +328,7 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati } /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 + * Used to reproduce bug reported in SPR-10997 */ @Test void buildContextHierarchyMapForTestClassHierarchyWithMultiLevelContextHierarchiesAndOverriddenInitializers() { @@ -505,7 +505,7 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati } /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 + * Used to reproduce bug reported in SPR-10997 */ @ContextHierarchy({// // @@ -516,7 +516,7 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati } /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 + * Used to reproduce bug reported in SPR-10997 */ @ContextHierarchy({// // @@ -528,7 +528,7 @@ class ContextLoaderUtilsContextHierarchyTests extends AbstractContextConfigurati } /** - * Used to reproduce bug reported in https://jira.spring.io/browse/SPR-10997 + * Used to reproduce bug reported in 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 f23f000afc..a6d9ced6eb 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. @@ -26,7 +26,7 @@ import org.springframework.web.testfixture.servlet.MockHttpServletRequest; /** - * Benchmarks for extracting parameters from {@libnk ServletRequest}. + * Benchmarks for extracting parameters from {@link jakarta.servlet.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 0223f1f274..d48990beb9 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. @@ -70,7 +70,7 @@ public abstract class AbstractListenerWriteProcessor implements Processorgh-273 */ private volatile boolean readyToCompleteAfterLastWrite; diff --git a/spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.java b/spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.java index ef637a9615..d28724c27d 100644 --- a/spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.java +++ b/spring-web/src/main/java/org/springframework/web/util/HtmlCharacterEntityReferences.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. @@ -31,7 +31,7 @@ import org.springframework.util.Assert; * HTML 4.0 standard. * *

A 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 cad5688020..1f260c993d 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 8cac515a6e..019488099c 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 b1762e6523..ce27f267a4 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,7 +170,7 @@ 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 + * @see SPR-2789 */ @PathPatternsParameterizedTest void nestedPathisUsedAsViewName_InBreakingChangeFromSpring12Line( 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 81d56abb71..2ec4ff6819 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 @@ -356,7 +356,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-2788 + * SPR-2788 */ @Test void asBodyTagWithErrorsAndExistingMessagesAttributeInNonPageScopeAreNotClobbered() throws Exception { @@ -380,7 +380,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-2788 + * SPR-2788 */ @Test void asBodyTagWithNoErrorsAndExistingMessagesAttributeInApplicationScopeAreNotClobbered() throws Exception { @@ -388,7 +388,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-2788 + * SPR-2788 */ @Test void asBodyTagWithNoErrorsAndExistingMessagesAttributeInSessionScopeAreNotClobbered() throws Exception { @@ -396,7 +396,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-2788 + * SPR-2788 */ @Test void asBodyTagWithNoErrorsAndExistingMessagesAttributeInPageScopeAreNotClobbered() throws Exception { @@ -404,7 +404,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-2788 + * SPR-2788 */ @Test void asBodyTagWithNoErrorsAndExistingMessagesAttributeInRequestScopeAreNotClobbered() throws Exception { @@ -412,7 +412,7 @@ public class ErrorsTagTests extends AbstractFormTagTests { } /** - * https://jira.spring.io/browse/SPR-4005 + * SPR-4005 */ @Test void omittedPathMatchesObjectErrorsOnly() throws Exception { 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 f009c85621..c8d9e6768e 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 @@ -32,7 +32,7 @@ import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException class PasswordInputTagTests extends InputTagTests { /** - * https://jira.spring.io/browse/SPR-2866 + * SPR-2866 */ @Test void passwordValueIsNotRenderedByDefault() throws Exception { @@ -49,7 +49,7 @@ class PasswordInputTagTests extends InputTagTests { } /** - * https://jira.spring.io/browse/SPR-2866 + * SPR-2866 */ @Test void passwordValueIsRenderedIfShowPasswordAttributeIsSetToTrue() throws Exception { @@ -67,7 +67,7 @@ class PasswordInputTagTests extends InputTagTests { } /** - * https://jira.spring.io/browse/SPR-2866 + * SPR-2866 */ @Test void passwordValueIsNotRenderedIfShowPasswordAttributeIsSetToFalse() throws Exception { From 4983a802a7a9c8b2bf2ecbea39733b85c0d4a5c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?St=C3=A9phane=20Nicoll?= Date: Sat, 9 Mar 2024 15:53:49 +0100 Subject: [PATCH 2/2] Polish "Fix Javadoc" See gh-32403 --- .../aop/testfixture/mixin/LockMixin.java | 9 ++---- .../factory/support/MethodDescriptor.java | 9 +++--- .../beans/ExtendedBeanInfoTests.java | 2 +- .../core/BridgeMethodResolver.java | 4 +-- .../NestedRepeatableAnnotationsTests.java | 1 - .../spel/ast/FunctionReference.java | 4 +-- ...ariosForSpringSecurityExpressionTests.java | 4 +-- .../init/H2DatabasePopulatorTests.java | 14 ++------- .../messaging/simp/SimpMessagingTemplate.java | 4 +-- ...agerTestExecutionEventPublishingTests.java | 1 - ...s.java => AutowiredRuleSpr15927Tests.java} | 4 +-- ...ntextLoaderUtilsContextHierarchyTests.java | 19 +++--------- .../bind/ServletRequestUtilsBenchmark.java | 4 ++- .../AbstractListenerWriteProcessor.java | 3 +- .../mvc/UrlFilenameViewControllerTests.java | 3 +- .../web/servlet/tags/form/ErrorsTagTests.java | 30 ++++--------------- .../tags/form/PasswordInputTagTests.java | 15 ++-------- 17 files changed, 38 insertions(+), 92 deletions(-) rename spring-test/src/test/java/org/springframework/test/context/junit4/rules/{AutowiredRuleTests.java => AutowiredRuleSpr15927Tests.java} (94%) 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 7d4ec1a6be..6c08f24ce2 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 @@ -44,9 +44,6 @@ public class LockMixin extends DelegatingIntroductionInterceptor implements Lock this.locked = false; } - /** - * @see org.springframework.aop.testfixture.mixin.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.intercept.MethodInterceptor#invoke(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 03ddf5c224..6b0e4e90a9 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 @@ -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}. * @@ -33,9 +35,8 @@ 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 java.lang.reflect.Method#getName() - * simple method name} or a - * {@linkplain org.springframework.util.ClassUtils#getQualifiedMethodName(java.lang.reflect.Method) + *

The supplied {@code methodName} may be a {@linkplain Method#getName() + * 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 a648bf23cf..99682ea16e 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 JDK-7023180 : Change in specified-to-be-unspecified ordering of getDeclaredMethods causes application problems + * @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 d4ad950520..c2bd54988d 100644 --- a/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java +++ b/spring-core/src/main/java/org/springframework/core/BridgeMethodResolver.java @@ -266,9 +266,9 @@ 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 JDK-6342411. + * introduced in Java 6 to fix + * JDK-6342411. * @return whether signatures match as described - * @see Java bridge methods explained */ public static boolean isVisibilityBridgeMethodPair(Method bridgeMethod, Method bridgedMethod) { if (bridgeMethod == 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 b6e88e7c13..6e20b59736 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 @@ -39,7 +39,6 @@ import static org.assertj.core.api.Assertions.assertThat; * * @author Sam Brannen * @since 5.3.24 - * @see gh-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 1bcc196f96..0387b65ef3 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(Object... var1)} which is the most - * efficient. Otherwise, {@link MethodHandle#invokeWithArguments(Object... arguments)} 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 8d57a86491..6f5b42af3c 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 Expressing Authorization with SpEL + * 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 1f897a8571..e3fb9fce4e 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; } - /** - * 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"); } - /** - * gh-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 5df97e571f..fdd5f357a7 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 @@ -130,8 +130,8 @@ public class SimpMessagingTemplate extends AbstractMessageSendingTemplateIf a destination header is not already present ,the message is sent - * to the configured {@link AbstractMessageSendingTemplate#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/EagerTestExecutionEventPublishingTests.java b/spring-test/src/test/java/org/springframework/test/context/event/EagerTestExecutionEventPublishingTests.java index 6d6dc4b444..feaa2c936f 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 @@ -50,7 +50,6 @@ import static org.junit.platform.engine.discovery.DiscoverySelectors.selectClass * * @author Sam Brannen * @since 5.3.17 - * @see gh-27757 */ class EagerTestExecutionEventPublishingTests { 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 94% 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 70e651968e..e4b26bfabd 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 @@ -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 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 ce715283d3..26769c38e1 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 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 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 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 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 a6d9ced6eb..47dcb1a599 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 @@ -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 {@link jakarta.servlet.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 d48990beb9..957e4fe51f 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 @@ -69,8 +69,7 @@ public abstract class AbstractListenerWriteProcessor implements Processorgh-273 + * place prior to calling AsyncContext.complete(). See also gh-273. */ private volatile boolean readyToCompleteAfterLastWrite; 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 ce27f267a4..b17ab58cd0 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 @@ -170,9 +170,8 @@ class UrlFilenameViewControllerTests { /** * This is the expected behavior, and it now has a test to prove it. - * @see 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 2ec4ff6819..34c1d96d79 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); } - /** - * 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); } - /** - * SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInApplicationScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.APPLICATION_SCOPE); } - /** - * SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInSessionScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.SESSION_SCOPE); } - /** - * SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInPageScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.PAGE_SCOPE); } - /** - * SPR-2788 - */ - @Test + @Test // SPR-2788 void asBodyTagWithNoErrorsAndExistingMessagesAttributeInRequestScopeAreNotClobbered() throws Exception { assertWhenNoErrorsExistingMessagesInScopeAreNotClobbered(PageContext.REQUEST_SCOPE); } - /** - * 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 c8d9e6768e..cb1cfaacbb 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 { - /** - * SPR-2866 - */ - @Test + @Test // SPR-2866 void passwordValueIsNotRenderedByDefault() throws Exception { this.getTag().setPath("name"); @@ -48,10 +45,7 @@ class PasswordInputTagTests extends InputTagTests { assertValueAttribute(output, ""); } - /** - * 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"); } - /** - * SPR-2866 - */ - @Test + @Test // >SPR-2866 void passwordValueIsNotRenderedIfShowPasswordAttributeIsSetToFalse() throws Exception { this.getTag().setPath("name"); this.getPasswordTag().setShowPassword(false);