use dictionary rather than list for json

Author: smoors

lib/vsc/utils/zabbix.py

@@ -44,7 +44,7 @@ class ZabbixReporter(NagiosReporter):
 
     def print_report_and_exit(self, timestamp, nagios_exit_code, nagios_exit_string, nagios_message):
         """Print the zabbix report and exit"""
-        print('[%f, "%s", %s]' % (timestamp, nagios_exit_string, nagios_message))
+        print('{"timestamp": %f, "exit_string": "%s", "message": %s}' % (timestamp, nagios_exit_string, nagios_message))
         self.log.info("Zabbix check cache file %s contents delivered: %s", self.filename, nagios_message)
         sys.exit(nagios_exit_code)
 

test/zabbix.py

@@ -87,7 +87,7 @@ def test_cache(self):
             buffer.close()
             self.assertTrue(err.code == nagios_exit[0])
             line = json.loads(line)
-            self.assertTrue(line[1] == nagios_exit[1])
-            self.assertTrue(line[2][0] == json.loads(message)[0])
+            self.assertTrue(line["exit_string"] == nagios_exit[1])
+            self.assertTrue(line["message"][0] == json.loads(message)[0])
 
         os.unlink(filename)