-
Notifications
You must be signed in to change notification settings - Fork 1
Signalement adresse transmise à l'IGN #1555
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1555 +/- ##
============================================
+ Coverage 97.27% 97.29% +0.02%
- Complexity 2579 2606 +27
============================================
Files 508 509 +1
Lines 10933 11032 +99
============================================
+ Hits 10635 10734 +99
Misses 298 298 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Est-ce que tu peux rajouter les tests manquants dans ton handler et ton IGNClient pour remonter le coverage ? 🙏 |
494925e to
65c8f80
Compare
65c8f80 to
5a240e7
Compare
tests/Unit/Application/User/Command/SaveReportAddressCommandHandlerTest.php
Outdated
Show resolved
Hide resolved
tests/Unit/Application/User/Command/SaveReportAddressCommandHandlerTest.php
Outdated
Show resolved
Hide resolved
3b2ea2b to
d6fc6db
Compare
d6fc6db to
9513219
Compare
mmarchois
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super taf, j'ai juste quelques petites suggestions
src/Application/User/Command/SaveReportAddressCommandHandler.php
Outdated
Show resolved
Hide resolved
c303169 to
52c455c
Compare
https://espacecollaboratif.ign.fr/gcms/api/doc/#generalites
https://espacecollaboratif.ign.fr/profile/