Fix @⁠API and @⁠since tag declarations

Author: sbrannen

junit-platform-commons/src/main/java/org/junit/platform/commons/util/KotlinReflectionUtils.java

@@ -29,9 +29,9 @@
 /**
  * Internal Kotlin-specific reflection utilities
  *
- * @since 5.13.3
+ * @since 1.13.3
  */
-@API(status = INTERNAL, since = "5.13.3")
+@API(status = INTERNAL, since = "1.13.3")
 public class KotlinReflectionUtils {
 
 	private static final String DEFAULT_IMPLS_CLASS_NAME = "DefaultImpls";
@@ -82,10 +82,7 @@ public static boolean isKotlinSuspendingFunction(Method method) {
 	 * <p>See
 	 * <a href="https://kotlinlang.org/docs/interfaces.html#jvm-default-method-generation-for-interface-functions">Kotlin documentation</a>
 	 * for details.
-	 *
-	 * @since 5.13.3
 	 */
-	@API(status = INTERNAL, since = "5.13.3")
 	public static boolean isKotlinInterfaceDefaultImplsClass(Class<?> clazz) {
 		if (!isKotlinType(clazz) || !DEFAULT_IMPLS_CLASS_NAME.equals(clazz.getSimpleName()) || !isStatic(clazz)) {
 			return false;

junit-platform-commons/src/main/java/org/junit/platform/commons/util/ReflectionUtils.java

@@ -14,7 +14,6 @@
 import static java.util.stream.Collectors.toCollection;
 import static java.util.stream.Collectors.toSet;
 import static org.apiguardian.api.API.Status.INTERNAL;
-import static org.apiguardian.api.API.Status.STABLE;
 import static org.junit.platform.commons.util.ReflectionUtils.HierarchyTraversalMode.BOTTOM_UP;
 import static org.junit.platform.commons.util.ReflectionUtils.HierarchyTraversalMode.TOP_DOWN;
 
@@ -1107,7 +1106,10 @@ public static List<Class<?>> findNestedClasses(Class<?> clazz, Predicate<Class<?
 		return findNestedClasses(clazz, predicate, CycleErrorHandling.THROW_EXCEPTION);
 	}
 
-	@API(status = INTERNAL, since = "5.13.2")
+	/**
+	 * @since 1.13.2
+	 */
+	@API(status = INTERNAL, since = "1.13.2")
 	public static List<Class<?>> findNestedClasses(Class<?> clazz, Predicate<Class<?>> predicate,
 			CycleErrorHandling errorHandling) {
 		Preconditions.notNull(clazz, "Class must not be null");
@@ -1136,6 +1138,7 @@ public static List<Class<?>> findNestedClasses(Class<?> clazz, Predicate<Class<?
 	 * checked; never {@code null}
 	 * @return {@code true} if such a nested class is present
 	 * @throws JUnitException if a cycle is detected within an inner class hierarchy
+	 * @since 1.13.2
 	 */
 	@API(status = INTERNAL, since = "1.13.2")
 	public static boolean isNestedClassPresent(Class<?> clazz, Predicate<Class<?>> predicate,
@@ -1150,14 +1153,18 @@ public static boolean isNestedClassPresent(Class<?> clazz, Predicate<Class<?>> p
 	}
 
 	/**
-	 * since 1.10
+	 * @since 1.10
 	 * @see org.junit.platform.commons.support.ReflectionSupport#streamNestedClasses(Class, Predicate)
 	 */
+	@API(status = INTERNAL, since = "1.10")
 	public static Stream<Class<?>> streamNestedClasses(Class<?> clazz, Predicate<Class<?>> predicate) {
 		return findNestedClasses(clazz, predicate).stream();
 	}
 
-	@API(status = INTERNAL, since = "5.13.2")
+	/**
+	 * @since 1.13.2
+	 */
+	@API(status = INTERNAL, since = "1.13.2")
 	public static Stream<Class<?>> streamNestedClasses(Class<?> clazz, Predicate<Class<?>> predicate,
 			CycleErrorHandling errorHandling) {
 		return findNestedClasses(clazz, predicate, errorHandling).stream();
@@ -1312,6 +1319,7 @@ public static List<Field> findFields(Class<?> clazz, Predicate<Field> predicate,
 	 * @since 1.10
 	 * @see org.junit.platform.commons.support.ReflectionSupport#streamFields(Class, Predicate, org.junit.platform.commons.support.HierarchyTraversalMode)
 	 */
+	@API(status = INTERNAL, since = "1.10")
 	public static Stream<Field> streamFields(Class<?> clazz, Predicate<Field> predicate,
 			HierarchyTraversalMode traversalMode) {
 
@@ -1453,6 +1461,9 @@ public static Optional<Method> findMethod(Class<?> clazz, String methodName, @Nu
 		return findMethod(clazz, methodName, resolveParameterTypes(clazz, methodName, parameterTypeNames));
 	}
 
+	/**
+	 * @since 1.10
+	 */
 	@API(status = INTERNAL, since = "1.10")
 	public static Class<?>[] resolveParameterTypes(Class<?> clazz, String methodName,
 			@Nullable String parameterTypeNames) {
@@ -1539,7 +1550,7 @@ private static Optional<Method> findMethod(Class<?> clazz, Predicate<Method> pre
 	 * @since 1.7
 	 * @see #findMethod(Class, String, Class...)
 	 */
-	@API(status = STABLE, since = "1.7")
+	@API(status = INTERNAL, since = "1.7")
 	public static Method getRequiredMethod(Class<?> clazz, String methodName, Class<?>... parameterTypes) {
 		return ReflectionUtils.findMethod(clazz, methodName, parameterTypes).orElseThrow(() -> new JUnitException(
 			"Could not find method [%s] in class [%s]".formatted(methodName, clazz.getName())));
@@ -1575,6 +1586,7 @@ public static List<Method> findMethods(Class<?> clazz, Predicate<Method> predica
 	 * @since 1.10
 	 * @see org.junit.platform.commons.support.ReflectionSupport#streamMethods(Class, Predicate, org.junit.platform.commons.support.HierarchyTraversalMode)
 	 */
+	@API(status = INTERNAL, since = "1.10")
 	public static Stream<Method> streamMethods(Class<?> clazz, Predicate<Method> predicate,
 			HierarchyTraversalMode traversalMode) {
 
@@ -1886,6 +1898,9 @@ private static boolean isGeneric(Type type) {
 		return type instanceof TypeVariable || type instanceof GenericArrayType;
 	}
 
+	/**
+	 * @since 1.11
+	 */
 	@API(status = INTERNAL, since = "1.11")
 	@SuppressWarnings("deprecation") // "AccessibleObject.isAccessible()" is deprecated in Java 9
 	public static <T extends Executable> T makeAccessible(T executable) {
@@ -1895,6 +1910,9 @@ public static <T extends Executable> T makeAccessible(T executable) {
 		return executable;
 	}
 
+	/**
+	 * @since 1.12
+	 */
 	@API(status = INTERNAL, since = "1.12")
 	@SuppressWarnings("deprecation") // "AccessibleObject.isAccessible()" is deprecated in Java 9
 	public static Field makeAccessible(Field field) {
@@ -1958,7 +1976,10 @@ static Throwable getUnderlyingCause(Throwable t) {
 		return t;
 	}
 
-	@API(status = INTERNAL, since = "5.13.2")
+	/**
+	 * @since 1.13.2
+	 */
+	@API(status = INTERNAL, since = "1.13.2")
 	public enum CycleErrorHandling {
 
 		THROW_EXCEPTION {

junit-platform-engine/src/main/java/org/junit/platform/engine/support/store/NamespacedHierarchicalStore.java

@@ -96,7 +96,7 @@ public NamespacedHierarchicalStore<N> newChild() {
 	 * <p>If this store does not have a parent, an empty {@code Optional} is returned.
 	 *
 	 * @return an {@code Optional} containing the parent store, or an empty {@code Optional} if there is no parent
-	 * @since 5.13
+	 * @since 1.13
 	 */
 	@API(status = EXPERIMENTAL, since = "6.0")
 	public Optional<NamespacedHierarchicalStore<N>> getParent() {

junit-platform-launcher/src/main/java/org/junit/platform/launcher/core/DelegatingLauncherDiscoveryRequest.java

@@ -22,7 +22,7 @@
 import org.junit.platform.launcher.PostDiscoveryFilter;
 
 /**
- * @since 5.13
+ * @since 1.13
  */
 class DelegatingLauncherDiscoveryRequest implements LauncherDiscoveryRequest {
 

junit-platform-reporting/src/main/java/org/junit/platform/reporting/open/xml/GitInfoCollector.java

@@ -28,7 +28,7 @@
 import org.junit.platform.commons.util.StringUtils;
 
 /**
- * @since 5.13.2
+ * @since 1.13.2
  */
 interface GitInfoCollector {