[#43] Rating model to unblock issues #42 #44 #45 #46 #101 #102

app/migrations/0014_rating.py

+# Generated by Django 3.0.3 on 2020-09-29 11:30
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+import django.utils.timezone
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('app', '0013_auto_20200919_2055'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='Rating',
+            fields=[
+                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('timestamp', models.DateTimeField(default=django.utils.timezone.now)),
+                ('score', models.IntegerField()),
+                ('materi', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='app.Materi')),
+                ('user', models.ForeignKey(blank=True, on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
+            ],
+            options={
+                'unique_together': {('materi', 'user')},
+            },
+        ),
+    ]

app/models.py

 import random
 
+from django.core.exceptions import ValidationError
 from django.db import models
 from django.utils import timezone
 
@@ -12,6 +13,7 @@ VERIFICATION_STATUS = [
     ("REVISION", "Perbaikan"),
 ]
 
+
 # Create your models here.
 
 
@@ -108,3 +110,22 @@ class DummyDownloadStatistics(models.Model):
 
 class DummyComment(models.Model):
     item = models.ForeignKey(Comment, on_delete=models.CASCADE)
+
+
+class Rating(models.Model):
+    materi = models.ForeignKey(Materi, models.CASCADE)
+    user = models.ForeignKey(User, on_delete=models.CASCADE, blank=True)
+    timestamp = models.DateTimeField(default=timezone.now)
+    score = models.IntegerField()
+
+    def save(self, force_insert=False, force_update=False, using=None, update_fields=None):
+        if 1 <= self.score <= 5:
+            super().save(force_insert, force_update, using, update_fields)
+        else:
+            raise ValidationError("Rating score must be integer between 1-5")
+
+    def __str__(self):
+        return "Material:{} | User:{} | Rating:{}".format(self.materi.title, self.user.name, self.score)
+
+    class Meta:
+        unique_together = ["materi", "user"]

app/tests.py

 import json
 
 from django.contrib.auth import get_user_model
-from django.core import serializers
-from django.core.files import File
+from django.core.exceptions import ValidationError
 from django.core.files.uploadedfile import SimpleUploadedFile
-from django.test import Client, RequestFactory, TestCase
+from django.db import IntegrityError
+from django.test import Client, TestCase
 from django.urls import resolve
 
 from administration.utils import id_generator
-from app.views import UploadMateriHTML, UploadMateriView
+from app.views import UploadMateriView
 from authentication.models import User
-
-from .models import Category, Comment, Materi, Like, ViewStatistics, DownloadStatistics
+from .models import Category, Comment, Materi, Like, Rating
 from .views import (DaftarKatalog, DashboardKontributorView, DetailMateri,
                     ProfilKontributorView, SuksesLoginAdminView,
                     SuksesLoginKontributorView, SuntingProfilView,
                     ProfilAdminView, CommentsView, SuntingProfilAdminView, RevisiMateriView)
 
-from app.views import UploadMateriHTML, UploadMateriView
-from authentication.models import User
-
 
 class DaftarKatalogTest(TestCase):
     def test_daftar_katalog_url_exist(self):
@@ -928,4 +924,98 @@ class RevisiMateriTest(TestCase):
         response = self.client.get(self.url)
         self.assertEqual(response.status_code, 200)
         # Logout
-        self.client.logout()
\ No newline at end of file
+        self.client.logout()
+
+
+class RatingMateriTest(TestCase):
+    def setUp(self):
+        self.url = '/administration/'
+        self.contributor_credential = {
+            "email": "kontributor@gov.id",
+            "password": id_generator()
+        }
+        self.user_one_credential = {
+            "email": "user_one@user.id",
+            "password": id_generator()
+        }
+        self.user_two_credential = {
+            "email": "user_two@user.id",
+            "password": id_generator()
+        }
+        self.contributor = get_user_model().objects.create_user(
+            **self.contributor_credential, name="Kontributor", is_contributor=True
+        )
+        self.user_one = get_user_model().objects.create_user(**self.user_one_credential, name="User One")
+        self.user_two = get_user_model().objects.create_user(**self.user_two_credential, name="User Two")
+        self.cover = SimpleUploadedFile(
+            "cover.jpg",
+            b"Test file"
+        )
+        self.content = SimpleUploadedFile(
+            "content.txt",
+            b"Test file"
+        )
+        Materi(title="Materi 1", author="Agas", uploader=self.contributor,
+               publisher="Kelas SC", descriptions="Deskripsi Materi 1",
+               status="PENDING", cover=self.cover, content=self.content).save()
+        Materi(title="Materi Dua", author="Author", uploader=self.contributor,
+               publisher="Publisher", descriptions="Deskripsi Materi Dua",
+               status="APPROVE", cover=self.cover, content=self.content).save()
+        self.materi1 = Materi.objects.all()[0]
+        self.materi2 = Materi.objects.all()[1]
+
+    def test_rating_model_can_be_created_with_proper_parameter(self):
+        Rating(materi=self.materi1, user=self.user_one, score=5).save()
+        rating = Rating.objects.first()
+        self.assertEqual(rating.materi, self.materi1)
+        self.assertEqual(rating.user, self.user_one)
+        self.assertTrue(0 < rating.score < 6)
+        self.assertEqual(rating.__str__(), "Material:Materi 1 | User:User One | Rating:5")
+
+    def test_rating_model_should_not_be_created_with_rating_more_than_5(self):
+        with self.assertRaises(ValidationError) as context:
+            Rating(materi=self.materi1, user=self.user_one, score=6).save()
+        self.assertTrue('Rating score must be integer between 1-5' in str(context.exception))
+
+    def test_rating_model_should_not_be_created_with_rating_less_than_1(self):
+        with self.assertRaises(ValidationError) as context:
+            Rating(materi=self.materi1, user=self.user_one, score=0).save()
+        self.assertTrue('Rating score must be integer between 1-5' in str(context.exception))
+
+    def test_one_materi_should_be_able_to_be_related_to_multiple_rating(self):
+        Rating(materi=self.materi1, user=self.user_one, score=1).save()
+        Rating(materi=self.materi1, user=self.user_two, score=2).save()
+        rating_one = Rating.objects.get(user=self.user_one)
+        rating_two = Rating.objects.get(user=self.user_two)
+        self.assertEqual(rating_one.materi, self.materi1)
+        self.assertEqual(rating_two.materi, self.materi1)
+        self.assertEqual(rating_one.user, self.user_one)
+        self.assertEqual(rating_two.user, self.user_two)
+
+    def test_one_user_should_be_able_to_be_related_to_two_rating(self):
+        Rating(materi=self.materi1, user=self.user_one, score=3).save()
+        Rating(materi=self.materi2, user=self.user_one, score=3).save()
+        rating_one = Rating.objects.filter(materi=self.materi1).first()
+        rating_two = Rating.objects.filter(materi=self.materi2).first()
+        self.assertEqual(rating_one.materi, self.materi1)
+        self.assertEqual(rating_two.materi, self.materi2)
+        self.assertEqual(rating_one.user, self.user_one)
+        self.assertEqual(rating_two.user, self.user_one)
+
+    def test_two_rating_should_not_have_same_user_and_materi(self):
+        with self.assertRaises(IntegrityError) as context:
+            Rating(materi=self.materi1, user=self.user_one, score=1).save()
+            Rating(materi=self.materi1, user=self.user_one, score=2).save()
+        self.assertTrue('already exists' in str(context.exception))
+
+    def test_materi_in_rating_should_not_be_null(self):
+        with self.assertRaises(IntegrityError):
+            Rating(user=self.user_one, score=1).save()
+
+    def test_user_in_rating_should_not_be_null(self):
+        with self.assertRaises(IntegrityError):
+            Rating(materi=self.materi1, score=1).save()
+
+    def test_score_in_rating_should_not_be_null(self):
+        with self.assertRaises(TypeError):
+            Rating(materi=self.materi1, user=self.user_one).save()