From 7453cdba1b62d6f6533f8c7bfdf6011524e11853 Mon Sep 17 00:00:00 2001 From: mame Date: Wed, 26 Dec 2018 07:02:31 +0000 Subject: spec/ruby/core/process/setpriority_spec.rb: allow to run under docker Docker container is not always able to lower the nice value even if the euid is a root. It depends upon the configuration of docker which cannot check from the container itself. This change does check it by actually trying to lower the value first. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@66570 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- spec/ruby/core/process/setpriority_spec.rb | 25 ++++++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) (limited to 'spec/ruby/core/process') diff --git a/spec/ruby/core/process/setpriority_spec.rb b/spec/ruby/core/process/setpriority_spec.rb index d3be02eb6d..118482eb4e 100644 --- a/spec/ruby/core/process/setpriority_spec.rb +++ b/spec/ruby/core/process/setpriority_spec.rb @@ -29,11 +29,26 @@ describe "Process.setpriority" do end as_superuser do - it "sets the scheduling priority for a specified user" do - p = Process.getpriority(Process::PRIO_USER, 0) - Process.setpriority(Process::PRIO_USER, 0, p + 1).should == 0 - Process.getpriority(Process::PRIO_USER, 0).should == (p + 1) - Process.setpriority(Process::PRIO_USER, 0, p).should == 0 + p = Process.getpriority(Process::PRIO_USER, 0) + # The nice value is a value in the range -20 to 19. + # This test tries to change the nice value to +-1, so it cannot run if p == -20 || p == 19. + if -20 < p && p < 19 + begin + # Check if we can lower the nice value or not. + # + # We are not always able to do it even as a root. + # Docker container is not always able to do it depending upon the configuration, + # which cannot know from the container itself. + Process.setpriority(Process::PRIO_USER, 0, p - 1) + Process.setpriority(Process::PRIO_USER, 0, p) + + it "sets the scheduling priority for a specified user" do + Process.setpriority(Process::PRIO_USER, 0, p + 1).should == 0 + Process.getpriority(Process::PRIO_USER, 0).should == (p + 1) + Process.setpriority(Process::PRIO_USER, 0, p).should == 0 + end + rescue Errno::EACCES + end end end end -- cgit v1.2.3