feat: ✨ implementation ocr module for blank guns

[nutfdt]

No description provided.

[leihuayi]

Reste à mettre à jour

• tests api dans le back
• tests e2e pour le front

[github-actions]

Code Coverage Summary

Filename               Stmts    Miss  Cover    Missing
-------------------  -------  ------  -------  --------------------------------------
src/__init__.py            0       0  100.00%
src/config.py             45       3  93.33%   79, 81, 83
src/main.py               13       0  100.00%
src/router.py            103      24  76.70%   55, 104-110, 137-140, 161-177, 184-200
src/utils.py              43       5  88.37%   72-74, 92, 96
tests/__init__.py          0       0  100.00%
tests/conftest.py         14       0  100.00%
tests/test_api.py        113       0  100.00%
tests/test_email.py        4       0  100.00%
TOTAL                    335      32  90.45%

Results for commit: f5a6f3f

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[thomashbrnrd]

C'est volontaire cette modification ?

[nutfdt]

Oui je trouve que c'est plus clair de spécifier AlarmGun plutôt que BlankGun, même si techniquement les armes d'alarmes sont des armes à blancs.

[thomashbrnrd]

Mais tu supprimes la clé "meta" c'est normal ?

[nutfdt]

Ah oui my bad, je l'ai reprise

[thomashbrnrd]

On utilise AlarmGun un peu partout, est ce que ce serait pas mieux d'utiliser isAlarmGun ?

[nutfdt]

On a une fonction qui se nomme isAlarmGun() donc j'ai préféré cette syntaxe pour éviter de potentiellement s'embrouiller

export function isAlarmGun() {
  if (
    store.selectedOptions[0] === "revolver_black_powder" ||
    !["pistolet_semi_auto_moderne", "revolver"].includes(store.typology) ||
    store.selectedAmmo === "billes"
  ) {
    return false;
  }
  return store.isAlarmModel ? true : undefined;
}

[thomashbrnrd]

Pourquoi ne pas utiliser la fonction isAlarmGun du coup ?

[nutfdt]

J'ai finalement remplacé par isAlarmGun après coup