Simplifier Announcement test_start_date_conflict_constraint #4717
Conversation
francoisfreitag
added
the
no-changelog
…traint The first existing_campaign is unused in the test and looks like an oversight.
francoisfreitag
force-pushed
the
ff/flakyx
branch
from
0d885b9 to
26ec374
Compare
francoisfreitag
changed the title
|
J’ai ouvert la PR suite au problème de flakiness, mais Xavier l’avait déjà corrigé avec d98c011. Comme nous avons eu une solution différente, je propose ma solution, plus simple. |
|
Je m'étais posé la question. |
| existing_campaign = AnnouncementCampaignFactory(start_date=date(2023, 1, 1)) | ||
|
|
||
| # can modify existing value without triggering constraint | ||
| existing_campaign = AnnouncementCampaignFactory(start_date=date(2024, 1, 1)) | ||
| existing_campaign.start_date = date(2024, 2, 1) |
There was a problem hiding this comment.
Hmm oui tout à fait ... désolé je pense il m'a loupé quand j'ai changé le comportement défaut du start_date de AnnouncementCampaignFactory ou pendant le rebase
Tiré par les cheveux et ce n’est plus le cas, les tests des vues de la page news génèrent plusieurs campagnes, comme les tests du cache de campagne. Je pense qu’on peut se passer de ce test maintenant. |
The first existing_campaign is unused in the test and looks like an oversight.