Fix tests

Author: Sushisource

temporalio/client.py

@@ -3461,8 +3461,16 @@ def __init__(
                 if pair.key.name in self.untyped_search_attributes:
                     # We know this is mutable here
                     del self.untyped_search_attributes[pair.key.name]  # type: ignore
-            self.static_summary = raw_info.user_metadata.summary
-            self.static_details = raw_info.user_metadata.details
+            self.static_summary = (
+                raw_info.user_metadata.summary
+                if raw_info.HasField("user_metadata") and raw_info.user_metadata.summary
+                else None
+            )
+            self.static_details = (
+                raw_info.user_metadata.details
+                if raw_info.HasField("user_metadata") and raw_info.user_metadata.details
+                else None
+            )
         else:
             if not id:
                 raise ValueError("ID required")

tests/test_client.py

@@ -780,8 +780,6 @@ async def test_schedule_basics(
         pytest.skip("Older proto library cannot compare repeated fields")
     await assert_no_schedules(client)
 
-    # TODO: Metadata
-
     # Create a schedule with a lot of stuff
     schedule = Schedule(
         action=ScheduleActionStartWorkflow(
@@ -794,6 +792,8 @@ async def test_schedule_basics(
             task_timeout=timedelta(hours=3),
             retry_policy=RetryPolicy(maximum_attempts=20),
             memo={"memokey1": "memoval1"},
+            static_summary="summary",
+            static_details="details",
         ),
         spec=ScheduleSpec(
             calendars=[
@@ -864,6 +864,15 @@ async def test_schedule_basics(
             day_of_week=(ScheduleRange(1),),
         )
     )
+    # Summary & description are encoded
+    assert schedule.action.static_summary
+    assert schedule.action.static_details
+    schedule.action.static_summary = (
+        await DataConverter.default.encode([schedule.action.static_summary])
+    )[0]
+    schedule.action.static_details = (
+        await DataConverter.default.encode([schedule.action.static_details])
+    )[0]
 
     # Describe it and confirm
     desc = await handle.describe()

tests/worker/test_workflow.py

@@ -44,9 +44,9 @@
 from temporalio.api.failure.v1 import Failure
 from temporalio.api.sdk.v1 import EnhancedStackTrace
 from temporalio.api.workflowservice.v1 import (
+    DescribeWorkflowExecutionRequest,
     GetWorkflowExecutionHistoryRequest,
     ResetStickyTaskQueueRequest,
-    DescribeWorkflowExecutionRequest
 )
 from temporalio.bridge.proto.workflow_activation import WorkflowActivation
 from temporalio.bridge.proto.workflow_completion import WorkflowActivationCompletion
@@ -505,7 +505,7 @@ async def test_workflow_signal_and_query_errors(client: Client):
             await handle.query("non-existent query")
         assert str(rpc_err.value) == (
             "Query handler for 'non-existent query' expected but not found,"
-            " known queries: [__enhanced_stack_trace __stack_trace bad_query other_query]"
+            " known queries: [__enhanced_stack_trace __stack_trace __temporal_workflow_metadata bad_query other_query]"
         )