[WIP] refactoring/location-club-statistics #71
|
@ -0,0 +1,13 @@
|
|||
#!/bin/bash
|
||||
|
||||
.PHONY: coverage errors
|
||||
|
||||
coverage:
|
||||
cd src/; coverage run --source='.' manage.py test; coverage report -m; cd ..
|
||||
@echo "Testing of coverage in the sources finished."
|
||||
|
||||
lint:
|
||||
pylint src/* --disable=django-not-configured
|
||||
|
||||
errors:
|
||||
pylint src/* --errors-only --disable=django-not-configured
|
|
@ -0,0 +1,16 @@
|
|||
[run]
|
||||
branch = true
|
||||
omit =
|
||||
*/tests/*
|
||||
*/test*
|
||||
*/migrations/*
|
||||
*/urls.py
|
||||
*/admin.py
|
||||
*/settings/*
|
||||
*/wsgi.py
|
||||
*/__init__.py
|
||||
manage.py
|
||||
source = .
|
||||
|
||||
[report]
|
||||
show_missing = true
|
|
@ -81,23 +81,8 @@ def __getEventInfo(request):
|
|||
.. todo:: il refuse mon 'filter' que ce soit avant ou après le 'next(5)'. Une idée ?
|
||||
next_event = Event.objects.filter(club__in=(request.session["clubid"], )).next(5)
|
||||
"""
|
||||
|
||||
# rest = 0
|
||||
# counted = 0
|
||||
# event_list = []
|
||||
# today = pendulum.now().date()
|
||||
next_event_list = Event.objects.next(5)
|
||||
|
||||
# for event in next_event:
|
||||
# counted = event.get_number_of_occurence_to_event(today)
|
||||
# # print('pouf !')
|
||||
|
||||
# unavailabilities = Unavailability.objects.filter(datebegin__lte=event.datebegin.date(), dateend__gte=today)
|
||||
# for unavailable in unavailabilities:
|
||||
# counted -= unavailable.get_total_occurence()
|
||||
|
||||
# event_list.append((event, counted, int((counted/16)*100)))
|
||||
|
||||
return next_event_list
|
||||
|
||||
|
||||
|
@ -130,18 +115,9 @@ def __getCourseInfo(request):
|
|||
else:
|
||||
rest = int((tmp.days + 1) / 7)
|
||||
|
||||
# # select tous les unavailables liés au cours
|
||||
# unavailabilities = Unavailability.objects.filter(course=course)
|
||||
# for unavailable in unavailabilities:
|
||||
# tmp = unavailable.get_total_occurence()
|
||||
# counted -= tmp
|
||||
# rest -= tmp # si un unavailability.date < today, on soustrait quand même de rest ??? Si oui => BUG !!!!
|
||||
|
||||
courses_left += rest
|
||||
courses_done += counted - rest
|
||||
|
||||
# courses.append((course, counted, (counted - rest)))
|
||||
|
||||
return courses, courses_done, courses_left
|
||||
|
||||
|
||||
|
|
|
@ -51,12 +51,46 @@ class Place(models.Model):
|
|||
return "%s (%s)" % (self.name, self.city if self.city else "?")
|
||||
|
||||
|
||||
class Club(models.Model):
|
||||
"""
|
||||
Représente un club. Un club est associé à un lieu. Pour faciliter les filtres,
|
||||
un club peut être actif ou non.
|
||||
class GymnastStatistics():
|
||||
def __init__(self, gymnast):
|
||||
self.gymnast = gymnast
|
||||
self.number_of_courses = 0
|
||||
self.number_of_hours_by_week = timedelta()
|
||||
self.number_of_trainings = 0
|
||||
self.number_of_attended_trainings = 0
|
||||
self.number_of_attendance_hours = timedelta()
|
||||
|
||||
.. todo:: Un club peut avoir plusieurs salle et une salle peut-être louée par plusieurs clubs... M2M ?
|
||||
self.number_of_absences = 0
|
||||
self.number_of_attendance = 0
|
||||
|
||||
def add_course(self, course):
|
||||
"""Ajoute le fait que ce gymnaste a assisté à un cours"""
|
||||
self.number_of_courses += 1
|
||||
|
||||
self.number_of_attended_trainings += self.gymnast.attendances(course).count()
|
||||
self.total_number_of_trainings += course.total_number_of_trainings
|
||||
|
||||
self.number_of_attendance_hours += self.gymnast.attendances(course).count() * course.duration
|
||||
self.number_of_total_hours += course.number_of_total_hours
|
||||
|
||||
def percentage_of_attended_hours(self):
|
||||
return int(self.number_of_attendance_hours / self.number_of_total_hours) * 100
|
||||
|
||||
def percentage_of_attended_trainings(self):
|
||||
return int(self.number_of_attended_trainings / self.total_number_of_trainings) * 100
|
||||
|
||||
|
||||
class Club(models.Model):
|
||||
"""Représentation d'un club.
|
||||
|
||||
Chaque club est associé à un lieu.
|
||||
|
||||
Remarks:
|
||||
Pour faciliter les filtres, un club peut être actif ou non.
|
||||
|
||||
Todo:
|
||||
* Un club peut avoir plusieurs salle
|
||||
* Une salle peut-être louée par plusieurs clubs... M2M ?
|
||||
"""
|
||||
|
||||
class Meta:
|
||||
|
@ -73,3 +107,59 @@ class Club(models.Model):
|
|||
|
||||
def __str__(self):
|
||||
return "%s (à %s)" % (self.name, self.place.city if self.place.city else "?")
|
||||
|
||||
@property
|
||||
def get_number_of_real_occurrences(self):
|
||||
"""Retourne le nombre de fois où un cours associé au club a été donné.
|
||||
"""
|
||||
courses = Course.objects.filter(club=self).order_by(
|
||||
"iso_day_number", "hour_begin"
|
||||
)
|
||||
|
||||
return sum([x.get_number_of_real_occurrences for x in courses])
|
||||
|
||||
@property
|
||||
def get_gymnasts(self):
|
||||
gymnasts = []
|
||||
|
||||
courses = Course.objects.filter(club=self).order_by(
|
||||
"iso_day_number", "hour_begin"
|
||||
)
|
||||
|
||||
for course in self.courses:
|
||||
gymnasts.extend(Gymnast.objects.filter(to_gym__in=course.to_subgroup.all()))
|
||||
|
||||
def build_statistics(self, season):
|
||||
"""Construit des statistiques liées à la saison passée en paramètre
|
||||
|
||||
Args:
|
||||
season_id (pk): L'identifiant de la saison.
|
||||
|
||||
Returns:
|
||||
Une structure (dict) reprenant les informations suivantes:
|
||||
|
||||
{
|
||||
"total_hours_paid": <Le nombre d'heures devant être payées pour ce cours>
|
||||
}
|
||||
|
||||
* La liste des cours associés à ce club et à la saison passée en paramètre
|
||||
* Des informations statistiques concernant les gymnastes associés à chacun de ces cours
|
||||
* Le nombre total de cours réellement donnés
|
||||
* Le nombre d'heures réellement passées pour l'ensemble des cours
|
||||
|
||||
"""
|
||||
|
||||
courses = self.courses.filter(season=season)
|
||||
|
||||
all_gymnasts = {}
|
||||
|
||||
|
||||
|
||||
return {
|
||||
"gymnasts": all_gymnasts,
|
||||
"courses": courses,
|
||||
"number_of_trainings": sum([x.number_of_trainings for x in courses])
|
||||
"number_of_occured_courses": sum([x.get_number_of_real_occurrences for x in courses]),
|
||||
"total_hours": sum([x.total_number_of_hours for x in courses]),
|
||||
"total_hours_paid": sum([x.total_number_of_paid_hours for x in courses]),
|
||||
}
|
||||
|
|
|
@ -1,22 +0,0 @@
|
|||
# coding=UTF-8
|
||||
|
||||
from .models import (
|
||||
Club,
|
||||
Place,
|
||||
Country
|
||||
)
|
||||
import pytest
|
||||
|
||||
# class GymnastTestCase():
|
||||
def test_country_tostring():
|
||||
c = Country(namefr="Belgique", iso2="56")
|
||||
assert str(c) == "Belgique (56)"
|
||||
|
||||
def test_place_tostring():
|
||||
p = Place(name="FATC", city="Lillois")
|
||||
assert str(p) == "FATC (Lillois)"
|
||||
|
||||
def test_club_tostring():
|
||||
p = Place(name="FATC", city="Lillois")
|
||||
club = Club(place=p, name="FATC2")
|
||||
assert str(club) == "FATC2 (à Lillois)"
|
|
@ -24,23 +24,14 @@ from planning.models import (
|
|||
from people.models import Gymnast, Accident # people model
|
||||
from .models import (
|
||||
Club,
|
||||
Place,
|
||||
Country,
|
||||
GymnastStatistics,
|
||||
Place,
|
||||
)
|
||||
from .forms import PlaceForm
|
||||
from objective.models import Skill, Routine # objective model
|
||||
|
||||
|
||||
def __diffTime(end, start):
|
||||
"""
|
||||
Prend deux `datetime.time` en paramètre et calcul la différence entre les deux.
|
||||
"""
|
||||
startdate = datetime(2000, 1, 1, start.hour, start.minute)
|
||||
enddate = datetime(2000, 1, 1, end.hour, end.minute)
|
||||
|
||||
return enddate - startdate
|
||||
|
||||
|
||||
@login_required
|
||||
@require_http_methods(["GET"])
|
||||
def place_lookup(request):
|
||||
|
@ -153,11 +144,133 @@ def chooseStatistics(request):
|
|||
return render(request, "club_statistics.html", context)
|
||||
|
||||
|
||||
@login_required
|
||||
def club_statistics_new(request, clubid):
|
||||
"""Construit les statistiques d'un club, pour une saison choisie
|
||||
|
||||
Questions:
|
||||
Tu dis que cela construit les stats d'un club __pour une saison__...
|
||||
Mais je ne vois pas la saison dans les paramètres...
|
||||
|
||||
Todo:
|
||||
* Tenir compte de la saison.
|
||||
"""
|
||||
club = Club.objects.get(pk=clubid)
|
||||
courses = Course.objects.filter(club__in=clubid).order_by(
|
||||
"iso_day_number", "hour_begin"
|
||||
)
|
||||
|
||||
total_hours = 0
|
||||
totalHoursByWeek = 0
|
||||
totalHoursPaid = 0
|
||||
gymnastsDict = {}
|
||||
gymnasts = []
|
||||
courseList = []
|
||||
|
||||
for course in courses:
|
||||
list_of_gymnasts = Gymnast.objects.filter(to_gym__in=course.to_subgroup.all())
|
||||
gymnasts.extend(list_of_gymnasts)
|
||||
number_of_gymnasts = len(list_of_gymnasts)
|
||||
|
||||
number_of_course_hours = course.number_of_hours
|
||||
totalHoursByWeek += number_of_course_hours.seconds
|
||||
|
||||
counted = course.get_number_of_real_occurrences
|
||||
|
||||
totalTimeForCourse = number_of_course_hours * counted # timedelta
|
||||
totalHourForCourse = (totalTimeForCourse.days * 24) + (
|
||||
totalTimeForCourse.seconds / 3600
|
||||
)
|
||||
totalHours += totalHourForCourse
|
||||
totalHoursPaidForCourse = totalHourForCourse * course.number_of_trainers
|
||||
totalHoursPaid += totalHoursPaidForCourse
|
||||
|
||||
hour = number_of_course_hours.seconds / 3600
|
||||
|
||||
courseList.append(
|
||||
(
|
||||
course,
|
||||
course.number_of_trainers,
|
||||
number_of_gymnasts,
|
||||
hour,
|
||||
course.get_number_of_real_occurrences,
|
||||
totalHourForCourse,
|
||||
totalHoursPaidForCourse,
|
||||
)
|
||||
)
|
||||
|
||||
for gymnast in list_of_gymnasts:
|
||||
|
||||
gymnast_stats = gymnastsDict.setdefault(gymnast.id, GymnastStatistics(gymnast))
|
||||
|
||||
attendance_list = Training.objects.filter(course=course, gymnast=gymnast)
|
||||
number_of_attendance = attendance_list.count()
|
||||
|
||||
gymnast_stats.number_of_courses_by_week += 1
|
||||
gymnast_stats.number_of_hours_by_week += nbhour
|
||||
gymnast_stats.number_of_trainings += counted
|
||||
gymnast_stats.number_of_attendance += number_of_attendance
|
||||
gymnast_stats.number_of_training_hours += totalHourForCourse
|
||||
gymnast_stats.number_of_attendance_hours += (
|
||||
nbhour * number_of_attendance
|
||||
)
|
||||
|
||||
# tous les cours ont été traités
|
||||
totalHoursByWeek = totalHoursByWeek / 3600
|
||||
|
||||
gymnasts = set(gymnasts)
|
||||
|
||||
for gymnast in gymnasts:
|
||||
tmp = int(gymnastsDict[gymnast.id]["nbhourbyweek"].seconds / 3600)
|
||||
gymnastsDict[gymnast.id]["nbhourbyweek"] = "%d:%02d" % (
|
||||
tmp,
|
||||
(gymnastsDict[gymnast.id]["nbhourbyweek"].seconds - (tmp * 3600)) / 60,
|
||||
)
|
||||
|
||||
gymnastsDict[gymnast.id]["nbabsence"] = (
|
||||
gymnastsDict[gymnast.id]["nbtraining"]
|
||||
- gymnastsDict[gymnast.id]["nbattendance"]
|
||||
)
|
||||
|
||||
# tmp = (gymnastsDict[gymnast.id]['nbhourattendance'].days * 24) + (gymnastsDict[gymnast.id]['nbhourattendance'].seconds/3600)
|
||||
gymnastsDict[gymnast.id]["nbhourattendance"] = (
|
||||
gymnastsDict[gymnast.id]["nbhourattendance"].days * 24
|
||||
) + (gymnastsDict[gymnast.id]["nbhourattendance"].seconds / 3600)
|
||||
|
||||
gymnastsDict[gymnast.id]["nbhourabsence"] = (
|
||||
gymnastsDict[gymnast.id]["nbhourtraining"]
|
||||
- gymnastsDict[gymnast.id]["nbhourattendance"]
|
||||
)
|
||||
|
||||
gymnastsDict[gymnast.id]["percentageattendance"] = int(
|
||||
(
|
||||
gymnastsDict[gymnast.id]["nbhourattendance"]
|
||||
/ gymnastsDict[gymnast.id]["nbhourtraining"]
|
||||
)
|
||||
* 100
|
||||
)
|
||||
gymnastsDict[gymnast.id]["percentageabsence"] = int(
|
||||
(
|
||||
gymnastsDict[gymnast.id]["nbhourabsence"]
|
||||
/ gymnastsDict[gymnast.id]["nbhourtraining"]
|
||||
)
|
||||
* 100
|
||||
)
|
||||
|
||||
context = {
|
||||
"courses": courseList,
|
||||
"gymnasts": gymnastsDict,
|
||||
"totalHoursByWeek": totalHoursByWeek,
|
||||
"totalCourses": club.get_number_of_real_occurrences,
|
||||
"totalHours": totalHours,
|
||||
"totalHoursPaid": totalHoursPaid,
|
||||
}
|
||||
return context
|
||||
|
||||
@login_required
|
||||
def club_statistics(request, clubid):
|
||||
"""
|
||||
Renvoie les statistiques d'un club pour une saison choisie.
|
||||
|
||||
.. todo:: tenir compte de la saison.
|
||||
"""
|
||||
|
||||
|
@ -172,8 +285,9 @@ def club_statistics(request, clubid):
|
|||
gymnastsDict = {}
|
||||
gymnasts = []
|
||||
courseList = []
|
||||
|
||||
for course in courses:
|
||||
nbtrainer = course.trainers.count()
|
||||
|
||||
list_of_gymnasts = Gymnast.objects.filter(to_gym__in=course.to_subgroup.all())
|
||||
gymnasts.extend(list_of_gymnasts)
|
||||
nbgymnast = len(list_of_gymnasts)
|
||||
|
@ -182,38 +296,8 @@ def club_statistics(request, clubid):
|
|||
nbhour = __diffTime(course.hour_end, course.hour_begin) # timedelta
|
||||
totalHoursByWeek += nbhour.seconds
|
||||
|
||||
counted = course.get_total_occurence()
|
||||
|
||||
# select tous les unavailables liés au cours
|
||||
unavailabilities = Unavailability.objects.filter(course=course)
|
||||
for unavailable in unavailabilities:
|
||||
counted -= unavailable.get_total_occurence()
|
||||
|
||||
totalCourses += counted
|
||||
totalTimeForCourse = nbhour * counted # timedelta
|
||||
totalHourForCourse = (totalTimeForCourse.days * 24) + (
|
||||
totalTimeForCourse.seconds / 3600
|
||||
)
|
||||
totalHours += totalHourForCourse
|
||||
totalHoursPaidForCourse = totalHourForCourse * nbtrainer
|
||||
totalHoursPaid += totalHoursPaidForCourse
|
||||
|
||||
# tmp = int(nbhour.seconds/3600)
|
||||
# hour = "%d:%02d" % (tmp, (nbhour.seconds - (tmp * 3600)) / 60)
|
||||
hour = nbhour.seconds / 3600
|
||||
|
||||
courseList.append(
|
||||
(
|
||||
course,
|
||||
nbtrainer,
|
||||
nbgymnast,
|
||||
hour,
|
||||
counted,
|
||||
totalHourForCourse,
|
||||
totalHoursPaidForCourse,
|
||||
)
|
||||
)
|
||||
|
||||
for gymnast in list_of_gymnasts:
|
||||
# print(gymnast)
|
||||
if gymnast.id not in gymnastsDict:
|
||||
|
@ -236,14 +320,10 @@ def club_statistics(request, clubid):
|
|||
|
||||
# print(str(gymnast) + ' : ' + str(nbattendance) + ' for ' + str(course) )
|
||||
|
||||
gymnastsDict[gymnast.id]["nbcoursebyweek"] += 1
|
||||
gymnastsDict[gymnast.id]["nbhourbyweek"] += nbhour # timedelta
|
||||
gymnastsDict[gymnast.id]["nbtraining"] += counted
|
||||
gymnastsDict[gymnast.id]["nbattendance"] += nbattendance
|
||||
gymnastsDict[gymnast.id]["nbhourtraining"] += totalHourForCourse
|
||||
gymnastsDict[gymnast.id]["nbhourattendance"] += (
|
||||
nbhour * nbattendance
|
||||
) # timedelta
|
||||
|
||||
|
||||
# print(gymnastsDict[gymnast.id])
|
||||
|
||||
|
@ -299,3 +379,4 @@ def club_statistics(request, clubid):
|
|||
"totalHoursPaid": totalHoursPaid,
|
||||
}
|
||||
return context
|
||||
|
||||
|
|
|
@ -62,8 +62,7 @@ class Gymnast(Markdownizable):
|
|||
"""
|
||||
|
||||
class Meta:
|
||||
verbose_name = "Gymnast"
|
||||
verbose_name_plural = "Gymnasts"
|
||||
verbose_name = "gymnaste"
|
||||
ordering = ["user__last_name", "user__first_name"]
|
||||
|
||||
GENDER_CHOICES = ((0, "Male"), (1, "Female"))
|
||||
|
@ -189,6 +188,12 @@ class Gymnast(Markdownizable):
|
|||
period = pendulum.now() - pendulum.instance(self.created_at, "Europe/Brussels")
|
||||
return int(self.year_of_practice + period.in_years())
|
||||
|
||||
def attendances(self, course):
|
||||
"""Retourne les séances d'un cours où le gymnaste était présent
|
||||
|
||||
"""
|
||||
return self.trainings.filter(course=course)
|
||||
|
||||
|
||||
class Accident(Markdownizable):
|
||||
"""La classe `Accident` permet d'indiquer qu'un gymnaste est tombé durant un saut.
|
||||
|
|
|
@ -0,0 +1,65 @@
|
|||
"""Tests liés aux personnes, aux gymnastes, ..."""
|
||||
|
||||
from django.contrib.auth import get_user_model
|
||||
from django.test import TestCase
|
||||
|
||||
from location.models import Club, Country, Place
|
||||
from planning.models import Course, Training
|
||||
|
||||
from ..models import Gymnast
|
||||
|
||||
|
||||
USER_MODEL = get_user_model()
|
||||
|
||||
|
||||
class TestGymnast(TestCase):
|
||||
def setUp(self):
|
||||
self.user1 = USER_MODEL.objects.create(username="james_bond", last_name="Bond", first_name="James")
|
||||
self.club = Club.objects.create(
|
||||
name="RCW",
|
||||
place=Place.objects.create(
|
||||
name="Somewhere",
|
||||
postal=1080,
|
||||
country=Country.objects.create(
|
||||
namefr="Belgique",
|
||||
isonum=56
|
||||
)
|
||||
)
|
||||
)
|
||||
self.course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin="2022-01-01",
|
||||
dateend="2022-10-20",
|
||||
hour_begin="08:30",
|
||||
hour_end="20:00",
|
||||
club=self.club
|
||||
)
|
||||
|
||||
def test_gymnast_should_return_lastname_and_firstname(self):
|
||||
gymnast = Gymnast(user=self.user1)
|
||||
self.assertEqual(str(gymnast), "Bond, James")
|
||||
|
||||
def test_gymnast_attendances_should_be_zero(self):
|
||||
"""Vérifie qu'un gymnaste n'assiste par défaut à aucun entrainement"""
|
||||
gymnast = Gymnast.objects.create(
|
||||
user=self.user1,
|
||||
birthdate="1980-01-01",
|
||||
gender=0,
|
||||
)
|
||||
|
||||
self.assertEqual(gymnast.attendances(self.course).count(), 0)
|
||||
|
||||
def test_attendances_should_be_equals_two(self):
|
||||
"""Vérifie que les entrainements auxquels le gymnaste a assisté sont bien pris en compte"""
|
||||
gymnast = Gymnast.objects.create(
|
||||
user=self.user1,
|
||||
birthdate="1980-01-01",
|
||||
gender=0,
|
||||
)
|
||||
Training.objects.create(gymnast=gymnast, course=self.course, trainingdate="2021-02-01")
|
||||
|
||||
self.assertEqual(gymnast.attendances(self.course).count(), 1)
|
||||
|
||||
Training.objects.create(gymnast=gymnast, course=self.course, trainingdate="2021-02-02")
|
||||
|
||||
self.assertEqual(gymnast.attendances(self.course).count(), 2)
|
|
@ -5,22 +5,6 @@ from .models import Gymnast, Accident
|
|||
from datetime import datetime
|
||||
import pytest
|
||||
|
||||
# class GymnastTestCase():
|
||||
def test_gymnast_tostring():
|
||||
g = Gymnast(lastname="Pauchou", firstname="Fred")
|
||||
assert str(g) == "Pauchou, Fred"
|
||||
|
||||
def test_gymnaste_get_age():
|
||||
g = Gymnast(lastname="Pauchou", firstname="Fred", birthdate=datetime.strptime('03/07/1985', '%d/%m/%Y'));
|
||||
assert g.age == 35
|
||||
|
||||
def test_gymnaste_get_next_age():
|
||||
g = Gymnast(lastname="Pauchou", firstname="Fred", birthdate=datetime.strptime('03/07/1985', '%d/%m/%Y'));
|
||||
assert g.next_age == 36
|
||||
|
||||
def test_gymnaste_next_birthday():
|
||||
g = Gymnast(lastname="Pauchou", firstname="Fred", birthdate=datetime.strptime('03/07/1985', '%d/%m/%Y'));
|
||||
assert g.next_birthday == datetime.strptime('03/07/2021', '%d/%m/%Y')
|
||||
|
||||
def test_gymnast_known_skills():
|
||||
# @Fred : Comment tester cela ?
|
||||
|
|
|
@ -11,6 +11,8 @@ from base.models import Markdownizable
|
|||
from location.models import Club
|
||||
from people.models import Gymnast
|
||||
|
||||
from .utils import time_diff
|
||||
|
||||
|
||||
def get_week(a_date):
|
||||
"""
|
||||
|
@ -271,16 +273,22 @@ class Course(Markdownizable, Temporizable):
|
|||
|
||||
Un cours est défini par :
|
||||
* une heure de début et une heure de fin,
|
||||
* une date de début et une date de fin (un cours est considéré comme donné hebdromadairement entre
|
||||
ces deux dates) (hérite de la classe `Temporizable`)
|
||||
* une date de début et une date de fin
|
||||
|
||||
Il est considéré comme donné hebdomadairement entre deux dates
|
||||
|
||||
* est associé à un ou plusieurs entraineurs,
|
||||
* est associé à un club
|
||||
* est associé à un jour de la semaine (numéro du jour dans la semaine : 0 = lundi, 6 = dimanche).
|
||||
|
||||
Remarks:
|
||||
Les cours par défaut triés sur les jours et sur les heures de début.
|
||||
"""
|
||||
|
||||
class Meta:
|
||||
verbose_name = "Course"
|
||||
verbose_name_plural = "Courses"
|
||||
verbose_name = "Cours"
|
||||
verbose_name_plural = "Cours"
|
||||
ordering = ("iso_day_number", "hour_begin")
|
||||
|
||||
DAY_CHOICE = (
|
||||
(1, "Lundi"),
|
||||
|
@ -293,7 +301,7 @@ class Course(Markdownizable, Temporizable):
|
|||
)
|
||||
|
||||
club = models.ForeignKey(
|
||||
"location.Club", verbose_name="Club", on_delete=models.CASCADE, default=None
|
||||
"location.Club", verbose_name="Club", on_delete=models.CASCADE, default=None, related_name="courses"
|
||||
)
|
||||
iso_day_number = models.PositiveSmallIntegerField(
|
||||
choices=DAY_CHOICE, verbose_name="Jour"
|
||||
|
@ -308,17 +316,80 @@ class Course(Markdownizable, Temporizable):
|
|||
Gymnast, verbose_name="Gymnasts", related_name="courses"
|
||||
)
|
||||
|
||||
def __str__(self):
|
||||
return "%s (%s à %s)" % (
|
||||
self.get_iso_day_number_display(),
|
||||
self.hour_begin.strftime("%H:%M"),
|
||||
self.hour_end.strftime("%H:%M"),
|
||||
)
|
||||
@property
|
||||
def number_of_trainers(self):
|
||||
"""Retourne le nombre d'entraineurs liés à ce cours"""
|
||||
return self.trainers.count()
|
||||
|
||||
@property
|
||||
def number_of_hours(self) -> timedelta:
|
||||
"""Retourne le temps que dure le cours
|
||||
|
||||
Returns:
|
||||
(datetime.timedelta) La durée du cours
|
||||
"""
|
||||
return time_diff(self.hour_begin, self.hour_end)
|
||||
|
||||
@property
|
||||
def total_number_of_hours(self) -> timedelta:
|
||||
"""Retourne le temps total a été consacré à ce cours
|
||||
"""
|
||||
return self.get_number_of_real_occurrences * self.number_of_hours
|
||||
|
||||
@property
|
||||
def total_number_of_paid_hours(self) -> timedelta:
|
||||
"""Retourne le temps total consacré à ce cours par le(s) entraineur(s)
|
||||
"""
|
||||
return self.total_number_of_hours * self.number_of_trainers
|
||||
|
||||
@property
|
||||
def get_number_of_planned_occurrences(self):
|
||||
return self.get_total_occurence()
|
||||
|
||||
@property
|
||||
def total_number_of_trainings(self):
|
||||
return Training.objects.filter(course=course).count()
|
||||
|
||||
@property
|
||||
def get_number_of_real_occurrences(self) -> int:
|
||||
"""Retourne le nombre de fois où le cours a réellement été donné.
|
||||
|
||||
Ce résultat est calculé entre le nombre de fois où le cours était prévu,
|
||||
auquel nous avons retiré les indisponibilités
|
||||
|
||||
Examples:
|
||||
>>>
|
||||
"""
|
||||
counted = self.get_number_of_planned_occurrences
|
||||
|
||||
unavailabilities = Unavailability.objects.filter(course=self)
|
||||
for unavailable in unavailabilities:
|
||||
counted -= unavailable.get_total_occurence()
|
||||
|
||||
return counted
|
||||
|
||||
@property
|
||||
def get_real_time_spent_during_season(self):
|
||||
"""Calcule le temps total dépensé pour ce cours
|
||||
|
||||
|
||||
|
||||
"""
|
||||
return self.get_number_of_real_occurrences * self.duration
|
||||
|
||||
|
||||
@property
|
||||
def get_gymnasts(self):
|
||||
return Gymnast.objects.filter(to_gym__in=self.to_subgroup.all())
|
||||
|
||||
@property
|
||||
def duration(self):
|
||||
"""
|
||||
Renvoie la durée d'un cours en heures
|
||||
"""Renvoie la durée d'un cours en heures
|
||||
|
||||
Examples:
|
||||
>>> course = Course(hour_begin=20, hour_end=22)
|
||||
>>> course.duration
|
||||
2
|
||||
"""
|
||||
date_begin = pendulum.datetime(
|
||||
2000, 1, 1, self.hour_begin.hour, self.hour_begin.minute
|
||||
|
@ -328,6 +399,13 @@ class Course(Markdownizable, Temporizable):
|
|||
)
|
||||
return date_end.diff(date_begin).in_hours()
|
||||
|
||||
def __str__(self):
|
||||
return "%s (%s à %s)" % (
|
||||
self.get_iso_day_number_display(),
|
||||
self.hour_begin.strftime("%H:%M"),
|
||||
self.hour_end.strftime("%H:%M"),
|
||||
)
|
||||
|
||||
|
||||
class Group(models.Model):
|
||||
"""Classe représentant les groupes (Loisir, D1, D2, A, B, …).
|
||||
|
@ -415,16 +493,12 @@ class Unavailability(Markdownizable, Temporizable):
|
|||
|
||||
|
||||
class Training(models.Model):
|
||||
"""Classe représentant les entraînements.
|
||||
"""Un entraînement est une occurence d'un cours pendant lequel des gymnastes sont présents.
|
||||
|
||||
Un entraînement est une occurence d'un cours pendant lequel des gmnastes sont présents.
|
||||
|
||||
Un objet de cette classe lie donc un cours et un gymnaste à une date donnée.
|
||||
"""
|
||||
|
||||
class Meta:
|
||||
verbose_name = "Training"
|
||||
verbose_name_plural = "Trainings"
|
||||
verbose_name = "entraînement"
|
||||
|
||||
gymnast = models.ForeignKey(
|
||||
"people.Gymnast",
|
||||
|
|
|
@ -1,7 +1,171 @@
|
|||
"""Tests liés au modèle de l'application planning"""
|
||||
|
||||
from datetime import datetime
|
||||
from datetime import datetime, time, timedelta
|
||||
from django.contrib.auth import get_user_model
|
||||
from django.test import TestCase
|
||||
from ..models import get_number_of_weeks_between, Season
|
||||
|
||||
from location.models import Club, Place, Country
|
||||
|
||||
from ..models import get_number_of_weeks_between, Season, Course, Unavailability
|
||||
|
||||
|
||||
USER_MODEL = get_user_model()
|
||||
|
||||
|
||||
class TestCourse(TestCase):
|
||||
def setUp(self):
|
||||
self.club = Club.objects.create(
|
||||
name="RCW",
|
||||
place=Place.objects.create(
|
||||
name="Somewhere",
|
||||
postal=1080,
|
||||
country=Country.objects.create(
|
||||
namefr="Belgique",
|
||||
isonum=56
|
||||
)
|
||||
)
|
||||
)
|
||||
|
||||
self.user1 = USER_MODEL.objects.create(username="james_bond")
|
||||
self.user2 = USER_MODEL.objects.create(username="doctor_no")
|
||||
|
||||
def test_number_of_planned_occurrences(self):
|
||||
"""Vérifie le calcul du nombre de séances planifiées durant une saison"""
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
self.assertEqual(course.get_number_of_real_occurrences, 39)
|
||||
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2022, 1, 1),
|
||||
dateend=datetime(2022, 10, 20),
|
||||
hour_begin=time(hour=8, minute=30),
|
||||
hour_end=time(hour=20, minute=0),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
self.assertEqual(course.get_number_of_real_occurrences, 42)
|
||||
|
||||
def test_number_of_real_occurrences_should_be_zero(self):
|
||||
"""Vérifie le calcul du nombre de séances réalisées durant une saison"""
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
unavailabiliy = Unavailability.objects.create(
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
)
|
||||
|
||||
unavailabiliy.course.add(course)
|
||||
|
||||
self.assertEqual(course.get_number_of_real_occurrences, 0)
|
||||
|
||||
def test_total_number_of_hours_should_be_39_times_3(self):
|
||||
"""Vérifie que le nombre total d'heures consacrées est bien égal à 39 séances * 3h
|
||||
|
||||
Remarks:
|
||||
39 séances * 3h = 117h = 4 jours + 21h = timedelta(days=3, seconds=75600) :-)
|
||||
"""
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=0),
|
||||
hour_end=time(hour=22, minute=0),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
self.assertEqual(course.total_number_of_hours, timedelta(days=4, seconds=75600))
|
||||
|
||||
def test_number_of_real_occurrences_should_be_21(self):
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
unavailabiliy = Unavailability.objects.create(
|
||||
datebegin=datetime(2021, 6, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
)
|
||||
|
||||
unavailabiliy.course.add(course)
|
||||
|
||||
self.assertEqual(course.get_number_of_real_occurrences, 21)
|
||||
|
||||
def test_number_of_trainers_should_be_calculated(self):
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
course.trainers.add(self.user1)
|
||||
course.trainers.add(self.user2)
|
||||
|
||||
self.assertEqual(course.number_of_trainers, 2)
|
||||
|
||||
def test_total_number_of_paid_hours(self):
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=0),
|
||||
hour_end=time(hour=22, minute=0),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
course.trainers.add(self.user1)
|
||||
course.trainers.add(self.user2)
|
||||
|
||||
self.assertEqual(course.total_number_of_paid_hours, timedelta(days=4, seconds=75600) * 2)
|
||||
|
||||
|
||||
|
||||
def test_number_of_course_hours(self):
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
self.assertEqual(course.number_of_hours, timedelta(seconds=11700))
|
||||
|
||||
|
||||
def test_course_duration(self):
|
||||
"""Vérifie le calcul de durée d'un cours"""
|
||||
course = Course.objects.create(
|
||||
iso_day_number=2,
|
||||
datebegin=datetime(2021, 1, 1),
|
||||
dateend=datetime(2021, 9, 30),
|
||||
hour_begin=time(hour=19, minute=30),
|
||||
hour_end=time(hour=22, minute=45),
|
||||
club=self.club
|
||||
)
|
||||
|
||||
self.assertEqual(course.duration, 2)
|
||||
|
||||
|
||||
class TestUtils(TestCase):
|
||||
|
|
|
@ -0,0 +1,14 @@
|
|||
"""Tests liés aux fonctions d'utilité publique :-)"""
|
||||
|
||||
from datetime import time, timedelta
|
||||
|
||||
from django.test import SimpleTestCase
|
||||
|
||||
from ..utils import time_diff
|
||||
|
||||
|
||||
class TestTimeDiff(SimpleTestCase):
|
||||
def test_time_should_be_timedelta(self):
|
||||
td = time_diff(time(8, 30), time(10, 15))
|
||||
|
||||
self.assertEqual(td, timedelta(seconds=6300))
|
|
@ -0,0 +1,24 @@
|
|||
"""Fonctions utilitaires liées aux plannings"""
|
||||
|
||||
from datetime import datetime
|
||||
|
||||
|
||||
def time_diff(start, end):
|
||||
"""Calcule la différence de temps entre deux moments
|
||||
|
||||
Args:
|
||||
start (datetime.time)
|
||||
end (datetime.time)
|
||||
|
||||
Returns:
|
||||
Le nombre de secondes de différence entre les paramètres `start` et `end`.
|
||||
|
||||
Examples:
|
||||
>>> from datetime import time
|
||||
>>> time_difference(time(8, 30), time(10, 15))
|
||||
datetime.timedelta(seconds=6300)
|
||||
"""
|
||||
startdate = datetime(2000, 1, 1, start.hour, start.minute)
|
||||
enddate = datetime(2000, 1, 1, end.hour, end.minute)
|
||||
|
||||
return enddate - startdate
|
Loading…
Reference in New Issue