Affichage des courbes de capteurs sur le dashboard

J’ai la réponse à cette énigme, c’est tout simple: -1 est censé représenter la valeur « illimitée », c’est ce qui est défini dans le code du signup lorsque tu clique sur le radio button « illimitée ».

Le souci, c’est qu’il y avait un double bug dans le code et dans les tests qui font que cette valeur -1 n’a jamais fonctionnée…

Ce qu’il s’est passé:

  • « -1 » s’est comportée comme si tu voulais garder que les états plus vieux que « dans 1 jour dans le futur » (NOW - (-1) = DEMAIN) :scream: Il n’y avait pas de if(-1) pour bloquer la purge des états, je pense j’ai du oublier en le codant. Comme on utilisait pas encore les valeurs de capteurs passée, ça s’est pas vu. Ce qui m’étonne c’est qu’il me semble que @Terdious a bien un historique illimité sur son instance. Tu confirme @Terdious ? Je pense tu dois avoir supprimé cette variable volontairement en DB non ?
  • Dans les tests, erreur assez bête, j’ai utilisé fake.resolves() pour mocker l’appel à variable.getValue(), sauf que j’ai mal utilisé fake.resolves() et du coup quand j’ai du faire mes tests à l’époque, le -1 se comportait comme voulu, car en gros je me retrouvais avec une fonction retournée au lieu de -1, et donc ça empêchait la purge, bref un truc très bête (c’était au tout début de la v4 il y a facile 1 an et demi je pense :D)

J’ai corrigé le bug et j’ai rajouté des tests plus solides pour s’assurer que ça bouge pas.

Dans l’UI, il y a maintenant la possibilité de modifier la valeur dans les paramètres: