Represent blood by enum

[tomaspavlin]

🩸 Represent blood by enum

Background

Currently, we store blood type as TEXT in DB and converting blood from db model to typed dataclasses is not working properly.

Needs to be done

We want to use enum for storing blood instead. This needs to be changed in the following db columns:

DonorModel.blood
RecipientModel.blood
RecipientAcceptableBloodModel.blood_type

This will also require to change few tests. Also please verify, that blood type conversion is working properly.

[kubantjan]

@tomaspavlin co to znamena ze nefunguje spravne? nekdy se A -> na jine pismenko? jen jak moc je tohle urgentni, jestli jsme mohli nekdy nejakymu pacientovi priradit spatnou krevni skubpinu?

[tomaspavlin]

@kubantjan Problém je, že na pár místech vytáhneme krevní skupinu z db a uložíme ji do enum atributu nesprávně jako string a ne jako enum. Mě kvůli tomu pak padaly testy, protože BloodGroup.A != 'A' a pokud se to takhle porovnává i jinde než v testech, potenciálně to nemusí fungovat správně. Ale nezkoumal jsem, jestli se to ve fungování vůbec někdy projeví.

[kubantjan]

Nejak jsme to nezamergovali, nevim proc. asi je ten kod ztraceny, kazdopadne neni prioritni, ale bude fajn to udelat

[kubantjan]

To co pise tomas neni pravda. BloodGroup.A == 'A'

viz pridany test. Ale upravil jsem ty sql alchemy objekty at fungujou spravne, ale mam pocit, ze to fakt neni issue.

[kubantjan]

V kódu jsou ještě stringy # TODO: #477 represent blood as enum. A ideální by bylo napsat migrační skript nebo aspoň někam připsat todo, teď by se nám mohlo stát, že se do db dostane jiná hodnota než je v enumu.

[jogobeny]

@kubantjan, je toto fixed kvuli #940?

[kubantjan]

@kristinagalik tohle se teda muze zavrit?