Fix mail template test (#2715)

ulischulte · SteKoe · web-flow · commit 6c6a2e836507 · 2023-08-25T09:52:07.000+02:00
* fixed thymeleaf template test

* formatting

* fix(MailNotifierIntegrationTest): enhanced tests

* fix missing mail notification image in docs

* removed unused import

* fix missing mail notification image in docs

* fix formatting

---------

Co-authored-by: Stephan Köninger &lt;stephan.koeninger@codecentric.de&gt;
diff --git a/spring-boot-admin-docs/src/site/asciidoc/server-notifications.adoc b/spring-boot-admin-docs/src/site/asciidoc/server-notifications.adoc
@@ -522,4 +522,4 @@ To enable https://www.rocket.chat/[Rocket.Chat] notifications you need a persona
 | spring.boot.admin.notify.rocketchat.message
 | Text to send. SpEL-expressions are supported.
 | `+++"*#{instance.registration.name}* (#{instance.id}) is *#{event.statusInfo.status}*"+++`
-|===
+|===
diff --git a/spring-boot-admin-docs/src/site/resources/images/mail-notification-sample.png b/spring-boot-admin-docs/src/site/resources/images/mail-notification-sample.png
diff --git a/spring-boot-admin-server/src/test/java/de/codecentric/boot/admin/server/notify/MailNotifierIntegrationTest.java b/spring-boot-admin-server/src/test/java/de/codecentric/boot/admin/server/notify/MailNotifierIntegrationTest.java
@@ -17,7 +17,6 @@
 package de.codecentric.boot.admin.server.notify;
 
 import java.io.FileNotFoundException;
-import java.io.IOException;
 import java.net.URL;
 
 import org.assertj.core.api.WithAssertions;
@@ -49,19 +48,28 @@ void fileProtocolIsNotAllowed() {
 	@Test
 	void httpProtocolIsNotAllowed() {
 		assertThatThrownBy(() -> {
-			URL resource = getClass().getClassLoader().getResource(".");
 			mailNotifier.setTemplate(
 					"https://raw.githubusercontent.com/codecentric/spring-boot-admin/gh-pages/vulnerable-file.html");
 			mailNotifier.getBody(new Context());
 		}).hasCauseInstanceOf(FileNotFoundException.class);
 	}
 
 	@Test
-	void classpathProtocolIsAllowed() throws IOException {
+	void classpathProtocolIsAllowed() {
+		assertThatNoException().isThrownBy(() -> {
+			mailNotifier.setTemplate("/de/codecentric/boot/admin/server/notify/allowed-file.html");
+			mailNotifier.getBody(new Context());
+		});
+	}
+
+	@Test
+	void callToReflectionUtilsAreNotAllwed() {
 		assertThatThrownBy(() -> {
 			mailNotifier.setTemplate("/de/codecentric/boot/admin/server/notify/vulnerable-file.html");
-			String body = mailNotifier.getBody(new Context());
-		}).rootCause().hasMessageContaining("error=2, No such file or directory");
+			mailNotifier.getBody(new Context());
+		}).rootCause()
+			.hasMessageContaining(
+					"Access is forbidden for type 'org.springframework.util.ReflectionUtils' in this expression context.");
 	}
 
 	@EnableAdminServer
diff --git a/spring-boot-admin-server/src/test/resources/de/codecentric/boot/admin/server/notify/allowed-file.html b/spring-boot-admin-server/src/test/resources/de/codecentric/boot/admin/server/notify/allowed-file.html
@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html xmlns:th="http://www.thymeleaf.org">
+<head>
+  <meta content="text/html; charset=UTF-8" http-equiv="Content-Type"/>
+</head>
+<body>
+
+I am fine!
+
+</body>
+</html>
