Fixing __doc__ and __name__ storage in Transportable Object (#1112)

* Fixing  and  storage in Transportable Object

* Fixing  and  storage in Transportable Object

* Added test

CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [UNRELEASED]
 
+### Fixed
+
+- Function's `__doc__` and `__name__` storage in dict/json for transportable object fixed.
+
+### Tests
+
+- Added unit test for the above fix.
+
 ## [0.184.0] - 2022-08-22
 
 ### Authors

covalent/_workflow/transport.py

@@ -56,7 +56,7 @@ def __init__(self, obj: Any) -> None:
         except TypeError as ex:
             self._json = ""
 
-        self.attrs = {"doc": getattr(obj, "doc", ""), "name": getattr(obj, "name", "")}
+        self.attrs = {"doc": getattr(obj, "__doc__", ""), "name": getattr(obj, "__name__", "")}
 
     def __eq__(self, obj) -> bool:
         if not isinstance(obj, TransportableObject):

tests/covalent_tests/workflow/transport_test.py

@@ -39,7 +39,7 @@ def subtask(x):
 
 
 def subtask_2(x):
-    """Workflow subtask."""
+    """Workflow subtask 2."""
 
     return x
 
@@ -133,6 +133,15 @@ def test_transportable_object_from_dict(transportable_object):
     assert to == to_new
 
 
+def test_transportable_object_to_dict_attributes(transportable_object):
+    """Test attributes from `to_dict` contain correct name and docstrings"""
+
+    tr_dict = transportable_object.to_dict()
+
+    assert tr_dict["attributes"]["attrs"]["doc"] == subtask.__doc__
+    assert tr_dict["attributes"]["attrs"]["name"] == subtask.__name__
+
+
 def test_transportable_object_serialize_to_json(transportable_object):
     import json