From 2b0a9b690bef4c4bbe9ad1bea17fc00500775b42 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Mon, 2 Dec 2019 19:19:04 +0100
Subject: [PATCH 1/4] add NewBoardActivity

- add repleceOnConflict test
---
 app/build.gradle                              |  1 -
 .../ids/wedroid/app/data/dao/BoardDaoTest.kt  | 24 +++++--
 app/src/main/AndroidManifest.xml              |  9 ++-
 .../app/data/repository/BoardRepository.kt    |  8 ++-
 .../wedroid/app/view/BoardsListsActivity.kt   | 38 +++++++++++-
 .../wedroid/app/view/NewBoardActivity.java    | 45 ++++++++++++++
 .../app/viewmodel/BoardsListViewModel.java    | 10 ++-
 .../main/res/layout/activity_new_board.xml    | 25 ++++++++
 app/src/main/res/layout/content_new_board.xml | 62 +++++++++++++++++++
 app/src/main/res/values/strings.xml           |  7 +++
 10 files changed, 213 insertions(+), 16 deletions(-)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivity.java
 create mode 100644 app/src/main/res/layout/activity_new_board.xml
 create mode 100644 app/src/main/res/layout/content_new_board.xml

diff --git a/app/build.gradle b/app/build.gradle
index 352ef63..7de7ce6 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -57,7 +57,6 @@ dependencies {
     implementation "androidx.swiperefreshlayout:swiperefreshlayout:1.0.0"
 
 
-
     // TESTING
     testImplementation 'junit:junit:4.12'
     androidTestImplementation 'androidx.test.ext:junit:1.1.1'
diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt
index dae8ff8..2655621 100644
--- a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt
@@ -62,15 +62,29 @@ class BoardDaoTest {
         }
     }
 
+    @Test
+    fun replaceOnConflict() {
+        val board0 = Board("id", "title0")
+        val board1 = Board("id", "title1")
+
+        runBlocking {
+            dao.insert(board0)
+            dao.insert(board1)
+        }
+
+        dao.getAllBoard().observeOnce {
+            assertEquals(1, it.size)
+            assertEquals("title1", it[0].title)
+        }
+    }
+
+    @Test
     fun getInAscendingOrder() {
         val board0 = Board("id0", "title0")
         val board1 = Board("id1", "title1")
 
         runBlocking {
             dao.insert(board1)
-        }
-
-        runBlocking {
             dao.insert(board0)
         }
 
@@ -81,14 +95,12 @@ class BoardDaoTest {
         }
     }
 
+    @Test
     fun delete() {
         val board = Board("id", "title")
 
         runBlocking {
             dao.insert(board)
-        }
-
-        runBlocking {
             dao.delete(board)
         }
 
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index e589c5d..591040a 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -2,7 +2,7 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="it.unisannio.ding.ids.wedroid.app">
 
-    <uses-permission android:name="android.permission.INTERNET"/>
+    <uses-permission android:name="android.permission.INTERNET" />
 
     <application
         android:allowBackup="true"
@@ -11,11 +11,14 @@
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
+        <activity
+            android:name=".view.NewBoardActivity"
+            android:label="@string/title_activity_new_board"
+            android:theme="@style/AppTheme.NoActionBar"></activity>
         <activity
             android:name=".view.BoardsListsActivity"
             android:label="@string/title_activity_boards_lists"
-            android:theme="@style/AppTheme.NoActionBar">
-        </activity>
+            android:theme="@style/AppTheme.NoActionBar"></activity>
         <activity android:name=".MainActivity">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
index 22179c0..8e7675d 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
@@ -7,6 +7,7 @@ import it.unisannio.ding.ids.wedroid.app.data.entity.convert
 import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader
 import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
 import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor
+import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardPermission
 import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardPrototype
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
@@ -28,12 +29,15 @@ class BoardRepository(
         synchronize()
     }
 
-    fun insertBoard(title: String) {
+    fun insertBoard(title: String, isPrivate: Boolean, color: BoardBackgroundColor) {
+        val permission = if (isPrivate) BoardPermission.PRIVATE else BoardPermission.PUBLIC
+
         service.newBoard(
             BoardPrototype.Builder()
                 .setOwner(reader.userId)
                 .setTitle(title)
-                .setBackgroundColor(BoardBackgroundColor.LIMEGREEN)
+                .setBackgroundColor(color)
+                .setBoardPermission(permission)
                 .build()
         ).enqueue(object : Callback<it.unisannio.ding.ids.wedroid.wrapper.entity.Board> {
             override fun onFailure(
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt
index 0e1064f..5b4681a 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt
@@ -1,6 +1,8 @@
 package it.unisannio.ding.ids.wedroid.app.view
 
+import android.content.Intent
 import android.os.Bundle
+import android.widget.Toast
 import androidx.appcompat.app.AppCompatActivity
 import androidx.lifecycle.Observer
 import androidx.lifecycle.ViewModelProvider
@@ -10,6 +12,7 @@ import androidx.recyclerview.widget.RecyclerView
 import it.unisannio.ding.ids.wedroid.app.R
 import it.unisannio.ding.ids.wedroid.app.view.adapter.BoardsListAdapter
 import it.unisannio.ding.ids.wedroid.app.viewmodel.BoardsListViewModel
+import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor
 
 import kotlinx.android.synthetic.main.activity_boards_lists.*
 import kotlinx.android.synthetic.main.content_boards_lists.*
@@ -37,7 +40,10 @@ class BoardsListsActivity : AppCompatActivity() {
         })
 
         fab.setOnClickListener {
-            viewModel.insertBoard("New board")
+            startActivityForResult(
+                Intent(this, NewBoardActivity::class.java),
+                NEW_BOARD_CODE
+            )
         }
 
         pullToRefresh.setOnRefreshListener {
@@ -67,5 +73,35 @@ class BoardsListsActivity : AppCompatActivity() {
 
         swipeToDelete.attachToRecyclerView(boardList)
     }
+
+    override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
+        super.onActivityResult(requestCode, resultCode, data)
+        when (requestCode) {
+            NEW_BOARD_CODE -> if (data != null) addBoard(resultCode, data)
+        }
+    }
+
+    private fun addBoard(resultCode: Int, data: Intent) {
+        if (resultCode != NewBoardActivity.RESULT_OK) {
+            Toast.makeText(this, R.string.on_add_new_board_error, Toast.LENGTH_LONG)
+                .show()
+            return
+        }
+
+        val title = data.getStringExtra(NewBoardActivity.BOARD_NAME)
+        if (title == null) {
+            Toast.makeText(this, R.string.on_null_new_board_name, Toast.LENGTH_LONG)
+                .show()
+            return
+        }
+
+        val isPrivate = data.getBooleanExtra(NewBoardActivity.BOARD_PRIVATE, true)
+
+        viewModel.insertBoard(title, isPrivate, BoardBackgroundColor.LIMEGREEN) //TODO
+    }
+
+    companion object {
+        const val NEW_BOARD_CODE = 17
+    }
 }
 
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivity.java b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivity.java
new file mode 100644
index 0000000..b0a48dd
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivity.java
@@ -0,0 +1,45 @@
+package it.unisannio.ding.ids.wedroid.app.view;
+
+import android.content.Intent;
+import android.os.Bundle;
+
+import androidx.appcompat.app.AppCompatActivity;
+import androidx.appcompat.widget.Toolbar;
+
+import android.view.View;
+import android.widget.EditText;
+import android.widget.Switch;
+
+import it.unisannio.ding.ids.wedroid.app.R;
+
+public class NewBoardActivity extends AppCompatActivity {
+    private EditText boardName;
+    private Switch isPrivate;
+
+    @Override
+    protected void onCreate(Bundle savedInstanceState) {
+        super.onCreate(savedInstanceState);
+        setContentView(R.layout.activity_new_board);
+        Toolbar toolbar = findViewById(R.id.toolbar);
+        setSupportActionBar(toolbar);
+
+        boardName = findViewById(R.id.newBoardName);
+        isPrivate = findViewById(R.id.newBoardPermission);
+    }
+
+    public void onDone(View v) {
+        Intent data = new Intent();
+        data.putExtra(BOARD_NAME, boardName.getText().toString());
+        data.putExtra(BOARD_PRIVATE, isPrivate.isChecked());
+        setResult(RESULT_OK, data);
+        finish();
+    }
+
+    public void onCancel(View v) {
+        finish();
+    }
+
+    public static final int RESULT_OK = 17;
+    public static final String BOARD_NAME = "BOARD_NAME";
+    public static final String BOARD_PRIVATE = "BOARD_PRIVATE";
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java
index 79ab9c2..05364cf 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java
@@ -13,6 +13,7 @@ import it.unisannio.ding.ids.wedroid.app.data.entity.Board;
 import it.unisannio.ding.ids.wedroid.app.data.repository.BoardRepository;
 import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader;
 import it.unisannio.ding.ids.wedroid.app.util.ServicesFactory;
+import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor;
 
 public class BoardsListViewModel extends AndroidViewModel {
     private BoardRepository repository;
@@ -34,12 +35,15 @@ public class BoardsListViewModel extends AndroidViewModel {
         return allBoards;
     }
 
-    public void insertBoard(String title) {
-        repository.insertBoard(title);
+    public void insertBoard(String title, boolean isPrivate, BoardBackgroundColor color) {
+        repository.insertBoard(title, isPrivate, color);
     }
 
     public void deleteBoard(int position) {
-        repository.deleteBoard(allBoards.getValue().get(position).getId());
+        List<Board> boards = allBoards.getValue();
+
+        if (boards != null)
+            repository.deleteBoard(boards.get(position).getId());
     }
 
     public void refresh() {
diff --git a/app/src/main/res/layout/activity_new_board.xml b/app/src/main/res/layout/activity_new_board.xml
new file mode 100644
index 0000000..d32de4d
--- /dev/null
+++ b/app/src/main/res/layout/activity_new_board.xml
@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.coordinatorlayout.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:context=".view.NewBoardActivity">
+
+    <com.google.android.material.appbar.AppBarLayout
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:theme="@style/AppTheme.AppBarOverlay">
+
+        <androidx.appcompat.widget.Toolbar
+            android:id="@+id/toolbar"
+            android:layout_width="match_parent"
+            android:layout_height="?attr/actionBarSize"
+            android:background="?attr/colorPrimary"
+            app:popupTheme="@style/AppTheme.PopupOverlay" />
+
+    </com.google.android.material.appbar.AppBarLayout>
+
+    <include layout="@layout/content_new_board" />
+
+</androidx.coordinatorlayout.widget.CoordinatorLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/content_new_board.xml b/app/src/main/res/layout/content_new_board.xml
new file mode 100644
index 0000000..ad7e6da
--- /dev/null
+++ b/app/src/main/res/layout/content_new_board.xml
@@ -0,0 +1,62 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    app:layout_behavior="@string/appbar_scrolling_view_behavior"
+    tools:context=".view.NewBoardActivity"
+    tools:showIn="@layout/activity_new_board">
+
+    <EditText
+        android:id="@+id/newBoardName"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:ems="10"
+        android:hint="@string/new_board_name_field"
+        android:inputType="textPersonName"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent"
+        app:layout_constraintVertical_bias="0.25"
+        android:importantForAutofill="no" />
+
+    <Switch
+        android:id="@+id/newBoardPermission"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:checked="true"
+        android:text="@string/new_board_switch"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="@+id/newBoardName"
+        app:layout_constraintStart_toStartOf="@+id/newBoardName"
+        app:layout_constraintTop_toBottomOf="@+id/newBoardName"
+        app:layout_constraintVertical_bias="0.1" />
+
+    <Button
+        android:id="@+id/newBoardDone"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:onClick="onDone"
+        android:text="@string/new_board_done_button"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toStartOf="@+id/newBoardPermission"
+        app:layout_constraintHorizontal_bias="1.0"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/newBoardPermission"
+        app:layout_constraintVertical_bias="0.40" />
+
+    <Button
+        android:id="@+id/newBoardCancel"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:onClick="onCancel"
+        android:text="@string/new_board_cancel_button"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintHorizontal_bias="0.0"
+        app:layout_constraintStart_toEndOf="@+id/newBoardPermission"
+        app:layout_constraintTop_toBottomOf="@+id/newBoardPermission"
+        app:layout_constraintVertical_bias="0.40" />
+</androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 4bd3ed4..142cd0b 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -2,4 +2,11 @@
     <string name="app_name">wedroid</string>
     <string name="title_activity_boards_list">BoardsListActivity</string>
     <string name="title_activity_boards_lists">Boards</string>
+    <string name="title_activity_new_board">NewBoardActivity</string>
+    <string name="new_board_name_field">Board name</string>
+    <string name="new_board_switch">Private</string>
+    <string name="new_board_cancel_button">Cancel</string>
+    <string name="new_board_done_button">Done</string>
+    <string name="on_null_new_board_name">There was a problem with the name of the new board</string>
+    <string name="on_add_new_board_error">It was not possible to add a new board</string>
 </resources>

From a645d5f64b91b8839b00c9415523b0476a17e905 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Tue, 3 Dec 2019 18:03:18 +0100
Subject: [PATCH 2/4] add BoardRepositoryTest

---
 app/build.gradle                              |   2 +
 .../ids/wedroid/app/BoardRepositoryTest.kt    | 171 ++++++++++++++++++
 2 files changed, 173 insertions(+)
 create mode 100644 app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt

diff --git a/app/build.gradle b/app/build.gradle
index 7de7ce6..dc54433 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -59,6 +59,8 @@ dependencies {
 
     // TESTING
     testImplementation 'junit:junit:4.12'
+    testImplementation "io.mockk:mockk:1.9.3"
+    testImplementation "com.squareup.okhttp3:mockwebserver:4.2.1"
     androidTestImplementation 'androidx.test.ext:junit:1.1.1'
     androidTestImplementation 'androidx.test.espresso:espresso-core:3.2.0'
     androidTestImplementation "androidx.room:room-testing:$rootProject.roomVersion"
diff --git a/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt
new file mode 100644
index 0000000..3e73d76
--- /dev/null
+++ b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt
@@ -0,0 +1,171 @@
+package it.unisannio.ding.ids.wedroid.app
+
+import io.mockk.coEvery
+import io.mockk.every
+import io.mockk.mockk
+import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+import it.unisannio.ding.ids.wedroid.app.data.repository.BoardRepository
+import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader
+import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
+import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor
+import junit.framework.TestCase.*
+import kotlinx.coroutines.runBlocking
+import okhttp3.mockwebserver.MockResponse
+import okhttp3.mockwebserver.MockWebServer
+import org.junit.After
+import org.junit.Before
+import org.junit.Test
+import retrofit2.Retrofit
+import retrofit2.converter.gson.GsonConverterFactory
+import java.net.HttpURLConnection
+import java.util.concurrent.CountDownLatch
+import java.util.concurrent.TimeUnit
+import kotlin.reflect.full.declaredFunctions
+import kotlin.reflect.jvm.isAccessible
+
+class BoardRepositoryTest {
+    private val reader = mockk<PreferenceReader>()
+    private val webServer = MockWebServer()
+    private lateinit var service: BoardService
+    private lateinit var latch: CountDownLatch
+
+    @Before
+    fun setUp() {
+        webServer.start()
+        service = Retrofit.Builder()
+            .baseUrl(webServer.url("/"))
+            .addConverterFactory(GsonConverterFactory.create())
+            .build()
+            .create(BoardService::class.java)
+
+        every { reader.userId } returns "user id"
+
+
+        latch = CountDownLatch(1)
+    }
+
+    @After
+    fun teardown() {
+        webServer.close()
+    }
+
+    @Test
+    fun insertWithOkOnServer() {
+        val dao = mockk<BoardDao>()
+        var inInsert = false
+
+        coEvery {
+            dao.insert(any())
+        } answers {
+            inInsert = true
+            assertEquals("id", arg<Board>(0).id)
+            assertEquals("title", arg<Board>(0).title)
+            latch.countDown()
+        }
+
+        webServer.enqueue(
+            MockResponse()
+                .setResponseCode(HttpURLConnection.HTTP_OK)
+                .setBody("{ \"_id\": \"id\" }")
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        repository.insertBoard(
+            "title", true, BoardBackgroundColor.LIMEGREEN
+        )
+
+        latch.await(5, TimeUnit.SECONDS)
+
+        assertTrue(inInsert)
+    }
+
+    @Test
+    fun notInsertWithErrorOnServer() {
+        val dao = mockk<BoardDao>()
+        var inInsert = false
+
+        coEvery {
+            dao.insert(any() as Board)
+        } answers {
+            inInsert = true
+            latch.countDown()
+        }
+
+        webServer.enqueue(
+            MockResponse()
+                .setResponseCode(HttpURLConnection.HTTP_BAD_REQUEST)
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        repository.insertBoard(
+            "title", true, BoardBackgroundColor.LIMEGREEN
+        )
+
+        latch.await(3, TimeUnit.SECONDS)
+
+        assertFalse(inInsert)
+    }
+
+    @Test
+    fun deleteWithOkOnServer() {
+        val dao = mockk<BoardDao>()
+        var inDelete = false
+
+        coEvery {
+            dao.delete(any() as Board)
+        } answers {
+            assertEquals("id", arg<Board>(0).id)
+            inDelete = true
+            latch.countDown()
+        }
+
+        webServer.enqueue(
+            MockResponse()
+                .setResponseCode(HttpURLConnection.HTTP_OK)
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        repository.deleteBoard("id")
+
+        latch.await(5, TimeUnit.SECONDS)
+        assertTrue(inDelete)
+    }
+
+    @Test
+    fun notDeleteWithErrorOnServer() {
+        val dao = mockk<BoardDao>()
+        var inDelete = false
+
+        coEvery {
+            dao.delete(any() as Board)
+        } answers {
+            assertEquals("id", arg<Board>(0).id)
+            inDelete = true
+            latch.countDown()
+        }
+
+        webServer.enqueue(
+            MockResponse()
+                .setResponseCode(HttpURLConnection.HTTP_BAD_REQUEST)
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        repository.deleteBoard("id")
+
+        latch.await(3, TimeUnit.SECONDS)
+        assertFalse(inDelete)
+    }
+}
\ No newline at end of file

From 44775d06192ed830816645133ad712e5460f31da Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Tue, 3 Dec 2019 19:48:29 +0100
Subject: [PATCH 3/4] refactor BoardRepository

---
 .../app/data/repository/BoardRepository.kt    | 120 ++++++++++--------
 1 file changed, 68 insertions(+), 52 deletions(-)

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
index 8e7675d..92eb29b 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
@@ -29,6 +29,39 @@ class BoardRepository(
         synchronize()
     }
 
+    fun synchronize() {
+        service.getBoardsFromUser(reader.userId)
+            .enqueue(object :
+                Callback<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>> {
+                override fun onFailure(
+                    call: Call<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>,
+                    t: Throwable
+                ) = logNetworkError(t.message)
+
+                override fun onResponse(
+                    call: Call<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>,
+                    response: Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>
+                ) = synchronizeCallback(response)
+            })
+    }
+
+    private fun synchronizeCallback(
+        response: Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>
+    ) {
+        if (!response.isSuccessful) {
+            logNetworkError("${response.code()} ${response.message()}")
+            return
+        }
+
+        // read boards from the body
+        val boards = (response.body() ?: return)
+            .map { it.convert() }
+
+        addNewBoardToDb(boards)
+
+        removeOldBoardsFromDb(boards)
+    }
+
     fun insertBoard(title: String, isPrivate: Boolean, color: BoardBackgroundColor) {
         val permission = if (isPrivate) BoardPermission.PRIVATE else BoardPermission.PUBLIC
 
@@ -48,54 +81,29 @@ class BoardRepository(
             override fun onResponse(
                 call: Call<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>,
                 response: Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>
-            ) {
-                if (!response.isSuccessful) {
-                    logNetworkError("${response.code()} ${response.message()}")
-                    return
-                }
-
-                val board = response.body()
-
-                if (board == null) {
-                    logNetworkError("empty body")
-                    return
-                }
-
-                CoroutineScope(Dispatchers.IO).launch {
-                    dao.insert(Board(board.id, title))
-                }
-            }
+            ) = insertBoardCallback(response, title)
         })
     }
 
-    fun synchronize() {
-        service.getBoardsFromUser(reader.userId)
-            .enqueue(object :
-                Callback<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>> {
+    private fun insertBoardCallback(
+        response: Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>,
+        title: String
+    ) {
+        if (!response.isSuccessful) {
+            logNetworkError("${response.code()} ${response.message()}")
+            return
+        }
 
-                override fun onFailure(
-                    call: Call<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>,
-                    t: Throwable
-                ) = logNetworkError(t.message)
+        val board = response.body()
 
-                override fun onResponse(
-                    call: Call<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>,
-                    response: Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>
-                ) {
-                    if (!response.isSuccessful) {
-                        logNetworkError("${response.code()} ${response.message()}")
-                        return
-                    }
+        if (board == null) {
+            logNetworkError("empty body")
+            return
+        }
 
-                    // read boards from the body
-                    val boards = (response.body() ?: return)
-                        .map { it.convert() }
-
-                    addNewBoardToDb(boards)
-
-                    removeOldBoardsFromDb(boards)
-                }
-            })
+        CoroutineScope(Dispatchers.IO).launch {
+            dao.insert(Board(board.id, title))
+        }
     }
 
     fun deleteBoard(id: String) {
@@ -105,19 +113,27 @@ class BoardRepository(
                     logNetworkError(t.message)
                 }
 
-                override fun onResponse(call: Call<Void>, response: Response<Void>) {
-                    if (!response.isSuccessful) {
-                        logNetworkError("${response.code()}, ${response.message()}")
-                        return
-                    }
-
-                    CoroutineScope(Dispatchers.IO).launch {
-                        dao.delete(Board(id))
-                    }
-                }
+                override fun onResponse(
+                    call: Call<Void>,
+                    response: Response<Void>
+                ) = deleteBoardCallback(response, id)
             })
     }
 
+    private fun deleteBoardCallback(
+        response: Response<Void>,
+        id: String
+    ) {
+        if (!response.isSuccessful) {
+            logNetworkError("${response.code()}, ${response.message()}")
+            return
+        }
+
+        CoroutineScope(Dispatchers.IO).launch {
+            dao.delete(Board(id))
+        }
+    }
+
     private fun addNewBoardToDb(boards: Collection<Board>) {
         boards.forEach {
             CoroutineScope(Dispatchers.IO).launch {

From 6d45b0bfae896c4e38f112ce26e0ca731b05caa7 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Wed, 4 Dec 2019 18:35:17 +0100
Subject: [PATCH 4/4] complete test BoardRepository

- refactoring BoardRepository for testing
---
 .../app/data/repository/BoardRepository.kt    |  44 +--
 .../ids/wedroid/app/BoardRepositoryTest.kt    | 171 ----------
 .../ding/ids/wedroid/app/TestHelper.kt        |  13 +
 .../data/repository/BoardRepositoryTest.kt    | 318 ++++++++++++++++++
 4 files changed, 355 insertions(+), 191 deletions(-)
 delete mode 100644 app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt
 create mode 100644 app/src/test/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
 create mode 100644 app/src/test/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepositoryTest.kt

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
index 92eb29b..204940d 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
@@ -41,11 +41,15 @@ class BoardRepository(
                 override fun onResponse(
                     call: Call<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>,
                     response: Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>
-                ) = synchronizeCallback(response)
+                ) {
+                    CoroutineScope(Dispatchers.IO).launch {
+                        synchronizeCallback(response)
+                    }
+                }
             })
     }
 
-    private fun synchronizeCallback(
+    private suspend fun synchronizeCallback(
         response: Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>
     ) {
         if (!response.isSuccessful) {
@@ -81,11 +85,15 @@ class BoardRepository(
             override fun onResponse(
                 call: Call<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>,
                 response: Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>
-            ) = insertBoardCallback(response, title)
+            ) {
+                CoroutineScope(Dispatchers.IO).launch {
+                    insertBoardCallback(response, title)
+                }
+            }
         })
     }
 
-    private fun insertBoardCallback(
+    private suspend fun insertBoardCallback(
         response: Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>,
         title: String
     ) {
@@ -101,9 +109,7 @@ class BoardRepository(
             return
         }
 
-        CoroutineScope(Dispatchers.IO).launch {
-            dao.insert(Board(board.id, title))
-        }
+        dao.insert(Board(board.id, title))
     }
 
     fun deleteBoard(id: String) {
@@ -116,38 +122,36 @@ class BoardRepository(
                 override fun onResponse(
                     call: Call<Void>,
                     response: Response<Void>
-                ) = deleteBoardCallback(response, id)
+                ) {
+                    CoroutineScope(Dispatchers.IO).launch {
+                        deleteBoardCallback(response, id)
+                    }
+                }
             })
     }
 
-    private fun deleteBoardCallback(
+    private suspend fun deleteBoardCallback(
         response: Response<Void>,
         id: String
     ) {
         if (!response.isSuccessful) {
-            logNetworkError("${response.code()}, ${response.message()}")
+            logNetworkError("${response.code()} ${response.message()}")
             return
         }
 
-        CoroutineScope(Dispatchers.IO).launch {
-            dao.delete(Board(id))
-        }
+        dao.delete(Board(id))
     }
 
-    private fun addNewBoardToDb(boards: Collection<Board>) {
+    private suspend fun addNewBoardToDb(boards: Collection<Board>) {
         boards.forEach {
-            CoroutineScope(Dispatchers.IO).launch {
-                dao.insert(it)
-            }
+            dao.insert(it)
         }
     }
 
-    private fun removeOldBoardsFromDb(boards: Collection<Board>) {
+    private suspend fun removeOldBoardsFromDb(boards: Collection<Board>) {
         allBoards.value?.minus(boards)
             ?.forEach {
-                CoroutineScope(Dispatchers.IO).launch {
                     dao.delete(it)
-                }
             }
     }
 
diff --git a/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt
deleted file mode 100644
index 3e73d76..0000000
--- a/app/src/test/java/it/unisannio/ding/ids/wedroid/app/BoardRepositoryTest.kt
+++ /dev/null
@@ -1,171 +0,0 @@
-package it.unisannio.ding.ids.wedroid.app
-
-import io.mockk.coEvery
-import io.mockk.every
-import io.mockk.mockk
-import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao
-import it.unisannio.ding.ids.wedroid.app.data.entity.Board
-import it.unisannio.ding.ids.wedroid.app.data.repository.BoardRepository
-import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader
-import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
-import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor
-import junit.framework.TestCase.*
-import kotlinx.coroutines.runBlocking
-import okhttp3.mockwebserver.MockResponse
-import okhttp3.mockwebserver.MockWebServer
-import org.junit.After
-import org.junit.Before
-import org.junit.Test
-import retrofit2.Retrofit
-import retrofit2.converter.gson.GsonConverterFactory
-import java.net.HttpURLConnection
-import java.util.concurrent.CountDownLatch
-import java.util.concurrent.TimeUnit
-import kotlin.reflect.full.declaredFunctions
-import kotlin.reflect.jvm.isAccessible
-
-class BoardRepositoryTest {
-    private val reader = mockk<PreferenceReader>()
-    private val webServer = MockWebServer()
-    private lateinit var service: BoardService
-    private lateinit var latch: CountDownLatch
-
-    @Before
-    fun setUp() {
-        webServer.start()
-        service = Retrofit.Builder()
-            .baseUrl(webServer.url("/"))
-            .addConverterFactory(GsonConverterFactory.create())
-            .build()
-            .create(BoardService::class.java)
-
-        every { reader.userId } returns "user id"
-
-
-        latch = CountDownLatch(1)
-    }
-
-    @After
-    fun teardown() {
-        webServer.close()
-    }
-
-    @Test
-    fun insertWithOkOnServer() {
-        val dao = mockk<BoardDao>()
-        var inInsert = false
-
-        coEvery {
-            dao.insert(any())
-        } answers {
-            inInsert = true
-            assertEquals("id", arg<Board>(0).id)
-            assertEquals("title", arg<Board>(0).title)
-            latch.countDown()
-        }
-
-        webServer.enqueue(
-            MockResponse()
-                .setResponseCode(HttpURLConnection.HTTP_OK)
-                .setBody("{ \"_id\": \"id\" }")
-        )
-
-        val repository = BoardRepository(
-            dao, service, reader
-        )
-
-        repository.insertBoard(
-            "title", true, BoardBackgroundColor.LIMEGREEN
-        )
-
-        latch.await(5, TimeUnit.SECONDS)
-
-        assertTrue(inInsert)
-    }
-
-    @Test
-    fun notInsertWithErrorOnServer() {
-        val dao = mockk<BoardDao>()
-        var inInsert = false
-
-        coEvery {
-            dao.insert(any() as Board)
-        } answers {
-            inInsert = true
-            latch.countDown()
-        }
-
-        webServer.enqueue(
-            MockResponse()
-                .setResponseCode(HttpURLConnection.HTTP_BAD_REQUEST)
-        )
-
-        val repository = BoardRepository(
-            dao, service, reader
-        )
-
-        repository.insertBoard(
-            "title", true, BoardBackgroundColor.LIMEGREEN
-        )
-
-        latch.await(3, TimeUnit.SECONDS)
-
-        assertFalse(inInsert)
-    }
-
-    @Test
-    fun deleteWithOkOnServer() {
-        val dao = mockk<BoardDao>()
-        var inDelete = false
-
-        coEvery {
-            dao.delete(any() as Board)
-        } answers {
-            assertEquals("id", arg<Board>(0).id)
-            inDelete = true
-            latch.countDown()
-        }
-
-        webServer.enqueue(
-            MockResponse()
-                .setResponseCode(HttpURLConnection.HTTP_OK)
-        )
-
-        val repository = BoardRepository(
-            dao, service, reader
-        )
-
-        repository.deleteBoard("id")
-
-        latch.await(5, TimeUnit.SECONDS)
-        assertTrue(inDelete)
-    }
-
-    @Test
-    fun notDeleteWithErrorOnServer() {
-        val dao = mockk<BoardDao>()
-        var inDelete = false
-
-        coEvery {
-            dao.delete(any() as Board)
-        } answers {
-            assertEquals("id", arg<Board>(0).id)
-            inDelete = true
-            latch.countDown()
-        }
-
-        webServer.enqueue(
-            MockResponse()
-                .setResponseCode(HttpURLConnection.HTTP_BAD_REQUEST)
-        )
-
-        val repository = BoardRepository(
-            dao, service, reader
-        )
-
-        repository.deleteBoard("id")
-
-        latch.await(3, TimeUnit.SECONDS)
-        assertFalse(inDelete)
-    }
-}
\ No newline at end of file
diff --git a/app/src/test/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
new file mode 100644
index 0000000..42a2ada
--- /dev/null
+++ b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
@@ -0,0 +1,13 @@
+package it.unisannio.ding.ids.wedroid.app
+
+import kotlin.reflect.KClass
+import kotlin.reflect.KFunction
+import kotlin.reflect.full.declaredFunctions
+import kotlin.reflect.jvm.isAccessible
+
+fun getPrivateFun(name: String, kClass: KClass<*>): KFunction<*>? {
+    return kClass.declaredFunctions
+        .find { it.name == name }
+        .also { it?.isAccessible = true }
+}
+
diff --git a/app/src/test/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepositoryTest.kt b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepositoryTest.kt
new file mode 100644
index 0000000..d8def75
--- /dev/null
+++ b/app/src/test/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepositoryTest.kt
@@ -0,0 +1,318 @@
+package it.unisannio.ding.ids.wedroid.app.data.repository
+
+import android.util.Log
+import androidx.lifecycle.LiveData
+import io.mockk.*
+import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+import it.unisannio.ding.ids.wedroid.app.getPrivateFun
+import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader
+import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
+import junit.framework.TestCase.*
+import kotlinx.coroutines.*
+import okhttp3.mockwebserver.MockWebServer
+import org.junit.After
+import org.junit.Before
+import org.junit.Test
+import retrofit2.Response
+import retrofit2.Retrofit
+import retrofit2.converter.gson.GsonConverterFactory
+import kotlin.reflect.full.callSuspend
+
+class BoardRepositoryTest {
+    private val reader = mockk<PreferenceReader>()
+    private val webServer = MockWebServer()
+    private lateinit var service: BoardService
+
+    @Before
+    fun setUp() {
+        webServer.start()
+        service = Retrofit.Builder()
+            .baseUrl(webServer.url("/"))
+            .addConverterFactory(GsonConverterFactory.create())
+            .build()
+            .create(BoardService::class.java)
+
+        mockkStatic(Log::class)
+        every { reader.userId } returns "user id"
+        every { Log.e(any(), any()) } returns 0
+    }
+
+    @After
+    fun teardown() {
+        webServer.close()
+    }
+
+    @Test
+    fun addNewBoardsToDb() {
+        val dao = mockk<BoardDao>()
+        var count = 0
+
+        coEvery {
+            dao.insert(any())
+        } answers {
+            count++
+        }
+
+        val addNewBoardToDb = getPrivateFun(
+            "addNewBoardToDb", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        val board0 = Board("id0", "title0")
+        val board1 = Board("id1", "title1")
+        val board2 = Board("id2", "title2")
+
+        runBlocking {
+            addNewBoardToDb?.callSuspend(
+                repository,
+                listOf(board0, board1, board2)
+            )
+        }
+
+        coVerifyAll {
+            dao.insert(board0)
+            dao.insert(board1)
+            dao.insert(board2)
+        }
+
+        assertEquals(3, count)
+    }
+
+    @Test
+    fun insertBoardCallbackSuccess() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>()
+        val board = mockk<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>()
+
+        every { response.isSuccessful } returns true
+        every { response.body() } returns board
+        every { board.id } returns "id"
+        coEvery { dao.insert(any()) } answers {}
+
+        val insertBoard = getPrivateFun(
+            "insertBoardCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            insertBoard?.callSuspend(
+                repository,
+                response,
+                "title"
+            )
+        }
+
+        coVerify { dao.insert(Board("id", "title")) }
+    }
+
+    @Test
+    fun insertBoardCallbackEmptyBody() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>()
+
+        every { response.isSuccessful } returns true
+        every { response.body() } returns null
+
+        val insertBoard = getPrivateFun(
+            "insertBoardCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            insertBoard?.callSuspend(
+                repository,
+                response,
+                "title"
+            )
+        }
+
+        verify { Log.e("RETROFIT", "empty body") }
+    }
+
+    @Test
+    fun insertBoardCallbackError() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>()
+
+        every { response.isSuccessful } returns false
+        every { response.code() } returns 400
+        every { response.message() } returns "Error"
+
+        val insertBoard = getPrivateFun(
+            "insertBoardCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            insertBoard?.callSuspend(
+                repository,
+                response,
+                "title"
+            )
+        }
+
+        verify { Log.e("RETROFIT", "400 Error") }
+    }
+
+    @Test
+    fun deleteBoardCallbackSuccess() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<Void>>()
+
+        every { response.isSuccessful } returns true
+        coEvery { dao.delete(any()) } answers {}
+
+        val deleteBoard = getPrivateFun(
+            "deleteBoardCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            deleteBoard?.callSuspend(
+                repository,
+                response,
+                "id"
+            )
+        }
+
+        coVerify { dao.delete(Board("id")) }
+    }
+
+    @Test
+    fun deleteBoardCallbackError() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<Void>>()
+
+        every { response.isSuccessful } returns false
+        every { response.code() } returns 400
+        every { response.message() } returns "Error"
+
+        val deleteBoard = getPrivateFun(
+            "deleteBoardCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            deleteBoard?.callSuspend(
+                repository,
+                response,
+                "id"
+            )
+        }
+
+        verify { Log.e("RETROFIT", "400 Error") }
+    }
+
+    @Test
+    fun deleteOldBoardFromDb() {
+        val dao = mockk<BoardDao>()
+        val dbBoards = mockk<LiveData<List<Board>>>()
+
+        val board0 = Board("id0", "title0")
+        val board1 = Board("id1", "title1")
+        val board2 = Board("id2", "title2")
+        val board3 = Board("id2", "title3")
+
+        every { dbBoards.value } returns listOf(
+            board0, board1, board2, board3
+        )
+        coEvery { dao.getAllBoard() } returns dbBoards
+        coEvery { dao.delete(any()) } answers {}
+
+        val removeOldBoards = getPrivateFun(
+            "removeOldBoardsFromDb", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            removeOldBoards?.callSuspend(
+                repository,
+                listOf(
+                    board0, board1, board3
+                )
+            )
+        }
+
+        coVerify { dao.delete(board2) }
+    }
+
+    @Test
+    fun synchronizeCallbackError() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>>()
+
+        every { response.isSuccessful } returns false
+        every { response.code() } returns 400
+        every { response.message() } returns "Error"
+
+        val synchronize = getPrivateFun(
+            "synchronizeCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            synchronize?.callSuspend(
+                repository,
+                response
+            )
+        }
+
+        verify { Log.e("RETROFIT", "400 Error") }
+    }
+
+    @Test
+    fun synchronizeCallbackEmptyBody() {
+        val dao = mockk<BoardDao>()
+        val response =
+            mockk<Response<MutableList<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>>>()
+
+        every { response.isSuccessful } returns true
+        every { response.body() } returns null
+
+        val synchronize = getPrivateFun(
+            "synchronizeCallback", BoardRepository::class
+        )
+
+        val repository = BoardRepository(
+            dao, service, reader
+        )
+
+        runBlocking {
+            synchronize?.callSuspend(
+                repository,
+                response
+            )
+        }
+    }
+}
\ No newline at end of file