You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
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
The text was updated successfully, but these errors were encountered: