None data fix (cloudevents#78)

cumason123 · web-flow · commit c70412081cbb · 2020-07-28T09:35:26.000-07:00
* fixed none data issue

Signed-off-by: Curtis Mason &lt;cumason@google.com&gt;

* added none data test for marshalling

Signed-off-by: Curtis Mason &lt;cumason@google.com&gt;

* lint fix

Signed-off-by: Curtis Mason &lt;cumason@google.com&gt;
diff --git a/cloudevents/sdk/http/event.py b/cloudevents/sdk/http/event.py
@@ -24,7 +24,7 @@
 
 
 def default_marshaller(content: any):
-    if len(content) == 0:
+    if content is None or len(content) == 0:
         return None
     try:
         return json.dumps(content)
diff --git a/cloudevents/tests/test_http_events.py b/cloudevents/tests/test_http_events.py
@@ -401,3 +401,16 @@ def test_cloudevent_repr(specversion):
     # we had issues in the past where event.__repr__() could run but
     # print(event) would fail.
     print(event)
+
+
+@pytest.mark.parametrize("specversion", ["1.0", "0.3"])
+def test_none_data_cloudevent(specversion):
+    event = CloudEvent(
+        {
+            "source": "<my-url>",
+            "type": "issue.example",
+            "specversion": specversion,
+        }
+    )
+    to_binary_http(event)
+    to_structured_http(event)
