Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Frontend testen #301

Merged
merged 47 commits into from
May 23, 2024
Merged

Frontend testen #301

merged 47 commits into from
May 23, 2024

Conversation

ArnoutAllaert
Copy link
Contributor

Voorlopig zijn er testen om als lesgever vakken en projecten aan te maken, bewerken en archiveren.

Er moeten nog testen komen voor:

  • groepen aanmaken en verdelen (lesgever)
  • scores toekennen (lesgever)
  • tot een groep toetreden (student)
  • een indiening aanmaken (student)

Normaal ging ik ook nog testen maken waar er verkeerde input werd meegegeven om te kijken of de juiste warnings/errors worden meegegeven, maar ik vrees dat er daar te weinig tijd voor zal zijn.

Ik zie ook net dat alle component testen falen met volgende error:
image
Dus als hier iemand toevallig een oplossing voor zou weten? 🙏 😭

ArnoutAllaert and others added 30 commits May 9, 2024 18:02
Copy link
Contributor

@sPAICEcake sPAICEcake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

frontend src files zijn in orde

@ArnoutAllaert
Copy link
Contributor Author

ArnoutAllaert commented May 23, 2024

Als je de e2e testen in deze volgorde uitvoert, startend van een lege databank dan zouden ze allemaal moeten slagen:

switchen naar lesgever test omgeving (sudo bash switch_test,sh true)

  • course (maak en wijzig een vak)
  • project (maak en wijzig een project)
  • groups (groepen aanmaken voor het project)

switchen naar student test omgeving (sudo bash switch_test,sh)

  • group (student verlaat en joint een groep)
  • submission (student dient in)

switchen naar lesgever test omgeving

  • score (lesgever geeft score op de indiening)
  • archive (lesgever archiveert het project en het verslag)

Het is mss een beetje omslachtig, maar ik denk dat dit het best de flow beschrijft van de belangrijkste zaken.

Momenteel is er nog een kleine bug die altijd punten op 20 toont waardoor de score test niet zal slagen, hier wordt aan gewerkt en vanaf die bug verholpen is zal ook de score test slagen.

@sPAICEcake
Copy link
Contributor

Als je de e2e testen in deze volgorde uitvoert, startend van een lege databank dan zouden ze allemaal moeten slagen:

switchen naar lesgever test omgeving (sudo bash switch_test,sh true)

  • course (maak en wijzig een vak)
  • project (maak en wijzig een project)
  • groups (groepen aanmaken voor het project)

switchen naar student test omgeving (sudo bash switch_test,sh)

  • group (student verlaat en joint een groep)
  • submission (student dient in)

switchen naar lesgever test omgeving

  • score (lesgever geeft score op de indiening)
  • archive (lesgever archiveert het project en het verslag)

Het is mss een beetje omslachtig, maar ik denk dat dit het best de flow beschrijft van de belangrijkste zaken.

Momenteel is er nog een kleine bug die altijd punten op 20 toont waardoor de score test niet zal slagen, hier wordt aan gewerkt en vanaf die bug verholpen is zal ook de score test slagen.

de score bug is ondertussen al aangepast

@sPAICEcake
Copy link
Contributor

sPAICEcake commented May 23, 2024

Als je de e2e testen in deze volgorde uitvoert, startend van een lege databank dan zouden ze allemaal moeten slagen:

switchen naar lesgever test omgeving (sudo bash switch_test,sh true)

  • course (maak en wijzig een vak)
  • project (maak en wijzig een project)
  • groups (groepen aanmaken voor het project)

switchen naar student test omgeving (sudo bash switch_test,sh)

  • group (student verlaat en joint een groep)
  • submission (student dient in)

switchen naar lesgever test omgeving

  • score (lesgever geeft score op de indiening)
  • archive (lesgever archiveert het project en het verslag)

Het is mss een beetje omslachtig, maar ik denk dat dit het best de flow beschrijft van de belangrijkste zaken.

Momenteel is er nog een kleine bug die altijd punten op 20 toont waardoor de score test niet zal slagen, hier wordt aan gewerkt en vanaf die bug verholpen is zal ook de score test slagen.

mss is het ook handig om dit in de readme te zetten, onder frontend testen

@sPAICEcake
Copy link
Contributor

sPAICEcake commented May 23, 2024

Na het mergen, moet de axiosConfig link terug aangepast worden

@ArnoutAllaert ArnoutAllaert marked this pull request as ready for review May 23, 2024 18:13
@ArnoutAllaert
Copy link
Contributor Author

De readme werd aangepast om uitleg te geven over hoe de testen kunnen uitgevoerd worden.

Copy link
Contributor

@LGDTimtou LGDTimtou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goed gedaan, ben blij dat dit af is

@LGDTimtou LGDTimtou merged commit 57ced79 into develop May 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants