fix: don't use deprecated Time.now

place-labs · Dec 12, 2019 · 79abbae · 79abbae
1 parent c1606db
commit 79abbae
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/spec/flux/data_point_spec.cr b/spec/flux/data_point_spec.cr
@@ -29,7 +29,7 @@ describe Flux::DataPoint do
 
   describe "to_s" do
     it "serializes to line procotol" do
-      time = Time.now
+      time = Time.utc
       ts = time.to_unix
       point = Flux::DataPoint.new "foo", time, a: 1
       point.tag :test, "bar"

diff --git a/spec/flux/line_protocol_spec.cr b/spec/flux/line_protocol_spec.cr
@@ -2,7 +2,7 @@ require "../spec_helper"
 
 describe Flux::LineProtocol do
   describe ".serialize" do
-    time = Time.now
+    time = Time.utc
     ts = time.to_unix
 
     it "serializes a simple point to line protocol" do

diff --git a/src/flux/data_point.cr b/src/flux/data_point.cr
@@ -15,7 +15,7 @@ struct Flux::DataPoint
   getter timestamp : Time
 
   # Creates a new data point that can be serialized for entry to InfluxDB.
-  def initialize(@measurement, @timestamp = Time.now, @tags = nil, **fields : **T) forall T
+  def initialize(@measurement, @timestamp = Time.local, @tags = nil, **fields : **T) forall T
     raise ArgumentError.new "points must include at least one field" if fields.empty?
 
     @fields = FieldSet.new