Zrychlení testů #21

Merged
ledoian merged 2 commits from faster_tests into master 1 year ago
ledoian commented 1 year ago
Owner

Několik optimalizací pro rychlejší běh testů. Pro detaily vizte commit message u af6628367f.

Budu rád, pokud posdílíte data o rychlosti před a po, ať vím, jak moc je zrychlení závislé na HW. (Nejpíš se kouknu na rozptyl poměrů zrychlení a víc to zkoumat nebudu, protože stejně na rigorózní testování nemáme dostatek dat (konfera na sous? :-D).) Pokud neumíte časovat přes time, tak klidně testům předhoďte --timing, ono se to pro tyhle účely moc neliší…

Několik optimalizací pro rychlejší běh testů. Pro detaily vizte commit message u af6628367f546a6. Budu rád, pokud posdílíte data o rychlosti před a po, ať vím, jak moc je zrychlení závislé na HW. (Nejpíš se kouknu na rozptyl poměrů zrychlení a víc to zkoumat nebudu, protože stejně na rigorózní testování nemáme dostatek dat (konfera na sous? :-D).) Pokud neumíte časovat přes `time`, tak klidně testům předhoďte `--timing`, ono se to pro tyhle účely moc neliší…
ledoian added 2 commits 1 year ago
af6628367f Testy: Schováváme si databázi napříč běhy
3a1aade3a2 Testy: bereme parametry
Poster
Owner

Zdá se, že zrychlení se koná. Tak si někdo prosím prohlédněte změny, posuďte, jestli to dává smysl a skoro bych to mergenul spíš dřïv – změna je to vesměs triviální.

Zdá se, že zrychlení se koná. Tak si někdo prosím prohlédněte změny, posuďte, jestli to dává smysl a skoro bych to mergenul spíš dřïv – změna je to vesměs triviální.
zelvuska reviewed 1 year ago
trap - ERR # Testy nejspíš selžou, ale nechceme kolem toho dělat další chybovou hlášku.
./manage.py test -v2
./manage.py test -v2 --keepdb "$@"
Poster
Owner

Proč "$@"?

Proč "$@"?
ledoian commented 1 year ago
Poster
Owner

Viz zprávu 3a1aade3a2bd215510b361eeb19c88fb2454628a: aby to umělo brát parametry a dalo se upravit, co chceme testovat. Zvlášť pokud jednou chceme mít spoustu testů, tak dává smysl si pustit jen jednu testovací sadu, když něco upravuji a kompletní testy pouštět až nakonec…

(A taky to jako milý bonus automaticky přidává podporu --help :-))

Viz zprávu 3a1aade3a2bd215510b361eeb19c88fb2454628a: aby to umělo brát parametry a dalo se upravit, co chceme testovat. Zvlášť pokud jednou chceme mít spoustu testů, tak dává smysl si pustit jen jednu testovací sadu, když něco upravuji a kompletní testy pouštět až nakonec… (A taky to jako milý bonus automaticky přidává podporu `--help` :-))
Owner

Za mě v pohodě :)

Za mě v pohodě :)
Poster
Owner

Merguji.

Merguji.
ledoian merged commit a6a57de3cf into master 1 year ago
ledoian deleted branch faster_tests 1 year ago
The pull request has been merged as a6a57de3cf.
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.