Zklidnění testů #23

Merged
zelvuska merged 2 commits from clean_tests into master 2 years ago
ledoian commented 2 years ago
Owner

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 2 years ago
Owner

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

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

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 2 years ago
e0eb12cf9e Školy jen zakomentovat, nepsat expectedFailure test
Poster
Owner

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 2 years ago
zelvuska deleted branch clean_tests 2 years ago
The pull request has been merged as c6ff17050b.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.