Removed __traits_version__ from json files

Created by: jjenthought

The workflow is a dictionary which contains lists of dictionaries, so only the traits_version keys in the top level dict were being removed. This changed the function so it should do it recursively. It seems fine but the only test is a on a single hardcoded example at the minute. Refs. issue https://github.com/force-h2020/force-bdss/issues/134

Fixes #134 (closed)

force_bdss/io/tests/test_workflow_writer.py

@@ -16,6 +16,7 @@ from force_bdss.tests.dummy_classes.factory_registry_plugin import \
 from force_bdss.io.workflow_writer import WorkflowWriter, traits_to_dict,\
     pop_recursive
 from force_bdss.core.workflow import Workflow
+from force_bdss.core.input_slot_info import InputSlotInfo
 
 
 class TestWorkflowWriter(unittest.TestCase):
@@ -89,7 +90,18 @@ class TestWorkflowWriter(unittest.TestCase):
 
         self.assertEqual(traits_to_dict(mock_traits), {"foo": "bar"})
 
-    def test_pop_recursive(self):
+    def test_traits_to_dict(self):
+
+        wfwriter = WorkflowWriter()
+        wf = self._create_workflow()
+        exec_layer = wf.execution_layers[0]
+        exec_layer.data_sources[0].input_slot_info = [InputSlotInfo()]
+        slotdata = exec_layer.data_sources[0].input_slot_info[0].__getstate__()
+        self.assertTrue("__traits_version__" in slotdata)
+        # Calls traits_to_dict for each data source
+        datastore_list = wfwriter._execution_layer_data(exec_layer)
+        new_slotdata = datastore_list[0]['model_data']['input_slot_info']
+        self.assertTrue("__traits_version__" not in new_slotdata)
 
         test_dictionary = {'K1': {'K1': 'V1', 'K2': 'V2', 'K3': 'V3'},
                            'K2': ['V1', 'V2', {'K1': 'V1', 'K2': 'V2',