fix: add celery tasks for idnits2 and bibxml file generation#7204
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7204 +/- ##
==========================================
- Coverage 88.98% 88.91% -0.08%
==========================================
Files 291 291
Lines 40717 41137 +420
==========================================
+ Hits 36233 36577 +344
- Misses 4484 4560 +76 ☔ View full report in Codecov by Sentry. |
ietf/doc/utils.py
Outdated
| write_bibxml_file_if_changed(ref_rev_file_name, bibxml) | ||
| except Exception as ee: | ||
| log.log('\n%s-%s: %s\n' % (e.doc.name, e.doc.rev, str(ee))) | ||
| raise ee |
There was a problem hiding this comment.
Here and in bibxml_for_recent_drafts(), the re-raise of the exception is a change from the management command. In those, the exception was written to stderr and the script continued. The change means that the task will stop after an error.
I think we want to keep the old behavior (i.e., remove the raise ee here and on line 1299). I hope this doesn't contradict something we discussed at the code sprint. @russhousley do you recall?
There was a problem hiding this comment.
Sounds fine to me.
ietf/doc/tests_tasks.py
Outdated
| content = f.read() | ||
| self.assertIn(draft.title, content) | ||
|
|
||
| def test_generate_bibxml_files_for_recent_drafts_task_with_bad_vakue(self): |
There was a problem hiding this comment.
Goof catch. I read past it many times.
Fix typo: bad_vakue -> bad_value
Don't raise error inbibxml_for_all_drafts
NGPixel
changed the title
Add celery tasks for idnits2 and bibxml file generation. Fixes #7192