P4Pillon/Krys4lide
4
2
Fork 0
Code Issues 27 Pull Requests 3 Actions Packages Projects 1 Releases Activity

Initialisation de la crate FSV, couche haut-niveau des accès aux fonctions SSV #71

Merged
florian_briand merged 6 commits from 38-fsv-high-level-lib into main 2024-10-09 22:46:42 +02:00
Conversation 0 Commits 6 Files Changed 6 +418
florian_briand commented 2024-10-02 01:07:17 +02:00
Owner
Copy Link

Détails

  • Création de la crate fsv, couche de haut niveau pour l'usage des librairies FSV
  • Implémentation partielle des appels "haut niveau" aux fonctions SSV_InitLIB2, SSV_LireCartePS et SSV_LireConfig
  • Implémentation de la gestion des erreurs numériques de la librairie C pour ces fonctions

Pourquoi ?

L'usage est de séparer les couches bas niveau, exposant des versions légèrement "rustifiées" des appels aux fonctions des librairies C, des couches "haut niveau", garantissant un usage "safe" et "user friendly".

Documentation

No response

Contribue à #38
Closes #49

### Détails - Création de la crate fsv, couche de haut niveau pour l'usage des librairies FSV - Implémentation partielle des appels "haut niveau" aux fonctions SSV_InitLIB2, SSV_LireCartePS et SSV_LireConfig - Implémentation de la gestion des erreurs numériques de la librairie C pour ces fonctions ### Pourquoi ? L'usage est de séparer les couches bas niveau, exposant des versions légèrement "rustifiées" des appels aux fonctions des librairies C, des couches "haut niveau", garantissant un usage "safe" et "user friendly". ### Documentation _No response_ Contribue à #38 Closes #49
florian_briand added the
enhancement
module/sesam-vitale
 labels 2024-10-02 01:07:17 +02:00
florian_briand self-assigned this 2024-10-02 01:07:17 +02:00
florian_briand added 4 commits 2024-10-02 01:07:17 +02:00
florian_briand added this to the 0 - POC project 2024-10-02 01:07:18 +02:00
florian_briand requested review from kosssi 2024-10-02 01:07:37 +02:00
florian_briand requested review from theo 2024-10-02 01:07:37 +02:00
florian_briand changed title from 38-fsv-high-level-lib to Initialisation de la crate FSV, couche haut-niveau des accès aux fonctions SSV 2024-10-02 01:09:13 +02:00
florian_briand force-pushed 38-fsv-high-level-lib from 45124db739 to 332fb01253 2024-10-02 01:17:22 +02:00 Compare
florian_briand force-pushed 38-fsv-high-level-lib from 332fb01253 to 79c16751e3 2024-10-02 12:20:49 +02:00 Compare
kosssi approved these changes 2024-10-07 23:07:24 +02:00
kosssi left a comment
Owner
Copy Link

Plus simple à comprendre, impressionnant le nombre d'erreurs différentes...

Plus simple à comprendre, impressionnant le nombre d'erreurs différentes...
crates/fsv/src/ssv/errors_ssv.rs Outdated
@ -0,0 +17,4 @@
/// - Sécurisation d'une série de lots en cours.
/// - Pour les fonctions TLA (sauf Identifier TLA) : Cette erreur survient lorsque le simulateur TLA est en mode 1.50.
/// - Lire Date Lecteur, Mettre à jour Date Lecteur, Lire Droits Vitale : Cette erreur peut survenir lorsque le Logiciel Lecteur ne connaît pas la fonction sollicitée, c'est-à-dire si la version du Logiciel Lecteur est antérieure à 2.00.
/// - Décharger Données Bénéficiaires : cette erreur peut survenir pour signaler que le
kosssi commented 2024-10-07 22:55:10 +02:00
Owner
Copy Link

Il manque la fin de la phrase ;)

  • Compléter la fin de la phrase
Il manque la fin de la phrase ;) - [x] Compléter la fin de la phrase
florian_briand marked this conversation as resolved
crates/fsv/src/ssv/mod.rs Outdated
@ -0,0 +182,4 @@
Ok(())
}
#[ignore]
kosssi commented 2024-10-07 23:05:01 +02:00
Owner
Copy Link

Pourquoi tu ignores ce test ?

Pourquoi tu ignores ce test ?
florian_briand commented 2024-10-07 23:48:46 +02:00
Author
Owner
Copy Link

C'est un test qui lit une carte réelle (donc il faut le lecteur et la carte), mais entre un faux code.

  • Le test ne peut pas tourner s'il n'y a pas un lecteur + CPS à dispo
  • Si on répète l'opération plusieurs fois, ça peut bloquer la carte

Ça fait donc que ça me parait pas un bon test à laisser en lancement "automatique".
Le #[ignore] fait que le test n'est pas inclu automatiquement quand on lance un cargo test, mais doit être lancé manuellement (ou avec le paramètre --ignored)

Dans les PR suivantes, j'ai itéré dessus pour mettre une "raison" au ignore, et généraliser ce ignore à tous les tests qui utilisent du hardware

C'est un test qui lit une carte réelle (donc il faut le lecteur et la carte), mais entre un faux code. - Le test ne peut pas tourner s'il n'y a pas un lecteur + CPS à dispo - Si on répète l'opération plusieurs fois, ça peut bloquer la carte Ça fait donc que ça me parait pas un bon test à laisser en lancement "automatique". Le `#[ignore]` fait que le test n'est pas inclu automatiquement quand on lance un `cargo test`, mais doit être lancé manuellement (ou avec le paramètre `--ignored`) Dans les PR suivantes, j'ai itéré dessus pour mettre une "raison" au ignore, et généraliser ce ignore à tous les tests qui utilisent du hardware
👍 1
kosssi marked this conversation as resolved
florian_briand force-pushed 38-fsv-high-level-lib from 79c16751e3 to d043915a29 2024-10-09 22:39:49 +02:00 Compare
florian_briand changed target branch from feat/38-fsv-sys-crate to main 2024-10-09 22:40:19 +02:00
florian_briand added 2 commits 2024-10-09 22:45:23 +02:00
florian_briand merged commit c83824ae34 into main 2024-10-09 22:46:42 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
theo
kosssi
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
independant

Tickets réalisables indépendaments des travaux prévus dans la roadmap - très approprié pour des contributions ponctuelles

  
invalid

Something is wrong

  
module/autre

Pour toute autre partie du produit non lié aux autres étiquettes disponibles

  
module/backend

Serveur backend, opérant les accès données et système

  
module/desktop

Encapsulation de la webview dans Tauri

  
module/docs

Documentation (docs)

  
module/frontend

Interfaces utilisateur⋅ices

  
module/scripts

Scripts utiles au setup et/ou au build (scripts)

  
module/sesam-vitale

Moteur SESAM-Vitale

  
module/utils

Librairie de fonctions utilitaires

  
open-source

Les tâches contribuant à l'ouverture et au partage du code

  
question

More information is needed

  
to-triage

À trier

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
independant
invalid
module/autre
module/backend
module/desktop
module/docs
module/frontend
module/scripts
module/sesam-vitale
module/utils
open-source
question
to-triage
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
0 - POC
Assignees
Clear assignees
No Assignees
florian_briand
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: P4Pillon/Krys4lide#71
No description provided.
Delete Branch "38-fsv-high-level-lib"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?