Minor changes to modinput tests, plus a critical fix to the data_open function

Author: Shakeel Mohamed

splunklib/modularinput/event_writer.py

@@ -76,7 +76,7 @@ def write_xml_document(self, document):
 
         :param document: An ``ElementTree`` object.
         """
-        self._out.write(ET.tostring(document, "utf-8", "xml"))
+        self._out.write(ET.tostring(document))
         self._out.flush()
 
     def close(self):

tests/modularinput/modularinput_testlib.py

@@ -26,4 +26,4 @@
 from splunklib.modularinput.utils import xml_compare, parse_xml_data, parse_parameters
 
 def data_open(filepath):
-    return os.path.join(os.path.dirname(os.path.abspath(__file__)), filepath)
+    return open(os.path.join(os.path.dirname(os.path.abspath(__file__)), filepath))

tests/modularinput/test_script.py

@@ -29,7 +29,7 @@
 except ImportError:
     import xml.etree.ElementTree as ET
 
-TEST_SCRIPT_PATH = "__IGNORED_SCRIPT_PATH__" #TODO: replace empty first args...
+TEST_SCRIPT_PATH = "__IGNORED_SCRIPT_PATH__"
 
 class ScriptTest(unittest.TestCase):