RDBC-747 FacetBase.to_json doesn't care if there are options or not provided

Author: poissoncorp

ravendb/documents/queries/facets/definitions.py

@@ -44,14 +44,16 @@ def to_facet_token(self, add_query_parameter: Callable[[object], str]) -> FacetT
         pass
 
     def to_json(self) -> Dict:
-        return {
+        json_dict = {
             "DisplayFieldName": self.display_field_name,
-            "Options": self.options.to_json(),
             "Aggregations": {
                 aggregation.to_json(): {aggregation_field.to_json() for aggregation_field in aggregation_fields_set}
                 for aggregation, aggregation_fields_set in self.aggregations.items()
             },
         }
+        if self.options:
+            json_dict["Options"] = self.options.to_json()
+        return json_dict
 
     @classmethod
     def from_json(cls, json_dict: Dict) -> FacetBase:

ravendb/documents/session/document_session.py

@@ -19,7 +19,8 @@
 from ravendb.documents.indexes.definitions import AbstractCommonApiForIndexes
 from ravendb.documents.operations.attachments import (
     GetAttachmentOperation,
-    AttachmentName, CloseableAttachmentResult,
+    AttachmentName,
+    CloseableAttachmentResult,
 )
 from ravendb.documents.operations.batch import BatchOperation
 from ravendb.documents.operations.executor import OperationExecutor, SessionOperationExecutor