Fix naming error

Author: Sushisource

temporalio/lib/temporalio/priority.rb

@@ -6,7 +6,7 @@ module Temporalio
   Priority = Data.define(
     :priority_key,
     :fairness_key,
-    :weight
+    :fairness_weight
   )
 
   # Priority contains metadata that controls relative ordering of task processing when tasks are
@@ -44,13 +44,13 @@ module Temporalio
   #
   #     Fairness keys are limited to 64 bytes.
   #
-  # @!attribute weight
+  # @!attribute fairness_weight
   #   @return [Float, nil] Weight for a task can come from multiple sources for
   #     flexibility. From highest to lowest precedence:
   #     1. Weights for a small set of keys can be overridden in task queue
   #        configuration with an API.
   #     2. It can be attached to the workflow/activity in this field.
-  #     3. The default weight of 1.0 will be used.
+  #     3. The default fairness_weight of 1.0 will be used.
   #
   #     Weight values are clamped to the range [0.001, 1000].
   class Priority
@@ -61,15 +61,15 @@ def self._from_proto(priority)
       new(
         priority_key: priority.priority_key.zero? ? nil : priority.priority_key,
         fairness_key: priority.fairness_key.empty? ? nil : priority.fairness_key,
-        weight: priority.fairness_weight.zero? ? nil : priority.fairness_weight
+        fairness_weight: priority.fairness_weight.zero? ? nil : priority.fairness_weight
       )
     end
 
     # The default priority instance.
     #
     # @return [Priority] The default priority
     def self.default
-      @default ||= new(priority_key: nil, fairness_key: nil, weight: nil)
+      @default ||= new(priority_key: nil, fairness_key: nil, fairness_weight: nil)
     end
 
     # @!visibility private
@@ -79,13 +79,13 @@ def _to_proto
       Temporalio::Api::Common::V1::Priority.new(
         priority_key: priority_key || 0,
         fairness_key: fairness_key || '',
-        fairness_weight: weight || 0.0
+        fairness_weight: fairness_weight || 0.0
       )
     end
 
     # @return [Boolean] True if this priority is empty/default
     def empty?
-      priority_key.nil? && fairness_key.nil? && weight.nil?
+      priority_key.nil? && fairness_key.nil? && fairness_weight.nil?
     end
   end
 end

temporalio/sig/temporalio/priority.rbs

@@ -2,9 +2,9 @@ module Temporalio
   class Priority
     attr_reader priority_key: Integer?
     attr_reader fairness_key: String?
-    attr_reader weight: Float?
+    attr_reader fairness_weight: Float?
 
-    def initialize: (priority_key: Integer?, ?fairness_key: String?, ?weight: Float?) -> void
+    def initialize: (priority_key: Integer?, ?fairness_key: String?, ?fairness_weight: Float?) -> void
 
     def self._from_proto: (untyped) -> Priority
 

temporalio/test/worker_workflow_priority_test.rb

@@ -11,7 +11,7 @@ def execute(expected_priority, expected_fairness_key = nil, expected_weight = ni
       info = Temporalio::Activity::Context.current.info
       actual_priority = info.priority&.priority_key
       actual_fairness_key = info.priority&.fairness_key
-      actual_weight = info.priority&.weight
+      actual_weight = info.priority&.fairness_weight
       [actual_priority, expected_priority, actual_fairness_key, expected_fairness_key, actual_weight, expected_weight]
     end
   end
@@ -28,7 +28,7 @@ def execute(expected_priority, stop_after_check, expected_fairness_key = nil, ex
       info = Temporalio::Workflow.info
       actual_priority = info.priority&.priority_key
       actual_fairness_key = info.priority&.fairness_key
-      actual_weight = info.priority&.weight
+      actual_weight = info.priority&.fairness_weight
 
       raise "Expected priority #{expected_priority}, got #{actual_priority}" if actual_priority != expected_priority
       if actual_fairness_key != expected_fairness_key
@@ -45,7 +45,7 @@ def execute(expected_priority, stop_after_check, expected_fairness_key = nil, ex
         true,
         'tenant-a',
         2.0,
-        priority: Temporalio::Priority.new(priority_key: 4, fairness_key: 'tenant-a', weight: 2.0)
+        priority: Temporalio::Priority.new(priority_key: 4, fairness_key: 'tenant-a', fairness_weight: 2.0)
       )
       raise 'child workflow failed' unless child_result == 'Done!'
 
@@ -55,7 +55,7 @@ def execute(expected_priority, stop_after_check, expected_fairness_key = nil, ex
         true,
         'tenant-b',
         0.5,
-        priority: Temporalio::Priority.new(priority_key: 2, fairness_key: 'tenant-b', weight: 0.5)
+        priority: Temporalio::Priority.new(priority_key: 2, fairness_key: 'tenant-b', fairness_weight: 0.5)
       )
       result = handle.result
       raise 'child workflow failed' unless result == 'Done!'
@@ -66,7 +66,7 @@ def execute(expected_priority, stop_after_check, expected_fairness_key = nil, ex
         'high',
         1.5,
         start_to_close_timeout: 5,
-        priority: Temporalio::Priority.new(priority_key: 5, fairness_key: 'high', weight: 1.5)
+        priority: Temporalio::Priority.new(priority_key: 5, fairness_key: 'high', fairness_weight: 1.5)
       )
       raise "Activity expected priority #{act_expected}, got #{act_actual}" if act_actual != act_expected
       if act_fairness_actual != act_fairness_expected
@@ -84,7 +84,7 @@ def test_workflow_priority
     execute_workflow(
       WorkflowUsingPriorities, 1, false, 'tenant-main', 0.8,
       activities: [ActivityWithPriority],
-      priority: Temporalio::Priority.new(priority_key: 1, fairness_key: 'tenant-main', weight: 0.8)
+      priority: Temporalio::Priority.new(priority_key: 1, fairness_key: 'tenant-main', fairness_weight: 0.8)
     ) do |handle|
       assert_equal 'Done!', handle.result
 
@@ -128,7 +128,7 @@ def test_workflow_priority
     execute_workflow(
       WorkflowUsingPriorities, nil, true, 'low', 0.2,
       activities: [ActivityWithPriority],
-      priority: Temporalio::Priority.new(priority_key: nil, fairness_key: 'low', weight: 0.2)
+      priority: Temporalio::Priority.new(priority_key: nil, fairness_key: 'low', fairness_weight: 0.2)
     ) do |handle|
       assert_equal 'Done!', handle.result