From e340988a2a308b3bcdcf5470c2538a329629e973 Mon Sep 17 00:00:00 2001 From: Frank Hunleth Date: Mon, 26 Feb 2024 20:25:00 -0500 Subject: [PATCH] Remove accidental superfluous braces; add test The change in 36d6d91bbfbbc3549611d7b8911b2c82b8480503 turned out to not fix the issue due to the extra braces. This fixes it and adds a test to be sure. --- lib/mdns_lite/responder.ex | 2 +- test/mdns_lite/responder_test.exs | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/mdns_lite/responder.ex b/lib/mdns_lite/responder.ex index aedeae3..30a5d4b 100644 --- a/lib/mdns_lite/responder.ex +++ b/lib/mdns_lite/responder.ex @@ -90,7 +90,7 @@ defmodule MdnsLite.Responder do def stop_server(ifname, address) do GenServer.stop(via_name({ifname, address})) catch - {:exit, {:noproc, _}} -> + :exit, {:noproc, _} -> # Ignore if the server already stopped. It already exited due to the # network going down. :ok diff --git a/test/mdns_lite/responder_test.exs b/test/mdns_lite/responder_test.exs index 8d6da1b..d9ee098 100644 --- a/test/mdns_lite/responder_test.exs +++ b/test/mdns_lite/responder_test.exs @@ -1,3 +1,9 @@ defmodule MdnsLite.ResponderTest do use ExUnit.Case, async: false + + alias MdnsLite.Responder + + test "stopping a nonexistent responder" do + Responder.stop_server("no_exist", {1, 2, 3, 4}) + end end