python:samba: Raise an exception if we can’t parse a Markdown operation

Otherwise we would continue with the wrong GUID and filename.

Signed-off-by: Jennifer Sutton <jennifersutton@catalyst.net.nz>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

diff --git a/python/samba/ms_forest_updates_markdown.py b/python/samba/ms_forest_updates_markdown.py
index 0a0d2111a7211f01a31289d975f972b75ba94506..12a2041459acb8a07b6db7e74c7070d7836a1cb3 100644 (file)
--- a/python/samba/ms_forest_updates_markdown.py
+++ b/python/samba/ms_forest_updates_markdown.py
@@ -253,10 +253,11 @@ def read_ms_markdown(in_file, out_folder=None, out_dict=None):
         if updates:
             for update in updates[2:]:
                 output = re.match(r'Operation (\d+): {(.*)}', update[0])
-                if output:
-                    # print output.group(1), output.group(2)
-                    guid = output.group(2)
-                    filename = "%s-{%s}.ldif" % (output.group(1).zfill(4), guid)
+                if not output:
+                    raise Exception(update)
+
+                guid = output.group(2)
+                filename = "%s-{%s}.ldif" % (output.group(1).zfill(4), guid)
 
                 found = False