Fix constant reference in ConsistentProbabilityTraceState (#1450)

* fix constant reference in cps sampler * try rubocop false (fingers crossed) * Revert "try rubocop false (fingers crossed)" This reverts commit 700650f.
open-telemetry · May 23, 2023 · 1ad4a21 · 1ad4a21
1 parent 24c8032
commit 1ad4a21
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/api/test/opentelemetry/trace/tracestate_test.rb b/api/test/opentelemetry/trace/tracestate_test.rb
@@ -156,4 +156,10 @@
       _(OpenTelemetry::Trace::Tracestate.from_hash('a' => 'b')).wont_be :empty?
     end
   end
+
+  describe '::DEFAULT' do
+    it 'returns empty tracestate' do
+      _(OpenTelemetry::Trace::Tracestate::DEFAULT.to_h).must_be :empty?
+    end
+  end
 end
diff --git a/sdk_experimental/lib/opentelemetry/sdk/trace/samplers/consistent_probability_tracestate.rb b/sdk_experimental/lib/opentelemetry/sdk/trace/samplers/consistent_probability_tracestate.rb
@@ -90,7 +90,7 @@ def update_tracestate(tracestate, p, r, rest) # rubocop:disable Naming/Uncommuni
 
           def new_tracestate(p: nil, r: nil) # rubocop:disable Naming/UncommunicativeMethodParamName
             if p.nil? && r.nil?
-              OpenTelemetry::Trace::Tracestate.DEFAULT
+              OpenTelemetry::Trace::Tracestate::DEFAULT
             elsif p.nil?
               OpenTelemetry::Trace::Tracestate.from_hash('ot' => "r:#{r}")
             elsif r.nil?