Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ccc_libs] Amélioration saveFiles et autres fonctions #908

Open
LucsT opened this issue Feb 1, 2023 · 0 comments
Open

[ccc_libs] Amélioration saveFiles et autres fonctions #908

LucsT opened this issue Feb 1, 2023 · 0 comments

Comments

@LucsT
Copy link
Contributor

LucsT commented Feb 1, 2023

Différents points permettant une amélioration/optimisation ont été remarqués sur la lib en général et saveFiles/saveBills/saveIdentities.
Cette issue permet de les documenter.

  • Client en premier argument
    Homogénéiser tous les fonctions pour qu'elle prennent le client en argument en 1er arg.
    Plus de client en variable global (addData, saveIdentities)

  • Mutation d'argument :
    Dans saveFiles des arguments sont modifiés puis retournés, il faut retourner un nouvel objet. (ex attachFileToEntry, sanitizeEntry )
    Probablement ailleurs aussi dans la lib

  • Fusion du mode create et update file de saveFiles
    L’utilisation du save de cozy-Client permettrait d'avoir un code moins différencier pour ces deux scénarios

  • Test d'existence des fichiers dans saveFiles
    Le comportement de cozy-client (de ne pas lever une erreur en cas d'inexistence) devrait permettre de nettoyer les fonctions de type getFileFromPath

  • Tests unitaires
    On peut importer (qques modifications seront nécéssaires) la base des tests de saveFiles cozy-client-js (dans la cozy-konnector-libs)
    La couverture n'est pas totale -> Essayer de l'améliorer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant