From 6239b3e68618cbbdc9cf18f6afa8cbdda483ec30 Mon Sep 17 00:00:00 2001
From: Maxime NATUREL <maxime.naturel@niji.fr>
Date: Thu, 3 Nov 2022 17:48:49 +0100
Subject: [PATCH] Adding some TODOs

---
 .../notification/CanTogglePushNotificationsViaPusherUseCase.kt   | 1 +
 .../CheckIfCanTogglePushNotificationsViaAccountDataUseCase.kt    | 1 +
 .../CheckIfCanTogglePushNotificationsViaPusherUseCase.kt         | 1 +
 .../devices/v2/notification/GetNotificationsStatusUseCase.kt     | 1 +
 .../devices/v2/notification/TogglePushNotificationUseCase.kt     | 1 +
 5 files changed, 5 insertions(+)

diff --git a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CanTogglePushNotificationsViaPusherUseCase.kt b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CanTogglePushNotificationsViaPusherUseCase.kt
index 963768ca04..af15e2f349 100644
--- a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CanTogglePushNotificationsViaPusherUseCase.kt
+++ b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CanTogglePushNotificationsViaPusherUseCase.kt
@@ -25,6 +25,7 @@ import javax.inject.Inject
 
 class CanTogglePushNotificationsViaPusherUseCase @Inject constructor() {
 
+    // TODO add unit tests
     fun execute(session: Session): Flow<Boolean> {
         return session
                 .homeServerCapabilitiesService()
diff --git a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaAccountDataUseCase.kt b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaAccountDataUseCase.kt
index 194a2aebbf..85abd7cd35 100644
--- a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaAccountDataUseCase.kt
+++ b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaAccountDataUseCase.kt
@@ -22,6 +22,7 @@ import javax.inject.Inject
 
 class CheckIfCanTogglePushNotificationsViaAccountDataUseCase @Inject constructor() {
 
+    // TODO update unit tests
     fun execute(session: Session, deviceId: String): Boolean {
         return session
                 .accountDataService()
diff --git a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaPusherUseCase.kt b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaPusherUseCase.kt
index ca314bf145..9c2a471120 100644
--- a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaPusherUseCase.kt
+++ b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/CheckIfCanTogglePushNotificationsViaPusherUseCase.kt
@@ -21,6 +21,7 @@ import javax.inject.Inject
 
 class CheckIfCanTogglePushNotificationsViaPusherUseCase @Inject constructor() {
 
+    // TODO update unit tests
     fun execute(session: Session): Boolean {
         return session
                 .homeServerCapabilitiesService()
diff --git a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/GetNotificationsStatusUseCase.kt b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/GetNotificationsStatusUseCase.kt
index 03e4e31f2e..ed1d54e118 100644
--- a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/GetNotificationsStatusUseCase.kt
+++ b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/GetNotificationsStatusUseCase.kt
@@ -34,6 +34,7 @@ class GetNotificationsStatusUseCase @Inject constructor(
         private val checkIfCanTogglePushNotificationsViaAccountDataUseCase: CheckIfCanTogglePushNotificationsViaAccountDataUseCase,
 ) {
 
+    // TODO update unit tests
     fun execute(session: Session, deviceId: String): Flow<NotificationsStatus> {
         return when {
             checkIfCanTogglePushNotificationsViaAccountDataUseCase.execute(session, deviceId) -> {
diff --git a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/TogglePushNotificationUseCase.kt b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/TogglePushNotificationUseCase.kt
index 7969bbbe9b..28018f9687 100644
--- a/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/TogglePushNotificationUseCase.kt
+++ b/vector/src/main/java/im/vector/app/features/settings/devices/v2/notification/TogglePushNotificationUseCase.kt
@@ -28,6 +28,7 @@ class TogglePushNotificationUseCase @Inject constructor(
         private val checkIfCanTogglePushNotificationsViaAccountDataUseCase: CheckIfCanTogglePushNotificationsViaAccountDataUseCase,
 ) {
 
+    // TODO update unit tests
     suspend fun execute(deviceId: String, enabled: Boolean) {
         val session = activeSessionHolder.getSafeActiveSession() ?: return