Remove Optional on Query Name (#33)

Remove optional on Query.name field (was unnecessary and annoying - since it has a default value). Also closes #30 (adding is_complete to ExecutionState).

Author: bh2smith

dune_client/models.py

@@ -51,6 +51,10 @@ def terminal_states(cls) -> set[ExecutionState]:
         """
         return {cls.COMPLETED, cls.CANCELLED, cls.FAILED}
 
+    def is_complete(self) -> bool:
+        """Returns True is state is completed, otherwise False."""
+        return self == ExecutionState.COMPLETED
+
 
 @dataclass
 class ExecutionResponse:

dune_client/query.py

@@ -13,7 +13,7 @@ class Query:
     """Basic data structure constituting a Dune Analytics Query."""
 
     query_id: int
-    name: Optional[str] = "unnamed"
+    name: str = "unnamed"
     params: Optional[List[QueryParameter]] = None
 
     def base_url(self) -> str: