Remove @Repeatable support for @DisabledOnOs

We'll add it back again when we need it, and delay the naming issue.

See gh-30082
pull/30505/head
Moritz Halbritter 3 years ago
parent be38311c5d
commit f3eb90046f

@ -17,7 +17,6 @@
package org.springframework.boot.testsupport.junit; package org.springframework.boot.testsupport.junit;
import java.lang.annotation.ElementType; import java.lang.annotation.ElementType;
import java.lang.annotation.Repeatable;
import java.lang.annotation.Retention; import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy; import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target; import java.lang.annotation.Target;
@ -27,7 +26,7 @@ import org.junit.jupiter.api.extension.ExtendWith;
/** /**
* Improves JUnit5's {@link org.junit.jupiter.api.condition.DisabledOnOs} by adding an * Improves JUnit5's {@link org.junit.jupiter.api.condition.DisabledOnOs} by adding an
* architecture check and making it repeatable. * architecture check.
* *
* @author Moritz Halbritter * @author Moritz Halbritter
* @since 2.5.11 * @since 2.5.11
@ -35,7 +34,6 @@ import org.junit.jupiter.api.extension.ExtendWith;
@Target({ ElementType.TYPE, ElementType.METHOD }) @Target({ ElementType.TYPE, ElementType.METHOD })
@Retention(RetentionPolicy.RUNTIME) @Retention(RetentionPolicy.RUNTIME)
@ExtendWith(DisabledOnOsCondition.class) @ExtendWith(DisabledOnOsCondition.class)
@Repeatable(DisabledOnOsRepeatable.class)
public @interface DisabledOnOs { public @interface DisabledOnOs {
/** /**

@ -16,7 +16,7 @@
package org.springframework.boot.testsupport.junit; package org.springframework.boot.testsupport.junit;
import java.util.List; import java.util.Optional;
import org.junit.jupiter.api.extension.ConditionEvaluationResult; import org.junit.jupiter.api.extension.ConditionEvaluationResult;
import org.junit.jupiter.api.extension.ExecutionCondition; import org.junit.jupiter.api.extension.ExecutionCondition;
@ -32,24 +32,21 @@ class DisabledOnOsCondition implements ExecutionCondition {
@Override @Override
public ConditionEvaluationResult evaluateExecutionCondition(ExtensionContext context) { public ConditionEvaluationResult evaluateExecutionCondition(ExtensionContext context) {
List<DisabledOnOs> annotations = AnnotationUtils.findRepeatableAnnotations(context.getElement(), Optional<DisabledOnOs> annotation = AnnotationUtils.findAnnotation(context.getElement(), DisabledOnOs.class);
DisabledOnOs.class); if (!annotation.isPresent()) {
if (annotations.isEmpty()) {
return ConditionEvaluationResult.enabled("No @DisabledOnOs found"); return ConditionEvaluationResult.enabled("No @DisabledOnOs found");
} }
return evaluate(annotations); return evaluate(annotation.get());
} }
private ConditionEvaluationResult evaluate(List<DisabledOnOs> annotations) { private ConditionEvaluationResult evaluate(DisabledOnOs annotation) {
String architecture = System.getProperty("os.arch"); String architecture = System.getProperty("os.arch");
String os = System.getProperty("os.name"); String os = System.getProperty("os.name");
for (DisabledOnOs annotation : annotations) { if (annotation.os().isCurrentOs() && annotation.architecture().equals(architecture)) {
if (annotation.os().isCurrentOs() && annotation.architecture().equals(architecture)) { String reason = annotation.disabledReason().isEmpty()
String reason = annotation.disabledReason().isEmpty() ? String.format("Disabled on OS = %s, architecture = %s", os, architecture)
? String.format("Disabled on OS = %s, architecture = %s", os, architecture) : annotation.disabledReason();
: annotation.disabledReason(); return ConditionEvaluationResult.disabled(reason);
return ConditionEvaluationResult.disabled(reason);
}
} }
return ConditionEvaluationResult return ConditionEvaluationResult
.enabled(String.format("Enabled on OS = %s, architecture = %s", os, architecture)); .enabled(String.format("Enabled on OS = %s, architecture = %s", os, architecture));

@ -1,39 +0,0 @@
/*
* Copyright 2012-2022 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.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.boot.testsupport.junit;
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;
import org.junit.jupiter.api.extension.ExtendWith;
/**
* Repeatable container for {@link DisabledOnOs}.
*
* @author Moritz Halbritter
* @since 2.5.10
*/
@Target({ ElementType.TYPE, ElementType.METHOD })
@Retention(RetentionPolicy.RUNTIME)
@ExtendWith(DisabledOnOsCondition.class)
public @interface DisabledOnOsRepeatable {
DisabledOnOs[] value();
}
Loading…
Cancel
Save