Merge 69a6641 into 0096714

docs/changes.rst

@@ -123,6 +123,11 @@ Released: not yet
   variables. They are no longer imported into the `pywbem` namespace.
   See issue #1302.
 
+* Fixed issue where `CIMInstance.tomof()` raised `IndexError` when it
+  contains an array property whose value is the empty array.
+  Added an according testcase.
+  See issue #1312.
+
 **Enhancements:**
 
 * Extend pywbem MOF compiler to search for dependent classes including:

pywbem/cim_obj.py

@@ -4782,7 +4782,8 @@ def tomof(
                 val_str, line_pos = _value_tomof(
                     self.value, self.type, indent + MOF_INDENT, maxline,
                     line_pos + 1, 1, True)
-                if val_str[0] != '\n':
+                # Empty arrays are represented as val_str=''
+                if val_str and val_str[0] != '\n':
                     # The extra space was actually needed
                     mof.append(u' ')
                 else:

testsuite/test_cim_obj.py

@@ -12062,6 +12062,19 @@ class Test_CIMProperty_tomof(object):
             P1 = 42;\n""",
             None, True
         ),
+        (
+            "instance uint32 array property, empty array",
+            CIMProperty(
+                name='P1',
+                value=list(),
+                type='uint32',
+                is_array=True,
+            ),
+            True, 12,
+            u"""\
+            P1 = { };\n""",
+            None, True
+        ),
         (
             "instance uint32 array property, single-line",
             CIMProperty(