Skip to content

Commit

Permalink
fix: I missed the result tuple!
Browse files Browse the repository at this point in the history
  • Loading branch information
hannahpurcell committed Feb 7, 2025
1 parent ed1c287 commit adc922c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/skate/telemetry.ex
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ defmodule Skate.Telemetry do
%{decode_time: decode_time, query_time: query_time, total_time: total_time},
%{
source: "detours",
result: %{connection_id: connection_id, num_rows: num_rows},
result: {:ok, %{connection_id: connection_id, num_rows: num_rows}},
query: query
},
_config
Expand Down
8 changes: 6 additions & 2 deletions test/skate/telemetry_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,12 @@ defmodule Skate.TelemetryTest do
end)

assert_receive {[:skate, :repo, :query], _ref,
%{decode_time: _, query_time: _, queue_time: _, total_time: _},
%{query: _, result: _, source: "detours"}}
%{decode_time: _, query_time: _, total_time: _},
%{
query: _,
result: {:ok, %{connection_id: _, num_rows: _}},
source: "detours"
}}

assert log =~ "Telemetry for Detours query"
end
Expand Down

0 comments on commit adc922c

Please sign in to comment.