From 112ac906dcae2c2af798750b7b993f6e1b2fac22 Mon Sep 17 00:00:00 2001 From: Jennifer Richards Date: Thu, 10 Aug 2023 16:01:34 -0300 Subject: [PATCH] test: Confirm changes are returned for draft and rfc --- ietf/sync/tests.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/ietf/sync/tests.py b/ietf/sync/tests.py index 8ca6362c7..345bf6eb6 100644 --- a/ietf/sync/tests.py +++ b/ietf/sync/tests.py @@ -347,11 +347,10 @@ class RFCSyncTests(TestCase): draft_abstract_before = draft_doc.abstract draft_pages_before = draft_doc.pages changes = [] - for cs, d, rfc_published in rfceditor.update_docs_from_rfc_index(data, errata, today - datetime.timedelta(days=30)): - changes.append(cs) + for _, d, rfc_published in rfceditor.update_docs_from_rfc_index(data, errata, today - datetime.timedelta(days=30)): + changes.append({"doc_pk": d.pk, "rfc_published": rfc_published}) # we ignore the actual change list draft_doc = Document.objects.get(name=draft_doc.name) - draft_events = draft_doc.docevent_set.all() self.assertEqual(len(draft_events) - event_count_before, 2) self.assertEqual(draft_events[0].type, "sync_from_rfc_editor") @@ -385,6 +384,13 @@ class RFCSyncTests(TestCase): self.assertEqual(rfc_doc.std_level_id, "ps") self.assertEqual(rfc_doc.pages, 42) + # check that we got the expected changes + self.assertEqual(len(changes), 2) + self.assertEqual(changes[0]["doc_pk"], draft_doc.pk) + self.assertEqual(changes[0]["rfc_published"], False) + self.assertEqual(changes[1]["doc_pk"], rfc_doc.pk) + self.assertEqual(changes[1]["rfc_published"], True) + # make sure we can apply it again with no changes changed = list(rfceditor.update_docs_from_rfc_index(data, errata, today - datetime.timedelta(days=30))) self.assertEqual(len(changed), 0)