Enhancement: Better display of timestamp in callback traces (#872)

* add timezone to traces timepstamp

Author: srzeszut

lib/live_debugger/app/debugger/callback_tracing/web/components/trace.ex

@@ -184,7 +184,7 @@ defmodule LiveDebugger.App.Debugger.CallbackTracing.Web.Components.Trace do
     assigns = assign(assigns, :timestamp, timestamp)
 
     ~H"""
-    <.tooltip id={@id <> "-tooltip"} content="timestamp" class="min-w-24">
+    <.tooltip id={@id <> "-tooltip"} content="Timestamp" class="min-w-12">
       <%= Parsers.parse_timestamp(@timestamp) %>
     </.tooltip>
     """

lib/live_debugger/app/utils/parsers.ex

@@ -42,6 +42,7 @@ defmodule LiveDebugger.App.Utils.Parsers do
   def parse_timestamp(timestamp) when is_integer(timestamp) and timestamp > 0 do
     timestamp
     |> DateTime.from_unix(:microsecond)
+    |> apply_timezone_diff()
     |> case do
       {:ok, %DateTime{hour: hour, minute: minute, second: second, microsecond: {micro, _}}} ->
         "~2..0B:~2..0B:~2..0B.~6..0B"
@@ -142,4 +143,18 @@ defmodule LiveDebugger.App.Utils.Parsers do
     |> Module.split()
     |> Enum.join(".")
   end
+
+  defp apply_timezone_diff({:ok, timestamp}) do
+    {:ok, DateTime.add(timestamp, get_utc_difference_in_minutes(), :minute)}
+  end
+
+  defp apply_timezone_diff(error), do: error
+
+  def get_utc_difference_in_minutes do
+    NaiveDateTime.diff(
+      NaiveDateTime.local_now(),
+      DateTime.utc_now() |> DateTime.to_naive(),
+      :minute
+    )
+  end
 end

test/app/utils/parsers_test.exs

@@ -6,10 +6,23 @@ defmodule LiveDebugger.App.Utils.ParsersTest do
   describe "parse_timestamp/1" do
     test "parses a valid timestamp" do
       timestamp1 = 1_000_000
-      assert Parsers.parse_timestamp(timestamp1) == "00:00:01.000000"
+
+      timezone_offset_minutes =
+        NaiveDateTime.diff(
+          NaiveDateTime.local_now(),
+          DateTime.utc_now()
+          |> DateTime.to_naive(),
+          :minute
+        )
+
+      assert Parsers.parse_timestamp(timestamp1) ==
+               "00:00:01.000000"
+               |> apply_timezone_diff(timezone_offset_minutes)
 
       timestamp2 = 60_000_000
-      assert Parsers.parse_timestamp(timestamp2) == "00:01:00.000000"
+
+      assert Parsers.parse_timestamp(timestamp2) ==
+               "00:01:00.000000" |> apply_timezone_diff(timezone_offset_minutes)
     end
 
     test "returns \"Invalid timestamp\" when timestamp is invalid" do
@@ -82,4 +95,11 @@ defmodule LiveDebugger.App.Utils.ParsersTest do
                ~s(%{key: "value"}, %{foo: :bar})
     end
   end
+
+  defp apply_timezone_diff(time, timezone_offset_minutes) do
+    time
+    |> Time.from_iso8601!()
+    |> Time.add(timezone_offset_minutes, :minute)
+    |> Calendar.strftime("%H:%M:%S.%6f")
+  end
 end