Skip to content

chore(elixir-client): Add test preventing regressions of Phoenix.Sync bug #2842

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

magnetised
Copy link
Contributor

electric-sql/phoenix_sync#63

Due to inappropriate casting of 16 byte binaries to UUIDs in the postgres query builder.

… bug

electric-sql/phoenix_sync#63

Due to inappropriate casting of 16 byte binaries to UUIDs in the
postgres query builder.
Copy link

codecov bot commented Jun 19, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
2226 2 2224 1007
View the top 2 failed test(s) by shortest run time
Elixir.Electric.Connection.ConnectionManagerTest::test status monitor resets the status when connection manager goes down
Stack Traces | 0.104s run time
2) test status monitor resets the status when connection manager goes down (Electric.Connection.ConnectionManagerTest)
     .../electric/connection/manager_test.exs:101
     Assertion with == failed
     code:  assert StatusMonitor.status(stack_id) == :waiting
     left:  :starting
     right: :waiting
     stacktrace:
       .../electric/connection/manager_test.exs:131: (test)
Elixir.Electric.Shapes.ConsumerTest::test event handling handles truncate without appending to log
Stack Traces | 0.439s run time
12) test event handling handles truncate without appending to log (Electric.Shapes.ConsumerTest)
     .../electric/shapes/consumer_test.exs:324
     Assertion failed, no matching message after 400ms
     Showing 10 of 12 messages in the mailbox
     code: assert_receive {Electric.Shapes.Monitor, :cleanup, @shape_handle1}
     mailbox:
       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :get_current_position, "Elixir.Electric.Shapes.ConsumerTest-shape1"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :snapshot_started?, "Elixir.Electric.Shapes.ConsumerTest-shape1"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :mark_snapshot_as_started, "Elixir.Electric.Shapes.ConsumerTest-shape1"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :for_shape, "Elixir.Electric.Shapes.ConsumerTest-shape2"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :initialise, "Elixir.Electric.Shapes.ConsumerTest-shape2"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :get_current_position, "Elixir.Electric.Shapes.ConsumerTest-shape2"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :snapshot_started?, "Elixir.Electric.Shapes.ConsumerTest-shape2"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :mark_snapshot_as_started, "Elixir.Electric.Shapes.ConsumerTest-shape2"}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Support.TestStorage, :set_pg_snapshot, "Elixir.Electric.Shapes.ConsumerTest-shape1", %{xmin: 100, xmax: 101, xip_list: ~c"d", filter_txns?: false}}

       pattern: {Electric.Shapes.Monitor, :cleanup, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
       value:   {Electric.Shapes.Monitor, :remove, "Elixir.Electric.Shapes.ConsumerTest-shape1"}
     stacktrace:
       .../electric/shapes/consumer_test.exs:343: (test)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants