codeforlife-portal 7.4.1__py2.py3-none-any.whl → 7.4.3__py2.py3-none-any.whl

{codeforlife_portal-7.4.1.dist-info → codeforlife_portal-7.4.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: codeforlife-portal
-Version: 7.4.1
+Version: 7.4.3
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Framework :: Django
@@ -21,7 +21,7 @@ Requires-Dist: django-classy-tags==2.0.0
 Requires-Dist: libsass==0.23.0
 Requires-Dist: phonenumbers==8.12.12
 Requires-Dist: more-itertools==8.7.0
-Requires-Dist: cfl-common==7.4.1
+Requires-Dist: cfl-common==7.4.3
 Requires-Dist: django-ratelimit==3.0.1
 Requires-Dist: django-preventconcurrentlogins==0.8.2
 Requires-Dist: django-csp==3.7

{codeforlife_portal-7.4.1.dist-info → codeforlife_portal-7.4.3.dist-info}/RECORD

@@ -108,7 +108,7 @@ example_project/portal_test_settings.py,sha256=_gI7-HMoPJ-cDGO6n5UlEIHKHuTR5SC_X
 example_project/settings.py,sha256=HgGSG0n6Xggd0NieFVoJgn8vKGqyRbCddoB3CRuoT-Y,5640
 example_project/urls.py,sha256=3SsP5jvPAXV5xmduka4zbSZB5PbXggjsalu1ojY5KIo,434
 example_project/wsgi.py,sha256=U1W6WzZxZaIdYZ5tks7w9fqp5WS5qvn2iThsVcskrWw,829
-portal/__init__.py,sha256=p8eq7jnQadKhC-wWoeEv-nRdPB1JN2Jan_o-R14Xz9w,22
+portal/__init__.py,sha256=olYEIni3LYPkdTYbQN19dX_D-AAeoyTsiUuVAO9gQPQ,22
 portal/admin.py,sha256=on1-zNRnZvf2cwBN6GVRVYRhkaksrCgfzX8XPWtkvz8,6062
 portal/app_settings.py,sha256=DhWLQOwM0zVOXE3O5TNKbMM9K6agfLuCsHOdr1J7xEI,651
 portal/backends.py,sha256=2Dss6_WoQwPuDzJUF1yEaTQTNG4eUrD12ujJQ5cp5Tc,812
@@ -126,7 +126,7 @@ portal/forms/invite_teacher.py,sha256=jkDNcCfkts4_lXRzhcI3xBam21Zn2yX9wMpMVhDtW1
 portal/forms/organisation.py,sha256=QcQyd7AiqBmvt4y8uQSQylguUbKOKqo2pjqWIkpWjDg,7433
 portal/forms/play.py,sha256=z9P5LzyS3jjYcnfco84d2x8ptgLxRmh94Dnj05plmbY,11505
 portal/forms/registration.py,sha256=gWcY7rllhWO3c9as6QHUDWZx1Jme7DqtGHYaKcvxe-U,5990
-portal/forms/teach.py,sha256=lFGM9gk1gC_N_qJhzV2VdAVArCiAYfhOmrxvPkYC-co,22446
+portal/forms/teach.py,sha256=Fd7zOdwpKTFmxxa3q_Tsj1E9B_M5DZlStF5YXiCA-ys,22688
 portal/helpers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 portal/helpers/captcha.py,sha256=Amwg3HZ9eIh1LGYVYWYruk1ccNj6P3nYP_evufOY8BY,254
 portal/helpers/decorators.py,sha256=AnHbcRg42cUYkpMnJMImEMueSKHZRBZ0o67B7_Wp_Zs,3695
@@ -435,7 +435,7 @@ portal/templates/captcha/includes/js_v2_invisible.html,sha256=FUyDf1RDyS7whzW2B_
 portal/templates/portal/about.html,sha256=_iD0GCP6q3-XuZ2LC-9O0KYY-mKL6c9qk3O-NRRqsRM,10321
 portal/templates/portal/base.html,sha256=4xLACNgKmRQTdEsdNNGYhLzMozzYzWIIzk31HrLGBf0,12109
 portal/templates/portal/base_no_userprofile.html,sha256=PlRufyYmUUGWBZ6CvbYhJWOMTqKqdcee4xnO5--AogA,447
-portal/templates/portal/coding_club.html,sha256=aaLx9UQ8p_7Vvy3Vlh1rOufeNN0KhJ6fj4oqI6QqJcM,4518
+portal/templates/portal/coding_club.html,sha256=A9_hiD-ChBRnictFvAaQiSgSANfrnQ_zFY6hqBtK2rg,5767
 portal/templates/portal/contribute.html,sha256=UIC_N3Lun9wB_6jEra0wvlT9fDiiIeMby7_onXYo6k0,4176
 portal/templates/portal/dotmailer_consent_form.html,sha256=UDdizPoKYZGybr6z9nzDV4WPhJPa-S3bIKywvVgFrxg,918
 portal/templates/portal/email_invitation_sent.html,sha256=hAMzQXE3NFGnOsQlCGuo3Aps-vazSJb5BhAN7bWT48M,641
@@ -490,7 +490,7 @@ portal/templates/portal/tag_manager/tag_manager_head.html,sha256=kWQ4ccJ062cWRJO
 portal/templates/portal/teach/base_registering.html,sha256=jpANyRQbsmWO404l9Mez91zolNP9ic1khWi27WW0FMQ,582
 portal/templates/portal/teach/class.html,sha256=zir7a7y4hbg9E6-zFIC0iv8kccIzF-qkni4DXlvkjgM,6726
 portal/templates/portal/teach/dashboard.html,sha256=SwNdLbTDPaz0O7wVlSVkU8KwIWPzBYsOo6ysOxkOwOc,32674
-portal/templates/portal/teach/invited.html,sha256=OJ_b6JY6n0iIh6CzWTYTSxfQMIk-RRVgrRFlOr57GLU,3908
+portal/templates/portal/teach/invited.html,sha256=lTsydONG5se1E099KaRgsxGuG12qVHxFUL2X0E3HBmQ,4188
 portal/templates/portal/teach/onboarding_classes.html,sha256=K212dunXG7eByyCg5lhv4ne-SniMz3kcKXrfianIY6g,2872
 portal/templates/portal/teach/onboarding_print.html,sha256=4vD2ZDNRxERNdCNj3ZNXJWYAFBQPn9C7Pzsz5TZjZHw,8862
 portal/templates/portal/teach/onboarding_school.html,sha256=EyvnQ13R4ovf3-152pCvjxJJIGeT47ygs0VU9ftvJlQ,2491
@@ -535,12 +535,11 @@ portal/tests/test_admin.py,sha256=AM2dgv8j9m4L-SDO-sMA9tQvQH9GwRBrlwRG9OgqtfI,14
 portal/tests/test_api.py,sha256=Yo5s_nEGOoG35jA39yZ6nuDOUZvuCZ8o8o8XhZos61w,13819
 portal/tests/test_captcha_forms.py,sha256=lirhIli-sHovun8VdrF0he7KRFTAd8DMCpkJ8cQNotg,1015
 portal/tests/test_class.py,sha256=43g2HslNosbSuSnnMBzM6VWk4LaV0HXkWS3f8VKVRmw,15930
-portal/tests/test_daily_activities.py,sha256=-siDCMGBD1ijjccHVk7eEmrk4bgTsvbh0B6hDoj2fo0,1803
 portal/tests/test_emails.py,sha256=pLr06j3uMBxP1raoZQWzUTBVFvsEDFtUh85J8OnqCwE,9238
 portal/tests/test_global_forms.py,sha256=A5JpAe4AYK-wpu0o1qU4THmeNv_wr7lhzaMbjz5czpY,1543
 portal/tests/test_helper_methods.py,sha256=-SQCDZm2XUtyXGEp0CHIb_SSC9CPD-XOSnpnY8QclHk,890
 portal/tests/test_independent_student.py,sha256=NrRjTEr6V4WXpCE74N8LYNVocvLSvddkjuo3dYpfAZc,27245
-portal/tests/test_invite_teacher.py,sha256=oeOaoJV1IqJSYPlaPFjnhVXdB2mq8otCTLp_lfjuCfk,12224
+portal/tests/test_invite_teacher.py,sha256=gUe1spFp60v3i6kMqGoNgJd0OlBEcwplPPNYLomTJS4,12269
 portal/tests/test_middleware.py,sha256=b6jfNmiRZ2snqLKsyJUG-RivoX5fmrqLlQkG9MeVnqM,8034
 portal/tests/test_organisation.py,sha256=kCMUNzLN6EzaMUBcFkqXwnqLGgOuQxQWIHHt63nhqBs,7574
 portal/tests/test_partials.py,sha256=cSLNLjdsriROjPxkZlM3HKD3CSKDuKgpaDIdL3fPyBs,1794
@@ -610,7 +609,7 @@ portal/views/admin.py,sha256=4Xt3zEyQH7sUwQSrwuRtoCodWidjOzd7gJUwWU96pXY,957
 portal/views/api.py,sha256=lCwiclR98G-yTgK55u8IjkueIH8iremeiZSa3jAvO-M,6990
 portal/views/dotmailer.py,sha256=x49p89TtwA1MLysRLtq5yRRzVtIpzGoU__Xb5hPuHak,3208
 portal/views/email.py,sha256=V3wXRxIjeZ4OJBVqGCQrPn-GQWKZK1PCXbR1f2Zpa_4,2174
-portal/views/home.py,sha256=GlNTJqSST_jQi0REuaV2aI2zram-KAhedmATdC6yU58,10247
+portal/views/home.py,sha256=qt5kW7TAaRFXz7BA2BrnHe5oYc69jjoLksEbfr0Aa3c,9715
 portal/views/legal.py,sha256=nUunsTHnhMcXBcDlg1GmUal86k9Vhinne4A2FWfq78M,342
 portal/views/organisation.py,sha256=sPDbiM7hdtpF8GKyh_4n4VPl2a-WnAgnF4q9aSvQCVI,3341
 portal/views/play_landing_page.py,sha256=FFmjUFub3ZdlbMqkB8yX3jAImCzqrUqgb8AZcpKywZ4,308
@@ -627,13 +626,13 @@ portal/views/student/edit_account_details.py,sha256=Ba-3D_zzKbX5N01NG5qqBS0ud10B
 portal/views/student/play.py,sha256=GMxk65bxWOe1Ds2kb6rvuOd1GoAtt5v_9AihLNXoUL0,8768
 portal/views/teacher/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 portal/views/teacher/dashboard.py,sha256=pjzOOCz29d4VHukaCI5QhkDsu-RPvy7SXC9dDcPo50k,27422
-portal/views/teacher/teach.py,sha256=MxheM_kxt_K1J7jEWlim8yZonnp63iYUTw5dCaLscLI,37056
+portal/views/teacher/teach.py,sha256=yWe2K0FHEQB_K3vzPJO8MR0gU2ZaDrXGw8gXMJBH0T0,35765
 portal/views/two_factor/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 portal/views/two_factor/core.py,sha256=O_wcBeFqdPYSGNGv-pT_vbs5-Dj1Z-Jfkd6f9-E5yZI,760
 portal/views/two_factor/form.py,sha256=lnHNKI-BMlpncTuW3zUzjPaJJNuEra2I_nOam0eOKFY,257
 portal/views/two_factor/profile.py,sha256=tkl_ludo8arMtd5LKNmohM66vpC_YQiP-0nspTSJiJ4,383
-codeforlife_portal-7.4.1.dist-info/LICENSE.md,sha256=9AbRlCDqD2D1tPibimysFv3zg3AIc49-eyv9aEsyq9w,115
-codeforlife_portal-7.4.1.dist-info/METADATA,sha256=6kN9QQduDOTlZTEwl5YTS2LiqxZijcWzKsBWa0qUYFQ,3317
-codeforlife_portal-7.4.1.dist-info/WHEEL,sha256=fS9sRbCBHs7VFcwJLnLXN1MZRR0_TVTxvXKzOnaSFs8,110
-codeforlife_portal-7.4.1.dist-info/top_level.txt,sha256=8e5pdsuIoTqEAMqpelHBjGjLbffcBtgOoggmd2q7nMw,41
-codeforlife_portal-7.4.1.dist-info/RECORD,,
+codeforlife_portal-7.4.3.dist-info/LICENSE.md,sha256=9AbRlCDqD2D1tPibimysFv3zg3AIc49-eyv9aEsyq9w,115
+codeforlife_portal-7.4.3.dist-info/METADATA,sha256=UWFujJxxfaXyqmfcwg4-DYbkouqg-eSRo3i0SWVNCTo,3317
+codeforlife_portal-7.4.3.dist-info/WHEEL,sha256=fS9sRbCBHs7VFcwJLnLXN1MZRR0_TVTxvXKzOnaSFs8,110
+codeforlife_portal-7.4.3.dist-info/top_level.txt,sha256=8e5pdsuIoTqEAMqpelHBjGjLbffcBtgOoggmd2q7nMw,41
+codeforlife_portal-7.4.3.dist-info/RECORD,,

portal/__init__.py

@@ -1 +1 @@
-__version__ = "7.4.1"
+__version__ = "7.4.3"

portal/forms/teach.py

@@ -18,6 +18,13 @@ from portal.helpers.ratelimit import clear_ratelimit_cache_for_user
 
 
 class InvitedTeacherForm(forms.Form):
+    prefix = 'teacher_signup'
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        for field_name, field in self.fields.items():
+            field.widget.attrs['id'] = f'id_teacher_signup-{field_name}'
+
     teacher_password = forms.CharField(
         help_text="Enter a password",
         widget=forms.PasswordInput(

portal/templates/portal/coding_club.html

@@ -86,12 +86,22 @@
                     </p>
                 </div>
                 <div>
-                    <form method="post", action="{% url 'download_student_pack' 4 %}" class="non-styled--form">
-                        {% csrf_token %}
-                        <button id="python_pack" download type="submit" class="button button--primary button--icon">
-                            Download the Python coding club pack<span class="iconify" data-icon="mdi:tray-arrow-down"></span>
-                        </button>
-                    </form>
+                    <!-- TODO: improve responsiveness -->
+                    <div style="margin-bottom: 10px;">
+                        <a id="grass_snakes_pack" download class="button button--primary button--icon" href="https://3289537671-files.gitbook.io/~/files/v0/b/gitbook-x-prod.appspot.com/o/spaces%2FS5kw31UTGL8CPHU9skBS%2Fuploads%2FKsC7tdyT6htj5JX1ig28%2FGrass%20Snakes%20Club.zip?alt=media&token=1f6e6f0e-1a10-4bef-a8f8-a20b0a59e4c0">
+                            Lvl 1: Grass Snakes<span class="iconify" data-icon="mdi:tray-arrow-down"></span>
+                        </a>
+                    </div>
+                    <div style="margin-bottom: 10px;">
+                        <a id="tiger_snakes_pack" download type="submit" class="button button--primary button--icon" href="https://3289537671-files.gitbook.io/~/files/v0/b/gitbook-x-prod.appspot.com/o/spaces%2FS5kw31UTGL8CPHU9skBS%2Fuploads%2FSIeSsvz348nXjNMcVPxx%2FTiger%20Snakes%20Club.zip?alt=media&token=872b383f-a209-4864-a0e0-26d7ba3a8d74">
+                            Lvl 2: Tiger Snakes<span class="iconify" data-icon="mdi:tray-arrow-down"></span>
+                        </a>
+                    </div>
+                    <div>
+                        <a id="cobra_snakes_pack" download type="submit" class="button button--primary button--icon" href="https://3289537671-files.gitbook.io/~/files/v0/b/gitbook-x-prod.appspot.com/o/spaces%2FS5kw31UTGL8CPHU9skBS%2Fuploads%2FITTbTC8DVwr8LdCkN82d%2FCobra%20Club.zip?alt=media&token=bfd94842-cba9-45c7-892b-331bc82c33e9">
+                            Lvl 3: Cobra Snakes<span class="iconify" data-icon="mdi:tray-arrow-down"></span>
+                        </a>
+                    </div>
                 </div>
             </div>
         </div>

portal/templates/portal/teach/invited.html

@@ -20,7 +20,7 @@
                 </p>
             </div>
 
-            <form class="d-flex flex-column" method="post" id="form-invited-teacher" autocomplete="off">
+            <form class="d-flex flex-column" method="post" id="teacher-register-form" autocomplete="off">
 
                 {% csrf_token %}
 
@@ -29,7 +29,9 @@
                 <div class="row form--row">
                     <label for="{{ invited_teacher_form.teacher_password.auto_id }}"></label>
                     <div class="input--icon">
-                        {{ invited_teacher_form.teacher_password }}<span class="iconify" data-icon="mdi:security"></span>
+                        {{ invited_teacher_form.teacher_password }}
+                        <span id="teacher-password-field-icon" class="iconify"
+                            data-icon="material-symbols:visibility-off" ></span>
                     </div>
                     <small>{{ invited_teacher_form.teacher_password.help_text }}</small>
                     {{ invited_teacher_form.teacher_password.errors }}
@@ -37,7 +39,9 @@
                 <div class="row form--row">
                     <label for="{{ invited_teacher_form.teacher_confirm_password.auto_id }}"></label>
                     <div class="input--icon">
-                        {{ invited_teacher_form.teacher_confirm_password }}<span class="iconify" data-icon="mdi:security"></span>
+                        {{ invited_teacher_form.teacher_confirm_password }}
+                        <span id="teacher-confirm-password-field-icon" class="iconify"
+                            data-icon="material-symbols:visibility-off" ></span>
                     </div>
                     <small>{{ invited_teacher_form.teacher_confirm_password.help_text }}</small>
                     {{ invited_teacher_form.teacher_confirm_password.errors }}
@@ -73,10 +77,14 @@
     </div>
 </div>
 
-<script type="text/javascript" src="{% static 'portal/js/passwordStrength.js' %}"></script>
+{% include "portal/partials/service_unavailable_popup.html" %}
+
 <script>
-    var TEACHER_PASSWORD_FIELD_ID = '{{ invited_teacher_form.teacher_password.auto_id }}';
-    var INDEP_STUDENT_PASSWORD_FIELD_ID = '{{ invited_teacher_form.teacher_password.auto_id }}';
+    const password_id = '{{ invited_teacher_form.teacher_password.auto_id }}';
+    var TEACHER_PASSWORD_FIELD_ID = password_id;
+    var INDEP_STUDENT_PASSWORD_FIELD_ID = password_id;
 </script>
 
-{% endblock content %}
+<script type="text/javascript" src="{% static 'portal/js/passwordStrength.js' %}"></script>
+
+{% endblock content %}

portal/tests/test_invite_teacher.py

@@ -63,9 +63,9 @@ class TestInviteTeacher(TestCase):
         response = client.post(
             invitation_url,
             {
-                "teacher_password": invited_teacher_password,
-                "teacher_confirm_password": invited_teacher_password,
-                "consent_ticked": "on",
+                "teacher_signup-teacher_password": invited_teacher_password,
+                "teacher_signup-teacher_confirm_password": invited_teacher_password,
+                "teacher_signup-consent_ticked": "on",
             },
         )
 

portal/views/home.py

@@ -33,15 +33,9 @@ from portal.helpers.ratelimit import (
 )
 from portal.strings.coding_club import CODING_CLUB_BANNER
 from portal.strings.home_learning import HOME_LEARNING_BANNER
-from portal.strings.ten_year_map import (
-    TEN_YEAR_MAP_BANNER,
-    TEN_YEAR_MAP_HEADLINE,
-)
+from portal.strings.ten_year_map import TEN_YEAR_MAP_BANNER, TEN_YEAR_MAP_HEADLINE
 from portal.templatetags.app_tags import cloud_storage
-from portal.views.teacher.teach import (
-    DownloadType,
-    count_student_pack_downloads_click,
-)
+from portal.views.teacher.teach import DownloadType, count_student_pack_downloads_click
 
 LOGGER = logging.getLogger(__name__)
 
@@ -71,15 +65,11 @@ def render_signup_form(request):
     invalid_form = False
 
     teacher_signup_form = TeacherSignupForm(prefix="teacher_signup")
-    independent_student_signup_form = IndependentStudentSignupForm(
-        prefix="independent_student_signup"
-    )
+    independent_student_signup_form = IndependentStudentSignupForm(prefix="independent_student_signup")
 
     if request.method == "POST":
         if "teacher_signup-teacher_email" in request.POST:
-            teacher_signup_form = TeacherSignupForm(
-                request.POST, prefix="teacher_signup"
-            )
+            teacher_signup_form = TeacherSignupForm(request.POST, prefix="teacher_signup")
 
             if not captcha.CAPTCHA_ENABLED:
                 remove_captcha_from_forms(teacher_signup_form)
@@ -133,15 +123,11 @@ def process_signup_form(request, data):
                 [email],
                 personalization_values={
                     "EMAIL": email,
-                    "LOGIN_URL": request.build_absolute_uri(
-                        reverse("teacher_login")
-                    ),
+                    "LOGIN_URL": request.build_absolute_uri(reverse("teacher_login")),
                 },
             )
         else:
-            LOGGER.warn(
-                f"Ratelimit teacher {RATELIMIT_USER_ALREADY_REGISTERED_EMAIL_GROUP}: {email}"
-            )
+            LOGGER.warn(f"Ratelimit teacher {RATELIMIT_USER_ALREADY_REGISTERED_EMAIL_GROUP}: {email}")
     else:
         teacher = Teacher.objects.factory(
             first_name=data["teacher_first_name"],
@@ -152,9 +138,7 @@ def process_signup_form(request, data):
 
         send_verification_email(request, teacher.user.user, data)
 
-        TotalActivity.objects.update(
-            teacher_registrations=F("teacher_registrations") + 1
-        )
+        TotalActivity.objects.update(teacher_registrations=F("teacher_registrations") + 1)
 
     return render(
         request,
@@ -182,15 +166,11 @@ def process_independent_student_signup_form(request, data):
                 [email],
                 personalization_values={
                     "EMAIL": email,
-                    "LOGIN_URL": request.build_absolute_uri(
-                        reverse("independent_student_login")
-                    ),
+                    "LOGIN_URL": request.build_absolute_uri(reverse("independent_student_login")),
                 },
             )
         else:
-            LOGGER.warning(
-                f"Ratelimit independent {RATELIMIT_USER_ALREADY_REGISTERED_EMAIL_GROUP}: {email}"
-            )
+            LOGGER.warning(f"Ratelimit independent {RATELIMIT_USER_ALREADY_REGISTERED_EMAIL_GROUP}: {email}")
         return render(
             request,
             "portal/email_verification_needed.html",
@@ -208,9 +188,7 @@ def process_independent_student_signup_form(request, data):
 
     send_verification_email(request, student.new_user, data, age=age)
 
-    TotalActivity.objects.update(
-        independent_registrations=F("independent_registrations") + 1
-    )
+    TotalActivity.objects.update(independent_registrations=F("independent_registrations") + 1)
 
     return render(
         request,
@@ -221,10 +199,7 @@ def process_independent_student_signup_form(request, data):
 
 
 def is_developer(request):
-    return (
-        hasattr(request.user, "userprofile")
-        and request.user.userprofile.developer
-    )
+    return hasattr(request.user, "userprofile") and request.user.userprofile.developer
 
 
 def redirect_teacher_to_correct_page(request, teacher):
@@ -254,14 +229,10 @@ def home(request):
     # tests where the first Selenium test passes, but any following test
     # fails because it cannot find the Maintenance banner instance.
     try:
-        maintenance_banner = DynamicElement.objects.get(
-            name="Maintenance banner"
-        )
+        maintenance_banner = DynamicElement.objects.get(name="Maintenance banner")
 
         if maintenance_banner.active:
-            messages.info(
-                request, format_html(maintenance_banner.text), extra_tags="safe"
-            )
+            messages.info(request, format_html(maintenance_banner.text), extra_tags="safe")
     except ObjectDoesNotExist:
         pass
 
@@ -279,19 +250,13 @@ def home(request):
 
 
 def coding_club(request):
-    return render(
-        request, "portal/coding_club.html", {"BANNER": CODING_CLUB_BANNER}
-    )
+    return render(request, "portal/coding_club.html", {"BANNER": CODING_CLUB_BANNER})
 
 
 def download_student_pack(request, student_pack_type):
     if request.method == "POST":
         count_student_pack_downloads_click(int(student_pack_type))
-        link = (
-            cloud_storage("club_packs/PythonCodingClub.zip")
-            if DownloadType(int(student_pack_type)) == DownloadType.PYTHON_PACK
-            else cloud_storage("club_packs/PrimaryCodingClub.zip")
-        )
+        link = cloud_storage("club_packs/PrimaryCodingClub.zip")
         return redirect(link)
 
 
@@ -304,9 +269,7 @@ def home_learning(request):
 
 
 def ten_year_map_page(request):
-    messages.info(
-        request, "This page is currently under construction.", extra_tags="safe"
-    )
+    messages.info(request, "This page is currently under construction.", extra_tags="safe")
     return render(
         request,
         "portal/ten_year_map.html",

portal/views/teacher/teach.py

@@ -1,11 +1,11 @@
 import csv
 import json
-import pytz
 from datetime import datetime, timedelta
 from enum import Enum
 from functools import partial, wraps
 from uuid import uuid4
 
+import pytz
 from common.helpers.emails import send_verification_email
 from common.helpers.generators import (
     generate_access_code,
@@ -34,12 +34,10 @@ from django.urls import reverse, reverse_lazy
 from django.utils import timezone
 from django.views.decorators.http import require_POST
 from game.views.level_selection import get_blockly_episodes, get_python_episodes
-from portal.views.registration import handle_reset_password_tracking
 from reportlab.lib.colors import black, red
 from reportlab.lib.pagesizes import A4
 from reportlab.lib.utils import ImageReader
 from reportlab.pdfgen import canvas
-from portal.helpers.ratelimit import clear_ratelimit_cache_for_user
 
 from portal.forms.teach import (
     BaseTeacherDismissStudentsFormSet,
@@ -55,13 +53,13 @@ from portal.forms.teach import (
     TeacherMoveStudentsDestinationForm,
     TeacherSetStudentPass,
 )
+from portal.helpers.ratelimit import clear_ratelimit_cache_for_user
+from portal.views.registration import handle_reset_password_tracking
 
 STUDENT_PASSWORD_LENGTH = 6
 REMINDER_CARDS_PDF_ROWS = 8
 REMINDER_CARDS_PDF_COLUMNS = 1
-REMINDER_CARDS_PDF_WARNING_TEXT = (
-    "Please ensure students keep login details in a secure place"
-)
+REMINDER_CARDS_PDF_WARNING_TEXT = "Please ensure students keep login details in a secure place"
 
 
 @login_required(login_url=reverse_lazy("teacher_login"))
@@ -71,9 +69,7 @@ def teacher_onboarding_create_class(request):
     Onboarding view for creating a class (and organisation if there isn't one, yet)
     """
     teacher = request.user.new_teacher
-    requests = Student.objects.filter(
-        pending_class_request__teacher=teacher, new_user__is_active=True
-    )
+    requests = Student.objects.filter(pending_class_request__teacher=teacher, new_user__is_active=True)
 
     if not teacher.school:
         return HttpResponseRedirect(reverse_lazy("onboarding-organisation"))
@@ -84,9 +80,7 @@ def teacher_onboarding_create_class(request):
             created_class = create_class(form, teacher)
             messages.success(
                 request,
-                "The class '{className}' has been created successfully.".format(
-                    className=created_class.name
-                ),
+                "The class '{className}' has been created successfully.".format(className=created_class.name),
             )
             return HttpResponseRedirect(
                 reverse_lazy(
@@ -133,9 +127,7 @@ def process_edit_class(request, access_code, onboarding_done, next_url):
     """
     klass = get_object_or_404(Class, access_code=access_code)
     teacher = request.user.new_teacher
-    students = Student.objects.filter(
-        class_field=klass, new_user__is_active=True
-    ).order_by("new_user__first_name")
+    students = Student.objects.filter(class_field=klass, new_user__is_active=True).order_by("new_user__first_name")
 
     check_teacher_authorised(request, klass.teacher)
 
@@ -156,9 +148,7 @@ def process_edit_class(request, access_code, onboarding_done, next_url):
                     login_id=hashed_login_id,
                 )
 
-                TotalActivity.objects.update(
-                    student_registrations=F("student_registrations") + 1
-                )
+                TotalActivity.objects.update(student_registrations=F("student_registrations") + 1)
 
                 login_url = generate_student_url(request, new_student, login_id)
                 students_info.append(
@@ -241,16 +231,12 @@ def teacher_delete_class(request, access_code):
     # check user authorised to see class
     check_teacher_authorised(request, klass.teacher)
 
-    if Student.objects.filter(
-        class_field=klass, new_user__is_active=True
-    ).exists():
+    if Student.objects.filter(class_field=klass, new_user__is_active=True).exists():
         messages.info(
             request,
             "This class still has students, please remove or delete them all before deleting the class.",
         )
-        return HttpResponseRedirect(
-            reverse_lazy("view_class", kwargs={"access_code": access_code})
-        )
+        return HttpResponseRedirect(reverse_lazy("view_class", kwargs={"access_code": access_code}))
 
     klass.anonymise()
 
@@ -267,9 +253,7 @@ def teacher_delete_students(request, access_code):
 
     # get student objects for students to be deleted, confirming they are in the class
     student_ids = json.loads(request.POST.get("transfer_students", "[]"))
-    students = [
-        get_object_or_404(Student, id=i, class_field=klass) for i in student_ids
-    ]
+    students = [get_object_or_404(Student, id=i, class_field=klass) for i in student_ids]
 
     def __anonymise(user):
         # Delete all personal data from inactive user and mark as inactive.
@@ -291,9 +275,7 @@ def teacher_delete_students(request, access_code):
         else:  # otherwise, just delete
             student.new_user.delete()
 
-    return HttpResponseRedirect(
-        reverse_lazy("view_class", kwargs={"access_code": access_code})
-    )
+    return HttpResponseRedirect(reverse_lazy("view_class", kwargs={"access_code": access_code}))
 
 
 @login_required(login_url=reverse_lazy("teacher_login"))
@@ -307,9 +289,7 @@ def teacher_edit_class(request, access_code):
     """
     klass = get_object_or_404(Class, access_code=access_code)
     old_teacher = klass.teacher
-    other_teachers = Teacher.objects.filter(school=old_teacher.school).exclude(
-        user=old_teacher.user
-    )
+    other_teachers = Teacher.objects.filter(school=old_teacher.school).exclude(user=old_teacher.user)
 
     # check user authorised to see class
     check_teacher_authorised(request, klass.teacher)
@@ -339,9 +319,7 @@ def teacher_edit_class(request, access_code):
         elif "level_control_submit" in request.POST:
             level_control_form = ClassLevelControlForm(request.POST)
             if level_control_form.is_valid():
-                return process_level_control_form(
-                    request, klass, blockly_episodes, python_episodes
-                )
+                return process_level_control_form(request, klass, blockly_episodes, python_episodes)
         elif "class_move_submit" in request.POST:
             class_move_form = ClassMoveForm(other_teachers, request.POST)
             if class_move_form.is_valid():
@@ -383,9 +361,7 @@ def process_edit_class_form(request, klass, form):
         elif hours < 1000:
             # Setting to number of hours
             klass.always_accept_requests = False
-            klass.accept_requests_until = timezone.now() + timedelta(
-                hours=hours
-            )
+            klass.accept_requests_until = timezone.now() + timedelta(hours=hours)
             messages.info(
                 request,
                 "Class set successfully to receive requests from external students until "
@@ -407,18 +383,12 @@ def process_edit_class_form(request, klass, form):
     klass.classmates_data_viewable = classmate_progress
     klass.save()
 
-    messages.success(
-        request, "The class's settings have been changed successfully."
-    )
+    messages.success(request, "The class's settings have been changed successfully.")
 
-    return HttpResponseRedirect(
-        reverse_lazy("view_class", kwargs={"access_code": klass.access_code})
-    )
+    return HttpResponseRedirect(reverse_lazy("view_class", kwargs={"access_code": klass.access_code}))
 
 
-def process_level_control_form(
-    request, klass, blockly_episodes, python_episodes
-):
+def process_level_control_form(request, klass, blockly_episodes, python_episodes):
     """
     Find the levels that the user wants to lock and lock them for the specific class.
     :param request: The request sent by the user submitting the form.
@@ -429,23 +399,14 @@ def process_level_control_form(
     """
     levels_to_lock_ids = []
 
-    mark_levels_to_lock_in_episodes(
-        request, blockly_episodes, levels_to_lock_ids
-    )
-    mark_levels_to_lock_in_episodes(
-        request, python_episodes, levels_to_lock_ids
-    )
+    mark_levels_to_lock_in_episodes(request, blockly_episodes, levels_to_lock_ids)
+    mark_levels_to_lock_in_episodes(request, python_episodes, levels_to_lock_ids)
 
     klass.locked_levels.clear()
-    [
-        klass.locked_levels.add(levels_to_lock_id)
-        for levels_to_lock_id in levels_to_lock_ids
-    ]
+    [klass.locked_levels.add(levels_to_lock_id) for levels_to_lock_id in levels_to_lock_ids]
 
     messages.success(request, "Your level preferences have been saved.")
-    activity_today = DailyActivity.objects.get_or_create(
-        date=datetime.now().date()
-    )[0]
+    activity_today = DailyActivity.objects.get_or_create(date=datetime.now().date())[0]
     activity_today.level_control_submits += 1
     activity_today.save()
 
@@ -468,14 +429,10 @@ def mark_levels_to_lock_in_episodes(request, episodes, levels_to_lock_ids):
             [
                 levels_to_lock_ids.append(episode_level["id"])
                 for episode_level in episode_levels
-                if str(episode_level["id"])
-                not in request.POST.getlist(episode_name)
+                if str(episode_level["id"]) not in request.POST.getlist(episode_name)
             ]
         else:
-            [
-                levels_to_lock_ids.append(episode_level["id"])
-                for episode_level in episode_levels
-            ]
+            [levels_to_lock_ids.append(episode_level["id"]) for episode_level in episode_levels]
 
 
 def process_move_class_form(request, klass, form):
@@ -501,9 +458,7 @@ def teacher_edit_student(request, pk):
     student = get_object_or_404(Student, id=pk)
     check_teacher_authorised(request, student.class_field.teacher)
 
-    name_form = TeacherEditStudentForm(
-        student, initial={"name": student.new_user.first_name}
-    )
+    name_form = TeacherEditStudentForm(student, initial={"name": student.new_user.first_name})
 
     password_form = TeacherSetStudentPass()
     set_password_mode = False
@@ -532,9 +487,7 @@ def teacher_edit_student(request, pk):
         else:
             password_form = TeacherSetStudentPass(request.POST)
             if password_form.is_valid():
-                return process_reset_password_form(
-                    request, student, password_form
-                )
+                return process_reset_password_form(request, student, password_form)
             set_password_mode = True
 
     return render(
@@ -577,9 +530,7 @@ def process_reset_password_form(request, student, password_form):
         student.new_user.set_password(new_password)
         student.new_user.save()
         student.login_id = login_id
-        clear_ratelimit_cache_for_user(
-            f"{student.new_user.first_name},{student.class_field.access_code}"
-        )
+        clear_ratelimit_cache_for_user(f"{student.new_user.first_name},{student.class_field.access_code}")
         student.blocked_time = datetime.now(tz=pytz.utc) - timedelta(days=1)
         student.save()
 
@@ -613,9 +564,7 @@ def teacher_dismiss_students(request, access_code):
 
     # get student objects for students to be dismissed, confirming they are in the class
     student_ids = json.loads(request.POST.get("transfer_students", "[]"))
-    students = [
-        get_object_or_404(Student, id=i, class_field=klass) for i in student_ids
-    ]
+    students = [get_object_or_404(Student, id=i, class_field=klass) for i in student_ids]
 
     TeacherDismissStudentsFormSet = formset_factory(
         wraps(TeacherDismissStudentsForm)(partial(TeacherDismissStudentsForm)),
@@ -626,9 +575,7 @@ def teacher_dismiss_students(request, access_code):
     if is_right_dismiss_form(request):
         formset = TeacherDismissStudentsFormSet(request.POST)
         if formset.is_valid():
-            return process_dismiss_student_form(
-                request, formset, klass, access_code
-            )
+            return process_dismiss_student_form(request, formset, klass, access_code)
 
     else:
         initial_data = [
@@ -679,14 +626,10 @@ def process_dismiss_student_form(request, formset, klass, access_code):
         student.user.save()
 
         # log the data
-        joinrelease = JoinReleaseStudent.objects.create(
-            student=student, action_type=JoinReleaseStudent.RELEASE
-        )
+        joinrelease = JoinReleaseStudent.objects.create(student=student, action_type=JoinReleaseStudent.RELEASE)
         joinrelease.save()
 
-        send_verification_email(
-            request, student.new_user, data, school=klass.teacher.school
-        )
+        send_verification_email(request, student.new_user, data, school=klass.teacher.school)
 
     if not failed_users:
         messages.success(
@@ -700,9 +643,7 @@ def process_dismiss_student_form(request, formset, klass, access_code):
             "Please make sure the email has not been registered to another account.",
         )
 
-    return HttpResponseRedirect(
-        reverse_lazy("view_class", kwargs={"access_code": access_code})
-    )
+    return HttpResponseRedirect(reverse_lazy("view_class", kwargs={"access_code": access_code}))
 
 
 @login_required(login_url=reverse_lazy("teacher_login"))
@@ -717,9 +658,7 @@ def teacher_class_password_reset(request, access_code):
     check_teacher_authorised(request, klass.teacher)
 
     student_ids = json.loads(request.POST.get("transfer_students", "[]"))
-    students = [
-        get_object_or_404(Student, id=i, class_field=klass) for i in student_ids
-    ]
+    students = [get_object_or_404(Student, id=i, class_field=klass) for i in student_ids]
 
     students_info = []
     handle_reset_password_tracking(request, "SCHOOL_STUDENT", access_code)
@@ -741,9 +680,7 @@ def teacher_class_password_reset(request, access_code):
         student.new_user.set_password(password)
         student.new_user.save()
         student.login_id = hashed_login_id
-        clear_ratelimit_cache_for_user(
-            f"{student.new_user.first_name},{access_code}"
-        )
+        clear_ratelimit_cache_for_user(f"{student.new_user.first_name},{access_code}")
         student.blocked_time = datetime.now(tz=pytz.utc) - timedelta(days=1)
         student.save()
 
@@ -757,9 +694,7 @@ def teacher_class_password_reset(request, access_code):
             "students_info": students_info,
             "query_data": json.dumps(students_info),
             "class_url": request.build_absolute_uri(
-                reverse(
-                    "student_login", kwargs={"access_code": klass.access_code}
-                )
+                reverse("student_login", kwargs={"access_code": klass.access_code})
             ),
         },
     )
@@ -809,37 +744,26 @@ def teacher_move_students_to_class(request, access_code):
 
     check_if_move_authorised(request, old_class, new_class)
 
-    transfer_students_ids = json.loads(
-        request.POST.get("transfer_students", "[]")
-    )
+    transfer_students_ids = json.loads(request.POST.get("transfer_students", "[]"))
 
     # get student objects for students to be transferred, confirming they are in the old class still
-    transfer_students = [
-        get_object_or_404(Student, id=i, class_field=old_class)
-        for i in transfer_students_ids
-    ]
+    transfer_students = [get_object_or_404(Student, id=i, class_field=old_class) for i in transfer_students_ids]
 
     # get new class' students
-    new_class_students = Student.objects.filter(
-        class_field=new_class, new_user__is_active=True
-    ).order_by("new_user__first_name")
+    new_class_students = Student.objects.filter(class_field=new_class, new_user__is_active=True).order_by(
+        "new_user__first_name"
+    )
 
     TeacherMoveStudentDisambiguationFormSet = formset_factory(
-        wraps(TeacherMoveStudentDisambiguationForm)(
-            partial(TeacherMoveStudentDisambiguationForm)
-        ),
+        wraps(TeacherMoveStudentDisambiguationForm)(partial(TeacherMoveStudentDisambiguationForm)),
         extra=0,
         formset=BaseTeacherMoveStudentsDisambiguationFormSet,
     )
 
     if is_right_move_form(request):
-        formset = TeacherMoveStudentDisambiguationFormSet(
-            new_class, request.POST
-        )
+        formset = TeacherMoveStudentDisambiguationFormSet(new_class, request.POST)
         if formset.is_valid():
-            return process_move_students_form(
-                request, formset, old_class, new_class
-            )
+            return process_move_students_form(request, formset, old_class, new_class)
     else:
         # format the students for the form
         initial_data = [
@@ -850,9 +774,7 @@ def teacher_move_students_to_class(request, access_code):
             for student in transfer_students
         ]
 
-        formset = TeacherMoveStudentDisambiguationFormSet(
-            new_class, initial=initial_data
-        )
+        formset = TeacherMoveStudentDisambiguationFormSet(new_class, initial=initial_data)
 
     return render(
         request,
@@ -872,9 +794,7 @@ def check_if_move_authorised(request, old_class, new_class):
 
     # check teacher has permission to edit old_class and that both classes
     # are in the same school
-    if (
-        not teacher.is_admin and teacher != old_class.teacher
-    ) or teacher.school != new_class.teacher.school:
+    if (not teacher.is_admin and teacher != old_class.teacher) or teacher.school != new_class.teacher.school:
         raise Http404
 
 
@@ -898,14 +818,8 @@ def process_move_students_form(request, formset, old_class, new_class):
         student.save()
         student.new_user.save()
 
-    messages.success(
-        request, "The students have been transferred successfully."
-    )
-    return HttpResponseRedirect(
-        reverse_lazy(
-            "view_class", kwargs={"access_code": old_class.access_code}
-        )
-    )
+    messages.success(request, "The students have been transferred successfully.")
+    return HttpResponseRedirect(reverse_lazy("view_class", kwargs={"access_code": old_class.access_code}))
 
 
 class DownloadType(Enum):
@@ -938,9 +852,7 @@ def teacher_print_reminder_cards(request, access_code):
 
     CARD_INNER_HEIGHT = CARD_HEIGHT - CARD_PADDING * 2
 
-    logo_image = ImageReader(
-        staticfiles_storage.path("portal/img/logo_cfl_reminder_cards.jpg")
-    )
+    logo_image = ImageReader(staticfiles_storage.path("portal/img/logo_cfl_reminder_cards.jpg"))
 
     klass = get_object_or_404(Class, access_code=access_code)
     # Check auth
@@ -948,12 +860,8 @@ def teacher_print_reminder_cards(request, access_code):
 
     # Use data from the query string if given
     student_data = get_student_data(request)
-    student_login_link = request.build_absolute_uri(
-        reverse("student_login_access_code")
-    )
-    class_login_link = request.build_absolute_uri(
-        reverse("student_login", kwargs={"access_code": access_code})
-    )
+    student_login_link = request.build_absolute_uri(reverse("student_login_access_code"))
+    class_login_link = request.build_absolute_uri(reverse("student_login", kwargs={"access_code": access_code}))
 
     # Now draw everything
     x = 0
@@ -965,17 +873,10 @@ def teacher_print_reminder_cards(request, access_code):
         if current_student_count % (NUM_X * NUM_Y) == 0:
             p.setFillColor(red)
             p.setFont("Helvetica-Bold", 10)
-            p.drawString(
-                PAGE_MARGIN, PAGE_MARGIN / 2, REMINDER_CARDS_PDF_WARNING_TEXT
-            )
+            p.drawString(PAGE_MARGIN, PAGE_MARGIN / 2, REMINDER_CARDS_PDF_WARNING_TEXT)
 
         left = PAGE_MARGIN + x * CARD_WIDTH + x * INTER_CARD_MARGIN * 2
-        bottom = (
-            PAGE_HEIGHT
-            - PAGE_MARGIN
-            - (y + 1) * CARD_HEIGHT
-            - y * INTER_CARD_MARGIN
-        )
+        bottom = PAGE_HEIGHT - PAGE_MARGIN - (y + 1) * CARD_HEIGHT - y * INTER_CARD_MARGIN
 
         inner_bottom = bottom + CARD_PADDING
 
@@ -995,12 +896,7 @@ def teacher_print_reminder_cards(request, access_code):
             anchor="w",
         )
 
-        text_left = (
-            left
-            + INTER_CARD_MARGIN
-            + (logo_image.getSize()[0] / logo_image.getSize()[1])
-            * card_logo_height
-        )
+        text_left = left + INTER_CARD_MARGIN + (logo_image.getSize()[0] / logo_image.getSize()[1]) * card_logo_height
 
         # student details
         p.setFillColor(black)
@@ -1023,9 +919,7 @@ def teacher_print_reminder_cards(request, access_code):
             inner_bottom + CARD_INNER_HEIGHT * 0.3,
             f"Name: {student['name']}",
         )
-        p.drawString(
-            text_left, inner_bottom, f"Password: {student['password']}"
-        )
+        p.drawString(text_left, inner_bottom, f"Password: {student['password']}")
 
         x = (x + 1) % NUM_X
         y = compute_show_page_character(p, x, y, NUM_Y)
@@ -1044,17 +938,13 @@ def teacher_print_reminder_cards(request, access_code):
 @user_passes_test(logged_in_as_teacher, login_url=reverse_lazy("teacher_login"))
 def teacher_download_csv(request, access_code):
     response = HttpResponse(content_type="text/csv")
-    response[
-        "Content-Disposition"
-    ] = 'attachment; filename="student_login_urls.csv"'
+    response["Content-Disposition"] = 'attachment; filename="student_login_urls.csv"'
 
     klass = get_object_or_404(Class, access_code=access_code)
     # Check auth
     check_teacher_authorised(request, klass.teacher)
 
-    class_url = request.build_absolute_uri(
-        reverse("student_login", kwargs={"access_code": access_code})
-    )
+    class_url = request.build_absolute_uri(reverse("student_login", kwargs={"access_code": access_code}))
 
     # Use data from the query string if given
     student_data = get_student_data(request)
@@ -1062,9 +952,7 @@ def teacher_download_csv(request, access_code):
         writer = csv.writer(response)
         writer.writerow([access_code, class_url])
         for student in student_data:
-            writer.writerow(
-                [student["name"], student["password"], student["login_url"]]
-            )
+            writer.writerow([student["name"], student["password"], student["login_url"]])
 
     count_student_details_click(DownloadType.CSV)
 
@@ -1092,22 +980,16 @@ def compute_show_page_end(p, x, y):
 
 
 def count_student_pack_downloads_click(student_pack_type):
-    activity_today = DailyActivity.objects.get_or_create(
-        date=datetime.now().date()
-    )[0]
+    activity_today = DailyActivity.objects.get_or_create(date=datetime.now().date())[0]
     if DownloadType(student_pack_type) == DownloadType.PRIMARY_PACK:
         activity_today.primary_coding_club_downloads += 1
-    elif DownloadType(student_pack_type) == DownloadType.PYTHON_PACK:
-        activity_today.python_coding_club_downloads += 1
     else:
         raise Exception("Unknown download type")
     activity_today.save()
 
 
 def count_student_details_click(download_type):
-    activity_today = DailyActivity.objects.get_or_create(
-        date=datetime.now().date()
-    )[0]
+    activity_today = DailyActivity.objects.get_or_create(date=datetime.now().date())[0]
 
     if download_type == DownloadType.CSV:
         activity_today.csv_click_count += 1

portal/tests/test_daily_activities.py

@@ -1,41 +0,0 @@
-from datetime import timedelta, datetime
-
-from common.models import DailyActivity
-from selenium.webdriver.common.by import By
-from selenium.webdriver.support import expected_conditions as EC
-from selenium.webdriver.support.ui import WebDriverWait
-
-from portal.tests.base_test import BaseTest
-
-
-class TestDailyActivities(BaseTest):
-    def test_coding_club_increment(self):
-
-        # first create dailyActivity one day before datetime.now()
-        # to check if it can handle incrementing on different days
-        # then check if increments are done on the same day
-        old_date = datetime.now() - timedelta(days=1)
-        old_daily_activity = DailyActivity(date=old_date)
-        old_daily_activity.save()
-
-        for i in range(4):
-            # check both buttons
-            self.go_to_homepage()
-            button_id = "primary_pack" if i < 2 else "python_pack"
-            find_out_more_button = WebDriverWait(self.selenium, 10).until(
-                EC.element_to_be_clickable((By.ID, "find_out_more"))
-            )
-            find_out_more_button.click()
-
-            daily_count_button = WebDriverWait(self.selenium, 10).until(
-                EC.visibility_of_element_located((By.ID, button_id))
-            )
-            daily_count_button.click()
-        # check the old_date is still the same
-        old_daily_activity = DailyActivity.objects.get(date=old_date)
-        assert old_daily_activity.primary_coding_club_downloads == 0
-        assert old_daily_activity.python_coding_club_downloads == 0
-        # check the current_date is incremented to 2
-        current_daily_activity = DailyActivity.objects.get(date=datetime.now())
-        assert current_daily_activity.primary_coding_club_downloads == 2
-        assert current_daily_activity.python_coding_club_downloads == 2