Mtl test

[busstoptaktik]

Første end-to-end test af MTL, men uden megen undervejs- og efterkontrol. Det lægger op til meget mere, men her er starten

[busstoptaktik]

... mangler lige at finde ud af hvordan man får testen til at køre i et specifikt bibliotek...

[kbevers]

Prøv at tage et kig på https://click.palletsprojects.com/en/7.x/testing/#file-system-isolation

Du må kunne tage dine test inputfiler og dumpe dem i det isolerede filsystem, hvorefter jeg går ud fra at du kan gøre som du vil (som jeg tror er, at manipulere filer i cwd).

[busstoptaktik]

Du må kunne tage dine test inputfiler og dumpe dem i det isolerede filsystem,

Ja - og jeg tror det bliver løsningen når jeg udvider, men lige her var problemet først og fremmest overhovedet at finde de faste inputfiler (testen er håndminimeret mhp. at kunne køre uden adgang til den fulde database. På længere sigt skal der nok hældes flere data i dit testdatasæt for at kunne køre mere omfattende test s hele vejen igennem systemet).

[kbevers]

Bare FYI, din nye test kaster et par warnings af sig:

/Users/kevers/miniconda3/envs/fire-dev/lib/python3.7/site-packages/patsy/constraint.py:13
  /Users/kevers/miniconda3/envs/fire-dev/lib/python3.7/site-packages/patsy/constraint.py:13: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working
    from collections import Mapping

test/mtl/test_mtl.py::test_cli
  /Users/kevers/miniconda3/envs/fire-dev/lib/python3.7/site-packages/scipy/stats/stats.py:1535: UserWarning: kurtosistest only valid for n>=20 ... continuing anyway, n=13
    "anyway, n=%i" % int(n))

Den første kan der vist ikke gøres så meget ved men den anden ligner en der bør tages lidt alvorligt?

[busstoptaktik]

Nej, den er ok: kurt-testen er ikke relevant, men kommer "med pakken" når man beder om en statistisk oversigt. Advarslen er bare en effekt af at testnettet er ganske lille (de to TAPAS-stationer på Djursland). Jeg har undladt at slå advarsler fra, fordi den første nok kræver at jeg tager mig sammen og checker om det allerede er rettet upstream, og i modsat fald begår verdens mindste PR, med en rettelse på skaldede 4 tegn ".abc". Eller "abc."

[busstoptaktik]

... men det burde selvfølgelig have fremgået af merge commitkommentaren, så beklager at have spildt din tid på den måde, @kbevers

[kbevers]

... og i modsat fald begår verdens mindste PR, med en rettelse på skaldede 4 tegn ".abc". Eller "abc."

Det er billige point at lave sådan et pull request: python-restx/flask-restx#54 :-) Det er i øvrigt ikke fikset upstream, og rettelsen ser ud til at være præcis den samme som jeg har lavet i mit pull request mod flask-restx. Tilsyneladende et velkendt trick for at understøtte både Python2 og 3.

Det er jo rart nok at warnings bliver fremhævet så man har en ærlig chance for reagere på dem. Men advarslen fra scipy.stats kunne nok være meget rar at undertrykke når det er bevidst at den kommer. Bare så den ikke støjer. Kan man gøre det via scipy? Ellers er det muligt at ignorere bestemte advarsler via en pytest opsætningsfil.

[kbevers]

Jeg prøver med et fiks i #135. Selvom jeg lokalt kun lykkes med at undertrykke den ene af de to warnings