chore: add test on TruncateMessageCommand #180 - #586
Open
guillaumedc wants to merge 2 commits into
Open
guillaumedc wants to merge 2 commits into
guillaumedc wants to merge 2 commits into
Conversation
Test Results110 tests 110 ✅ 22s ⏱️ Results for commit f0093d9. ♻️ This comment has been updated with latest results. |
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
August 26, 2026 14:18
602a4e1 to
f2f66c1
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
August 26, 2026 14:44
1a38b5a to
b75352e
Compare
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
August 28, 2026 14:34
c65fbd8 to
9a29aa7
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
August 28, 2026 14:35
b75352e to
d3d99d4
Compare
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
August 28, 2026 15:34
9a29aa7 to
2466d6b
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
August 28, 2026 15:34
d3d99d4 to
195f415
Compare
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
August 28, 2026 15:56
2466d6b to
2865288
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
August 28, 2026 15:56
195f415 to
78efad6
Compare
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
September 4, 2026 07:17
2865288 to
cb65447
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
September 4, 2026 07:18
78efad6 to
cd79b33
Compare
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
September 7, 2026 07:53
cb65447 to
473c554
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
September 7, 2026 08:20
cd79b33 to
b214462
Compare
leandrg
reviewed
Sep 7, 2026
| @@ -0,0 +1,68 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Cette review ajoute une entité Doctrine et modifie donc du code de production. Je serais d'avis soit d'extraire ces changements dans une PR dédiée, puisque celle-ci est centrée sur les tests, soit d'ajouter des tests autour de ce mapping, notamment sur la relation entre OutMessage et OutQuarantine, ainsi que sur la suppression des quarantaines associées.
guillaumedc
force-pushed
the
chore/report-mail-tests-#180
branch
from
September 8, 2026 15:25
473c554 to
3df21f6
Compare
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
September 14, 2026 12:14
b214462 to
02e737c
Compare
guillaumedc
changed the base branch from
chore/report-mail-tests-#180
to
main
September 14, 2026 12:15
guillaumedc
force-pushed
the
chore/truncate-message-tests-#180
branch
from
September 14, 2026 12:17
02e737c to
9e6366d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related issue(s)
#180
How to test manually
make testReviewer checklist