fix and test InfluxDB.now

Author: dmke

lib/influxdb/timestamp_conversion.rb

@@ -31,7 +31,7 @@ def self.convert_timestamp(time, precision = "s")
   #  InfluxDB.now("m")    #=> 25726868
   #  InfluxDB.now("h")    #=> 428781
   def self.now(precision = "s")
-    name, divisor = TIME_PRECISION_FACTORS.fetch(precision) do
+    name, divisor = CLOCK_NAMES.fetch(precision) do
       raise ArgumentError, "invalid time precision: #{precision}"
     end
 
@@ -56,8 +56,8 @@ def self.now(precision = "s")
     "u"  => [:microsecond, 1],
     "ms" => [:millisecond, 1],
     "s"  => [:second, 1],
-    "m"  => [:second, 1.to_r / 60],
-    "h"  => [:second, 1.to_r / 60 / 60],
+    "m"  => [:second, 60.to_r],
+    "h"  => [:second, (60 * 60).to_r],
   }.freeze
   private_constant :CLOCK_NAMES
 end

spec/influxdb/time_conversion_spec.rb

@@ -23,4 +23,23 @@
         .to raise_exception(/invalid time precision.*whatever/i)
     end
   end
+
+  describe ".now" do
+    {
+      "ns" => [:nanosecond,   1_513_009_229_111_222_333],
+      nil  => [:nanosecond,   1_513_009_229_111_222_333],
+      "u"  => [:microsecond,  1_513_009_229_111_222],
+      "ms" => [:millisecond,  1_513_009_229_111],
+      "s"  => [:second,       1_513_009_229],
+      "m"  => [:second,       25_216_820,   1_513_009_229],
+      "h"  => [:second,       420_280,      1_513_009_229],
+    }.each do |precision, (name, expected, stub)|
+      it "should return the current time in #{precision.inspect}" do
+        expect(Process).to receive(:clock_gettime)
+          .with(Process::CLOCK_REALTIME, name)
+          .and_return(stub || expected)
+        expect(described_class.now(precision)).to eq(expected)
+      end
+    end
+  end
 end