Implement suggestions

Author: MartelliEnrico

src/main/java/com/github/stickerifier/stickerify/bot/Stickerify.java

@@ -20,6 +20,7 @@
 import com.github.stickerifier.stickerify.telegram.model.TelegramRequest;
 import com.pengrad.telegrambot.ExceptionHandler;
 import com.pengrad.telegrambot.TelegramBot;
+import com.pengrad.telegrambot.TelegramException;
 import com.pengrad.telegrambot.UpdatesListener;
 import com.pengrad.telegrambot.model.LinkPreviewOptions;
 import com.pengrad.telegrambot.model.Update;
@@ -47,7 +48,7 @@
  *
  * @author Roberto Cella
  */
-public record Stickerify(TelegramBot bot, Executor executor) {
+public record Stickerify(TelegramBot bot, Executor executor) implements ExceptionHandler {
 
 	private static final Logger LOGGER = LoggerFactory.getLogger(Stickerify.class);
 	private static final String BOT_TOKEN = System.getenv("STICKERIFY_TOKEN");
@@ -67,13 +68,13 @@ public Stickerify() {
 	 *
 	 * @see Stickerify
 	 */
-	public Stickerify(TelegramBot bot, Executor executor) {
-		this.bot = bot;
-		this.executor = executor;
-
-		ExceptionHandler exceptionHandler = e -> LOGGER.atError().log("There was an unexpected failure: {}", e.getMessage());
+	public Stickerify {
+		bot.setUpdatesListener(this::handleUpdates, this, new GetUpdates().timeout(50));
+	}
 
-		bot.setUpdatesListener(this::handleUpdates, exceptionHandler, new GetUpdates().timeout(50));
+	@Override
+	public void onException(TelegramException e) {
+		LOGGER.atError().log("There was an unexpected failure: {}", e.getMessage());
 	}
 
 	private int handleUpdates(List<Update> updates) {

src/test/java/com/github/stickerifier/stickerify/bot/StickerifyTest.java

@@ -8,6 +8,7 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 import com.github.stickerifier.stickerify.junit.ClearTempFiles;
+import com.github.stickerifier.stickerify.junit.Tags;
 import com.github.stickerifier.stickerify.telegram.Answer;
 import com.pengrad.telegrambot.TelegramBot;
 import mockwebserver3.MockWebServer;
@@ -18,7 +19,7 @@
 
 import java.net.URLEncoder;
 
-@Tag("telegramApi")
+@Tag(Tags.TELEGRAM_API)
 @ClearTempFiles
 class StickerifyTest {
 

src/test/java/com/github/stickerifier/stickerify/junit/Tags.java

@@ -0,0 +1,12 @@
+package com.github.stickerifier.stickerify.junit;
+
+public interface Tags {
+	String ANIMATED_STICKER = "AnimatedSticker";
+	String CONCURRENT = "concurrent";
+	String IMAGE = "Image";
+	String LOG = "Log";
+	String MEDIA = "Media";
+	String TELEGRAM_API = "TelegramApi";
+	String UNSUPPORTED_FILE = "unsupportedFile";
+	String VIDEO = "video";
+}

src/test/java/com/github/stickerifier/stickerify/logger/ExceptionHighlighterTest.java

@@ -9,12 +9,13 @@
 import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.is;
 
+import com.github.stickerifier.stickerify.junit.Tags;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
-@Tag("log")
+@Tag(Tags.LOG)
 class ExceptionHighlighterTest {
 
 	private static final String LOG_MESSAGE = "Received request";

src/test/java/com/github/stickerifier/stickerify/logger/MessageHighlighterTest.java

@@ -8,12 +8,13 @@
 import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.is;
 
+import com.github.stickerifier.stickerify.junit.Tags;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
-@Tag("log")
+@Tag(Tags.LOG)
 class MessageHighlighterTest {
 
 	private static final String LOG_MESSAGE = "Received request";