changes

Author: marcoacierno

backend/organizers/tests/factories.py

@@ -9,4 +9,4 @@ class Meta:
         model = Organizer
 
     name = factory.Faker("word")
-    slug = factory.Faker("word")
+    slug = factory.Sequence(lambda n: "slug{}".format(n))

backend/visa/models.py

@@ -114,7 +114,8 @@ def grant_approved_type(self):
     def user_grant(self):
         return Grant.objects.for_conference(self.conference).of_user(self.user).first()
 
-    def get_role(self):
+    @cached_property
+    def role(self):
         user = self.user
 
         if not user:

backend/visa/tasks.py

@@ -140,7 +140,7 @@ def _render_content(content, invitation_letter_request, config):
                 "date_of_birth": invitation_letter_request.date_of_birth,
                 "passport_number": invitation_letter_request.passport_number,
                 "embassy_name": invitation_letter_request.embassy_name,
-                "role": invitation_letter_request.get_role(),
+                "role": invitation_letter_request.role,
                 "grant_approved_type": invitation_letter_request.grant_approved_type,
                 "has_accommodation_via_grant": invitation_letter_request.has_accommodation_via_grant(),
                 "has_travel_via_grant": invitation_letter_request.has_travel_via_grant(),

backend/visa/tests/test_models.py

@@ -19,7 +19,7 @@ def test_request_on_behalf_of_other():
     assert request.on_behalf_of_other
     assert request.email == "[email protected]"
     assert request.user is None
-    assert request.get_role() == "Attendee"
+    assert request.role == "Attendee"
 
     # With matching user, it is found
     user = UserFactory(email="[email protected]")
@@ -74,7 +74,7 @@ def test_role_for_speakers():
         status=Submission.STATUS.accepted,
     )
 
-    assert request.get_role() == "Speaker"
+    assert request.role == "Speaker"
 
 
 def test_schedule_processing(django_capture_on_commit_callbacks, mocker):