search

etalab / rncs_worker_api_entreprise

API pour récupérer et mettre à disposition les données du Répertoire National du Commerce et des Société
https://entreprise.data.gouv.fr
MIT License
23 stars 13 forks source link

[TITMC] indexes #76

Closed Haelle closed 5 years ago

Haelle commented 5 years ago

Que fait cette PR

Ajout de l'étape de suppression/création des indexes pour les TITMC selon la même logique que pour les TC. Le fichier de conf a été séparé en 2 parties, la configuration pour la table commune DossierEntreprise est dupliqué volontairement.

Commentaires

Suppression des tests sur le bon fonctionnement des drop/create indexes. Actuellement c'était expect(Postgres).to receive(:execute).exactly(X).times et X c'était nombre d'indexes + nombre opération de setup de BDD pour RSpec ce type de tests n'est pas vraiment généralisable, car le nombre d'opération en BDD de setup de RSpec varie d'un test à l'autre (6 ou 8 opérations) Du coup j'ai abandonné cette idée.

Comment faire la review

Commits par commits, mais ça se fait bien aussi d'un coup