From 6c3139b891843603d235b80ed300f898fde34335 Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Tue, 26 Nov 2019 17:41:00 +0100
Subject: [PATCH 01/29] add login activity

---
 app/build.gradle                              |  2 +
 app/src/main/AndroidManifest.xml              |  2 +
 .../ding/ids/wedroid/app/LoginActivity.kt     | 75 +++++++++++++++++++
 .../ding/ids/wedroid/app/MainActivity.kt      |  9 +++
 app/src/main/res/layout/activity_login.xml    | 56 ++++++++++++++
 app/src/main/res/layout/activity_main.xml     | 11 +++
 6 files changed, 155 insertions(+)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
 create mode 100644 app/src/main/res/layout/activity_login.xml

diff --git a/app/build.gradle b/app/build.gradle
index c0fc443..cc33006 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -32,4 +32,6 @@ dependencies {
     testImplementation 'junit:junit:4.12'
     androidTestImplementation 'androidx.test.ext:junit:1.1.1'
     androidTestImplementation 'androidx.test.espresso:espresso-core:3.2.0'
+    implementation "com.squareup.retrofit2:retrofit:2.6.2"
+    implementation "com.squareup.retrofit2:converter-gson:2.6.2"
 }
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 253fb67..558d39e 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="it.unisannio.ding.ids.wedroid.app">
+    <uses-permission android:name="android.permission.INTERNET"/>
 
     <application
         android:allowBackup="true"
@@ -9,6 +10,7 @@
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
+        <activity android:name=".LoginActivity"></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/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
new file mode 100644
index 0000000..4107b45
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
@@ -0,0 +1,75 @@
+package it.unisannio.ding.ids.wedroid.app
+
+import androidx.appcompat.app.AppCompatActivity
+import android.os.Bundle
+import android.util.Log
+import android.view.View
+import android.widget.Toast
+import it.unisannio.ding.ids.wedroid.wrapper.api.LoginService
+import it.unisannio.ding.ids.wedroid.wrapper.entity.UserPrototype
+import kotlinx.android.synthetic.main.activity_login.*
+import retrofit2.Call
+import retrofit2.Callback
+import retrofit2.Response
+import retrofit2.Retrofit
+import retrofit2.converter.gson.GsonConverterFactory
+
+class LoginActivity : AppCompatActivity() {
+
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_login)
+
+
+
+    }
+
+    fun loginButton(v: View) {
+        if (username.text.isBlank() || istanceServer.text.isBlank() ||  password.text.isBlank()) {
+            Toast.makeText(this, "Riempire tutti i campi", Toast.LENGTH_LONG)
+                .show()
+            return
+        }
+
+
+
+        val userNameText = username.text.toString()
+        val passwordText = password.text.toString()
+
+        val service = Retrofit.Builder()
+            .addConverterFactory(GsonConverterFactory.create())
+            .baseUrl(istanceServer.text.toString())
+            .build()
+            .create(LoginService::class.java)
+
+
+
+        service.login(userNameText, passwordText).enqueue(object : Callback<UserPrototype> {
+            override fun onFailure(call: Call<UserPrototype>, t: Throwable) {
+                Toast.makeText(
+                    applicationContext,
+                    "Controlla la tua connessione internet",
+                    Toast.LENGTH_LONG
+                ).show()
+            }
+
+            override fun onResponse(call: Call<UserPrototype>, response: Response<UserPrototype>) {
+                if (response.code() != 200) {
+                    Toast.makeText(
+                        applicationContext,
+                        "Credenziali non corrette",
+                        Toast.LENGTH_LONG
+                    ).show()
+                    return
+                }
+                Toast.makeText(
+                    applicationContext,
+                    "Login effettuato con successo",
+                    Toast.LENGTH_LONG
+                ).show()
+
+            }
+
+        })
+    }
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
index e5390c0..97c3848 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
@@ -1,7 +1,9 @@
 package it.unisannio.ding.ids.wedroid.app
 
+import android.content.Intent
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
+import android.view.View
 import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
 
 class MainActivity : AppCompatActivity() {
@@ -10,5 +12,12 @@ class MainActivity : AppCompatActivity() {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_main)
         val service : BoardService? = null
+
+    }
+
+    fun login(v: View) {
+        startActivity(
+            Intent(this, LoginActivity::class.java)
+        )
     }
 }
diff --git a/app/src/main/res/layout/activity_login.xml b/app/src/main/res/layout/activity_login.xml
new file mode 100644
index 0000000..b48dab0
--- /dev/null
+++ b/app/src/main/res/layout/activity_login.xml
@@ -0,0 +1,56 @@
+<?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"
+    tools:context=".LoginActivity">
+
+    <EditText
+        android:id="@+id/username"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginBottom="100dp"
+        android:ems="10"
+        android:hint="UserName"
+        android:inputType="textPersonName"
+        app:layout_constraintBottom_toTopOf="@+id/password"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent" />
+
+    <EditText
+        android:id="@+id/password"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginBottom="50dp"
+        android:ems="10"
+        android:hint="Password"
+        android:inputType="textPassword"
+        app:layout_constraintBottom_toTopOf="@+id/button"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent" />
+
+    <Button
+        android:id="@+id/button"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginBottom="200dp"
+        android:onClick="loginButton"
+        android:text="LOGIN"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent" />
+
+    <EditText
+        android:id="@+id/istanceServer"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginBottom="50dp"
+        android:ems="10"
+        android:hint="Istance Server"
+        android:inputType="textPersonName"
+        app:layout_constraintBottom_toTopOf="@+id/username"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent" />
+
+</androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index 4fc2444..af20ea2 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -7,6 +7,7 @@
     tools:context=".MainActivity">
 
     <TextView
+        android:id="@+id/textView"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:text="Hello World!"
@@ -15,4 +16,14 @@
         app:layout_constraintRight_toRightOf="parent"
         app:layout_constraintTop_toTopOf="parent" />
 
+    <Button
+        android:id="@+id/button2"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:onClick="login"
+        android:text="Button"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/textView"
+        tools:layout_editor_absoluteX="168dp" />
+
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file

From 923db743ac6919ff6f286b71a5e970dfad0d0d64 Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Tue, 26 Nov 2019 18:16:34 +0100
Subject: [PATCH 02/29] fixed login button

---
 app/src/main/res/layout/activity_main.xml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index af20ea2..f6fd27e 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -23,7 +23,8 @@
         android:onClick="login"
         android:text="Button"
         app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintTop_toBottomOf="@+id/textView"
-        tools:layout_editor_absoluteX="168dp" />
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/textView" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file

From 8dae5179435060e6228fc201987194f45b8f955c Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Thu, 28 Nov 2019 12:08:37 +0100
Subject: [PATCH 03/29] add util

- add PreferenceReader interface
- add PreferenceWriter interface
- add lazy singleton ServicesFactory
- add retrofit dependencies into app module
---
 app/build.gradle                              |  4 ++
 .../wedroid/app/util/PreferenceReader.java    |  7 +++
 .../wedroid/app/util/PreferenceWriter.java    |  7 +++
 .../ids/wedroid/app/util/ServicesFactory.kt   | 59 +++++++++++++++++++
 4 files changed, 77 insertions(+)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceReader.java
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceWriter.java
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt

diff --git a/app/build.gradle b/app/build.gradle
index c0fc443..5fb3211 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -24,6 +24,10 @@ android {
 }
 
 dependencies {
+    // retrofit
+    implementation "com.squareup.retrofit2:retrofit:2.6.2"
+    implementation "com.squareup.retrofit2:converter-gson:2.6.2"
+
     implementation project(':wrapper')
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation 'androidx.appcompat:appcompat:1.1.0'
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceReader.java b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceReader.java
new file mode 100644
index 0000000..3fe9de0
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceReader.java
@@ -0,0 +1,7 @@
+package it.unisannio.ding.ids.wedroid.app.util;
+
+public interface PreferenceReader {
+    String getBaseUrl();
+    String getUserId();
+    String getToken();
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceWriter.java b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceWriter.java
new file mode 100644
index 0000000..264b507
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/PreferenceWriter.java
@@ -0,0 +1,7 @@
+package it.unisannio.ding.ids.wedroid.app.util;
+
+public interface PreferenceWriter {
+    void setBaseUrl(String baseUrl);
+    void setUserId(String userId);
+    void setToken(String token);
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
new file mode 100644
index 0000000..aa0c3d2
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
@@ -0,0 +1,59 @@
+package it.unisannio.ding.ids.wedroid.app.util
+
+import it.unisannio.ding.ids.wedroid.wrapper.api.*
+import okhttp3.OkHttpClient
+import retrofit2.Retrofit
+import retrofit2.converter.gson.GsonConverterFactory
+
+object ServicesFactory {
+    private val retrofit: Retrofit
+    private lateinit var reader: PreferenceReader //TODO
+
+    init {
+        val httpClient = OkHttpClient.Builder()
+            .addInterceptor {
+                val request = it.request().newBuilder()
+                    .addHeader("Authorization", "Bearer ${reader.token}")
+                    .addHeader("Accept", "application/json")
+                    .addHeader("Content-type", "application/json")
+                    .build()
+
+                it.proceed(request)
+            }.build()
+
+        retrofit = Retrofit.Builder()
+            .baseUrl(reader.baseUrl)
+            .addConverterFactory(GsonConverterFactory.create())
+            .client(httpClient)
+            .build()
+    }
+
+    val boardService by lazy {
+        retrofit.create(BoardService::class.java)
+    }
+
+    val cardCommentService by lazy {
+        retrofit.create(CardCommentService::class.java)
+    }
+
+    val cardService by lazy {
+        retrofit.create(CardService::class.java)
+    }
+
+    val checklistService by lazy {
+        retrofit.create(ChecklistService::class.java)
+    }
+
+    val listService by lazy {
+        retrofit.create(ListService::class.java)
+    }
+
+    val swimlanesService by lazy {
+        retrofit.create(SwimlanesService::class.java)
+    }
+
+    val userService by lazy {
+        retrofit.create(UserService::class.java)
+    }
+}
+

From d1636ac9d996c543f787fd683a92d6c3b331157e Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Fri, 29 Nov 2019 09:51:46 +0100
Subject: [PATCH 04/29] add shared preference

---
 .../ding/ids/wedroid/app/LoginActivity.kt     | 20 ++++++++---
 .../app/util/SharedPreferenceHelper.kt        | 35 +++++++++++++++++++
 2 files changed, 51 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
index 4107b45..da250cf 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
@@ -1,10 +1,13 @@
 package it.unisannio.ding.ids.wedroid.app
 
+import android.content.Context
+import android.content.SharedPreferences
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
 import android.util.Log
 import android.view.View
 import android.widget.Toast
+import it.unisannio.ding.ids.wedroid.app.util.SharedPreferenceHelper
 import it.unisannio.ding.ids.wedroid.wrapper.api.LoginService
 import it.unisannio.ding.ids.wedroid.wrapper.entity.UserPrototype
 import kotlinx.android.synthetic.main.activity_login.*
@@ -15,24 +18,27 @@ import retrofit2.Retrofit
 import retrofit2.converter.gson.GsonConverterFactory
 
 class LoginActivity : AppCompatActivity() {
+    lateinit var sph: SharedPreferenceHelper
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_login)
+        sph = SharedPreferenceHelper(this)
 
-
-
+        val id = sph.userId
+        val token = sph.token
+        val url = sph.baseUrl
+        Log.d("sharedPreference", "userId: $id\n token: $token\n url: $url")
     }
 
     fun loginButton(v: View) {
-        if (username.text.isBlank() || istanceServer.text.isBlank() ||  password.text.isBlank()) {
+        if (username.text.isBlank() || istanceServer.text.isBlank() || password.text.isBlank()) {
             Toast.makeText(this, "Riempire tutti i campi", Toast.LENGTH_LONG)
                 .show()
             return
         }
 
 
-
         val userNameText = username.text.toString()
         val passwordText = password.text.toString()
 
@@ -54,6 +60,7 @@ class LoginActivity : AppCompatActivity() {
             }
 
             override fun onResponse(call: Call<UserPrototype>, response: Response<UserPrototype>) {
+
                 if (response.code() != 200) {
                     Toast.makeText(
                         applicationContext,
@@ -62,6 +69,11 @@ class LoginActivity : AppCompatActivity() {
                     ).show()
                     return
                 }
+                val users = response.body()
+                sph.baseUrl = istanceServer.text.toString()
+                sph.token = users?.token
+                sph.userId = users?.id
+
                 Toast.makeText(
                     applicationContext,
                     "Login effettuato con successo",
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt
new file mode 100644
index 0000000..2f150c8
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt
@@ -0,0 +1,35 @@
+package it.unisannio.ding.ids.wedroid.app.util
+
+import android.content.Context
+
+class SharedPreferenceHelper(context : Context) : PreferenceReader, PreferenceWriter {
+    private val sp = context.getSharedPreferences("userinfo", Context.MODE_PRIVATE)
+
+    override fun getBaseUrl(): String? {
+        return sp.getString("url", "")
+    }
+
+    override fun getUserId(): String? {
+        return sp.getString("id", "")
+    }
+
+    override fun getToken(): String? {
+        return sp.getString("token", "")
+    }
+
+    override fun setBaseUrl(baseUrl: String?) {
+        val editor = sp.edit()
+        editor.putString("url", baseUrl).apply()
+
+           }
+
+    override fun setUserId(userId: String?) {
+        val editor = sp.edit()
+        editor.putString("id", userId).apply()
+    }
+
+    override fun setToken(token: String?) {
+        val editor = sp.edit()
+        editor.putString("token", token).apply()
+    }
+}
\ No newline at end of file

From 41505a0ca6f4ac552ebfeadbc9dedbe4a8293a6a Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sat, 30 Nov 2019 11:51:09 +0100
Subject: [PATCH 05/29] init BoardsListActivity

- add room database for Board
- add BoardsListActivity
- add BoardsListViewModel
- add BoardRepository
  - add insertBoard
  - add deleteBoard
  - add syncronize
---
 app/build.gradle                              |  37 ++++-
 app/src/main/AndroidManifest.xml              |   7 +
 .../ding/ids/wedroid/app/MainActivity.kt      |  11 +-
 .../ding/ids/wedroid/app/data/dao/BoardDao.kt |  17 +++
 .../app/data/database/BoardDatabase.kt        |  35 +++++
 .../ding/ids/wedroid/app/data/entity/Board.kt |  16 ++
 .../app/data/repository/BoardRepository.kt    | 138 ++++++++++++++++++
 .../wedroid/app/view/BoardsListsActivity.kt   |  38 +++++
 .../app/view/adapter/BoardsListAdapter.kt     |  44 ++++++
 .../app/viewmodel/BoardListViewModel.kt       |  25 ++++
 .../main/res/layout/activity_boards_lists.xml |  33 +++++
 app/src/main/res/layout/activity_main.xml     |  12 ++
 .../main/res/layout/board_recycle_item.xml    |  12 ++
 .../main/res/layout/content_boards_lists.xml  |  19 +++
 app/src/main/res/values/dimens.xml            |   3 +
 app/src/main/res/values/strings.xml           |   2 +
 app/src/main/res/values/styles.xml            |   9 ++
 build.gradle                                  |   9 ++
 18 files changed, 460 insertions(+), 7 deletions(-)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
 create mode 100644 app/src/main/res/layout/activity_boards_lists.xml
 create mode 100644 app/src/main/res/layout/board_recycle_item.xml
 create mode 100644 app/src/main/res/layout/content_boards_lists.xml
 create mode 100644 app/src/main/res/values/dimens.xml

diff --git a/app/build.gradle b/app/build.gradle
index 5fb3211..1112cf7 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -4,6 +4,8 @@ apply plugin: 'kotlin-android'
 
 apply plugin: 'kotlin-android-extensions'
 
+apply plugin: 'kotlin-kapt'
+
 android {
     compileSdkVersion 29
     buildToolsVersion "29.0.2"
@@ -21,19 +23,44 @@ android {
             proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
         }
     }
+    packagingOptions {
+        exclude 'META-INF/atomicfu.kotlin_module'
+    }
 }
 
 dependencies {
-    // retrofit
-    implementation "com.squareup.retrofit2:retrofit:2.6.2"
-    implementation "com.squareup.retrofit2:converter-gson:2.6.2"
-
-    implementation project(':wrapper')
+    // standard
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation 'androidx.appcompat:appcompat:1.1.0'
     implementation 'androidx.core:core-ktx:1.1.0'
     implementation 'androidx.constraintlayout:constraintlayout:1.1.3'
+    implementation 'com.google.android.material:material:1.0.0'
+    // wrapper
+    implementation project(':wrapper')
+    // retrofit
+    implementation "com.squareup.retrofit2:retrofit:2.6.2"
+    implementation "com.squareup.retrofit2:converter-gson:2.6.2"
+    // room database
+    implementation "androidx.room:room-runtime:$rootProject.roomVersion"
+    implementation "androidx.room:room-ktx:$rootProject.roomVersion"
+    kapt "androidx.room:room-compiler:$rootProject.roomVersion"
+    // lifecycle components
+    implementation "androidx.lifecycle:lifecycle-extensions:$rootProject.archLifecycleVersion"
+    //noinspection LifecycleAnnotationProcessorWithJava8
+    kapt "androidx.lifecycle:lifecycle-compiler:$rootProject.archLifecycleVersion"
+    // ViewModel Kotlin support
+    implementation "androidx.lifecycle:lifecycle-viewmodel-ktx:$rootProject.archLifecycleVersion"
+    // Coroutines
+    api "org.jetbrains.kotlinx:kotlinx-coroutines-android:$rootProject.coroutines"
+    // UI
+    implementation "com.google.android.material:material:$rootProject.materialVersion"
+
+
+    // TESTING
     testImplementation 'junit:junit:4.12'
     androidTestImplementation 'androidx.test.ext:junit:1.1.1'
     androidTestImplementation 'androidx.test.espresso:espresso-core:3.2.0'
+    androidTestImplementation "androidx.room:room-testing:$rootProject.roomVersion"
+    androidTestImplementation "androidx.arch.core:core-testing:$rootProject.androidxArchVersion"
+    androidTestImplementation "androidx.arch.core:core-testing:$rootProject.coreTestingVersion"
 }
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 253fb67..e589c5d 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -2,6 +2,8 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="it.unisannio.ding.ids.wedroid.app">
 
+    <uses-permission android:name="android.permission.INTERNET"/>
+
     <application
         android:allowBackup="true"
         android:icon="@mipmap/ic_launcher"
@@ -9,6 +11,11 @@
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
+        <activity
+            android:name=".view.BoardsListsActivity"
+            android:label="@string/title_activity_boards_lists"
+            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/MainActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
index e5390c0..4f79a49 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/MainActivity.kt
@@ -1,14 +1,21 @@
 package it.unisannio.ding.ids.wedroid.app
 
+import android.content.Intent
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
-import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
+import android.view.View
+import it.unisannio.ding.ids.wedroid.app.view.BoardsListsActivity
 
 class MainActivity : AppCompatActivity() {
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_main)
-        val service : BoardService? = null
+    }
+
+    fun open(v: View) {
+        startActivity(
+            Intent(this, BoardsListsActivity::class.java)
+        )
     }
 }
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
new file mode 100644
index 0000000..ca88fa4
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
@@ -0,0 +1,17 @@
+package it.unisannio.ding.ids.wedroid.app.data.dao
+
+import androidx.lifecycle.LiveData
+import androidx.room.*
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+
+@Dao
+interface BoardDao {
+    @Query("SELECT * from board_table ORDER BY title ASC")
+    fun getAllBoard(): LiveData<List<Board>>
+
+    @Insert(onConflict = OnConflictStrategy.REPLACE)
+    suspend fun insert(board: Board)
+
+    @Delete
+    suspend fun delete(board: Board)
+}
\ No newline at end of file
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
new file mode 100644
index 0000000..e99fc50
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
@@ -0,0 +1,35 @@
+package it.unisannio.ding.ids.wedroid.app.data.database
+
+import android.content.Context
+import androidx.room.Database
+import androidx.room.Room
+import androidx.room.RoomDatabase
+import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+
+@Database(entities = [Board::class], version = 1, exportSchema = false)
+abstract class BoardDatabase : RoomDatabase() {
+
+    abstract fun boardDao(): BoardDao
+
+    companion object {
+        @Volatile
+        private var INSTANCE: BoardDatabase? = null
+
+        fun getDatabase(context: Context): BoardDatabase {
+            val tempInstance = INSTANCE
+            if (tempInstance != null) {
+                return tempInstance
+            }
+            synchronized(this) {
+                val instance = Room.databaseBuilder(
+                    context.applicationContext,
+                    BoardDatabase::class.java,
+                    "board_database"
+                ).build()
+                INSTANCE = instance
+                return instance
+            }
+        }
+    }
+}
\ No newline at end of file
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
new file mode 100644
index 0000000..0cd1f70
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
@@ -0,0 +1,16 @@
+package it.unisannio.ding.ids.wedroid.app.data.entity
+
+import androidx.room.ColumnInfo
+import androidx.room.Entity
+import androidx.room.PrimaryKey
+
+@Entity(tableName = "board_table")
+data class Board(
+    @PrimaryKey @ColumnInfo(name = "id") val id: String,
+    @ColumnInfo(name = "title") val title: String = ""
+)
+
+fun it.unisannio.ding.ids.wedroid.wrapper.entity.Board.convert(): Board {
+    return Board(this.id, this.title)
+}
+
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
new file mode 100644
index 0000000..22179c0
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/repository/BoardRepository.kt
@@ -0,0 +1,138 @@
+package it.unisannio.ding.ids.wedroid.app.data.repository
+
+import android.util.Log
+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.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.BoardPrototype
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.launch
+import retrofit2.Call
+import retrofit2.Callback
+import retrofit2.Response
+
+class BoardRepository(
+    private val dao: BoardDao,
+    private val service: BoardService,
+    private val reader: PreferenceReader
+) {
+    val allBoards by lazy {
+        dao.getAllBoard()
+    }
+
+    init {
+        synchronize()
+    }
+
+    fun insertBoard(title: String) {
+        service.newBoard(
+            BoardPrototype.Builder()
+                .setOwner(reader.userId)
+                .setTitle(title)
+                .setBackgroundColor(BoardBackgroundColor.LIMEGREEN)
+                .build()
+        ).enqueue(object : Callback<it.unisannio.ding.ids.wedroid.wrapper.entity.Board> {
+            override fun onFailure(
+                call: Call<it.unisannio.ding.ids.wedroid.wrapper.entity.Board>,
+                t: Throwable
+            ) = logNetworkError(t.message)
+
+            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))
+                }
+            }
+        })
+    }
+
+    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>>
+                ) {
+                    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 deleteBoard(id: String) {
+        service.deleteBoard(id).enqueue(
+            object : Callback<Void> {
+                override fun onFailure(call: Call<Void>, t: Throwable) {
+                    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))
+                    }
+                }
+            })
+    }
+
+    private fun addNewBoardToDb(boards: Collection<Board>) {
+        boards.forEach {
+            CoroutineScope(Dispatchers.IO).launch {
+                dao.insert(it)
+            }
+        }
+    }
+
+    private fun removeOldBoardsFromDb(boards: Collection<Board>) {
+        allBoards.value?.minus(boards)
+            ?.forEach {
+                CoroutineScope(Dispatchers.IO).launch {
+                    dao.delete(it)
+                }
+            }
+    }
+
+    private fun logNetworkError(message: String?) {
+        Log.e("RETROFIT", message)
+    }
+}
+
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
new file mode 100644
index 0000000..702e7ac
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivity.kt
@@ -0,0 +1,38 @@
+package it.unisannio.ding.ids.wedroid.app.view
+
+import android.os.Bundle
+import androidx.appcompat.app.AppCompatActivity
+import androidx.lifecycle.Observer
+import androidx.lifecycle.ViewModelProvider
+import androidx.recyclerview.widget.LinearLayoutManager
+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.BoardListViewModel
+
+import kotlinx.android.synthetic.main.activity_boards_lists.*
+import kotlinx.android.synthetic.main.content_boards_lists.*
+
+class BoardsListsActivity : AppCompatActivity() {
+    private lateinit var viewModel: BoardListViewModel
+
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_boards_lists)
+        setSupportActionBar(toolbar)
+        viewModel = ViewModelProvider(this).get(BoardListViewModel::class.java)
+
+        val adapter = BoardsListAdapter(this)
+        boardList.adapter = adapter
+        boardList.layoutManager = LinearLayoutManager(this)
+
+        viewModel.allBoards.observe(this, Observer {
+            it.let { adapter.setBoards(it) }
+        })
+
+        fab.setOnClickListener { view ->
+            viewModel.insertBoard("New board")
+        }
+    }
+
+}
+
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
new file mode 100644
index 0000000..56d5ddb
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
@@ -0,0 +1,44 @@
+package it.unisannio.ding.ids.wedroid.app.view.adapter
+
+import android.content.Context
+import android.view.LayoutInflater
+import android.view.View
+import android.view.ViewGroup
+import android.widget.TextView
+import androidx.recyclerview.widget.RecyclerView
+import it.unisannio.ding.ids.wedroid.app.R
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+
+
+class BoardsListAdapter internal constructor(
+    context: Context
+) : RecyclerView.Adapter<BoardsListAdapter.BoardViewHolder>() {
+
+    private val inflater = LayoutInflater.from(context)
+    private var boards = emptyList<Board>()
+
+    inner class BoardViewHolder(
+        view: View
+    ) : RecyclerView.ViewHolder(view) {
+        val boardTitle: TextView = view.findViewById(R.id.boardTitle)
+    }
+
+    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): BoardViewHolder {
+        val view = inflater.inflate(R.layout.board_recycle_item, parent, false)
+        return BoardViewHolder(view)
+    }
+
+    override fun getItemCount(): Int {
+        return boards.size
+    }
+
+    override fun onBindViewHolder(holder: BoardViewHolder, position: Int) {
+        val board = boards[position]
+        holder.boardTitle.text = board.title
+    }
+
+    internal fun setBoards(boards: List<Board>) {
+        this.boards = boards
+        notifyDataSetChanged()
+    }
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
new file mode 100644
index 0000000..582051d
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
@@ -0,0 +1,25 @@
+package it.unisannio.ding.ids.wedroid.app.viewmodel
+
+import android.app.Application
+import androidx.lifecycle.AndroidViewModel
+import androidx.lifecycle.LiveData
+import it.unisannio.ding.ids.wedroid.app.util.ServicesFactory
+import it.unisannio.ding.ids.wedroid.app.data.database.BoardDatabase
+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.DumReader
+
+class BoardListViewModel(application: Application) : AndroidViewModel(application) {
+
+    private val repository: BoardRepository = BoardRepository(
+        BoardDatabase.getDatabase(application).boardDao(),
+        ServicesFactory.boardService,
+        DumReader()
+    )
+
+    val allBoards: LiveData<List<Board>> = repository.allBoards
+
+    fun insertBoard(title: String) = repository.insertBoard(title)
+
+    fun deleteBoard(id: String) = repository.deleteBoard(id)
+}
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_boards_lists.xml b/app/src/main/res/layout/activity_boards_lists.xml
new file mode 100644
index 0000000..985531a
--- /dev/null
+++ b/app/src/main/res/layout/activity_boards_lists.xml
@@ -0,0 +1,33 @@
+<?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.BoardsListsActivity">
+
+    <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_boards_lists" />
+
+    <com.google.android.material.floatingactionbutton.FloatingActionButton
+        android:id="@+id/fab"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_gravity="bottom|end"
+        android:layout_margin="@dimen/fab_margin"
+        app:srcCompat="@android:drawable/ic_dialog_email" />
+
+</androidx.coordinatorlayout.widget.CoordinatorLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index 4fc2444..dca2774 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -7,6 +7,7 @@
     tools:context=".MainActivity">
 
     <TextView
+        android:id="@+id/textView"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:text="Hello World!"
@@ -15,4 +16,15 @@
         app:layout_constraintRight_toRightOf="parent"
         app:layout_constraintTop_toTopOf="parent" />
 
+    <Button
+        android:id="@+id/button"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:onClick="open"
+        android:text="Button"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/textView" />
+
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/board_recycle_item.xml b/app/src/main/res/layout/board_recycle_item.xml
new file mode 100644
index 0000000..fa14d0d
--- /dev/null
+++ b/app/src/main/res/layout/board_recycle_item.xml
@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical" android:layout_width="match_parent"
+    android:layout_height="wrap_content">
+
+    <TextView
+        android:id="@+id/boardTitle"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:text="TextView" />
+
+</LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/content_boards_lists.xml b/app/src/main/res/layout/content_boards_lists.xml
new file mode 100644
index 0000000..cb2ba98
--- /dev/null
+++ b/app/src/main/res/layout/content_boards_lists.xml
@@ -0,0 +1,19 @@
+<?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.BoardsListsActivity"
+    tools:showIn="@layout/activity_boards_lists">
+
+    <androidx.recyclerview.widget.RecyclerView
+        android:layout_width="0dp"
+        android:layout_height="0dp"
+        android:id="@+id/boardList"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+</androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
new file mode 100644
index 0000000..59a0b0c
--- /dev/null
+++ b/app/src/main/res/values/dimens.xml
@@ -0,0 +1,3 @@
+<resources>
+    <dimen name="fab_margin">16dp</dimen>
+</resources>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 908274d..4bd3ed4 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -1,3 +1,5 @@
 <resources>
     <string name="app_name">wedroid</string>
+    <string name="title_activity_boards_list">BoardsListActivity</string>
+    <string name="title_activity_boards_lists">Boards</string>
 </resources>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 5885930..545b9c6 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -8,4 +8,13 @@
         <item name="colorAccent">@color/colorAccent</item>
     </style>
 
+    <style name="AppTheme.NoActionBar">
+        <item name="windowActionBar">false</item>
+        <item name="windowNoTitle">true</item>
+    </style>
+
+    <style name="AppTheme.AppBarOverlay" parent="ThemeOverlay.AppCompat.Dark.ActionBar" />
+
+    <style name="AppTheme.PopupOverlay" parent="ThemeOverlay.AppCompat.Light" />
+
 </resources>
diff --git a/build.gradle b/build.gradle
index f4ed563..033fb3c 100644
--- a/build.gradle
+++ b/build.gradle
@@ -23,6 +23,15 @@ allprojects {
     }
 }
 
+ext {
+    roomVersion = '2.2.2'
+    archLifecycleVersion = '2.2.0-rc02'
+    androidxArchVersion = '2.1.0'
+    coreTestingVersion = "2.1.0"
+    coroutines = '1.3.2'
+    materialVersion = "1.0.0"
+}
+
 task clean(type: Delete) {
     delete rootProject.buildDir
 }

From 8c1b0cb6a07b87023238a488424ba40a27396811 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sat, 30 Nov 2019 12:13:01 +0100
Subject: [PATCH 06/29] edit ServicesFactory

---
 .../ids/wedroid/app/util/ServicesFactory.kt   | 23 +++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
index aa0c3d2..b64032e 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
@@ -5,9 +5,10 @@ import okhttp3.OkHttpClient
 import retrofit2.Retrofit
 import retrofit2.converter.gson.GsonConverterFactory
 
-object ServicesFactory {
+class ServicesFactory private constructor(
+    reader: PreferenceReader
+) {
     private val retrofit: Retrofit
-    private lateinit var reader: PreferenceReader //TODO
 
     init {
         val httpClient = OkHttpClient.Builder()
@@ -55,5 +56,23 @@ object ServicesFactory {
     val userService by lazy {
         retrofit.create(UserService::class.java)
     }
+
+    companion object {
+        @Volatile
+        private var INSTANCE: ServicesFactory? = null
+
+        fun getInstance(reader: PreferenceReader): ServicesFactory {
+            val tempInstance = INSTANCE
+
+            if (tempInstance != null)
+                return tempInstance
+
+            synchronized(this) {
+                val instance = ServicesFactory(reader)
+                INSTANCE = instance
+                return instance
+            }
+        }
+    }
 }
 

From 8e9e001c4b7e1eae1e0d32dce855c8f56fec163b Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sat, 30 Nov 2019 14:43:52 +0100
Subject: [PATCH 07/29] refactoring

---
 .../app/data/database/BoardDatabase.java      | 30 +++++++++++++
 .../app/data/database/BoardDatabase.kt        | 35 ---------------
 .../wedroid/app/view/BoardsListsActivity.kt   |  6 +--
 .../app/viewmodel/BoardListViewModel.kt       | 25 -----------
 .../app/viewmodel/BoardsListViewModel.java    | 44 +++++++++++++++++++
 5 files changed, 77 insertions(+), 63 deletions(-)
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.java
 delete mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
 delete mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
 create mode 100644 app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.java b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.java
new file mode 100644
index 0000000..07181ba
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.java
@@ -0,0 +1,30 @@
+package it.unisannio.ding.ids.wedroid.app.data.database;
+
+import android.content.Context;
+
+import androidx.room.Database;
+import androidx.room.Room;
+import androidx.room.RoomDatabase;
+
+import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao;
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board;
+
+@Database(entities = Board.class, version = 1, exportSchema = false)
+public abstract class BoardDatabase extends RoomDatabase {
+    private static volatile BoardDatabase INSTANCE;
+    public abstract BoardDao boardDao();
+
+    public static BoardDatabase getDatabase(Context context) {
+        if (INSTANCE != null)
+            return INSTANCE;
+        synchronized (BoardDatabase.class) {
+            INSTANCE = Room.databaseBuilder(
+                    context.getApplicationContext(),
+                    BoardDatabase.class,
+                    "board_database"
+            ).build();
+
+            return INSTANCE;
+        }
+    }
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
deleted file mode 100644
index e99fc50..0000000
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/database/BoardDatabase.kt
+++ /dev/null
@@ -1,35 +0,0 @@
-package it.unisannio.ding.ids.wedroid.app.data.database
-
-import android.content.Context
-import androidx.room.Database
-import androidx.room.Room
-import androidx.room.RoomDatabase
-import it.unisannio.ding.ids.wedroid.app.data.dao.BoardDao
-import it.unisannio.ding.ids.wedroid.app.data.entity.Board
-
-@Database(entities = [Board::class], version = 1, exportSchema = false)
-abstract class BoardDatabase : RoomDatabase() {
-
-    abstract fun boardDao(): BoardDao
-
-    companion object {
-        @Volatile
-        private var INSTANCE: BoardDatabase? = null
-
-        fun getDatabase(context: Context): BoardDatabase {
-            val tempInstance = INSTANCE
-            if (tempInstance != null) {
-                return tempInstance
-            }
-            synchronized(this) {
-                val instance = Room.databaseBuilder(
-                    context.applicationContext,
-                    BoardDatabase::class.java,
-                    "board_database"
-                ).build()
-                INSTANCE = instance
-                return instance
-            }
-        }
-    }
-}
\ No newline at end of file
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 702e7ac..6c36ccd 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
@@ -7,19 +7,19 @@ import androidx.lifecycle.ViewModelProvider
 import androidx.recyclerview.widget.LinearLayoutManager
 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.BoardListViewModel
+import it.unisannio.ding.ids.wedroid.app.viewmodel.BoardsListViewModel
 
 import kotlinx.android.synthetic.main.activity_boards_lists.*
 import kotlinx.android.synthetic.main.content_boards_lists.*
 
 class BoardsListsActivity : AppCompatActivity() {
-    private lateinit var viewModel: BoardListViewModel
+    private lateinit var viewModel: BoardsListViewModel
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_boards_lists)
         setSupportActionBar(toolbar)
-        viewModel = ViewModelProvider(this).get(BoardListViewModel::class.java)
+        viewModel = ViewModelProvider(this).get(BoardsListViewModel::class.java)
 
         val adapter = BoardsListAdapter(this)
         boardList.adapter = adapter
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
deleted file mode 100644
index 50f7161..0000000
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardListViewModel.kt
+++ /dev/null
@@ -1,25 +0,0 @@
-package it.unisannio.ding.ids.wedroid.app.viewmodel
-
-import android.app.Application
-import androidx.lifecycle.AndroidViewModel
-import androidx.lifecycle.LiveData
-import it.unisannio.ding.ids.wedroid.app.util.ServicesFactory
-import it.unisannio.ding.ids.wedroid.app.data.database.BoardDatabase
-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.DumReader
-
-class BoardListViewModel(application: Application) : AndroidViewModel(application) {
-
-    private val repository: BoardRepository = BoardRepository(
-        BoardDatabase.getDatabase(application).boardDao(),
-        ServicesFactory.getInstance(TODO()).boardService,
-        TODO()
-    )
-
-    val allBoards: LiveData<List<Board>> = repository.allBoards
-
-    fun insertBoard(title: String) = repository.insertBoard(title)
-
-    fun deleteBoard(id: String) = repository.deleteBoard(id)
-}
\ No newline at end of file
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
new file mode 100644
index 0000000..f8f3a9f
--- /dev/null
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/viewmodel/BoardsListViewModel.java
@@ -0,0 +1,44 @@
+package it.unisannio.ding.ids.wedroid.app.viewmodel;
+
+import android.app.Application;
+
+import androidx.annotation.NonNull;
+import androidx.lifecycle.AndroidViewModel;
+import androidx.lifecycle.LiveData;
+
+import java.util.List;
+
+import it.unisannio.ding.ids.wedroid.app.data.database.BoardDatabase;
+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;
+
+public class BoardsListViewModel extends AndroidViewModel {
+    private BoardRepository repository;
+    private LiveData<List<Board>> allBoards;
+
+    public BoardsListViewModel(@NonNull Application application) {
+        super(application);
+        PreferenceReader reader = null; //TODO
+        repository = new BoardRepository(
+                BoardDatabase.getDatabase(application).boardDao(),
+                ServicesFactory.Companion.getInstance(reader).getBoardService(),
+                reader
+        );
+
+        allBoards = repository.getAllBoards();
+    }
+
+    public LiveData<List<Board>> getAllBoards() {
+        return allBoards;
+    }
+
+    public void insertBoard(String title) {
+        repository.insertBoard(title);
+    }
+
+    public void deleteBoard(String id) {
+        repository.deleteBoard(id);
+    }
+}

From 90e31fbf168b3d745be49b68f126b45c46ec1a14 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 1 Dec 2019 16:24:44 +0100
Subject: [PATCH 08/29] add database testing

---
 .../ding/ids/wedroid/app/TestHelper.kt        |  22 ++++
 .../ids/wedroid/app/data/dao/BoardDaoTest.kt  | 103 ++++++++++++++++++
 2 files changed, 125 insertions(+)
 create mode 100644 app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
 create mode 100644 app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt

diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
new file mode 100644
index 0000000..017874f
--- /dev/null
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt
@@ -0,0 +1,22 @@
+package it.unisannio.ding.ids.wedroid.app
+
+import androidx.lifecycle.*
+
+class OneTimeObserver<T>(private val handler: (T) -> Unit) : Observer<T>, LifecycleOwner {
+    private val lifecycle = LifecycleRegistry(this)
+    init {
+        lifecycle.handleLifecycleEvent(Lifecycle.Event.ON_RESUME)
+    }
+
+    override fun getLifecycle(): Lifecycle = lifecycle
+
+    override fun onChanged(t: T) {
+        handler(t)
+        lifecycle.handleLifecycleEvent(Lifecycle.Event.ON_DESTROY)
+    }
+}
+
+fun <T> LiveData<T>.observeOnce(onChangeHandler: (T) -> Unit) {
+    val observer = OneTimeObserver(handler = onChangeHandler)
+    observe(observer, observer)
+}
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
new file mode 100644
index 0000000..dae8ff8
--- /dev/null
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDaoTest.kt
@@ -0,0 +1,103 @@
+package it.unisannio.ding.ids.wedroid.app.data.dao
+
+import android.content.Context
+import androidx.arch.core.executor.testing.InstantTaskExecutorRule
+import androidx.room.Room
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import it.unisannio.ding.ids.wedroid.app.data.database.BoardDatabase
+import it.unisannio.ding.ids.wedroid.app.data.entity.Board
+import it.unisannio.ding.ids.wedroid.app.observeOnce
+import junit.framework.TestCase.assertEquals
+import kotlinx.coroutines.runBlocking
+import org.junit.After
+import org.junit.Before
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+import java.io.IOException
+
+
+@RunWith(AndroidJUnit4::class)
+class BoardDaoTest {
+    private lateinit var dao: BoardDao
+    private lateinit var db: BoardDatabase
+
+    @get:Rule
+    val instantTaskExecutorRule = InstantTaskExecutorRule()
+
+    @Before
+    fun createDb() {
+        val context = ApplicationProvider.getApplicationContext<Context>()
+        db = Room.inMemoryDatabaseBuilder(
+            context, BoardDatabase::class.java
+        ).build()
+        dao = db.boardDao()
+    }
+
+    @After
+    @Throws(IOException::class)
+    fun closeDb() {
+        db.close()
+    }
+
+    @Test
+    fun emptyDatabaseOnCreation() {
+        dao.getAllBoard().observeOnce {
+            assertEquals(0, it.size)
+        }
+    }
+
+    @Test
+    fun insert() {
+        val board = Board("id", "title")
+
+        runBlocking {
+            dao.insert(board)
+        }
+
+        dao.getAllBoard().observeOnce {
+            assertEquals(1, it.size)
+            assertEquals(board, it[0])
+        }
+    }
+
+    fun getInAscendingOrder() {
+        val board0 = Board("id0", "title0")
+        val board1 = Board("id1", "title1")
+
+        runBlocking {
+            dao.insert(board1)
+        }
+
+        runBlocking {
+            dao.insert(board0)
+        }
+
+        dao.getAllBoard().observeOnce {
+            assertEquals(2, it.size)
+            assertEquals(board0, it[0])
+            assertEquals(board1, it[1])
+        }
+    }
+
+    fun delete() {
+        val board = Board("id", "title")
+
+        runBlocking {
+            dao.insert(board)
+        }
+
+        runBlocking {
+            dao.delete(board)
+        }
+
+        dao.getAllBoard().observeOnce {
+            assertEquals(0, it.size)
+        }
+    }
+}
+
+
+
+

From 75e88ef617a65ab7449d763b669241eea0693f06 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 1 Dec 2019 19:39:41 +0100
Subject: [PATCH 09/29] add delete and refresh

- add swipe to delete
- add pull to refresh
---
 app/build.gradle                              |  2 ++
 .../wedroid/app/view/BoardsListsActivity.kt   | 35 ++++++++++++++++++-
 .../app/viewmodel/BoardsListViewModel.java    |  8 +++--
 .../main/res/layout/content_boards_lists.xml  | 10 ++++++
 4 files changed, 52 insertions(+), 3 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index 1112cf7..352ef63 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -54,6 +54,8 @@ dependencies {
     api "org.jetbrains.kotlinx:kotlinx-coroutines-android:$rootProject.coroutines"
     // UI
     implementation "com.google.android.material:material:$rootProject.materialVersion"
+    implementation "androidx.swiperefreshlayout:swiperefreshlayout:1.0.0"
+
 
 
     // TESTING
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 6c36ccd..0e1064f 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
@@ -4,7 +4,9 @@ import android.os.Bundle
 import androidx.appcompat.app.AppCompatActivity
 import androidx.lifecycle.Observer
 import androidx.lifecycle.ViewModelProvider
+import androidx.recyclerview.widget.ItemTouchHelper
 import androidx.recyclerview.widget.LinearLayoutManager
+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
@@ -25,14 +27,45 @@ class BoardsListsActivity : AppCompatActivity() {
         boardList.adapter = adapter
         boardList.layoutManager = LinearLayoutManager(this)
 
+        swipeLeftToDelete()
+
+        pullToRefresh.setColorSchemeColors(getColor(R.color.colorAccent))
+
         viewModel.allBoards.observe(this, Observer {
             it.let { adapter.setBoards(it) }
+            pullToRefresh.isRefreshing = false
         })
 
-        fab.setOnClickListener { view ->
+        fab.setOnClickListener {
             viewModel.insertBoard("New board")
         }
+
+        pullToRefresh.setOnRefreshListener {
+            viewModel.refresh()
+        }
+
     }
 
+    private fun swipeLeftToDelete() {
+        val swipeToDelete = ItemTouchHelper(
+            object : ItemTouchHelper.SimpleCallback(
+                0, ItemTouchHelper.LEFT
+            ) {
+                override fun onMove(
+                    recyclerView: RecyclerView,
+                    viewHolder: RecyclerView.ViewHolder,
+                    target: RecyclerView.ViewHolder
+                ): Boolean {
+                    return false
+                }
+
+                override fun onSwiped(viewHolder: RecyclerView.ViewHolder, direction: Int) {
+                    val pos = viewHolder.adapterPosition
+                    viewModel.deleteBoard(pos)
+                }
+            })
+
+        swipeToDelete.attachToRecyclerView(boardList)
+    }
 }
 
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 f8f3a9f..79ab9c2 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
@@ -38,7 +38,11 @@ public class BoardsListViewModel extends AndroidViewModel {
         repository.insertBoard(title);
     }
 
-    public void deleteBoard(String id) {
-        repository.deleteBoard(id);
+    public void deleteBoard(int position) {
+        repository.deleteBoard(allBoards.getValue().get(position).getId());
+    }
+
+    public void refresh() {
+        repository.synchronize();
     }
 }
diff --git a/app/src/main/res/layout/content_boards_lists.xml b/app/src/main/res/layout/content_boards_lists.xml
index cb2ba98..dc3adf9 100644
--- a/app/src/main/res/layout/content_boards_lists.xml
+++ b/app/src/main/res/layout/content_boards_lists.xml
@@ -7,6 +7,14 @@
     app:layout_behavior="@string/appbar_scrolling_view_behavior"
     tools:context=".view.BoardsListsActivity"
     tools:showIn="@layout/activity_boards_lists">
+    <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
+        android:layout_width="0dp"
+        android:layout_height="0dp"
+        android:id="@+id/pullToRefresh"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent">
 
     <androidx.recyclerview.widget.RecyclerView
         android:layout_width="0dp"
@@ -16,4 +24,6 @@
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent" />
+
+    </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file

From 861a9e32360c8305efa321f400d8b3ee56f8813a Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Mon, 2 Dec 2019 13:30:48 +0100
Subject: [PATCH 10/29] connected LoginActivity and BoardsListActivity

---
 app/src/main/AndroidManifest.xml              |  2 +-
 .../wedroid/app/view/BoardsListsActivity.kt   | 30 ++++++++++++++++++-
 .../wedroid/app/{ => view}/LoginActivity.kt   | 16 +++++++---
 .../app/viewmodel/BoardsListViewModel.java    |  4 ++-
 app/src/main/res/layout/activity_login.xml    |  2 +-
 5 files changed, 46 insertions(+), 8 deletions(-)
 rename app/src/main/java/it/unisannio/ding/ids/wedroid/app/{ => view}/LoginActivity.kt (89%)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index a3bab34..0e63a51 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -11,7 +11,7 @@
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
 
-        <activity android:name=".LoginActivity"></activity>
+        <activity android:name=".view.LoginActivity"></activity>
 
         <activity
             android:name=".view.BoardsListsActivity"
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..8074c54 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.util.Log
 import androidx.appcompat.app.AppCompatActivity
 import androidx.lifecycle.Observer
 import androidx.lifecycle.ViewModelProvider
@@ -8,6 +10,8 @@ import androidx.recyclerview.widget.ItemTouchHelper
 import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import it.unisannio.ding.ids.wedroid.app.R
+import it.unisannio.ding.ids.wedroid.app.util.PreferenceReader
+import it.unisannio.ding.ids.wedroid.app.util.SharedPreferenceHelper
 import it.unisannio.ding.ids.wedroid.app.view.adapter.BoardsListAdapter
 import it.unisannio.ding.ids.wedroid.app.viewmodel.BoardsListViewModel
 
@@ -21,6 +25,17 @@ class BoardsListsActivity : AppCompatActivity() {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_boards_lists)
         setSupportActionBar(toolbar)
+
+        val reader: PreferenceReader = SharedPreferenceHelper(this)
+        if (reader.token == "")
+            startActivityForResult(
+                Intent(this, LoginActivity::class.java),
+                LoginActivity.LOGIN_REQUEST_CODE
+            )
+        else initializeUi()
+    }
+
+    private fun initializeUi() {
         viewModel = ViewModelProvider(this).get(BoardsListViewModel::class.java)
 
         val adapter = BoardsListAdapter(this)
@@ -43,7 +58,6 @@ class BoardsListsActivity : AppCompatActivity() {
         pullToRefresh.setOnRefreshListener {
             viewModel.refresh()
         }
-
     }
 
     private fun swipeLeftToDelete() {
@@ -67,5 +81,19 @@ class BoardsListsActivity : AppCompatActivity() {
 
         swipeToDelete.attachToRecyclerView(boardList)
     }
+
+    override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
+        super.onActivityResult(requestCode, resultCode, data)
+        when (requestCode) {
+            LoginActivity.LOGIN_REQUEST_CODE -> {
+                when (resultCode) {
+                    LoginActivity.LOGIN_ERROR -> finish()
+                    LoginActivity.LOGIN_OK -> initializeUi()
+                    else -> finish()
+                }
+            }
+            else -> finish()
+        }
+    }
 }
 
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
similarity index 89%
rename from app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
rename to app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
index da250cf..82b6806 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
@@ -1,12 +1,11 @@
-package it.unisannio.ding.ids.wedroid.app
+package it.unisannio.ding.ids.wedroid.app.view
 
-import android.content.Context
-import android.content.SharedPreferences
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
 import android.util.Log
 import android.view.View
 import android.widget.Toast
+import it.unisannio.ding.ids.wedroid.app.R
 import it.unisannio.ding.ids.wedroid.app.util.SharedPreferenceHelper
 import it.unisannio.ding.ids.wedroid.wrapper.api.LoginService
 import it.unisannio.ding.ids.wedroid.wrapper.entity.UserPrototype
@@ -25,10 +24,11 @@ class LoginActivity : AppCompatActivity() {
         setContentView(R.layout.activity_login)
         sph = SharedPreferenceHelper(this)
 
+        setResult(LOGIN_ERROR)
+
         val id = sph.userId
         val token = sph.token
         val url = sph.baseUrl
-        Log.d("sharedPreference", "userId: $id\n token: $token\n url: $url")
     }
 
     fun loginButton(v: View) {
@@ -80,8 +80,16 @@ class LoginActivity : AppCompatActivity() {
                     Toast.LENGTH_LONG
                 ).show()
 
+                setResult(LOGIN_OK)
+                finish()
             }
 
         })
     }
+
+    companion object {
+        const val LOGIN_REQUEST_CODE = 13
+        const val LOGIN_OK = 0
+        const val LOGIN_ERROR = 1
+    }
 }
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..847a760 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
@@ -3,6 +3,7 @@ package it.unisannio.ding.ids.wedroid.app.viewmodel;
 import android.app.Application;
 
 import androidx.annotation.NonNull;
+import androidx.core.app.SharedElementCallback;
 import androidx.lifecycle.AndroidViewModel;
 import androidx.lifecycle.LiveData;
 
@@ -13,6 +14,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.app.util.SharedPreferenceHelper;
 
 public class BoardsListViewModel extends AndroidViewModel {
     private BoardRepository repository;
@@ -20,7 +22,7 @@ public class BoardsListViewModel extends AndroidViewModel {
 
     public BoardsListViewModel(@NonNull Application application) {
         super(application);
-        PreferenceReader reader = null; //TODO
+        PreferenceReader reader = new SharedPreferenceHelper(application);
         repository = new BoardRepository(
                 BoardDatabase.getDatabase(application).boardDao(),
                 ServicesFactory.Companion.getInstance(reader).getBoardService(),
diff --git a/app/src/main/res/layout/activity_login.xml b/app/src/main/res/layout/activity_login.xml
index b48dab0..0d5a258 100644
--- a/app/src/main/res/layout/activity_login.xml
+++ b/app/src/main/res/layout/activity_login.xml
@@ -4,7 +4,7 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    tools:context=".LoginActivity">
+    tools:context=".view.LoginActivity">
 
     <EditText
         android:id="@+id/username"

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 11/29] 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 12/29] 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 13/29] 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 14/29] 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

From 7e2a2dec1bd46cc2b532c45130cfdc08c223e99a Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Thu, 12 Dec 2019 18:50:54 +0100
Subject: [PATCH 15/29] add detekt

---
 app/build.gradle         |   4 +
 build.gradle             |   5 +-
 config/detekt/detekt.yml | 580 +++++++++++++++++++++++++++++++++++++++
 3 files changed, 587 insertions(+), 2 deletions(-)
 create mode 100644 config/detekt/detekt.yml

diff --git a/app/build.gradle b/app/build.gradle
index c0fc443..071bc34 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -4,6 +4,8 @@ apply plugin: 'kotlin-android'
 
 apply plugin: 'kotlin-android-extensions'
 
+apply plugin: 'io.gitlab.arturbosch.detekt'
+
 android {
     compileSdkVersion 29
     buildToolsVersion "29.0.2"
@@ -24,6 +26,8 @@ android {
 }
 
 dependencies {
+    detektPlugins "io.gitlab.arturbosch.detekt:detekt-formatting:1.2.2"
+
     implementation project(':wrapper')
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation 'androidx.appcompat:appcompat:1.1.0'
diff --git a/build.gradle b/build.gradle
index f4ed563..bf30e7a 100644
--- a/build.gradle
+++ b/build.gradle
@@ -5,11 +5,13 @@ buildscript {
     repositories {
         google()
         jcenter()
-        
+        maven { url "https://plugins.gradle.org/m2/" }
     }
+
     dependencies {
         classpath 'com.android.tools.build:gradle:3.5.2'
         classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
+        classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.2.2"
         // NOTE: Do not place your application dependencies here; they belong
         // in the individual module build.gradle files
     }
@@ -19,7 +21,6 @@ allprojects {
     repositories {
         google()
         jcenter()
-        
     }
 }
 
diff --git a/config/detekt/detekt.yml b/config/detekt/detekt.yml
new file mode 100644
index 0000000..3545f2f
--- /dev/null
+++ b/config/detekt/detekt.yml
@@ -0,0 +1,580 @@
+build:
+  maxIssues: 10
+  weights:
+    # complexity: 2
+    # LongParameterList: 1
+    # style: 1
+    # comments: 1
+
+config:
+  validation: true
+  # when writing own rules with new properties, exclude the property path e.g.: "my_rule_set,.*>.*>[my_property]"
+  excludes: ""
+
+processors:
+  active: true
+  exclude:
+  # - 'DetektProgressListener'
+  # - 'FunctionCountProcessor'
+  # - 'PropertyCountProcessor'
+  # - 'ClassCountProcessor'
+  # - 'PackageCountProcessor'
+  # - 'KtFileCountProcessor'
+
+console-reports:
+  active: true
+  exclude:
+  #  - 'ProjectStatisticsReport'
+  #  - 'ComplexityReport'
+  #  - 'NotificationReport'
+  #  - 'FindingsReport'
+     - 'FileBasedFindingsReport'
+  #  - 'BuildFailureReport'
+
+comments:
+  active: true
+  excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+  CommentOverPrivateFunction:
+    active: false
+  CommentOverPrivateProperty:
+    active: false
+  EndOfSentenceFormat:
+    active: false
+    endOfSentenceFormat: ([.?!][ \t\n\r\f<])|([.?!:]$)
+  UndocumentedPublicClass:
+    active: false
+    searchInNestedClass: true
+    searchInInnerClass: true
+    searchInInnerObject: true
+    searchInInnerInterface: true
+  UndocumentedPublicFunction:
+    active: false
+  UndocumentedPublicProperty:
+    active: false
+
+complexity:
+  active: true
+  ComplexCondition:
+    active: true
+    threshold: 4
+  ComplexInterface:
+    active: false
+    threshold: 10
+    includeStaticDeclarations: false
+  ComplexMethod:
+    active: true
+    threshold: 15
+    ignoreSingleWhenExpression: false
+    ignoreSimpleWhenEntries: false
+    ignoreNestingFunctions: false
+    nestingFunctions: run,let,apply,with,also,use,forEach,isNotNull,ifNull
+  LabeledExpression:
+    active: false
+    ignoredLabels: ""
+  LargeClass:
+    active: true
+    threshold: 600
+  LongMethod:
+    active: true
+    threshold: 60
+  LongParameterList:
+    active: true
+    threshold: 6
+    ignoreDefaultParameters: false
+  MethodOverloading:
+    active: false
+    threshold: 6
+  NestedBlockDepth:
+    active: true
+    threshold: 4
+  StringLiteralDuplication:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    threshold: 3
+    ignoreAnnotation: true
+    excludeStringsWithLessThan5Characters: true
+    ignoreStringsRegex: '$^'
+  TooManyFunctions:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    thresholdInFiles: 11
+    thresholdInClasses: 11
+    thresholdInInterfaces: 11
+    thresholdInObjects: 11
+    thresholdInEnums: 11
+    ignoreDeprecated: false
+    ignorePrivate: false
+    ignoreOverridden: false
+
+empty-blocks:
+  active: true
+  EmptyCatchBlock:
+    active: true
+    allowedExceptionNameRegex: "^(_|(ignore|expected).*)"
+  EmptyClassBlock:
+    active: true
+  EmptyDefaultConstructor:
+    active: true
+  EmptyDoWhileBlock:
+    active: true
+  EmptyElseBlock:
+    active: true
+  EmptyFinallyBlock:
+    active: true
+  EmptyForBlock:
+    active: true
+  EmptyFunctionBlock:
+    active: true
+    ignoreOverriddenFunctions: false
+  EmptyIfBlock:
+    active: true
+  EmptyInitBlock:
+    active: true
+  EmptyKtFile:
+    active: true
+  EmptySecondaryConstructor:
+    active: true
+  EmptyWhenBlock:
+    active: true
+  EmptyWhileBlock:
+    active: true
+
+exceptions:
+  active: true
+  ExceptionRaisedInUnexpectedLocation:
+    active: false
+    methodNames: 'toString,hashCode,equals,finalize'
+  InstanceOfCheckForException:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+  NotImplementedDeclaration:
+    active: false
+  PrintStackTrace:
+    active: false
+  RethrowCaughtException:
+    active: false
+  ReturnFromFinally:
+    active: false
+    ignoreLabeled: false
+  SwallowedException:
+    active: false
+    ignoredExceptionTypes: 'InterruptedException,NumberFormatException,ParseException,MalformedURLException'
+    allowedExceptionNameRegex: "^(_|(ignore|expected).*)"
+  ThrowingExceptionFromFinally:
+    active: false
+  ThrowingExceptionInMain:
+    active: false
+  ThrowingExceptionsWithoutMessageOrCause:
+    active: false
+    exceptions: 'IllegalArgumentException,IllegalStateException,IOException'
+  ThrowingNewInstanceOfSameException:
+    active: false
+  TooGenericExceptionCaught:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    exceptionNames:
+     - ArrayIndexOutOfBoundsException
+     - Error
+     - Exception
+     - IllegalMonitorStateException
+     - NullPointerException
+     - IndexOutOfBoundsException
+     - RuntimeException
+     - Throwable
+    allowedExceptionNameRegex: "^(_|(ignore|expected).*)"
+  TooGenericExceptionThrown:
+    active: true
+    exceptionNames:
+     - Error
+     - Exception
+     - Throwable
+     - RuntimeException
+
+formatting:
+  active: true
+  android: false
+  autoCorrect: true
+  AnnotationOnSeparateLine:
+    active: false
+    autoCorrect: true
+  ChainWrapping:
+    active: true
+    autoCorrect: true
+  CommentSpacing:
+    active: true
+    autoCorrect: true
+  Filename:
+    active: true
+  FinalNewline:
+    active: true
+    autoCorrect: true
+  ImportOrdering:
+    active: false
+    autoCorrect: true
+  Indentation:
+    active: false
+    autoCorrect: true
+    indentSize: 4
+    continuationIndentSize: 4
+  MaximumLineLength:
+    active: true
+    maxLineLength: 120
+  ModifierOrdering:
+    active: true
+    autoCorrect: true
+  MultiLineIfElse:
+    active: true
+    autoCorrect: true
+  NoBlankLineBeforeRbrace:
+    active: true
+    autoCorrect: true
+  NoConsecutiveBlankLines:
+    active: true
+    autoCorrect: true
+  NoEmptyClassBody:
+    active: true
+    autoCorrect: true
+  NoLineBreakAfterElse:
+    active: true
+    autoCorrect: true
+  NoLineBreakBeforeAssignment:
+    active: true
+    autoCorrect: true
+  NoMultipleSpaces:
+    active: true
+    autoCorrect: true
+  NoSemicolons:
+    active: true
+    autoCorrect: true
+  NoTrailingSpaces:
+    active: true
+    autoCorrect: true
+  NoUnitReturn:
+    active: true
+    autoCorrect: true
+  NoUnusedImports:
+    active: true
+    autoCorrect: true
+  NoWildcardImports:
+    active: true
+  PackageName:
+    active: true
+    autoCorrect: true
+  ParameterListWrapping:
+    active: true
+    autoCorrect: true
+    indentSize: 4
+  SpacingAroundColon:
+    active: true
+    autoCorrect: true
+  SpacingAroundComma:
+    active: true
+    autoCorrect: true
+  SpacingAroundCurly:
+    active: true
+    autoCorrect: true
+  SpacingAroundDot:
+    active: true
+    autoCorrect: true
+  SpacingAroundKeyword:
+    active: true
+    autoCorrect: true
+  SpacingAroundOperators:
+    active: true
+    autoCorrect: true
+  SpacingAroundParens:
+    active: true
+    autoCorrect: true
+  SpacingAroundRangeOperator:
+    active: true
+    autoCorrect: true
+  StringTemplate:
+    active: true
+    autoCorrect: true
+
+naming:
+  active: true
+  ClassNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    classPattern: '[A-Z$][a-zA-Z0-9$]*'
+  ConstructorParameterNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    parameterPattern: '[a-z][A-Za-z0-9]*'
+    privateParameterPattern: '[a-z][A-Za-z0-9]*'
+    excludeClassPattern: '$^'
+    ignoreOverridden: true
+  EnumNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    enumEntryPattern: '^[A-Z][_a-zA-Z0-9]*'
+  ForbiddenClassName:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    forbiddenName: ''
+  FunctionMaxLength:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    maximumFunctionNameLength: 30
+  FunctionMinLength:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    minimumFunctionNameLength: 3
+  FunctionNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    functionPattern: '^([a-z$][a-zA-Z$0-9]*)|(`.*`)$'
+    excludeClassPattern: '$^'
+    ignoreOverridden: true
+  FunctionParameterNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    parameterPattern: '[a-z][A-Za-z0-9]*'
+    excludeClassPattern: '$^'
+    ignoreOverriddenFunctions: true
+  InvalidPackageDeclaration:
+    active: false
+    rootPackage: ''
+  MatchingDeclarationName:
+    active: true
+  MemberNameEqualsClassName:
+    active: true
+    ignoreOverriddenFunction: true
+  ObjectPropertyNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    constantPattern: '[A-Za-z][_A-Za-z0-9]*'
+    propertyPattern: '[A-Za-z][_A-Za-z0-9]*'
+    privatePropertyPattern: '(_)?[A-Za-z][_A-Za-z0-9]*'
+  PackageNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    packagePattern: '^[a-z]+(\.[a-z][A-Za-z0-9]*)*$'
+  TopLevelPropertyNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    constantPattern: '[A-Z][_A-Z0-9]*'
+    propertyPattern: '[A-Za-z][_A-Za-z0-9]*'
+    privatePropertyPattern: '_?[A-Za-z][_A-Za-z0-9]*'
+  VariableMaxLength:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    maximumVariableNameLength: 64
+  VariableMinLength:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    minimumVariableNameLength: 1
+  VariableNaming:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    variablePattern: '[a-z][A-Za-z0-9]*'
+    privateVariablePattern: '(_)?[a-z][A-Za-z0-9]*'
+    excludeClassPattern: '$^'
+    ignoreOverridden: true
+
+performance:
+  active: true
+  ArrayPrimitive:
+    active: true
+  ForEachOnRange:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+  SpreadOperator:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+  UnnecessaryTemporaryInstantiation:
+    active: true
+
+potential-bugs:
+  active: true
+  Deprecation:
+    active: false
+  DuplicateCaseInWhenExpression:
+    active: true
+  EqualsAlwaysReturnsTrueOrFalse:
+    active: true
+  EqualsWithHashCodeExist:
+    active: true
+  ExplicitGarbageCollectionCall:
+    active: true
+  HasPlatformType:
+    active: false
+  ImplicitDefaultLocale:
+    active: false
+  InvalidRange:
+    active: true
+  IteratorHasNextCallsNextMethod:
+    active: true
+  IteratorNotThrowingNoSuchElementException:
+    active: true
+  LateinitUsage:
+    active: false
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    excludeAnnotatedProperties: ""
+    ignoreOnClassesPattern: ""
+  MissingWhenCase:
+    active: true
+  RedundantElseInWhen:
+    active: true
+  UnconditionalJumpStatementInLoop:
+    active: false
+  UnreachableCode:
+    active: true
+  UnsafeCallOnNullableType:
+    active: true
+  UnsafeCast:
+    active: false
+  UselessPostfixExpression:
+    active: false
+  WrongEqualsTypeParameter:
+    active: true
+
+style:
+  active: true
+  CollapsibleIfStatements:
+    active: false
+  DataClassContainsFunctions:
+    active: false
+    conversionFunctionPrefix: 'to'
+  DataClassShouldBeImmutable:
+    active: false
+  EqualsNullCall:
+    active: true
+  EqualsOnSignatureLine:
+    active: false
+  ExplicitItLambdaParameter:
+    active: false
+  ExpressionBodySyntax:
+    active: false
+    includeLineWrapping: false
+  ForbiddenComment:
+    active: true
+    values: 'TODO:,FIXME:,STOPSHIP:'
+    allowedPatterns: ""
+  ForbiddenImport:
+    active: false
+    imports: ''
+    forbiddenPatterns: ""
+  ForbiddenVoid:
+    active: false
+    ignoreOverridden: false
+    ignoreUsageInGenerics: false
+  FunctionOnlyReturningConstant:
+    active: true
+    ignoreOverridableFunction: true
+    excludedFunctions: 'describeContents'
+    excludeAnnotatedFunction: "dagger.Provides"
+  LibraryCodeMustSpecifyReturnType:
+    active: true
+  LoopWithTooManyJumpStatements:
+    active: true
+    maxJumpCount: 1
+  MagicNumber:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    ignoreNumbers: '-1,0,1,2'
+    ignoreHashCodeFunction: true
+    ignorePropertyDeclaration: false
+    ignoreLocalVariableDeclaration: false
+    ignoreConstantDeclaration: true
+    ignoreCompanionObjectPropertyDeclaration: true
+    ignoreAnnotation: false
+    ignoreNamedArgument: true
+    ignoreEnums: false
+    ignoreRanges: false
+  MandatoryBracesIfStatements:
+    active: false
+  MaxLineLength:
+    active: true
+    maxLineLength: 120
+    excludePackageStatements: true
+    excludeImportStatements: true
+    excludeCommentStatements: false
+  MayBeConst:
+    active: true
+  ModifierOrder:
+    active: true
+  NestedClassesVisibility:
+    active: false
+  NewLineAtEndOfFile:
+    active: true
+  NoTabs:
+    active: false
+  OptionalAbstractKeyword:
+    active: true
+  OptionalUnit:
+    active: false
+  OptionalWhenBraces:
+    active: false
+  PreferToOverPairSyntax:
+    active: false
+  ProtectedMemberInFinalClass:
+    active: true
+  RedundantExplicitType:
+    active: false
+  RedundantVisibilityModifierRule:
+    active: false
+  ReturnCount:
+    active: true
+    max: 2
+    excludedFunctions: "equals"
+    excludeLabeled: false
+    excludeReturnFromLambda: true
+    excludeGuardClauses: false
+  SafeCast:
+    active: true
+  SerialVersionUIDInSerializableClass:
+    active: false
+  SpacingBetweenPackageAndImports:
+    active: false
+  ThrowsCount:
+    active: true
+    max: 2
+  TrailingWhitespace:
+    active: false
+  UnderscoresInNumericLiterals:
+    active: false
+    acceptableDecimalLength: 5
+  UnnecessaryAbstractClass:
+    active: true
+    excludeAnnotatedClasses: "dagger.Module"
+  UnnecessaryApply:
+    active: false
+  UnnecessaryInheritance:
+    active: true
+  UnnecessaryLet:
+    active: false
+  UnnecessaryParentheses:
+    active: false
+  UntilInsteadOfRangeTo:
+    active: false
+  UnusedImports:
+    active: false
+  UnusedPrivateClass:
+    active: true
+  UnusedPrivateMember:
+    active: false
+    allowedNames: "(_|ignored|expected|serialVersionUID)"
+  UseArrayLiteralsInAnnotations:
+    active: false
+  UseCheckOrError:
+    active: false
+  UseDataClass:
+    active: false
+    excludeAnnotatedClasses: ""
+    allowVars: false
+  UseIfInsteadOfWhen:
+    active: false
+  UseRequire:
+    active: false
+  UselessCallOnNotNull:
+    active: true
+  UtilityClassWithPublicConstructor:
+    active: true
+  VarCouldBeVal:
+    active: false
+  WildcardImport:
+    active: true
+    excludes: "**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
+    excludeImports: 'java.util.*,kotlinx.android.synthetic.*'

From 27113a530fa0804550e221c7a27cd80ae9e5c136 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Thu, 12 Dec 2019 19:31:38 +0100
Subject: [PATCH 16/29] fix formatting

---
 .../ding/ids/wedroid/app/data/dao/BoardDaoTest.kt        | 7 +------
 .../unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt  | 8 ++++++--
 .../unisannio/ding/ids/wedroid/app/data/entity/Board.kt  | 1 -
 .../ids/wedroid/app/data/repository/BoardRepository.kt   | 1 -
 .../ding/ids/wedroid/app/util/ServicesFactory.kt         | 9 +++++++--
 .../ding/ids/wedroid/app/util/SharedPreferenceHelper.kt  | 7 +++----
 .../ding/ids/wedroid/app/view/BoardsListsActivity.kt     | 3 +--
 .../unisannio/ding/ids/wedroid/app/view/LoginActivity.kt | 4 ----
 .../ids/wedroid/app/view/adapter/BoardsListAdapter.kt    | 1 -
 .../java/it/unisannio/ding/ids/wedroid/app/TestHelper.kt | 1 -
 10 files changed, 18 insertions(+), 24 deletions(-)

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 2655621..e6151bb 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
@@ -17,7 +17,6 @@ import org.junit.Test
 import org.junit.runner.RunWith
 import java.io.IOException
 
-
 @RunWith(AndroidJUnit4::class)
 class BoardDaoTest {
     private lateinit var dao: BoardDao
@@ -108,8 +107,4 @@ class BoardDaoTest {
             assertEquals(0, it.size)
         }
     }
-}
-
-
-
-
+}
\ No newline at end of file
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
index ca88fa4..adcfa85 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/dao/BoardDao.kt
@@ -1,7 +1,11 @@
 package it.unisannio.ding.ids.wedroid.app.data.dao
 
 import androidx.lifecycle.LiveData
-import androidx.room.*
+import androidx.room.Dao
+import androidx.room.Delete
+import androidx.room.Insert
+import androidx.room.OnConflictStrategy
+import androidx.room.Query
 import it.unisannio.ding.ids.wedroid.app.data.entity.Board
 
 @Dao
@@ -14,4 +18,4 @@ interface BoardDao {
 
     @Delete
     suspend fun delete(board: Board)
-}
\ No newline at end of file
+}
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
index 0cd1f70..eb61071 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/data/entity/Board.kt
@@ -13,4 +13,3 @@ data class Board(
 fun it.unisannio.ding.ids.wedroid.wrapper.entity.Board.convert(): Board {
     return Board(this.id, this.title)
 }
-
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 204940d..1218371 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
@@ -159,4 +159,3 @@ class BoardRepository(
         Log.e("RETROFIT", message)
     }
 }
-
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
index b64032e..eece43a 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/ServicesFactory.kt
@@ -1,6 +1,12 @@
 package it.unisannio.ding.ids.wedroid.app.util
 
-import it.unisannio.ding.ids.wedroid.wrapper.api.*
+import it.unisannio.ding.ids.wedroid.wrapper.api.BoardService
+import it.unisannio.ding.ids.wedroid.wrapper.api.CardCommentService
+import it.unisannio.ding.ids.wedroid.wrapper.api.CardService
+import it.unisannio.ding.ids.wedroid.wrapper.api.ChecklistService
+import it.unisannio.ding.ids.wedroid.wrapper.api.ListService
+import it.unisannio.ding.ids.wedroid.wrapper.api.SwimlanesService
+import it.unisannio.ding.ids.wedroid.wrapper.api.UserService
 import okhttp3.OkHttpClient
 import retrofit2.Retrofit
 import retrofit2.converter.gson.GsonConverterFactory
@@ -75,4 +81,3 @@ class ServicesFactory private constructor(
         }
     }
 }
-
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt
index 2f150c8..9f4f37d 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/util/SharedPreferenceHelper.kt
@@ -2,7 +2,7 @@ package it.unisannio.ding.ids.wedroid.app.util
 
 import android.content.Context
 
-class SharedPreferenceHelper(context : Context) : PreferenceReader, PreferenceWriter {
+class SharedPreferenceHelper(context: Context) : PreferenceReader, PreferenceWriter {
     private val sp = context.getSharedPreferences("userinfo", Context.MODE_PRIVATE)
 
     override fun getBaseUrl(): String? {
@@ -20,8 +20,7 @@ class SharedPreferenceHelper(context : Context) : PreferenceReader, PreferenceWr
     override fun setBaseUrl(baseUrl: String?) {
         val editor = sp.edit()
         editor.putString("url", baseUrl).apply()
-
-           }
+    }
 
     override fun setUserId(userId: String?) {
         val editor = sp.edit()
@@ -32,4 +31,4 @@ class SharedPreferenceHelper(context : Context) : PreferenceReader, PreferenceWr
         val editor = sp.edit()
         editor.putString("token", token).apply()
     }
-}
\ No newline at end of file
+}
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 b67f7b6..f5be060 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
@@ -118,7 +118,7 @@ class BoardsListsActivity : AppCompatActivity() {
 
         val isPrivate = data.getBooleanExtra(NewBoardActivity.BOARD_PRIVATE, true)
 
-        viewModel.insertBoard(title, isPrivate, BoardBackgroundColor.LIMEGREEN) //TODO
+        viewModel.insertBoard(title, isPrivate, BoardBackgroundColor.LIMEGREEN) // TODO
     }
 
     companion object {
@@ -126,4 +126,3 @@ class BoardsListsActivity : AppCompatActivity() {
         const val LOGIN_CODE = 19
     }
 }
-
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
index d3a15bf..222dd8b 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
@@ -37,7 +37,6 @@ class LoginActivity : AppCompatActivity() {
             return
         }
 
-
         val userNameText = username.text.toString()
         val passwordText = password.text.toString()
 
@@ -47,8 +46,6 @@ class LoginActivity : AppCompatActivity() {
             .build()
             .create(LoginService::class.java)
 
-
-
         service.login(userNameText, passwordText).enqueue(object : Callback<UserPrototype> {
             override fun onFailure(call: Call<UserPrototype>, t: Throwable) {
                 Toast.makeText(
@@ -82,7 +79,6 @@ class LoginActivity : AppCompatActivity() {
                 setResult(LOGIN_OK)
                 finish()
             }
-
         })
     }
 
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
index 56d5ddb..f4fb929 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
@@ -9,7 +9,6 @@ import androidx.recyclerview.widget.RecyclerView
 import it.unisannio.ding.ids.wedroid.app.R
 import it.unisannio.ding.ids.wedroid.app.data.entity.Board
 
-
 class BoardsListAdapter internal constructor(
     context: Context
 ) : RecyclerView.Adapter<BoardsListAdapter.BoardViewHolder>() {
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
index 42a2ada..f2a68ee 100644
--- 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
@@ -10,4 +10,3 @@ fun getPrivateFun(name: String, kClass: KClass<*>): KFunction<*>? {
         .find { it.name == name }
         .also { it?.isAccessible = true }
 }
-

From 57768bee3807489110b7b37b4d12c3dbc5165bf0 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Fri, 13 Dec 2019 15:08:01 +0100
Subject: [PATCH 17/29] add checkstyle

---
 build.gradle                     |   2 +
 config/checkstyle/checkstyle.xml | 247 +++++++++++++++++++++++++++++++
 2 files changed, 249 insertions(+)
 create mode 100644 config/checkstyle/checkstyle.xml

diff --git a/build.gradle b/build.gradle
index bf30e7a..fc197f5 100644
--- a/build.gradle
+++ b/build.gradle
@@ -18,6 +18,8 @@ buildscript {
 }
 
 allprojects {
+    apply plugin: 'checkstyle'
+
     repositories {
         google()
         jcenter()
diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml
new file mode 100644
index 0000000..28f19a7
--- /dev/null
+++ b/config/checkstyle/checkstyle.xml
@@ -0,0 +1,247 @@
+<?xml version="1.0"?>
+<!DOCTYPE module PUBLIC
+    "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
+    "https://checkstyle.org/dtds/configuration_1_3.dtd">
+
+<!--
+    Checkstyle configuration that checks the Google coding conventions from Google Java Style
+    that can be found at https://google.github.io/styleguide/javaguide.html
+
+    Checkstyle is very configurable. Be sure to read the documentation at
+    http://checkstyle.sf.net (or in your downloaded distribution).
+
+    To completely disable a check, just comment it out or delete it from the file.
+
+    Authors: Max Vetrenko, Ruslan Diachenko, Roman Ivanov.
+ -->
+
+<module name = "Checker">
+    <property name="charset" value="UTF-8"/>
+
+    <property name="severity" value="warning"/>
+
+    <property name="fileExtensions" value="java, properties, xml"/>
+    <!-- Excludes all 'module-info.java' files              -->
+    <!-- See https://checkstyle.org/config_filefilters.html -->
+    <module name="BeforeExecutionExclusionFileFilter">
+        <property name="fileNamePattern" value="module\-info\.java$"/>
+    </module>
+    <!-- Checks for whitespace                               -->
+    <!-- See http://checkstyle.sf.net/config_whitespace.html -->
+    <module name="FileTabCharacter">
+        <property name="eachLine" value="true"/>
+    </module>
+
+    <module name="LineLength">
+        <property name="fileExtensions" value="java"/>
+        <property name="max" value="100"/>
+        <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
+    </module>
+
+    <module name="TreeWalker">
+        <module name="OuterTypeFilename"/>
+        <module name="IllegalTokenText">
+            <property name="tokens" value="STRING_LITERAL, CHAR_LITERAL"/>
+            <property name="format"
+                value="\\u00(09|0(a|A)|0(c|C)|0(d|D)|22|27|5(C|c))|\\(0(10|11|12|14|15|42|47)|134)"/>
+            <property name="message"
+                value="Consider using special escape sequence instead of octal value or Unicode escaped value."/>
+        </module>
+        <module name="AvoidEscapedUnicodeCharacters">
+            <property name="allowEscapesForControlCharacters" value="true"/>
+            <property name="allowByTailComment" value="true"/>
+            <property name="allowNonPrintableEscapes" value="true"/>
+        </module>
+        <module name="AvoidStarImport"/>
+        <module name="OneTopLevelClass"/>
+        <module name="NoLineWrap"/>
+        <module name="EmptyBlock">
+            <property name="option" value="TEXT"/>
+            <property name="tokens"
+                value="LITERAL_TRY, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE, LITERAL_SWITCH"/>
+        </module>
+        <module name="NeedBraces"/>
+        <module name="LeftCurly"/>
+        <module name="RightCurly">
+            <property name="id" value="RightCurlySame"/>
+            <property name="tokens"
+                value="LITERAL_TRY, LITERAL_CATCH, LITERAL_FINALLY, LITERAL_IF, LITERAL_ELSE,
+                    LITERAL_DO"/>
+        </module>
+        <module name="RightCurly">
+            <property name="id" value="RightCurlyAlone"/>
+            <property name="option" value="alone"/>
+            <property name="tokens"
+                value="CLASS_DEF, METHOD_DEF, CTOR_DEF, LITERAL_FOR, LITERAL_WHILE, STATIC_INIT,
+                    INSTANCE_INIT"/>
+        </module>
+        <module name="WhitespaceAround">
+            <property name="allowEmptyConstructors" value="true"/>
+            <property name="allowEmptyLambdas" value="true"/>
+            <property name="allowEmptyMethods" value="true"/>
+            <property name="allowEmptyTypes" value="true"/>
+            <property name="allowEmptyLoops" value="true"/>
+            <message key="ws.notFollowed"
+                value="WhitespaceAround: ''{0}'' is not followed by whitespace. Empty blocks may only be represented as '{}' when not part of a multi-block statement (4.1.3)"/>
+            <message key="ws.notPreceded"
+                value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/>
+        </module>
+        <module name="OneStatementPerLine"/>
+        <module name="MultipleVariableDeclarations"/>
+        <module name="ArrayTypeStyle"/>
+        <module name="MissingSwitchDefault"/>
+        <module name="FallThrough"/>
+        <module name="UpperEll"/>
+        <module name="ModifierOrder"/>
+        <module name="EmptyLineSeparator">
+            <property name="allowNoEmptyLineBetweenFields" value="true"/>
+        </module>
+        <module name="SeparatorWrap">
+            <property name="id" value="SeparatorWrapDot"/>
+            <property name="tokens" value="DOT"/>
+            <property name="option" value="nl"/>
+        </module>
+        <module name="SeparatorWrap">
+            <property name="id" value="SeparatorWrapComma"/>
+            <property name="tokens" value="COMMA"/>
+            <property name="option" value="EOL"/>
+        </module>
+        <module name="SeparatorWrap">
+            <!-- ELLIPSIS is EOL until https://github.com/google/styleguide/issues/258 -->
+            <property name="id" value="SeparatorWrapEllipsis"/>
+            <property name="tokens" value="ELLIPSIS"/>
+            <property name="option" value="EOL"/>
+        </module>
+        <module name="SeparatorWrap">
+            <!-- ARRAY_DECLARATOR is EOL until https://github.com/google/styleguide/issues/259 -->
+            <property name="id" value="SeparatorWrapArrayDeclarator"/>
+            <property name="tokens" value="ARRAY_DECLARATOR"/>
+            <property name="option" value="EOL"/>
+        </module>
+        <module name="SeparatorWrap">
+            <property name="id" value="SeparatorWrapMethodRef"/>
+            <property name="tokens" value="METHOD_REF"/>
+            <property name="option" value="nl"/>
+        </module>
+        <module name="PackageName">
+            <property name="format" value="^[a-z]+(\.[a-z][a-z0-9]*)*$"/>
+            <message key="name.invalidPattern"
+                value="Package name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="TypeName">
+            <message key="name.invalidPattern"
+                value="Type name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="MemberName">
+            <property name="format" value="^[a-z][a-z0-9][a-zA-Z0-9]*$"/>
+            <message key="name.invalidPattern"
+                value="Member name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="ParameterName">
+            <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/>
+            <message key="name.invalidPattern"
+                value="Parameter name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="LambdaParameterName">
+            <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/>
+            <message key="name.invalidPattern"
+                value="Lambda parameter name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="CatchParameterName">
+            <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/>
+            <message key="name.invalidPattern"
+                value="Catch parameter name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="LocalVariableName">
+            <property name="tokens" value="VARIABLE_DEF"/>
+            <property name="format" value="^[a-z]([a-z0-9][a-zA-Z0-9]*)?$"/>
+            <message key="name.invalidPattern"
+                value="Local variable name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="ClassTypeParameterName">
+            <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/>
+            <message key="name.invalidPattern"
+                value="Class type name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="MethodTypeParameterName">
+            <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/>
+            <message key="name.invalidPattern"
+                value="Method type name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="InterfaceTypeParameterName">
+            <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/>
+            <message key="name.invalidPattern"
+                value="Interface type name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="NoFinalizer"/>
+        <module name="GenericWhitespace">
+            <message key="ws.followed"
+                value="GenericWhitespace ''{0}'' is followed by whitespace."/>
+            <message key="ws.preceded"
+                value="GenericWhitespace ''{0}'' is preceded with whitespace."/>
+            <message key="ws.illegalFollow"
+                value="GenericWhitespace ''{0}'' should followed by whitespace."/>
+            <message key="ws.notPreceded"
+                value="GenericWhitespace ''{0}'' is not preceded with whitespace."/>
+        </module>
+        <module name="Indentation">
+            <property name="basicOffset" value="4"/>
+            <property name="braceAdjustment" value="0"/>
+            <property name="caseIndent" value="4"/>
+            <property name="throwsIndent" value="4"/>
+            <property name="lineWrappingIndentation" value="4"/>
+            <property name="arrayInitIndent" value="4"/>
+        </module>
+        <module name="AbbreviationAsWordInName">
+            <property name="ignoreFinal" value="false"/>
+            <property name="allowedAbbreviationLength" value="1"/>
+        </module>
+        <module name="OverloadMethodsDeclarationOrder"/>
+        <module name="VariableDeclarationUsageDistance"/>
+        <module name="CustomImportOrder">
+            <property name="sortImportsInGroupAlphabetically" value="true"/>
+            <property name="separateLineBetweenGroups" value="true"/>
+            <property name="customImportOrderRules" value="STATIC###THIRD_PARTY_PACKAGE"/>
+        </module>
+        <module name="MethodParamPad"/>
+        <module name="NoWhitespaceBefore">
+            <property name="tokens"
+                value="COMMA, SEMI, POST_INC, POST_DEC, DOT, ELLIPSIS, METHOD_REF"/>
+            <property name="allowLineBreaks" value="true"/>
+        </module>
+        <module name="ParenPad"/>
+        <module name="OperatorWrap">
+            <property name="option" value="eol"/>
+            <property name="tokens"
+                value="BAND, BOR, BSR, BXOR, DIV, EQUAL, GE, GT, LAND, LE, LITERAL_INSTANCEOF, LOR,
+                    LT, MINUS, MOD, NOT_EQUAL, PLUS, QUESTION, SL, SR, STAR, METHOD_REF "/>
+        </module>
+        <module name="AnnotationLocation">
+            <property name="id" value="AnnotationLocationMostCases"/>
+            <property name="tokens"
+                value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, METHOD_DEF, CTOR_DEF"/>
+        </module>
+        <module name="AnnotationLocation">
+            <property name="id" value="AnnotationLocationVariables"/>
+            <property name="tokens" value="VARIABLE_DEF"/>
+            <property name="allowSamelineMultipleAnnotations" value="true"/>
+        </module>
+        <module name="AtclauseOrder">
+            <property name="tagOrder" value="@param, @return, @throws, @deprecated"/>
+            <property name="target"
+                value="CLASS_DEF, INTERFACE_DEF, ENUM_DEF, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/>
+        </module>
+        <module name="MethodName">
+            <property name="format" value="^[a-z][a-z0-9][a-zA-Z0-9_]*$"/>
+            <message key="name.invalidPattern"
+                value="Method name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="SingleLineJavadoc">
+            <property name="ignoreInlineTags" value="false"/>
+        </module>
+        <module name="EmptyCatchBlock">
+            <property name="exceptionVariableName" value="expected"/>
+        </module>
+        <module name="CommentsIndentation"/>
+    </module>
+</module>

From c846f04066b735016a5042395cf5127062657843 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Fri, 13 Dec 2019 15:09:26 +0100
Subject: [PATCH 18/29] edit detekt rules

---
 config/detekt/detekt.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/config/detekt/detekt.yml b/config/detekt/detekt.yml
index 3545f2f..05426c6 100644
--- a/config/detekt/detekt.yml
+++ b/config/detekt/detekt.yml
@@ -192,7 +192,7 @@ exceptions:
 
 formatting:
   active: true
-  android: false
+  android: true
   autoCorrect: true
   AnnotationOnSeparateLine:
     active: false
@@ -206,7 +206,7 @@ formatting:
   Filename:
     active: true
   FinalNewline:
-    active: true
+    active: false
     autoCorrect: true
   ImportOrdering:
     active: false
@@ -223,7 +223,7 @@ formatting:
     active: true
     autoCorrect: true
   MultiLineIfElse:
-    active: true
+    active: false
     autoCorrect: true
   NoBlankLineBeforeRbrace:
     active: true

From 161068769583aca10274e66dfe3b98c228f2aa9a Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Fri, 13 Dec 2019 19:18:48 +0100
Subject: [PATCH 19/29] remove LineLength

It not work
---
 config/checkstyle/checkstyle.xml | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml
index 28f19a7..ca6d646 100644
--- a/config/checkstyle/checkstyle.xml
+++ b/config/checkstyle/checkstyle.xml
@@ -32,12 +32,6 @@
         <property name="eachLine" value="true"/>
     </module>
 
-    <module name="LineLength">
-        <property name="fileExtensions" value="java"/>
-        <property name="max" value="100"/>
-        <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
-    </module>
-
     <module name="TreeWalker">
         <module name="OuterTypeFilename"/>
         <module name="IllegalTokenText">

From f2908ed101fd1cda3e82f1aeeb557f34a28968bc Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Sat, 28 Dec 2019 19:39:25 +0100
Subject: [PATCH 20/29] test login activity

---
 app/build.gradle                              |  4 ++
 .../ids/wedroid/app/view/LoginActivityTest.kt | 69 +++++++++++++++++++
 .../ids/wedroid/app/view/LoginActivity.kt     |  6 +-
 app/src/main/res/layout/activity_login.xml    |  2 +-
 4 files changed, 77 insertions(+), 4 deletions(-)
 create mode 100644 app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt

diff --git a/app/build.gradle b/app/build.gradle
index 6a93023..5a3cea8 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -70,6 +70,10 @@ dependencies {
     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.test:runner:1.2.0'
+    androidTestImplementation 'androidx.test:rules:1.2.0'
+
+
 
     implementation "com.squareup.retrofit2:retrofit:2.6.2"
     implementation "com.squareup.retrofit2:converter-gson:2.6.2"
diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt
new file mode 100644
index 0000000..da13ca8
--- /dev/null
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt
@@ -0,0 +1,69 @@
+package it.unisannio.ding.ids.wedroid.app.view
+
+import androidx.test.espresso.Espresso.closeSoftKeyboard
+import androidx.test.espresso.Espresso.onView
+import androidx.test.espresso.action.ViewActions.click
+import androidx.test.espresso.action.ViewActions.typeText
+import androidx.test.espresso.assertion.ViewAssertions.matches
+import androidx.test.espresso.matcher.RootMatchers.withDecorView
+import androidx.test.espresso.matcher.ViewMatchers.*
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import androidx.test.filters.LargeTest
+import androidx.test.rule.ActivityTestRule
+import it.unisannio.ding.ids.wedroid.app.R
+import org.hamcrest.CoreMatchers.not
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+
+@RunWith(AndroidJUnit4::class)
+@LargeTest
+class LoginActivityTest {
+    @get:Rule
+    val activityRule = ActivityTestRule(LoginActivity::class.java)
+
+    @Test
+    fun loginWithEmptyUrl() {
+        onView(withId(R.id.username))
+            .perform(typeText("username"))
+        onView(withId(R.id.password))
+            .perform(typeText("password"))
+        closeSoftKeyboard()
+        onView(withId(R.id.button))
+            .perform(click())
+
+        onView(withText("Riempire tutti i campi"))
+            .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
+           .check(matches(isDisplayed()))
+    }
+
+    @Test
+    fun loginWithEmptyUsername() {
+        onView(withId(R.id.instanceServer))
+            .perform(typeText("instanceServer"))
+        onView(withId(R.id.password))
+            .perform(typeText("password"))
+        closeSoftKeyboard()
+        onView(withId(R.id.button))
+            .perform(click())
+        onView(withText("Riempire tutti i campi"))
+            .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
+            .check(matches(isDisplayed()))
+
+    }
+
+    @Test
+    fun loginWithEmptyPassword() {
+        onView(withId(R.id.instanceServer))
+            .perform(typeText("instanceServer"))
+        onView(withId(R.id.username))
+            .perform(typeText("username"))
+        closeSoftKeyboard()
+        onView(withId(R.id.button))
+            .perform(click())
+        onView(withText("Riempire tutti i campi"))
+            .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
+            .check(matches(isDisplayed()))
+    }
+    
+}
\ No newline at end of file
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
index d3a15bf..53e7c66 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
@@ -31,7 +31,7 @@ class LoginActivity : AppCompatActivity() {
     }
 
     fun loginButton(v: View) {
-        if (username.text.isBlank() || istanceServer.text.isBlank() || password.text.isBlank()) {
+        if (username.text.isBlank() || instanceServer.text.isBlank() || password.text.isBlank()) {
             Toast.makeText(this, "Riempire tutti i campi", Toast.LENGTH_LONG)
                 .show()
             return
@@ -43,7 +43,7 @@ class LoginActivity : AppCompatActivity() {
 
         val service = Retrofit.Builder()
             .addConverterFactory(GsonConverterFactory.create())
-            .baseUrl(istanceServer.text.toString())
+            .baseUrl(instanceServer.text.toString())
             .build()
             .create(LoginService::class.java)
 
@@ -69,7 +69,7 @@ class LoginActivity : AppCompatActivity() {
                     return
                 }
                 val users = response.body()
-                sph.baseUrl = istanceServer.text.toString()
+                sph.baseUrl = instanceServer.text.toString()
                 sph.token = users?.token
                 sph.userId = users?.id
 
diff --git a/app/src/main/res/layout/activity_login.xml b/app/src/main/res/layout/activity_login.xml
index 0d5a258..1d6bf2a 100644
--- a/app/src/main/res/layout/activity_login.xml
+++ b/app/src/main/res/layout/activity_login.xml
@@ -42,7 +42,7 @@
         app:layout_constraintStart_toStartOf="parent" />
 
     <EditText
-        android:id="@+id/istanceServer"
+        android:id="@+id/instanceServer"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginBottom="50dp"

From 4253074679cc3b1977450efb43552d833899990e Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 29 Dec 2019 23:09:37 +0100
Subject: [PATCH 21/29] add espresso test for BoardsListActivity

---
 app/build.gradle                              |  4 ++
 .../app/view/BoardsListsActivityTest.kt       | 52 +++++++++++++++++++
 2 files changed, 56 insertions(+)
 create mode 100644 app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivityTest.kt

diff --git a/app/build.gradle b/app/build.gradle
index a4e2310..e09008e 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -67,6 +67,10 @@ dependencies {
     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.test.espresso:espresso-contrib:3.2.0'
+    androidTestImplementation 'androidx.test:runner:1.2.0'
+    androidTestImplementation 'androidx.test:rules:1.2.0'
+
 
     implementation "com.squareup.retrofit2:retrofit:2.6.2"
     implementation "com.squareup.retrofit2:converter-gson:2.6.2"
diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivityTest.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivityTest.kt
new file mode 100644
index 0000000..6fe7e0a
--- /dev/null
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/BoardsListsActivityTest.kt
@@ -0,0 +1,52 @@
+package it.unisannio.ding.ids.wedroid.app.view
+
+import androidx.test.espresso.Espresso.onView
+import androidx.test.espresso.action.ViewActions.click
+import androidx.test.espresso.action.ViewActions.swipeDown
+import androidx.test.espresso.action.ViewActions.swipeLeft
+import androidx.test.espresso.assertion.ViewAssertions.matches
+import androidx.test.espresso.contrib.RecyclerViewActions
+import androidx.test.espresso.matcher.ViewMatchers.isDisplayed
+import androidx.test.espresso.matcher.ViewMatchers.withId
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import androidx.test.filters.LargeTest
+import androidx.test.rule.ActivityTestRule
+import it.unisannio.ding.ids.wedroid.app.R
+import it.unisannio.ding.ids.wedroid.app.view.adapter.BoardsListAdapter
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+
+@RunWith(AndroidJUnit4::class)
+@LargeTest
+class BoardsListsActivityTest {
+
+    @get:Rule
+    val activityRule = ActivityTestRule(BoardsListsActivity::class.java)
+
+    fun swipeLeftToDelete() {
+        onView(withId(R.id.boardList))
+            .perform(
+                RecyclerViewActions.actionOnItemAtPosition<BoardsListAdapter.BoardViewHolder>(
+                    0, swipeLeft()
+                )
+            )
+    }
+
+    fun pullToRefresh() {
+        onView(withId(R.id.pullToRefresh))
+            .perform(swipeDown())
+
+    }
+
+    @Test
+    fun openNewBoardActivity() {
+        onView(withId(R.id.fab))
+            .perform(click())
+
+        onView(withId(R.id.newBoardName))
+            .check(matches(isDisplayed()))
+    }
+
+}
+

From 419b8c1faa7df7a9d08d32dea506f0d5396eebec Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 29 Dec 2019 23:15:09 +0100
Subject: [PATCH 22/29] add toast on empty name

- show toast if the board name is empty
- add test for NewBoardActivity
---
 .../wedroid/app/view/NewBoardActivityTest.kt  | 37 +++++++++++++++++++
 .../wedroid/app/view/NewBoardActivity.java    |  7 ++++
 app/src/main/res/values/strings.xml           |  1 +
 3 files changed, 45 insertions(+)
 create mode 100644 app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivityTest.kt

diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivityTest.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivityTest.kt
new file mode 100644
index 0000000..4f49761
--- /dev/null
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/NewBoardActivityTest.kt
@@ -0,0 +1,37 @@
+package it.unisannio.ding.ids.wedroid.app.view
+
+import androidx.test.espresso.Espresso.onView
+import androidx.test.espresso.action.ViewActions.click
+import androidx.test.espresso.assertion.ViewAssertions.matches
+import androidx.test.espresso.matcher.RootMatchers.withDecorView
+import androidx.test.espresso.matcher.ViewMatchers.isDisplayed
+import androidx.test.espresso.matcher.ViewMatchers.withId
+import androidx.test.espresso.matcher.ViewMatchers.withText
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import androidx.test.filters.LargeTest
+import androidx.test.rule.ActivityTestRule
+import it.unisannio.ding.ids.wedroid.app.R
+import org.hamcrest.core.IsNot.not
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+
+@RunWith(AndroidJUnit4::class)
+@LargeTest
+class NewBoardActivityTest {
+
+    @get:Rule
+    val activityRule = ActivityTestRule(NewBoardActivity::class.java)
+
+    @Test
+    fun showToastOnEmptyName() {
+        onView(withId(R.id.newBoardDone))
+            .perform(click())
+
+        onView(withText(R.string.on_add_new_board_empty_name))
+            .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
+            .check(matches(isDisplayed()))
+    }
+
+}
+
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
index b0a48dd..85eb857 100644
--- 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
@@ -9,6 +9,7 @@ import androidx.appcompat.widget.Toolbar;
 import android.view.View;
 import android.widget.EditText;
 import android.widget.Switch;
+import android.widget.Toast;
 
 import it.unisannio.ding.ids.wedroid.app.R;
 
@@ -28,6 +29,12 @@ public class NewBoardActivity extends AppCompatActivity {
     }
 
     public void onDone(View v) {
+        if (boardName.getText().toString().equals("")) {
+            Toast.makeText(this, R.string.on_add_new_board_empty_name, Toast.LENGTH_LONG)
+                    .show();
+            return;
+        }
+
         Intent data = new Intent();
         data.putExtra(BOARD_NAME, boardName.getText().toString());
         data.putExtra(BOARD_PRIVATE, isPrivate.isChecked());
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 215138e..1cb93e8 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -8,4 +8,5 @@
     <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>
+    <string name="on_add_new_board_empty_name">Name cannot be empty</string>
 </resources>

From 583a19091e8dddcea26d232d22409f4812aa40b7 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 29 Dec 2019 23:18:26 +0100
Subject: [PATCH 23/29] add onClickListener on list item

---
 .../ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt     | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
index f4fb929..b9c995c 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
@@ -33,6 +33,9 @@ class BoardsListAdapter internal constructor(
 
     override fun onBindViewHolder(holder: BoardViewHolder, position: Int) {
         val board = boards[position]
+        holder.itemView.setOnClickListener {
+            // TODO start board activity
+        }
         holder.boardTitle.text = board.title
     }
 

From 9bd5b01faf2b1a0be7b5cb27e7b20bfbf9252430 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Sun, 29 Dec 2019 23:31:37 +0100
Subject: [PATCH 24/29] change fab icon

---
 app/src/main/res/drawable/ic_add_black_24dp.xml   |  9 +++++++++
 app/src/main/res/layout/activity_boards_lists.xml | 12 ++++++------
 2 files changed, 15 insertions(+), 6 deletions(-)
 create mode 100644 app/src/main/res/drawable/ic_add_black_24dp.xml

diff --git a/app/src/main/res/drawable/ic_add_black_24dp.xml b/app/src/main/res/drawable/ic_add_black_24dp.xml
new file mode 100644
index 0000000..6b6146e
--- /dev/null
+++ b/app/src/main/res/drawable/ic_add_black_24dp.xml
@@ -0,0 +1,9 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+        android:width="24dp"
+        android:height="24dp"
+        android:viewportWidth="24.0"
+        android:viewportHeight="24.0">
+    <path
+        android:fillColor="#FFFFFF"
+        android:pathData="M19,13h-6v6h-2v-6H5v-2h6V5h2v6h6v2z"/>
+</vector>
diff --git a/app/src/main/res/layout/activity_boards_lists.xml b/app/src/main/res/layout/activity_boards_lists.xml
index 985531a..7282fa5 100644
--- a/app/src/main/res/layout/activity_boards_lists.xml
+++ b/app/src/main/res/layout/activity_boards_lists.xml
@@ -23,11 +23,11 @@
     <include layout="@layout/content_boards_lists" />
 
     <com.google.android.material.floatingactionbutton.FloatingActionButton
-        android:id="@+id/fab"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_gravity="bottom|end"
-        android:layout_margin="@dimen/fab_margin"
-        app:srcCompat="@android:drawable/ic_dialog_email" />
+            android:id="@+id/fab"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="bottom|end"
+            android:layout_margin="@dimen/fab_margin"
+            app:srcCompat="@drawable/ic_add_black_24dp" />
 
 </androidx.coordinatorlayout.widget.CoordinatorLayout>
\ No newline at end of file

From e99559a67552382f2279e3265b16d293f2fe6ffb Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Mon, 30 Dec 2019 11:10:30 +0100
Subject: [PATCH 25/29] enhancement board list UI

---
 app/src/main/res/layout/board_recycle_item.xml | 9 +++++----
 app/src/main/res/values/dimens.xml             | 1 +
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/src/main/res/layout/board_recycle_item.xml b/app/src/main/res/layout/board_recycle_item.xml
index fa14d0d..3f472f3 100644
--- a/app/src/main/res/layout/board_recycle_item.xml
+++ b/app/src/main/res/layout/board_recycle_item.xml
@@ -4,9 +4,10 @@
     android:layout_height="wrap_content">
 
     <TextView
-        android:id="@+id/boardTitle"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:text="TextView" />
+            android:id="@+id/boardTitle"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:padding="@dimen/board_item_padding"
+            android:textSize="24sp" />
 
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index 59a0b0c..4896a1e 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -1,3 +1,4 @@
 <resources>
     <dimen name="fab_margin">16dp</dimen>
+    <dimen name="board_item_padding">8dp</dimen>
 </resources>

From 842573ece92608972b98a2a7783893dff06f91b8 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Mon, 30 Dec 2019 15:13:14 +0100
Subject: [PATCH 26/29] add backgroudColorPicker

- add color picker for new board (close #22)
- edit typo of pumpkin
---
 .../wedroid/app/view/BoardsListsActivity.kt   |  7 +-
 .../wedroid/app/view/NewBoardActivity.java    | 17 ++++
 app/src/main/res/layout/content_new_board.xml | 78 +++++++++++--------
 app/src/main/res/values/strings.xml           | 15 ++++
 .../wrapper/entity/BoardBackgroundColor.java  |  2 +-
 5 files changed, 83 insertions(+), 36 deletions(-)

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 f5be060..d3c6924 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
@@ -18,6 +18,7 @@ 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.*
+import java.util.Locale
 
 class BoardsListsActivity : AppCompatActivity() {
     private lateinit var viewModel: BoardsListViewModel
@@ -118,7 +119,11 @@ class BoardsListsActivity : AppCompatActivity() {
 
         val isPrivate = data.getBooleanExtra(NewBoardActivity.BOARD_PRIVATE, true)
 
-        viewModel.insertBoard(title, isPrivate, BoardBackgroundColor.LIMEGREEN) // TODO
+        val colorName = data.getStringExtra(NewBoardActivity.BOARD_BACKGROUND_COLOR) ?: "belize"
+        val backgroundColor = BoardBackgroundColor
+            .valueOf(colorName.toUpperCase(Locale.ROOT))
+
+        viewModel.insertBoard(title, isPrivate, backgroundColor)
     }
 
     companion object {
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
index 85eb857..887b245 100644
--- 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
@@ -6,16 +6,21 @@ import android.os.Bundle;
 import androidx.appcompat.app.AppCompatActivity;
 import androidx.appcompat.widget.Toolbar;
 
+import android.util.Log;
 import android.view.View;
+import android.widget.ArrayAdapter;
 import android.widget.EditText;
+import android.widget.Spinner;
 import android.widget.Switch;
 import android.widget.Toast;
 
 import it.unisannio.ding.ids.wedroid.app.R;
+import it.unisannio.ding.ids.wedroid.wrapper.entity.BoardBackgroundColor;
 
 public class NewBoardActivity extends AppCompatActivity {
     private EditText boardName;
     private Switch isPrivate;
+    private Spinner colorPicker;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
@@ -26,6 +31,16 @@ public class NewBoardActivity extends AppCompatActivity {
 
         boardName = findViewById(R.id.newBoardName);
         isPrivate = findViewById(R.id.newBoardPermission);
+
+        colorPicker = findViewById(R.id.newBoardColor);
+        ArrayAdapter<CharSequence> adapter = ArrayAdapter.createFromResource(
+                this,
+                R.array.board_background_colors,
+                android.R.layout.simple_spinner_item
+        );
+        adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
+        colorPicker.setAdapter(adapter);
+
     }
 
     public void onDone(View v) {
@@ -38,6 +53,7 @@ public class NewBoardActivity extends AppCompatActivity {
         Intent data = new Intent();
         data.putExtra(BOARD_NAME, boardName.getText().toString());
         data.putExtra(BOARD_PRIVATE, isPrivate.isChecked());
+        data.putExtra(BOARD_BACKGROUND_COLOR, colorPicker.getSelectedItem().toString());
         setResult(RESULT_OK, data);
         finish();
     }
@@ -49,4 +65,5 @@ public class NewBoardActivity extends AppCompatActivity {
     public static final int RESULT_OK = 17;
     public static final String BOARD_NAME = "BOARD_NAME";
     public static final String BOARD_PRIVATE = "BOARD_PRIVATE";
+    public static final String BOARD_BACKGROUND_COLOR = "BOARD_BACKGROUND_COLOR";
 }
diff --git a/app/src/main/res/layout/content_new_board.xml b/app/src/main/res/layout/content_new_board.xml
index ad7e6da..851e4dd 100644
--- a/app/src/main/res/layout/content_new_board.xml
+++ b/app/src/main/res/layout/content_new_board.xml
@@ -9,18 +9,18 @@
     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" />
+            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:importantForAutofill="no"
+            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.20999998" />
 
     <Switch
         android:id="@+id/newBoardPermission"
@@ -35,28 +35,38 @@
         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" />
+            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_toTopOf="@+id/newBoardColor"
+            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" />
+            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_toTopOf="@+id/newBoardColor"
+            app:layout_constraintVertical_bias="0.40" />
+
+    <Spinner
+            android:id="@+id/newBoardColor"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintEnd_toEndOf="@+id/newBoardPermission"
+            app:layout_constraintStart_toStartOf="@+id/newBoardPermission"
+            app:layout_constraintTop_toBottomOf="@+id/newBoardPermission"
+            app:layout_constraintVertical_bias="0.120000005" />
 </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 1cb93e8..4cb76fa 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -9,4 +9,19 @@
     <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>
     <string name="on_add_new_board_empty_name">Name cannot be empty</string>
+
+    <string-array name="board_background_colors">
+        <item>Belize</item>
+        <item>Nephritis</item>
+        <item>Pomegranate</item>
+        <item>Pumpkin</item>
+        <item>Wisteria</item>
+        <item>Moderatepink</item>
+        <item>Strongcyan</item>
+        <item>Limegreen</item>
+        <item>Midnight</item>
+        <item>Dark</item>
+        <item>Relax</item>
+        <item>Corteza</item>
+    </string-array>
 </resources>
diff --git a/wrapper/src/main/java/it/unisannio/ding/ids/wedroid/wrapper/entity/BoardBackgroundColor.java b/wrapper/src/main/java/it/unisannio/ding/ids/wedroid/wrapper/entity/BoardBackgroundColor.java
index 32401d9..aee9e1c 100644
--- a/wrapper/src/main/java/it/unisannio/ding/ids/wedroid/wrapper/entity/BoardBackgroundColor.java
+++ b/wrapper/src/main/java/it/unisannio/ding/ids/wedroid/wrapper/entity/BoardBackgroundColor.java
@@ -10,7 +10,7 @@ public enum BoardBackgroundColor {
     @SerializedName("pomegranate")
     POMEGRANATE,
     @SerializedName("pumpkin")
-    PUMPIK,
+    PUMPKIN,
     @SerializedName("wisteria")
     WISTERIA,
     @SerializedName("moderatepink")

From 2b060f7d38692862d8b9675e1d48a08f85143e16 Mon Sep 17 00:00:00 2001
From: norangebit <git@norangeb.it>
Date: Mon, 30 Dec 2019 15:33:27 +0100
Subject: [PATCH 27/29] add startActivity onClick on item of boardsList

---
 .../wedroid/app/view/adapter/BoardsListAdapter.kt  | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
index b9c995c..7ea269c 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/adapter/BoardsListAdapter.kt
@@ -1,6 +1,7 @@
 package it.unisannio.ding.ids.wedroid.app.view.adapter
 
 import android.content.Context
+import android.content.Intent
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
@@ -33,14 +34,21 @@ class BoardsListAdapter internal constructor(
 
     override fun onBindViewHolder(holder: BoardViewHolder, position: Int) {
         val board = boards[position]
-        holder.itemView.setOnClickListener {
-            // TODO start board activity
-        }
         holder.boardTitle.text = board.title
+
+        holder.itemView.setOnClickListener {
+            val intent = Intent(it.context, TODO())
+            intent.putExtra(BOARD_ID, board.id)
+            it.context.startActivity(intent)
+        }
     }
 
     internal fun setBoards(boards: List<Board>) {
         this.boards = boards
         notifyDataSetChanged()
     }
+
+    companion object {
+        const val BOARD_ID = "BOARD_ID"
+    }
 }

From ac590e418dd9e36b6a1c61b70cc96da183daca9d Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Tue, 7 Jan 2020 10:51:33 +0100
Subject: [PATCH 28/29] fix #21

---
 .../ding/ids/wedroid/app/view/LoginActivity.kt         | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
index 8e92d4f..1bcfd1d 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
@@ -3,6 +3,7 @@ package it.unisannio.ding.ids.wedroid.app.view
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
 import android.view.View
+import android.webkit.URLUtil
 import android.widget.Toast
 import it.unisannio.ding.ids.wedroid.app.R
 import it.unisannio.ding.ids.wedroid.app.util.SharedPreferenceHelper
@@ -39,10 +40,17 @@ class LoginActivity : AppCompatActivity() {
 
         val userNameText = username.text.toString()
         val passwordText = password.text.toString()
+        val instanceServerText = instanceServer.text.toString()
+
+        if (!URLUtil.isValidUrl(instanceServerText)){
+            Toast.makeText(this, "Formato URL non valido", Toast.LENGTH_LONG)
+                .show()
+            return 
+        }
 
         val service = Retrofit.Builder()
             .addConverterFactory(GsonConverterFactory.create())
-            .baseUrl(instanceServer.text.toString())
+            .baseUrl(instanceServerText)
             .build()
             .create(LoginService::class.java)
 

From 82ee1f5f7eee70d351b62bf0e3bcb82a0a531161 Mon Sep 17 00:00:00 2001
From: noemi3 <n.mincol@gmail.com>
Date: Tue, 7 Jan 2020 11:04:15 +0100
Subject: [PATCH 29/29] fix #23

---
 .../ids/wedroid/app/view/LoginActivityTest.kt | 30 +++++++++++++++----
 .../ids/wedroid/app/view/LoginActivity.kt     | 12 ++++----
 app/src/main/res/values/strings.xml           |  5 ++++
 3 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt
index da13ca8..62af7b7 100644
--- a/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt
+++ b/app/src/androidTest/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivityTest.kt
@@ -32,7 +32,7 @@ class LoginActivityTest {
         onView(withId(R.id.button))
             .perform(click())
 
-        onView(withText("Riempire tutti i campi"))
+        onView(withText(R.string.login_empty_field))
             .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
            .check(matches(isDisplayed()))
     }
@@ -40,13 +40,14 @@ class LoginActivityTest {
     @Test
     fun loginWithEmptyUsername() {
         onView(withId(R.id.instanceServer))
-            .perform(typeText("instanceServer"))
+            .perform(typeText("https://wekan.com"))
         onView(withId(R.id.password))
             .perform(typeText("password"))
         closeSoftKeyboard()
         onView(withId(R.id.button))
             .perform(click())
-        onView(withText("Riempire tutti i campi"))
+
+        onView(withText(R.string.login_empty_field))
             .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
             .check(matches(isDisplayed()))
 
@@ -55,15 +56,32 @@ class LoginActivityTest {
     @Test
     fun loginWithEmptyPassword() {
         onView(withId(R.id.instanceServer))
-            .perform(typeText("instanceServer"))
+            .perform(typeText("https://wekan.com"))
         onView(withId(R.id.username))
             .perform(typeText("username"))
         closeSoftKeyboard()
         onView(withId(R.id.button))
             .perform(click())
-        onView(withText("Riempire tutti i campi"))
+
+        onView(withText(R.string.login_empty_field))
+            .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
+            .check(matches(isDisplayed()))
+    }
+
+    @Test
+    fun loginWithUnformedInstance(){
+        onView(withId(R.id.instanceServer))
+            .perform(typeText("not an URL"))
+        onView(withId(R.id.username))
+            .perform(typeText("username"))
+        onView(withId(R.id.password))
+            .perform(typeText("password"))
+        closeSoftKeyboard()
+        onView(withId(R.id.button))
+            .perform(click())
+
+        onView(withText(R.string.login_unformed_instance))
             .inRoot(withDecorView(not(activityRule.activity.window.decorView)))
             .check(matches(isDisplayed()))
     }
-    
 }
\ No newline at end of file
diff --git a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
index 1bcfd1d..811db1a 100644
--- a/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
+++ b/app/src/main/java/it/unisannio/ding/ids/wedroid/app/view/LoginActivity.kt
@@ -33,7 +33,7 @@ class LoginActivity : AppCompatActivity() {
 
     fun loginButton(v: View) {
         if (username.text.isBlank() || instanceServer.text.isBlank() || password.text.isBlank()) {
-            Toast.makeText(this, "Riempire tutti i campi", Toast.LENGTH_LONG)
+            Toast.makeText(this, R.string.login_empty_field, Toast.LENGTH_LONG)
                 .show()
             return
         }
@@ -43,9 +43,9 @@ class LoginActivity : AppCompatActivity() {
         val instanceServerText = instanceServer.text.toString()
 
         if (!URLUtil.isValidUrl(instanceServerText)){
-            Toast.makeText(this, "Formato URL non valido", Toast.LENGTH_LONG)
+            Toast.makeText(this, R.string.login_unformed_instance, Toast.LENGTH_LONG)
                 .show()
-            return 
+            return
         }
 
         val service = Retrofit.Builder()
@@ -58,7 +58,7 @@ class LoginActivity : AppCompatActivity() {
             override fun onFailure(call: Call<UserPrototype>, t: Throwable) {
                 Toast.makeText(
                     applicationContext,
-                    "Controlla la tua connessione internet",
+                    R.string.login_network_error,
                     Toast.LENGTH_LONG
                 ).show()
             }
@@ -68,7 +68,7 @@ class LoginActivity : AppCompatActivity() {
                 if (response.code() != 200) {
                     Toast.makeText(
                         applicationContext,
-                        "Credenziali non corrette",
+                        R.string.login_wrong_field,
                         Toast.LENGTH_LONG
                     ).show()
                     return
@@ -80,7 +80,7 @@ class LoginActivity : AppCompatActivity() {
 
                 Toast.makeText(
                     applicationContext,
-                    "Login effettuato con successo",
+                    R.string.login_success,
                     Toast.LENGTH_LONG
                 ).show()
 
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 215138e..bc2e5a0 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -8,4 +8,9 @@
     <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>
+    <string name="login_empty_field">Riempire tutti i campi</string>
+    <string name="login_unformed_instance">Formato URL non valido</string>
+    <string name="login_network_error">Controlla la tua connessione internet</string>
+    <string name="login_wrong_field">Credenziali non corrette</string>
+    <string name="login_success">Login effettuato con successo</string>
 </resources>