From 9a18bcc41467b29eb597c2e584925ff78dfa0afc Mon Sep 17 00:00:00 2001
From: Lukas Barragan Torres <lukas.barragantorres@ugent.be>
Date: Wed, 26 Apr 2023 14:34:44 +0200
Subject: [PATCH] preview + visitor refactor

---
 .../sessionScreens/AbstractSessionScreen.kt   | 19 ++++++++++++++++---
 .../sessionScreens/GetSessionScreen.kt        | 15 ++++++---------
 2 files changed, 22 insertions(+), 12 deletions(-)

diff --git a/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/AbstractSessionScreen.kt b/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/AbstractSessionScreen.kt
index 3db5b36..98a5871 100644
--- a/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/AbstractSessionScreen.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/AbstractSessionScreen.kt
@@ -15,8 +15,10 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.text.style.TextAlign
+import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
+import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalEndlessTimer
 import be.ugent.sel.studeez.navigation.StudeezDestinations
 import be.ugent.sel.studeez.screens.session.SessionActions
 import kotlinx.coroutines.delay
@@ -66,7 +68,7 @@ abstract class AbstractSessionScreen {
     }
 
     @Composable
-    private fun Timer(
+    fun Timer(
         sessionActions: SessionActions,
     ) {
         var tikker by remember { mutableStateOf(false) }
@@ -85,8 +87,8 @@ abstract class AbstractSessionScreen {
 
         if (!timerEnd && sessionActions.getTimer().hasEnded()) {
 //        sessionActions.prepareMediaPlayer()
-            timerEnd =
-                true // Placeholder, vanaf hier moet het report opgestart worden en de sessie afgesloten
+             timerEnd =
+                 true // Placeholder, vanaf hier moet het report opgestart worden en de sessie afgesloten
         }
 
         val hms = sessionActions.getTimer().getHoursMinutesSeconds()
@@ -135,4 +137,15 @@ abstract class AbstractSessionScreen {
     @Composable
     abstract fun motivationString(): String
 
+}
+
+@Preview
+@Composable
+fun TimerPreview() {
+    val sessionScreen = object : AbstractSessionScreen() {
+        @Composable
+        override fun motivationString(): String = "Test"
+
+    }
+    sessionScreen.Timer(sessionActions = SessionActions({ FunctionalEndlessTimer() }, { "Preview" }, {}, {}))
 }
\ No newline at end of file
diff --git a/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/GetSessionScreen.kt b/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/GetSessionScreen.kt
index e5ee42c..e378661 100644
--- a/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/GetSessionScreen.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/GetSessionScreen.kt
@@ -6,15 +6,12 @@ import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalPomodor
 import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalTimerVisitor
 
 class GetSessionScreen : FunctionalTimerVisitor<AbstractSessionScreen> {
-    override fun visitFunctionalCustomTimer(functionalCustomTimer: FunctionalCustomTimer): AbstractSessionScreen {
-        return CustomSessionScreen(functionalCustomTimer)
-    }
+    override fun visitFunctionalCustomTimer(functionalCustomTimer: FunctionalCustomTimer): AbstractSessionScreen =
+        CustomSessionScreen(functionalCustomTimer)
 
-    override fun visitFunctionalEndlessTimer(functionalEndlessTimer: FunctionalEndlessTimer): AbstractSessionScreen {
-        return EndlessSessionScreen()
-    }
+    override fun visitFunctionalEndlessTimer(functionalEndlessTimer: FunctionalEndlessTimer): AbstractSessionScreen =
+        EndlessSessionScreen()
 
-    override fun visitFunctionalBreakTimer(functionalPomodoroTimer: FunctionalPomodoroTimer): AbstractSessionScreen {
-        return BreakSessionScreen(functionalPomodoroTimer)
-    }
+    override fun visitFunctionalBreakTimer(functionalPomodoroTimer: FunctionalPomodoroTimer): AbstractSessionScreen =
+        BreakSessionScreen(functionalPomodoroTimer)
 }
\ No newline at end of file