Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Interfaçage - OILHI] Envoi de dossier relevant du RSD #2112

Merged
merged 6 commits into from
Jan 18, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Jan 15, 2024

Ticket

#1977

Description

Envoi des dossiers du pas de calais relevant du RSD
Voir tableau de correspondance
https://confused-galette-6e0.notion.site/Interconnexion-OILHI-d7138d7ea19c43c58238ee2bcfa622e4

Workflow

image

Changements apportés

  • Ajout feature flipping
  • Ajout fixtures, mise à jour tests
  • Création message pour traitement asynchrone
  • Création de la factory créant le dossier

Pré-requis

Editer le fichier .env.local

FEATURE_OILHI_ENABLE=1 
ZAPIER_HOOK_URL=https://hooks.zapier.com/
ZAPIER_OILHI_TOKEN=xxxxxxxxx
ZAPIER_OILHI_USER_ID=xxxxxxxxxx
ZAPIER_OILHI_CREATE_AIRTABLE_RECORD_ZAP_ID=xxxxxx

Exécuter la commande pour mettre à jour le fichier .env.local.php

composer dump-env dev 

Démarrer le worker

make worker-start
make mock

Tests

  • Dossier nouveau formulaire
  • Dossier formulaire actuelle

Tests de non régression

Désaffecter les partenaires de la fiche ci joint

@sfinx13 sfinx13 marked this pull request as draft January 15, 2024 08:30
@sfinx13 sfinx13 changed the title [Interfaçage - OILHI] Envoi de données vers Oilhi [Interfaçage - OILHI] Envoi de données Jan 15, 2024
@sfinx13 sfinx13 changed the title [Interfaçage - OILHI] Envoi de données [Interfaçage - OILHI] Envoi de dossier relevant du RSD Jan 15, 2024
@sfinx13 sfinx13 marked this pull request as ready for review January 15, 2024 17:33
@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented Jan 16, 2024

Affecter le partenaire 62 à un dossier du pas de calais relevant du RSD http://localhost:8080/bo/signalements/00000000-0000-0000-2024-000000000001 et vérifier que l'appel soit bien enregistré dans la table job_event : KO, rien dans la table job_event

Affecter le partenaire 62 à un dossier du pas de calais relevant du RSD http://localhost:8080/bo/signalements/00000000-0000-0000-2024-000000000002 et vérifier que l'appel soit bien enregistré dans la table job_event : OK

Affecter un dossier relevant du RSD d'un autre territoire et vérifier que l'appel vers OILHI ne se fait pas : OK

Désactiver la feature via la variable d'environnement FEATURE_OILHI_ENABLE et vérifier que les appels ne se font plus : OK

Vérifier que les messages correspondent bien à la table de correspondance
https://confused-galette-6e0.notion.site/Interconnexion-OILHI-d7138d7ea19c43c58238ee2bcfa622e4 : OK

@hmeneuvrier
Copy link
Collaborator

Affecter le partenaire 62 à un dossier du pas de calais relevant du RSD http://localhost:8080/bo/signalements/00000000-0000-0000-2024-000000000001 et vérifier que l'appel soit bien enregistré dans la table job_event : KO, rien dans la table job_event

En refaisant le test, ça a fonctionné... je ne sais pas ce qui j'ai oublié de faire lors de mes premiers essais.

@sfinx13
Copy link
Collaborator Author

sfinx13 commented Jan 16, 2024

Affecter le partenaire 62 à un dossier du pas de calais relevant du RSD http://localhost:8080/bo/signalements/00000000-0000-0000-2024-000000000001 et vérifier que l'appel soit bien enregistré dans la table job_event : KO, rien dans la table job_event

En refaisant le test, ça a fonctionné... je ne sais pas ce qui j'ai oublié de faire lors de mes premiers essais.

OK Cool mais aucune idée du pourquoi

src/DataFixtures/Files/NewSignalement.yml Show resolved Hide resolved
src/DataFixtures/Files/Partner.yml Show resolved Hide resolved
src/DataFixtures/Files/Partner.yml Show resolved Hide resolved
src/DataFixtures/Files/Territory.yml Show resolved Hide resolved
src/Controller/Back/AffectationController.php Outdated Show resolved Hide resolved
src/Factory/Oilhi/DossierMessageFactory.php Outdated Show resolved Hide resolved
src/Factory/Oilhi/DossierMessageFactory.php Outdated Show resolved Hide resolved
src/Factory/Oilhi/DossierMessageFactory.php Outdated Show resolved Hide resolved
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests OK, j'ai mis deux mini remarques

src/Entity/Signalement.php Outdated Show resolved Hide resolved
src/Entity/Signalement.php Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Jan 17, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

3 New issues
0 Security Hotspots
No data about Coverage
0.1% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-relecture OK, tests OK, TNR OK

@hmeneuvrier hmeneuvrier merged commit d7f3f2d into develop Jan 18, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/1977-workflow-oilhi branch January 30, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants