greenwave

#438 Fix handling date/time format for fedora-messaging
Merged by gnaponie. Opened by lholecek.
lholecek/greenwave fix-resultsdb-fedora-messaging  into  master

Download 438.patch

Fixes: https://pagure.io/fedora-infrastructure/issue/7818

Signed-off-by: Lukas Holecek hluk@email.cz

:+1:

For the test: can we just use an existing one with "parameterized" with the other type of submit_time?
Beside that: +1!

For the test: can we just use an existing one with "parameterized" with the other type of submit_time?

The test is more about ensuring the real-world message and data from resultsdb work well and won't break in future.

I would rather keep this test. That said, I noticed that lot of tests use similar code, would be nice idea to abstract it to functions/fixtures (for future refactoring).

Sounds good! Let's keep it this way and maybe let's think about some refactoring in the future.

Commit 6fde4210 fixes this pull-request

Pull-Request has been merged by gnaponie

Pull-Request has been merged by gnaponie
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.