Skip to content

Commit 65cff52

Browse files
test: make the runs the after_hook after forking more predictable (#1939)
Co-authored-by: Kayla Reopelle <[email protected]>
1 parent 7047f09 commit 65cff52

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

metrics_sdk/test/opentelemetry/sdk/metrics/fork_hooks_test.rb

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,15 @@ def with_pipe
5555
after_fork_write_io.puts message
5656
end
5757

58-
forking_pid, forked_pid = fork_with_fork_hooks(after_fork_lambda)
59-
pid_from_after_fork = JSON.parse(after_fork_read_io.gets.chomp)['after_fork_pid'].to_i
58+
meter_provider = OpenTelemetry::SDK::Metrics::MeterProvider.new
59+
::OpenTelemetry.stub(:meter_provider, meter_provider) do
60+
forking_pid, forked_pid = fork_with_fork_hooks(after_fork_lambda)
61+
pid_from_after_fork = JSON.parse(after_fork_read_io.gets.chomp)['after_fork_pid'].to_i
6062

61-
refute_equal(pid_from_after_fork, Process.pid)
62-
refute_equal(pid_from_after_fork, forking_pid)
63-
assert_equal(forked_pid, pid_from_after_fork)
63+
refute_equal(pid_from_after_fork, Process.pid)
64+
refute_equal(pid_from_after_fork, forking_pid)
65+
assert_equal(forked_pid, pid_from_after_fork)
66+
end
6467
end
6568
end
6669

0 commit comments

Comments
 (0)