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

@@ -13,7 +13,8 @@ from force_bdss.io.workflow_reader import WorkflowReader
 from force_bdss.tests.dummy_classes.factory_registry_plugin import \
     DummyFactoryRegistryPlugin
 
-from force_bdss.io.workflow_writer import WorkflowWriter, traits_to_dict
+from force_bdss.io.workflow_writer import WorkflowWriter, traits_to_dict,\
+    pop_traits
 from force_bdss.core.workflow import Workflow
 
 
@@ -87,3 +88,14 @@ class TestWorkflowWriter(unittest.TestCase):
         mock_traits.__getstate__ = mock.Mock(return_value={"foo": "bar"})
 
         self.assertEqual(traits_to_dict(mock_traits), {"foo": "bar"})
+
+    def test_pop_traits_version(self):
+
+        test_dictionary = {'Entry1': {'Entry1-1': 4, '__traits_version__':67},
+                           'Entry2': [3, 'a', {'Entry2-1': 5,
+                                               '__traits_version__': 9001}],
+                           '__traits_version__': 13}
+        result_dictionary = {'Entry1': {'Entry1-1': 4, },
+                             'Entry2': [3, 'a', {'Entry2-1': 5, }], }
+        traitless_dictionary = pop_traits(test_dictionary)
+        self.assertEqual(traitless_dictionary,result_dictionary)