mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

Zklidnění testů #23

Merged
zelvuska merged 2 commits from clean_tests into master 2023-02-27 20:12:45 +01:00
Conversation 3 Commits 2 Files changed 1 +6 -5
ledoian commented 2023-02-06 22:03:26 +01:00
Owner
Copy link

Malá úprava testu pro autocomplete škol, aby make/test by default házelo čistý výsledek a dalo se tedy snadno vyznat ve výsledku celkových testů.

Neříkám, že tohle je nutně to správné řešení (alternativa: otagovat testy a ty s tagem, že failují, by default nepouštět…), ale docela bych se chtěl blížit stavu, kdy ty testy spíš běží než failují a když udělám velkou změnu, tak se zejména dozvím, co se rozbilo. (Moje velká změna byla mergenutí aktuálního masteru do zadavatko_problemu…)

Malá úprava testu pro autocomplete škol, aby `make/test` by default házelo čistý výsledek a dalo se tedy snadno vyznat ve výsledku celkových testů. Neříkám, že tohle je nutně to správné řešení (alternativa: otagovat testy a ty s tagem, že failují, by default nepouštět…), ale docela bych se chtěl blížit stavu, kdy ty testy spíš běží než failují a když udělám velkou změnu, tak se zejména dozvím, co se rozbilo. (Moje velká změna byla mergenutí aktuálního masteru do `zadavatko_problemu`…)
ledoian added 1 commit 2023-02-06 22:03:27 +01:00 
… moc nahlas.
zelvuska commented 2023-02-06 22:13:01 +01:00
Owner
Copy link

Zakomentování mi přijde dostatečné…

Zakomentování mi přijde dostatečné…
ledoian commented 2023-02-06 22:30:22 +01:00
Author
Owner
Copy link

Tam je otázka, kde držet informaci o tom, že je potřeba to opravit. Asi by stačilo to držet v Kanci, to je pravda.

Mám to jen zakomentovat?

Tam je otázka, kde držet informaci o tom, že je potřeba to opravit. Asi by stačilo to držet v Kanci, to je pravda. Mám to jen zakomentovat?
ledoian added 1 commit 2023-02-06 22:32:29 +01:00 
Podle toho, jak dopadne PR se to buď aplikuje takto, nebo se tenhle
commit revertne.
ledoian commented 2023-02-06 22:33:03 +01:00
Author
Owner
Copy link

V aktuálním commitu už jsou jen zakomentované, kdyžtak revertněme e0eb12cf9e.

V aktuálním commitu už jsou jen zakomentované, kdyžtak revertněme e0eb12cf9e3645cd8265b2c4b454bca12ce16a13.
zelvuska merged commit c6ff17050b into master 2023-02-27 20:12:45 +01:00
zelvuska deleted branch clean_tests 2023-02-27 20:12:45 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#23
No description provided.
Delete branch "clean_tests"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?