launchpad-reviewers team mailing list archive

[Enrique Sánchez <mp+484942@xxxxxxxxxxxxxxxxxx>]

Enrique Sánchez has proposed merging ~enriqueesanchz/launchpad:add-priority-explanation into launchpad:master.

Commit message:
Fix CVE-2023-32637 Mitigation is an optional field

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~enriqueesanchz/launchpad/+git/launchpad/+merge/484942
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~enriqueesanchz/launchpad:add-priority-explanation into launchpad:master.

diff --git a/lib/lp/bugs/scripts/tests/sampledata/CVE-2023-32637 b/lib/lp/bugs/scripts/tests/sampledata/CVE-2023-32637
index 161dc4d..8b88352 100644
--- a/lib/lp/bugs/scripts/tests/sampledata/CVE-2023-32637
+++ b/lib/lp/bugs/scripts/tests/sampledata/CVE-2023-32637
@@ -12,7 +12,6 @@ Description:
 Ubuntu-Description:
 Notes:
  ccdm94> this has likely been fixed in all 2.x versions.
-Mitigation:
 Bugs:
 Priority: high
  This has a high priority because it is a vulnerability that allows a remote
diff --git a/lib/lp/bugs/scripts/tests/test_uct.py b/lib/lp/bugs/scripts/tests/test_uct.py
index 66f3b71..8ac253a 100644
--- a/lib/lp/bugs/scripts/tests/test_uct.py
+++ b/lib/lp/bugs/scripts/tests/test_uct.py
@@ -200,7 +200,7 @@ class TestUCTRecord(TestCase):
                     "code on the server."
                 ),
                 discovered_by="",
-                mitigation="",
+                mitigation=None,
                 notes=(
                     "ccdm94> this has likely been fixed in all 2.x "
                     "versions."