From 7c140bb97dcff0373e84635e171e956a7da45318 Mon Sep 17 00:00:00 2001
From: Matias Vallejos <matiasvallejosdev@outlook.com>
Date: Mon, 29 Jan 2024 16:19:11 -0300
Subject: [PATCH] Updating my django todo api

---
 .DS_Store                                     | Bin 0 -> 6148 bytes
 app/core/migrations/0001_initial.py           |   4 +-
 .../0002_rename_user_id_user_userid.py        |  18 ------
 app/todo_api/migrations/0001_initial.py       |   7 +-
 .../0002_tasklist_task_list_uuid.py           |  19 ------
 ...ename_task_list_uuid_tasklist_list_uuid.py |  18 ------
 .../migrations/0004_task_task_uuid.py         |  19 ------
 .../migrations/0005_alter_task_task_list.py   |  19 ------
 app/todo_api/permissions.py                   |   0
 app/todo_api/tests/test_task_api.py           |   2 -
 app/todo_api/tests/test_task_list_api.py      |  47 +++++++++++++-
 app/todo_api/views.py                         |   3 +-
 app/todo_project/settings.py                  |  60 +++++++++---------
 requirements.txt                              |   6 +-
 14 files changed, 86 insertions(+), 136 deletions(-)
 create mode 100644 .DS_Store
 delete mode 100644 app/core/migrations/0002_rename_user_id_user_userid.py
 delete mode 100644 app/todo_api/migrations/0002_tasklist_task_list_uuid.py
 delete mode 100644 app/todo_api/migrations/0003_rename_task_list_uuid_tasklist_list_uuid.py
 delete mode 100644 app/todo_api/migrations/0004_task_task_uuid.py
 delete mode 100644 app/todo_api/migrations/0005_alter_task_task_list.py
 delete mode 100644 app/todo_api/permissions.py

diff --git a/.DS_Store b/.DS_Store
new file mode 100644
index 0000000000000000000000000000000000000000..844352c7f63c6eba95401795ed28f0bbbe615a81
GIT binary patch
literal 6148
zcmeHKu};G<5Iv`nD6kZP5Mw?7Av&>!sz~e|SfQ<GfmVrtsz@yP0{9dLK7q2Z!OX|-
z&S$DPNxOj%x|4j*@!k2(PqG~ok!el(O`--7)zKKkHK+<<JGYVqZe$B+c#hGa+36+Y
zwB)G+%78NP7a8Ea8`2&P=z?y&{TsY*<Za`m-R_Q(F2-;eWS6^-uZO<9zto-%b7#YL
zgiu=f&Pzj@P}j>&!E@QeuYnHkKWAJ+VV-f4uVj&zSC2jas77a$QtH(aoV+1}vlQ~r
z9*!*sbE}uPVD<9V_c1AW?^8lOy74lyC1)$-e0ln?Yo5mzR$H=rd1dZZnj-HtUC}v>
z(3@6PP~lamW%Ew>uk`e|SmIexpH!*(ry0PV%~D?%RILms1Ioah0X`ohG{)FrCTNci
zDEkBeh8VVjKEEY6CUO`%%miV97zqVRsBupWBjFg2oL}rP6O?c=?(tz<X5(Hc#%9O-
zk##2%3#wKIlmVZCntV3+{y)MVUhw|+gY-)oPzL@L114;D+AS=}-L0j?@m(9CouaX@
nUnXcnpxkwAJA4)Ipjm+};tnu&m<eJ8qCWzb235+yuQKojXgHOi

literal 0
HcmV?d00001

diff --git a/app/core/migrations/0001_initial.py b/app/core/migrations/0001_initial.py
index 6b5d7de..1002d99 100644
--- a/app/core/migrations/0001_initial.py
+++ b/app/core/migrations/0001_initial.py
@@ -1,4 +1,4 @@
-# Generated by Django 4.0 on 2023-07-14 21:57
+# Generated by Django 4.0 on 2024-01-25 22:44
 
 from django.db import migrations, models
 import uuid
@@ -18,7 +18,7 @@ class Migration(migrations.Migration):
             fields=[
                 ('last_login', models.DateTimeField(blank=True, null=True, verbose_name='last login')),
                 ('is_superuser', models.BooleanField(default=False, help_text='Designates that this user has all permissions without explicitly assigning them.', verbose_name='superuser status')),
-                ('user_id', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False)),
+                ('userId', models.UUIDField(default=uuid.uuid4, editable=False, primary_key=True, serialize=False)),
                 ('email', models.EmailField(max_length=255, unique=True)),
                 ('username', models.CharField(max_length=45, unique=True)),
                 ('password', models.CharField(max_length=255)),
diff --git a/app/core/migrations/0002_rename_user_id_user_userid.py b/app/core/migrations/0002_rename_user_id_user_userid.py
deleted file mode 100644
index bfa803d..0000000
--- a/app/core/migrations/0002_rename_user_id_user_userid.py
+++ /dev/null
@@ -1,18 +0,0 @@
-# Generated by Django 4.0 on 2023-07-19 18:35
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('core', '0001_initial'),
-    ]
-
-    operations = [
-        migrations.RenameField(
-            model_name='user',
-            old_name='user_id',
-            new_name='userId',
-        ),
-    ]
diff --git a/app/todo_api/migrations/0001_initial.py b/app/todo_api/migrations/0001_initial.py
index cde208d..5879e96 100644
--- a/app/todo_api/migrations/0001_initial.py
+++ b/app/todo_api/migrations/0001_initial.py
@@ -1,7 +1,8 @@
-# Generated by Django 4.0 on 2023-07-14 21:57
+# Generated by Django 4.0 on 2024-01-25 22:44
 
 from django.db import migrations, models
 import django.db.models.deletion
+import uuid
 
 
 class Migration(migrations.Migration):
@@ -17,6 +18,7 @@ class Migration(migrations.Migration):
             name='TaskList',
             fields=[
                 ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('list_uuid', models.UUIDField(default=uuid.uuid4, editable=False, unique=True)),
                 ('name', models.CharField(max_length=255)),
                 ('created_at', models.DateTimeField(auto_now_add=True, null=True)),
                 ('created_by', models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, to='core.user')),
@@ -26,12 +28,13 @@ class Migration(migrations.Migration):
             name='Task',
             fields=[
                 ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('task_uuid', models.UUIDField(default=uuid.uuid4, editable=False, unique=True)),
                 ('title', models.CharField(max_length=255)),
                 ('completed', models.BooleanField(default=False)),
                 ('due_date', models.DateTimeField(blank=True, null=True)),
                 ('created_at', models.DateTimeField(auto_now_add=True, null=True)),
                 ('created_by', models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, to='core.user')),
-                ('task_list', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='todo_api.tasklist')),
+                ('task_list', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='todo_api.tasklist', to_field='list_uuid')),
             ],
         ),
     ]
diff --git a/app/todo_api/migrations/0002_tasklist_task_list_uuid.py b/app/todo_api/migrations/0002_tasklist_task_list_uuid.py
deleted file mode 100644
index a2a1a08..0000000
--- a/app/todo_api/migrations/0002_tasklist_task_list_uuid.py
+++ /dev/null
@@ -1,19 +0,0 @@
-# Generated by Django 4.0 on 2023-07-24 23:37
-
-from django.db import migrations, models
-import uuid
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('todo_api', '0001_initial'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='tasklist',
-            name='task_list_uuid',
-            field=models.UUIDField(default=uuid.uuid4, editable=False, unique=True),
-        ),
-    ]
diff --git a/app/todo_api/migrations/0003_rename_task_list_uuid_tasklist_list_uuid.py b/app/todo_api/migrations/0003_rename_task_list_uuid_tasklist_list_uuid.py
deleted file mode 100644
index 1fe4d32..0000000
--- a/app/todo_api/migrations/0003_rename_task_list_uuid_tasklist_list_uuid.py
+++ /dev/null
@@ -1,18 +0,0 @@
-# Generated by Django 4.0 on 2023-07-24 23:40
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('todo_api', '0002_tasklist_task_list_uuid'),
-    ]
-
-    operations = [
-        migrations.RenameField(
-            model_name='tasklist',
-            old_name='task_list_uuid',
-            new_name='list_uuid',
-        ),
-    ]
diff --git a/app/todo_api/migrations/0004_task_task_uuid.py b/app/todo_api/migrations/0004_task_task_uuid.py
deleted file mode 100644
index a6f3499..0000000
--- a/app/todo_api/migrations/0004_task_task_uuid.py
+++ /dev/null
@@ -1,19 +0,0 @@
-# Generated by Django 4.0 on 2023-07-24 23:43
-
-from django.db import migrations, models
-import uuid
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('todo_api', '0003_rename_task_list_uuid_tasklist_list_uuid'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='task',
-            name='task_uuid',
-            field=models.UUIDField(default=uuid.uuid4, editable=False, unique=True),
-        ),
-    ]
diff --git a/app/todo_api/migrations/0005_alter_task_task_list.py b/app/todo_api/migrations/0005_alter_task_task_list.py
deleted file mode 100644
index 1b1d748..0000000
--- a/app/todo_api/migrations/0005_alter_task_task_list.py
+++ /dev/null
@@ -1,19 +0,0 @@
-# Generated by Django 4.0 on 2023-07-25 01:02
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('todo_api', '0004_task_task_uuid'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='task',
-            name='task_list',
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, to='todo_api.tasklist', to_field='list_uuid'),
-        ),
-    ]
diff --git a/app/todo_api/permissions.py b/app/todo_api/permissions.py
deleted file mode 100644
index e69de29..0000000
diff --git a/app/todo_api/tests/test_task_api.py b/app/todo_api/tests/test_task_api.py
index f49ffad..297c04c 100644
--- a/app/todo_api/tests/test_task_api.py
+++ b/app/todo_api/tests/test_task_api.py
@@ -71,7 +71,6 @@ def test_retrieve_tasks(self):
         create_task(user=self.user)
 
         res = self.client.get(TASKS_URL)
-        print(res.data)
         tasks = Task.objects.all().order_by("created_at")
         serializer = TaskSerializer(tasks, many=True)
 
@@ -93,7 +92,6 @@ def test_retrieve_tasks_limited_to_user(self):
         create_task(user=self.user)
 
         res = self.client.get(TASKS_URL)
-        print(res.data)
 
         tasks = Task.objects.filter(created_by=self.user).order_by("created_at")
         serializer = TaskSerializer(tasks, many=True)
diff --git a/app/todo_api/tests/test_task_list_api.py b/app/todo_api/tests/test_task_list_api.py
index ee4ba1c..9cb40ad 100644
--- a/app/todo_api/tests/test_task_list_api.py
+++ b/app/todo_api/tests/test_task_list_api.py
@@ -1,3 +1,4 @@
+from venv import create
 from django.test import TestCase
 from django.urls import reverse
 from django.contrib.auth import get_user_model
@@ -11,7 +12,7 @@
 TASKS_LISTS_URL = reverse("todo_api:lists-list")
 
 
-def list_detail_url_list(list_uuid):
+def list_detail_url(list_uuid):
     return reverse(
         "todo_api:lists-detail",
         args=[
@@ -88,16 +89,56 @@ def test_retrieve_list_pk_from_name(self):
         list_2 = create_task_list(user=self.user, name="List 2")
         create_task_list(user=self.user, name="List 3")
 
-        res = self.client.get(list_detail_url_list(list_2.list_uuid))
+        res = self.client.get(list_detail_url(list_2.list_uuid))
 
         self.assertEqual(res.status_code, status.HTTP_200_OK)
         self.assertEqual(list_2.id, res.data["id"])
+    
 
     def test_retrieve_inbox_from_name(self):
         """Test retrieve and create if inbox not exists using slug name"""
-        url = list_detail_url_list("inbox")
+        url = list_detail_url("inbox")
         res = self.client.get(url)
         exists = TaskList.objects.filter(name="inbox").exists()
 
         self.assertEqual(res.status_code, status.HTTP_200_OK)
         self.assertTrue(exists)
+        
+    def test_retrieve_unique_inbox_for_each_account(self):
+        """Test retrieve and create unique inbox for account"""
+        user_two = get_user_model().objects.create(
+            email="usertwo@example.com", password="userexample123"
+        )
+        list_inbox = create_task_list(user=user_two, name="inbox")
+        url = list_detail_url("inbox")
+        res = self.client.get(url)
+        exists = TaskList.objects.filter(name="inbox", created_by=self.user).exists()
+        list = TaskList.objects.filter(name="inbox", created_by=self.user)
+
+        self.assertEqual(res.status_code, status.HTTP_200_OK)
+        self.assertTrue(exists)
+        self.assertNotEqual(list_inbox.list_uuid, list[0].list_uuid)
+        
+    def test_fully_update_task_list(self):
+        """Test partial update list with patch"""
+        list = create_task_list(user=self.user)
+        payload = {
+            "name": "New name"
+        }
+        url = list_detail_url(list.list_uuid)
+        res = self.client.patch(url, payload)
+        self.assertEqual(res.status_code, status.HTTP_200_OK)
+        list.refresh_from_db()
+        self.assertEqual(list.name, payload["name"])
+
+    def test_update_unauthorized_failure(self):
+        new_user = get_user_model().objects.create_user(
+            email="user@new.com", password="usernew123"
+        )
+        list = create_task_list(user=new_user)
+        url = list_detail_url(list.list_uuid)
+        res = self.client.patch(url, {
+            "name": "new name"
+        })
+        self.assertEqual(res.status_code, status.HTTP_404_NOT_FOUND)
+        self.assertNotEqual(list.name, "new name")
\ No newline at end of file
diff --git a/app/todo_api/views.py b/app/todo_api/views.py
index 3c1b0fc..653fbba 100644
--- a/app/todo_api/views.py
+++ b/app/todo_api/views.py
@@ -143,7 +143,7 @@ def retrieve(self, request, list_uuid=None, *args, **kwargs):
         try:
             if list_uuid == "inbox":
                 # case inbox os is not created
-                inbox = TaskList.objects.filter(name="inbox").exists()
+                inbox = TaskList.objects.filter(name="inbox", created_by=self.request.user).exists()  # noqa: E501
                 if not inbox:
                     TaskList.objects.create(name="inbox", created_by=self.request.user)
                 queryset = queryset.get(name__iexact="inbox")
@@ -155,3 +155,4 @@ def retrieve(self, request, list_uuid=None, *args, **kwargs):
             )
         serializer = self.get_serializer(queryset, many=False)
         return Response(serializer.data)
+    
\ No newline at end of file
diff --git a/app/todo_project/settings.py b/app/todo_project/settings.py
index f7ed6ee..3746d1e 100644
--- a/app/todo_project/settings.py
+++ b/app/todo_project/settings.py
@@ -90,46 +90,46 @@
 # Database
 # https://docs.djangoproject.com/en/4.0/ref/settings/#databases
 
-# MYSQL_DATABASE = os.getenv('MYSQL_DATABASE')
-# MYSQL_ROOT_PASSWORD = os.getenv('MYSQL_ROOT_PASSWORD')
-# MYSQL_DATABASE_HOST = os.getenv('MYSQL_DATABASE_HOST')
-# MYSQL_DATABASE_PORT = os.getenv('MYSQL_DATABASE_PORT')
-
-# DATABASES = {
-#     'default': {
-#         'ENGINE': 'django.db.backends.mysql',
-#         'NAME': MYSQL_DATABASE,
-#         'USER': 'root',
-#         'PASSWORD': MYSQL_ROOT_PASSWORD,
-#         'HOST': MYSQL_DATABASE_HOST,
-#         'PORT': MYSQL_DATABASE_PORT,
-#         'TEST': {
-#             'NAME': 'test_tododjango',
-#         },
-#     }
-# }
-
-# if 'test' in sys.argv or 'test_coverage' in sys.argv:
-#     DATABASES = {
-#         'default': {
-#             'ENGINE': 'django.db.backends.sqlite3',
-#             'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
-#         }
-#     }
+MYSQL_DATABASE = os.getenv('MYSQL_DATABASE')
+MYSQL_ROOT_PASSWORD = os.getenv('MYSQL_ROOT_PASSWORD')
+MYSQL_DATABASE_HOST = os.getenv('MYSQL_DATABASE_HOST')
+MYSQL_DATABASE_PORT = os.getenv('MYSQL_DATABASE_PORT')
 
 DATABASES = {
     'default': {
-        'ENGINE': 'django.db.backends.sqlite3',
-        'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
+        'ENGINE': 'django.db.backends.mysql',
+        'NAME': MYSQL_DATABASE,
+        'USER': 'root',
+        'PASSWORD': MYSQL_ROOT_PASSWORD,
+        'HOST': MYSQL_DATABASE_HOST,
+        'PORT': MYSQL_DATABASE_PORT,
+        'TEST': {
+            'NAME': 'test_tododjango',
+        },
     }
 }
 
+if 'test' in sys.argv or 'test_coverage' in sys.argv:
+    DATABASES = {
+        'default': {
+            'ENGINE': 'django.db.backends.sqlite3',
+            'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
+        }
+    }
+
+# DATABASES = {
+#     'default': {
+#         'ENGINE': 'django.db.backends.sqlite3',
+#         'NAME': os.path.join(BASE_DIR, 'db.sqlite3'),
+#     }
+# }
+
 # Password validation
 # https://docs.djangoproject.com/en/4.0/ref/settings/#auth-password-validators
 
 AUTH_PASSWORD_VALIDATORS = [
     {
-        'NAME': 'django.contrib.auth.password_validation.UserAttributeSimilarityValidator',
+        'NAME': 'django.contrib.auth.password_validation.UserAttributeSimilarityValidator',  # noqa: E501
     },
     {
         'NAME': 'django.contrib.auth.password_validation.MinimumLengthValidator',
@@ -177,7 +177,7 @@
 TEST_RUNNER = "redgreenunittest.django.runner.RedGreenDiscoverRunner"
 
 # CORS configuration
-CORS_ORIGIN_ALLOW_ALL = True  # only for dev environment!, this should be changed before you push to production
+CORS_ORIGIN_ALLOW_ALL = True  # only for dev environment!, this should be changed before you push to production  # noqa: E501
 
 # Default user model
 AUTH_USER_MODEL = 'core.User'
diff --git a/requirements.txt b/requirements.txt
index b302c5d..a4a029f 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -13,12 +13,12 @@ pytz==2022.7.1
 
 psycopg2-binary==2.9.5
 urllib3~=1.26.13
-# mysqlclient==2.1.1
-# django-mysql==3.8.1
+mysqlclient==2.1.1
+django-mysql==3.8.1
 
 dj-rest-auth==4.0.1
 django-allauth==0.50.0
 djangorestframework-simplejwt==5.2.2
 
 PyJWT==2.7.0
-flake8==4.0.1
+flake8==4.0.1
\ No newline at end of file