Tech(perf): frozen string literals partout

[colinux]

Maintenant qu'on connait notre bonne couverture de tests, on peut être confiant sur ce genre de changements (+ canary release).

Rappels:

• ne concerne que les string définies dans des fichiers .rb (donc pas venant de la db ou autre)
• ne concerne pas les views
• n'impacte pas les string qui contiennent une interpolation
• + devant un string le définit comme mutable

Cas où des strings doivent rester mutables :

• app/models/concerns/tags_substitution_concern.rb: parser de tags (parsby modifie l'IO)
• dans quelques tests , soit par construction du test, soit parce-qu'on y hardcode un literal qui doit être traité comme une variable (par exemple: param d'une requête)

Dans un commit isolé j'ai supprimé du code dans Dolist::API à propos de l'envoi d'email avec des attachments, car :

• ce code est inutilisé et on ne rentrait jamais dedans , car notre compte dolist ne permet pas l'envoi de PJ
• ce code contenait un string qui était mutable
  J'en ai profité pour rajouter quelques tests sur cette classe qui n'en avait aucun.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.34%. Comparing base (1750641) to head (d5baa2d).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10386      +/-   ##
==========================================
- Coverage   80.79%   79.34%   -1.46%     
==========================================
  Files        1193     1058     -135     
  Lines       25139    23625    -1514     
  Branches     4526     4218     -308     
==========================================
- Hits        20312    18745    -1567     
- Misses       4827     4880      +53     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.