all: use test.sh and fix.sh to encapsulate tests and formating/linting fix

Pinheiro-Reis · Pinheiro-Reis · commit 464647444d47 · 2025-11-12T14:47:59.000-03:00
diff --git a/api/School-Secretary/settings.py b/api/School-Secretary/settings.py
@@ -1,5 +1,4 @@
 # Permitir mais campos em POST para deleção em massa no admin
-DATA_UPLOAD_MAX_NUMBER_FIELDS = 10000
 """
 Django settings for School-Secretary project.
 
@@ -16,6 +15,8 @@
 from datetime import timedelta
 from pathlib import Path
 
+DATA_UPLOAD_MAX_NUMBER_FIELDS = 10000
+
 # Build paths inside the project like this: BASE_DIR / 'subdir'.
 BASE_DIR = Path(__file__).resolve().parent.parent
 
diff --git a/api/fix.sh b/api/fix.sh
@@ -0,0 +1,13 @@
+#!/bin/bash
+
+set -e
+
+SCRIPT_DIR="$(dirname "$(realpath "$0")")"
+
+cd $SCRIPT_DIR
+
+echo "--- Executing API code fix ---"
+uv run ruff format .
+
+echo "--- Executing API tests ---"
+uv run ruff check --fix .
diff --git a/api/manage.py b/api/manage.py
@@ -16,7 +16,7 @@
 
 def main():
     """Run administrative tasks."""
-    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'School-Secretary.settings')
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "School-Secretary.settings")
     try:
         from django.core.management import execute_from_command_line
     except ImportError as exc:
@@ -28,5 +28,5 @@ def main():
     execute_from_command_line(sys.argv)
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()
diff --git a/api/pyproject.toml b/api/pyproject.toml
@@ -62,3 +62,8 @@ dependencies = [
     "xhtml2pdf==0.2.17",
     "zopfli==0.2.3.post1",
 ]
+
+[dependency-groups]
+dev = [
+    "ruff>=0.14.4",
+]
diff --git a/api/resources/tests/test_models.py b/api/resources/tests/test_models.py
@@ -6,7 +6,6 @@
 
 
 class ResourceModelsTest(TestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user_student = self.User.objects.create_user(
diff --git a/api/resources/tests/test_serializers.py b/api/resources/tests/test_serializers.py
@@ -7,7 +7,6 @@
 
 
 class ResourceSerializersTest(APITestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user_student = self.User.objects.create_user(
@@ -33,8 +32,8 @@ def setUp(self):
     def test_resource_serializer(self):
         serializer = ResourceSerializer(self.resource)
         data = serializer.data
-        self.assertEqual(data['name'], 'Test Resource')
-        self.assertEqual(data['resource_type'], 'COMPUTER')
+        self.assertEqual(data["name"], "Test Resource")
+        self.assertEqual(data["resource_type"], "COMPUTER")
 
     def test_resource_loan_serializer(self):
         loan = ResourceLoan.objects.create(
@@ -45,7 +44,7 @@ def test_resource_loan_serializer(self):
         )
         serializer = ResourceLoanSerializer(loan)
         data = serializer.data
-        self.assertEqual(data['resource'], self.resource.id)
-        self.assertEqual(data['student'], self.student.id)
-        self.assertIn('resource_details', data)
-        self.assertIn('student_name', data)
+        self.assertEqual(data["resource"], self.resource.id)
+        self.assertEqual(data["student"], self.student.id)
+        self.assertIn("resource_details", data)
+        self.assertIn("student_name", data)
diff --git a/api/school/management/commands/monitoring_log.py b/api/school/management/commands/monitoring_log.py
@@ -3,13 +3,14 @@
 from datetime import datetime
 from django.core.management.base import BaseCommand
 
+
 class Command(BaseCommand):
-    help = 'Loga periodicamente a hora atual com a mensagem monitorando...'
+    help = "Loga periodicamente a hora atual com a mensagem monitorando..."
 
     def handle(self, *args, **options):
-        self.stdout.write(self.style.SUCCESS('Monitoramento iniciado.'))
+        self.stdout.write(self.style.SUCCESS("Monitoramento iniciado."))
         while True:
-            timestamp = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
+            timestamp = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
             msg = f"[{timestamp}] monitorando..."
             print(msg)
             logging.info(msg)
diff --git a/api/school/tests/test_models.py b/api/school/tests/test_models.py
@@ -17,7 +17,6 @@
 
 
 class SchoolModelsTest(TestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user = self.User.objects.create_user(
diff --git a/api/school/tests/test_serializers.py b/api/school/tests/test_serializers.py
@@ -1,5 +1,4 @@
 from rest_framework.test import APITestCase
-from rest_framework import serializers
 from school.serializers import (
     SubjectCompactSerializer,
     ItineraryCompactSerializer,
@@ -35,7 +34,6 @@
 
 
 class SchoolSerializersTest(APITestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user = self.User.objects.create_user(
diff --git a/api/school/urls.py b/api/school/urls.py
@@ -1,4 +1,3 @@
-from django.urls import include, path
 from rest_framework.routers import DefaultRouter
 
 from .views import (
diff --git a/api/school/views.py b/api/school/views.py
@@ -7,8 +7,6 @@
 from users.permissions import IsProfessor, IsStaff
 from utils.date import get_day_name
 from utils.reports import (
-    calculate_approval_rate,
-    calculate_dropout_rate,
     generate_efficiency_analysis,
     generate_group_performance_report,
 )
diff --git a/api/students/management/commands/send_notifications.py b/api/students/management/commands/send_notifications.py
@@ -2,6 +2,7 @@
 Django management command to send periodic notifications
 Run this command daily via cron or scheduler to check and send notifications
 """
+
 from django.core.management.base import BaseCommand
 from utils.notifications import (
     check_and_send_absence_notifications,
@@ -12,62 +13,70 @@
 
 
 class Command(BaseCommand):
-    help = 'Check students and send notifications for absences, grades, payments, and events'
+    help = "Check students and send notifications for absences, grades, payments, and events"
 
     def add_arguments(self, parser):
         parser.add_argument(
-            '--absences',
-            action='store_true',
-            help='Check and send absence notifications',
+            "--absences",
+            action="store_true",
+            help="Check and send absence notifications",
         )
         parser.add_argument(
-            '--grades',
-            action='store_true',
-            help='Check and send low grade notifications',
+            "--grades",
+            action="store_true",
+            help="Check and send low grade notifications",
         )
         parser.add_argument(
-            '--payments',
-            action='store_true',
-            help='Check and send overdue payment notifications',
+            "--payments",
+            action="store_true",
+            help="Check and send overdue payment notifications",
         )
         parser.add_argument(
-            '--events',
-            action='store_true',
-            help='Send upcoming event notifications',
+            "--events",
+            action="store_true",
+            help="Send upcoming event notifications",
         )
         parser.add_argument(
-            '--all',
-            action='store_true',
-            help='Send all types of notifications',
+            "--all",
+            action="store_true",
+            help="Send all types of notifications",
         )
 
     def handle(self, *args, **options):
         notifications_sent = 0
 
-        if options['all'] or options['absences']:
-            self.stdout.write('Checking absence rates...')
+        if options["all"] or options["absences"]:
+            self.stdout.write("Checking absence rates...")
             notifs = check_and_send_absence_notifications()
             notifications_sent += len(notifs)
-            self.stdout.write(self.style.SUCCESS(f'  Sent {len(notifs)} absence notifications'))
+            self.stdout.write(
+                self.style.SUCCESS(f"  Sent {len(notifs)} absence notifications")
+            )
 
-        if options['all'] or options['grades']:
-            self.stdout.write('Checking grades...')
+        if options["all"] or options["grades"]:
+            self.stdout.write("Checking grades...")
             notifs = check_and_send_grade_notifications()
             notifications_sent += len(notifs)
-            self.stdout.write(self.style.SUCCESS(f'  Sent {len(notifs)} grade notifications'))
+            self.stdout.write(
+                self.style.SUCCESS(f"  Sent {len(notifs)} grade notifications")
+            )
 
-        if options['all'] or options['payments']:
-            self.stdout.write('Checking overdue payments...')
+        if options["all"] or options["payments"]:
+            self.stdout.write("Checking overdue payments...")
             notifs = check_and_send_overdue_payment_notifications()
             notifications_sent += len(notifs)
-            self.stdout.write(self.style.SUCCESS(f'  Sent {len(notifs)} payment notifications'))
+            self.stdout.write(
+                self.style.SUCCESS(f"  Sent {len(notifs)} payment notifications")
+            )
 
-        if options['all'] or options['events']:
-            self.stdout.write('Checking upcoming events...')
+        if options["all"] or options["events"]:
+            self.stdout.write("Checking upcoming events...")
             notifs = send_upcoming_event_notifications(days_ahead=7)
             notifications_sent += len(notifs)
-            self.stdout.write(self.style.SUCCESS(f'  Sent {len(notifs)} event notifications'))
+            self.stdout.write(
+                self.style.SUCCESS(f"  Sent {len(notifs)} event notifications")
+            )
 
         self.stdout.write(
-            self.style.SUCCESS(f'\nTotal notifications sent: {notifications_sent}')
+            self.style.SUCCESS(f"\nTotal notifications sent: {notifications_sent}")
         )
diff --git a/api/students/tests/test_models.py b/api/students/tests/test_models.py
@@ -6,7 +6,6 @@
 
 
 class StudentModelsTest(TestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user_student = self.User.objects.create_user(
diff --git a/api/students/tests/test_serializers.py b/api/students/tests/test_serializers.py
@@ -1,5 +1,4 @@
 from rest_framework.test import APITestCase
-from rest_framework import serializers
 from students.serializers import (
     StudentCompactSerializer,
     GuardianCompactSerializer,
@@ -15,11 +14,9 @@
 from school.models import Group, Subject
 from django.contrib.auth import get_user_model
 from django.utils import timezone
-import datetime
 
 
 class StudentSerializersTest(APITestCase):
-
     def setUp(self):
         self.User = get_user_model()
         self.user_student = self.User.objects.create_user(
diff --git a/api/students/views.py b/api/students/views.py
@@ -86,7 +86,9 @@ def download_grades_pdf(self, request, pk=None):
             # Initialize with None for each bimester
             bimester_grades = [None, None, None, None]
             for grade in all_grades.filter(subject__full_name=subject):
-                bimester_num = int(grade.bimester[0])  # Extract the numeric part (e.g., "1B" -> 1)
+                bimester_num = int(
+                    grade.bimester[0]
+                )  # Extract the numeric part (e.g., "1B" -> 1)
                 if 1 <= bimester_num <= 4:
                     bimester_grades[bimester_num - 1] = grade.value
             data[subject] = bimester_grades
@@ -102,7 +104,7 @@ def download_presence_pdf(self, request, pk=None):
         current_year = timezone.now().year
         presence_records = Presence.objects.filter(
             student=student, date__year=current_year
-        ).order_by('date')
+        ).order_by("date")
         return pdfgen(
             "presence.html",
             {"student": student, "data": presence_records},
@@ -128,7 +130,7 @@ def download_academic_report(self, request, pk=None):
             "grades": report["grades"],
             "attendance": report["attendance"],
             "discipline": report["discipline"],
-            "now": tz.now(),
+            "now": timezone.now(),
         }
 
         return pdfgen(
@@ -375,7 +377,7 @@ def download_financial_report(self, request):
             "student": student,
             "summary": report["summary"],
             "payment_history": report["payment_history"],
-            "now": tz.now(),
+            "now": timezone.now(),
         }
 
         filename = (
diff --git a/api/test.sh b/api/test.sh
@@ -0,0 +1,13 @@
+#!/bin/bash
+
+set -e
+
+SCRIPT_DIR="$(dirname "$(realpath "$0")")"
+
+cd $SCRIPT_DIR
+
+echo "--- Executing API code checking ---"
+uv run ruff check .
+
+echo "--- Executing API tests ---"
+uv run python manage.py test
diff --git a/api/users/forms.py b/api/users/forms.py
@@ -1,4 +1,3 @@
-from django import forms
 from django.contrib.auth.forms import UserChangeForm, UserCreationForm
 
 from .models import User
diff --git a/api/users/serializers.py b/api/users/serializers.py
@@ -1,9 +1,7 @@
 from django.contrib.auth import get_user_model
 from rest_framework import serializers
 
-from school.models import Professor
 from school.serializers import ProfessorSerializer
-from students.models import Guardian, Student
 from students.serializers import GuardianSerializer, StudentSerializer
 
 
@@ -40,7 +38,15 @@ class UserProfileSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = get_user_model()
-        fields = ["id", "email", "name", "role", "is_staff", "is_superuser", "profile_details"]
+        fields = [
+            "id",
+            "email",
+            "name",
+            "role",
+            "is_staff",
+            "is_superuser",
+            "profile_details",
+        ]
 
     def get_profile_details(self, obj):
         profile = obj.profile
diff --git a/api/users/signals.py b/api/users/signals.py
@@ -1,14 +1,6 @@
 import random
 import string
-
-
-def generate_random_password(length=12):
-    chars = string.ascii_letters + string.digits + "!@#$%^&*()_+-="
-    return "".join(random.choice(chars) for _ in range(length))
-
-
 from django.contrib.auth import get_user_model
-from django.contrib.auth.models import UserManager
 from django.db.models.signals import post_delete, post_save
 from django.dispatch import receiver
 
@@ -17,7 +9,10 @@ def generate_random_password(length=12):
 
 User = get_user_model()
 
-# Função utilitária para criar usuário
+
+def generate_random_password(length=12):
+    chars = string.ascii_letters + string.digits + "!@#$%^&*()_+-="
+    return "".join(random.choice(chars) for _ in range(length))
 
 
 def create_related_user(instance, role):
diff --git a/api/users/tests/test_models.py b/api/users/tests/test_models.py
diff --git a/api/users/tests/test_serializers.py b/api/users/tests/test_serializers.py
diff --git a/api/utils/notifications.py b/api/utils/notifications.py
diff --git a/api/utils/reports.py b/api/utils/reports.py
diff --git a/app/fix.sh b/app/fix.sh
diff --git a/app/package.json b/app/package.json
diff --git a/app/test.sh b/app/test.sh
diff --git a/controller.sh b/controller.sh
diff --git a/scripts/test.sh b/scripts/test.sh

Original file line number	Diff line number	Diff line change
`@@ -62,3 +62,8 @@ dependencies = [`
`62`	`62`	`"xhtml2pdf==0.2.17",`
`63`	`63`	`"zopfli==0.2.3.post1",`
`64`	`64`	`]`
	`65`	`+`
	`66`	`+[dependency-groups]`
	`67`	`+dev = [`
	`68`	`+ "ruff>=0.14.4",`
	`69`	`+]`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-from django.urls import include, path`
`2`	`1`	`from rest_framework.routers import DefaultRouter`
`3`	`2`
`4`	`3`	`from .views import (`