Make displayName optional just in case not every engine provides one

ls1intum · Mar 24, 2022 · 582371d · 582371d
1 parent c08f7bd
commit 582371d
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/src/main/java/de/tum/in/test/api/internal/TestContext.java b/src/main/java/de/tum/in/test/api/internal/TestContext.java
@@ -22,7 +22,7 @@ public abstract class TestContext {
 
 	public abstract Optional<Object> testInstance();
 
-	public abstract String displayName();
+	public abstract Optional<String> displayName();
 
 	public abstract Optional<AnnotatedElement> annotatedElement();
 

diff --git a/src/main/java/de/tum/in/test/api/jqwik/JqwikContext.java b/src/main/java/de/tum/in/test/api/jqwik/JqwikContext.java
@@ -42,8 +42,8 @@ public Optional<AnnotatedElement> annotatedElement() {
 	}
 
 	@Override
-	public String displayName() {
-		return lifecycleContext.label();
+	public Optional<String> displayName() {
+		return Optional.of(lifecycleContext.label());
 	}
 
 	public PropertyLifecycleContext getPropertyLifecycleContext() {

diff --git a/src/main/java/de/tum/in/test/api/jupiter/JupiterContext.java b/src/main/java/de/tum/in/test/api/jupiter/JupiterContext.java
@@ -44,8 +44,8 @@ public Optional<AnnotatedElement> annotatedElement() {
 	}
 
 	@Override
-	public String displayName() {
-		return extensionContext.getDisplayName();
+	public Optional<String> displayName() {
+		return Optional.of(extensionContext.getDisplayName());
 	}
 
 	@Override

diff --git a/src/test/java/de/tum/in/test/api/security/AresSecurityConfigurationTest.java b/src/test/java/de/tum/in/test/api/security/AresSecurityConfigurationTest.java
@@ -160,8 +160,8 @@ public Optional<Object> testInstance() {
 		}
 
 		@Override
-		public String displayName() {
-			return "Some test";
+		public Optional<String> displayName() {
+			return Optional.of("Some test");
 		}
 
 		@Override