konnectors/libs

[ccc_libs] Amélioration saveFiles et autres fonctions

Opened this issue · 0 comments

LucsT commented

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