Use TIME_TOLERANCE in specs instead of hardcoded numbers

Author: eregon

spec/ruby/core/file/mtime_spec.rb

@@ -12,7 +12,7 @@
 
   it "returns the modification Time of the file" do
     File.mtime(@filename).should be_kind_of(Time)
-    File.mtime(@filename).should be_close(@mtime, 60.0)
+    File.mtime(@filename).should be_close(@mtime, TIME_TOLERANCE)
   end
 
   guard -> { platform_is :linux or (platform_is :windows and ruby_version_is '2.5') } do

spec/ruby/core/file/utime_spec.rb

@@ -27,10 +27,10 @@
       File.atime(@file2).should be_close(@atime, 0.0001)
       File.mtime(@file2).should be_close(@mtime, 0.0001)
     else
-      File.atime(@file1).to_i.should be_close(@atime.to_i, 2)
-      File.mtime(@file1).to_i.should be_close(@mtime.to_i, 2)
-      File.atime(@file2).to_i.should be_close(@atime.to_i, 2)
-      File.mtime(@file2).to_i.should be_close(@mtime.to_i, 2)
+      File.atime(@file1).to_i.should be_close(@atime.to_i, TIME_TOLERANCE)
+      File.mtime(@file1).to_i.should be_close(@mtime.to_i, TIME_TOLERANCE)
+      File.atime(@file2).to_i.should be_close(@atime.to_i, TIME_TOLERANCE)
+      File.mtime(@file2).to_i.should be_close(@mtime.to_i, TIME_TOLERANCE)
     end
   end
 
@@ -43,10 +43,10 @@
       File.atime(@file2).should be_close(tn, 0.050)
       File.mtime(@file2).should be_close(tn, 0.050)
     else
-      File.atime(@file1).to_i.should be_close(Time.now.to_i, 2)
-      File.mtime(@file1).to_i.should be_close(Time.now.to_i, 2)
-      File.atime(@file2).to_i.should be_close(Time.now.to_i, 2)
-      File.mtime(@file2).to_i.should be_close(Time.now.to_i, 2)
+      File.atime(@file1).to_i.should be_close(Time.now.to_i, TIME_TOLERANCE)
+      File.mtime(@file1).to_i.should be_close(Time.now.to_i, TIME_TOLERANCE)
+      File.atime(@file2).to_i.should be_close(Time.now.to_i, TIME_TOLERANCE)
+      File.mtime(@file2).to_i.should be_close(Time.now.to_i, TIME_TOLERANCE)
     end
   end
 
@@ -63,10 +63,10 @@
       File.mtime(@file2).should be_close(@mtime, 0.0001)
     else
       File.utime(@atime.to_i, @mtime.to_i, @file1, @file2)
-      File.atime(@file1).to_i.should be_close(@atime.to_i, 2)
-      File.mtime(@file1).to_i.should be_close(@mtime.to_i, 2)
-      File.atime(@file2).to_i.should be_close(@atime.to_i, 2)
-      File.mtime(@file2).to_i.should be_close(@mtime.to_i, 2)
+      File.atime(@file1).to_i.should be_close(@atime.to_i, TIME_TOLERANCE)
+      File.mtime(@file1).to_i.should be_close(@mtime.to_i, TIME_TOLERANCE)
+      File.atime(@file2).to_i.should be_close(@atime.to_i, TIME_TOLERANCE)
+      File.mtime(@file2).to_i.should be_close(@mtime.to_i, TIME_TOLERANCE)
     end
   end
 

spec/ruby/core/process/clock_gettime_spec.rb

@@ -36,15 +36,17 @@
       t2 = Process.clock_gettime(Process::CLOCK_MONOTONIC, :float_second)
 
       t1.should be_an_instance_of(Float)
-      t2.should be_close(t1, 2.0)  # 2.0 is chosen arbitrarily to allow for time skew without admitting failure cases, which would be off by an order of magnitude.
+      t2.should be_an_instance_of(Float)
+      t2.should be_close(t1, TIME_TOLERANCE)
     end
 
     it 'uses the default time unit (:float_second) when passed nil' do
       t1 = Process.clock_gettime(Process::CLOCK_MONOTONIC, nil)
       t2 = Process.clock_gettime(Process::CLOCK_MONOTONIC, :float_second)
 
       t1.should be_an_instance_of(Float)
-      t2.should be_close(t1, 2.0) # 2.0 is chosen arbitrarily to allow for time skew without admitting failure cases, which would be off by an order of magnitude.
+      t2.should be_an_instance_of(Float)
+      t2.should be_close(t1, TIME_TOLERANCE)
     end
   end
 end

spec/ruby/core/time/shared/now.rb

@@ -8,7 +8,7 @@
 
   it "sets the current time" do
     now = TimeSpecs::MethodHolder.send(@method)
-    now.to_f.should be_close(Process.clock_gettime(Process::CLOCK_REALTIME), 10.0)
+    now.to_f.should be_close(Process.clock_gettime(Process::CLOCK_REALTIME), TIME_TOLERANCE)
   end
 
   it "uses the local timezone" do

spec/ruby/library/datetime/now_spec.rb

@@ -7,13 +7,13 @@
   end
 
   it "sets the current date" do
-    (DateTime.now - Date.today).to_f.should be_close(0.0, 2.0)
+    (DateTime.now - Date.today).to_f.should be_close(0.0, TIME_TOLERANCE)
   end
 
   it "sets the current time" do
     dt = DateTime.now
     now = Time.now
-    (dt.to_time - now).should be_close(0.0, 10.0)
+    (dt.to_time - now).should be_close(0.0, TIME_TOLERANCE)
   end
 
   it "grabs the local timezone" do

spec/ruby/optional/capi/thread_spec.rb

@@ -24,7 +24,7 @@ def call_capi_rb_thread_wakeup
     it "sleeps the current thread for the give amount of time" do
       start = Time.now
       @t.rb_thread_wait_for(0, 100_000)
-      (Time.now - start).should be_close(0.1, 0.2)
+      (Time.now - start).should be_close(0.1, TIME_TOLERANCE)
     end
   end
 

spec/ruby/optional/capi/time_spec.rb

@@ -294,7 +294,7 @@
       now = Time.now
       time = @s.rb_time_from_timespec(now.utc_offset)
       time.should be_an_instance_of(Time)
-      (time - now).should be_close(0, 10)
+      (time - now).should be_close(0, TIME_TOLERANCE)
     end
   end
 end