Hoppa till innehållet

Ämne på Diskussion:Developer guidelines/Flow

Manadatory testing of relatively small batchuploads?

2
Mattias Östmar (WMSE) (diskussionbidrag)

Is it reasonable to add mandatory tests and testdriven development to batchuploads that might be as small as 500-1000 images? I think not. The extra workload of teaching and learning unit testing this far into the KÖK project (11 out of 18 months) should also be considered. The use of unit testing could instead be decided upon per project.

The LSH batchupload added 919 sloc (equivalent or more that the total sum of GAR, DechArch) to the work load, while SMM and Nationalmuseum did not include tests.

https://github.com/lokal-profil/upload_batches/tree/master/SMM-images.

André Costa (WMSE) (diskussionbidrag)

Jag håller med om att vi kan behöva ta oss en funderare kring detta. Bra att komma ihåg är att tester även kan ses som extra dokumentation.

Min första tanke är tester (som krav) för:

  • Kod som går i produktion (här kod som kör i bakgrunden, typ Fikarummet-bot/Wikispeech)
  • Kod som återanvänds av flera projekt (typ BatchUploadTools kanske även huvudklassen i Wikidata-migrationen). Generellt sett ser jag att vi bryter ut fler sådana bibliotek allt eftersom.
  • "Komplicerad kod", luddigt som tusan men om du måste hålla tungan rätt i mun när du försöker klura ut vad koden gör kan det vara bra med tester.

Ser gärna att vi bollar detta tillsammans innan vi gör någon ändring dock.