search

TK-IT / web

TÅGEKAMMERETs hjemmeside i Django. Både offentlig og intern
https://TAAGEKAMMERET.dk
Other
1 stars 3 forks source link

Add test and fix for #223 #225

Closed Mortal closed 6 years ago

Mortal commented 6 years ago

Fixes #223 in regnskab ("Når man skriver Dorte, skal Dorte komme før Anne Dorte").

Also add JavaScript tests, run using the dukpy JS interpreter.

Also add tools/babel-compile.py which uses dukpy to run babeljs on the file regnskab.es6x to compile it into regnskab.js (for running in the browser).

neic commented 6 years ago

Når jeg kører manage.py test med dukpy installeret fejler den med _dukpy.JSRuntimeError: Error: cannot find module: babel-polyfill.

Mortal commented 6 years ago

@neic Beklager - det var noget bøvl at få til at virke og jeg havde i frustration manuelt tilføjet babel-polyfill i .venv/lib/python3.6/site-packages/dukpy/jsmodules. Jeg har fikset det nu - vil du teste igen?

neic commented 6 years ago

De automatiske test fejler ikke mere. Jeg har ikke noget data at teste manuelt på og har ikke ~så meget~ nogen erfaring med apps/regnskab. Hvis du sender mig noget kan jeg godt gøre det, ellers så tænker jeg at du bare kan merge.

neic commented 6 years ago

Måske burde du tilføje dukpy til [dev-packages] i Pipfile.

Mortal commented 6 years ago

Tak! Jeg burde skrive nogle automatiske tests af regnskab en dag.