mangaki · Elarnon · Nov 12, 2016 · Nov 12, 2016 · Nov 12, 2016 · Nov 12, 2016
diff --git a/mangaki/mangaki/__init__.py b/mangaki/mangaki/__init__.py
@@ -0,0 +1 @@
+default_app_config = 'mangaki.apps.MangakiConfig'
diff --git a/mangaki/mangaki/apps.py b/mangaki/mangaki/apps.py
@@ -0,0 +1,11 @@
+from django.apps import AppConfig
+
+
+class MangakiConfig(AppConfig):
+    name = 'mangaki'
+    verbose_name = 'Mangaki'
+
+    def ready(self):
+        # Ensure signal receivers decorated with `@receiver` are connected by
+        # importing the `receivers` module.
+        from . import receivers
diff --git a/mangaki/mangaki/factories.py b/mangaki/mangaki/factories.py
@@ -1,8 +1,9 @@
 import factory
-from factory.django import DjangoModelFactory
+from factory.django import DjangoModelFactory, mute_signals
 
 from .models import Profile, Work, Category
 from django.contrib.auth.models import User
+from django.db.models.signals import post_save
 
 class ProfileFactory(DjangoModelFactory):
     class Meta:
@@ -15,6 +16,7 @@ class Meta:
     newsletter_ok = factory.Faker('boolean')
     avatar_url = factory.LazyAttribute(lambda o: '{}{}.png'.format(factory.Faker('url').generate({}), o.mal_username))
 
+@mute_signals(post_save)
 class UserFactory(DjangoModelFactory):
     class Meta:
         model = User

diff --git a/mangaki/mangaki/models.py b/mangaki/mangaki/models.py
@@ -281,10 +281,9 @@ def update_scores(self):
         score = suggestions_score + recommendations_score
         Profile.objects.filter(user=self.user).update(score=score)
 
-
-def suggestion_saved(sender, instance, *args, **kwargs):
-    instance.update_scores()
-models.signals.post_save.connect(suggestion_saved, sender=Suggestion)
+    def save(self, *args, **kwargs):
+        super().save(*args, **kwargs)
+        self.update_scores()
 
 
 class Neighborship(models.Model):

diff --git a/mangaki/mangaki/receivers.py b/mangaki/mangaki/receivers.py
@@ -0,0 +1,11 @@
+from django.db.models.signals import post_save
+from django.dispatch import receiver
+from django.conf import settings
+
+from mangaki.models import Profile
+
+
+@receiver(post_save, sender=settings.AUTH_USER_MODEL)
+def create_profile(sender, instance, created, **kwargs):
+    if created:
+        Profile.objects.create(user=instance)
diff --git a/mangaki/mangaki/tests/test_user.py b/mangaki/mangaki/tests/test_user.py
@@ -0,0 +1,13 @@
+from django.test import TestCase
+from django.contrib.auth import get_user_model
+
+from mangaki.models import Profile
+
+
+class UserTest(TestCase):
+    def setUp(self):
+        self.User = get_user_model()
+
+    def test_user_creation_creates_profile(self):
+        u = self.User.objects.create_user('testuser')
+        self.assertIsInstance(u.profile, Profile)
diff --git a/mangaki/mangaki/views.py b/mangaki/mangaki/views.py
@@ -19,8 +19,6 @@
 from django.dispatch import receiver
 from django.db.models import Count, Case, When, F, Value, Sum, IntegerField
 from django.db import connection
-from allauth.account.signals import user_signed_up
-from allauth.socialaccount.signals import social_account_added
 from mangaki.models import Work, Rating, Page, Profile, Artist, Suggestion, SearchIssue, Announcement, Recommendation, Pairing, Top, Ranking, Staff, Category, FAQTheme
 from mangaki.mixins import AjaxableResponseMixin, JSONResponseMixin
 from mangaki.mixins import AjaxableResponseMixin
@@ -696,13 +694,6 @@ def add_pairing(request, artist_id, work_id):
     return HttpResponse()
 
 
-@receiver(user_signed_up)
-@receiver(social_account_added)
-def register_profile(sender, **kwargs):
-    user = kwargs['user']
-    Profile(user=user).save()
-
-
 def faq_index(request):
     latest_theme_list = FAQTheme.objects.order_by('order')
     all_information = [[faqtheme.theme, [(entry.question, entry.answer) for entry in faqtheme.entries.filter(is_active=True).order_by('-pub_date')]] for faqtheme in latest_theme_list]
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		default_app_config = 'mangaki.apps.MangakiConfig'