Merge pull request coinbase#72 from stripe-private-oss-forks/calum-fixing-list-workflows-bug

Added fix for nil search attributes when listing workflows

Author: calum-stripe

lib/temporal/workflow/execution_info.rb

@@ -17,6 +17,7 @@ class ExecutionInfo < Struct.new(:workflow, :workflow_id, :run_id, :start_time,
       ]
 
       def self.generate_from(response)
+        search_attributes = self.from_payload_map(response.search_attributes.indexed_fields) if !response.search_attributes.nil?
         new(
           workflow: response.type.name,
           workflow_id: response.execution.workflow_id,
@@ -26,7 +27,7 @@ def self.generate_from(response)
           status: Temporal::Workflow::Status::API_STATUS_MAP.fetch(response.status),
           history_length: response.history_length,
           memo: self.from_payload_map(response.memo.fields),
-          search_attributes: self.from_payload_map(response.search_attributes.indexed_fields),
+          search_attributes: search_attributes,
         ).freeze
       end
 

spec/unit/lib/temporal/workflow/execution_info_spec.rb

@@ -20,6 +20,13 @@
     it 'freezes the info' do
       expect(subject).to be_frozen
     end
+
+    it 'deserializes if search_attributes is nil' do
+      api_info.search_attributes = nil
+
+      result = described_class.generate_from(api_info)
+      expect(result.search_attributes).to be_nil
+    end
   end
 
   describe 'statuses' do