PGL/Clyde
PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

Leo/Backend #82

Merged
Maxime merged 11 commits from Leo/Backend into master 2024-03-11 19:16:20 +01:00
Conversation 0 Commits 11 Files Changed 10 +8 -25
2 changed files with 8 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 94d86ef252 - Show all commits

27
backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
View File

@ -53,31 +53,14 @@ public class MockController {
userRepo.saveAll(mockUsers);
for (User user: mockUsers){
//Petit test pour être sur de delete les bons tokens
if (user.getLastName().equals("brine")){
Calendar c = Calendar.getInstance();
c.setTime(new Date());
c.add(Calendar.DAY_OF_WEEK, 1);
tokenService.saveToken(new Token(user,user.getPassword(), c.getTime()));
}else{
tokenService.saveToken(new Token(user,user.getPassword(), new Date()));
}
}
}
//Nous allons donner des tokens a herobrine pour tester la limite de token
@PostMapping("/herobrine")
public void giveHerobrineTokens(){
User herobrine = userRepo.findById(1);
Calendar c = Calendar.getInstance();
for (int i = 1; i <= 7; i++){
Token t = new Token(herobrine, herobrine.getPassword(), c.getTime());
c.add(Calendar.DAY_OF_YEAR, 1);
tokenService.saveToken(t);
c.setTime(new Date());
for (User user: mockUsers){
tokenService.saveToken(new Token(user,user.getPassword(), new Date()));
}
}
@DeleteMapping("/mock")
public void deleteMock(){
for (User user:mockUsers){
tonitch commented 2024-03-09 14:56:45 +01:00
Outdated
Review
Copy Link

Junit est installé normalement, ça serait mieux de faire ce genre de chose dans les test !
(Et en plus ça peut rester vu que on sera noté dessu)

Junit est installé normalement, ça serait mieux de faire ce genre de chose dans les test ! (Et en plus ça peut rester vu que on sera noté dessu)
👍 1
LeoMoulin commented 2024-03-09 19:29:22 +01:00
Outdated
Review
Copy Link

J'ai essayé et j'avoues galérer un peu a setup les tests unitaires sur la DB, je vais m'y mettre demain a tête reposée en attendant je nettoies les tests et je les referai propres !

J'ai essayé et j'avoues galérer un peu a setup les tests unitaires sur la DB, je vais m'y mettre demain a tête reposée en attendant je nettoies les tests et je les referai propres !
tonitch commented 2024-03-09 19:34:23 +01:00
Outdated
Review
Copy Link

en vrai au pire c'est pas grave mais je ne pense pas que ça ai sa place dans le projet final, genre il faut juste pas le commit et le garder ailleur quoi ^^

en vrai au pire c'est pas grave mais je ne pense pas que ça ai sa place dans le projet final, genre il faut juste pas le commit et le garder ailleur quoi ^^
LeoMoulin commented 2024-03-10 13:46:36 +01:00
Outdated
Review
Copy Link

entiérement d'accord j'aurai du le nettoyer

entiérement d'accord j'aurai du le nettoyer

6
backend/src/main/java/ovh/herisson/Clyde/Services/TokenService.java
View File

@ -14,7 +14,6 @@ import java.util.Date;
@Service
public class TokenService {
TokenRepository tokenRepo;
public TokenService(TokenRepository tokenRepo){
@ -52,13 +51,14 @@ public class TokenService {
//Tous les jours a minuit
tonitch commented 2024-03-09 14:49:38 +01:00
Outdated
Review
Copy Link

Calendar.getInstance retourne déjà l'instant ou il est crée
https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html

Calendar.getInstance retourne déjà l'instant ou il est crée https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html
👍 1
LeoMoulin commented 2024-03-09 19:30:07 +01:00
Outdated
Review
Copy Link

C'est bien vu ca

C'est bien vu ca
@Scheduled(cron = "0 0 0 * * ?")
public void autoDeleteToken() {
System.out.println("triggered");
for (Token t: tokenRepo.findAll()){
Maxime marked this conversation as resolved Outdated
tonitch commented 2024-03-09 14:52:04 +01:00
Outdated
Review
Copy Link

peut être faire des >= au lieu des == pour éviter que le programme ne passe au dessus si imaginons le serveur est fermé pendant 2 jours

Mais encore mieux tu pourrais comparer les deux Calendar avec la fonction ´.compareTo(Calendar)`
https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html#compareTo(java.util.Calendar) et juste voir si le résultat est négatif

peut être faire des >= au lieu des == pour éviter que le programme ne passe au dessus si imaginons le serveur est fermé pendant 2 jours Mais encore mieux tu pourrais comparer les deux `Calendar` avec la fonction ´.compareTo(Calendar)` https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html#compareTo(java.util.Calendar) et juste voir si le résultat est négatif
👍 1
LeoMoulin commented 2024-03-09 19:30:35 +01:00
Outdated
Review
Copy Link

Excellente remarque ! Ca sera changé dans mon prochain commit !

Excellente remarque ! Ca sera changé dans mon prochain commit !
Calendar cal = Calendar.getInstance();
cal.setTime(new Date());
Calendar cal2 = Calendar.getInstance();
cal2.setTime(t.getExpirationDate());
if (cal.get(Calendar.YEAR) == cal2.get(Calendar.YEAR) && cal.get(Calendar.DAY_OF_YEAR) == cal2.get(Calendar.DAY_OF_YEAR)){
if (cal.compareTo(cal2) >= 0){
tokenRepo.delete(t);
}
}