Dedup TodosNeedTickets

theblitzapp · Nov 28, 2023 · 6afa266 · 6afa266
1 parent 729e43a
commit 6afa266
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/blitz_credo_checks/todos_need_tickets.ex b/lib/blitz_credo_checks/todos_need_tickets.ex
@@ -32,6 +32,7 @@ defmodule BlitzCredoChecks.TodosNeedTickets do
  [] ->
  tags
  |> Stream.flat_map(&TagHelper.tags(source_file, &1, true))
+ |> Stream.uniq_by(&elem(&1, 0))
  |> Enum.map(&issue_for(issue_meta, &1))
 
  # We found ticket urls so ignore every todo in file

diff --git a/test/blitz_credo_checks/todos_need_tickets_test.exs b/test/blitz_credo_checks/todos_need_tickets_test.exs
@@ -14,7 +14,7 @@ defmodule BlitzCredoChecks.TodosNeedTicketsTest do
  """
  |> to_source_file()
  |> TodosNeedTickets.run(ticket_url: "https://linear.app/blitz/issue/")
- |> assert_issues()
+ |> assert_issue()
  end
 
  test "accepts todos with a ticket attached" do