Enable quokka with :module_directives rule

test/music_brainz_test.exs

@@ -1,10 +1,11 @@
 defmodule MusicBrainzTest do
   use ExUnit.Case, async: true
 
-  alias MusicBrainz.ReleaseGroupSearchResult
   import MusicBrainz.Fixtures.Release
   import MusicBrainz.Fixtures.ReleaseGroup
 
+  alias MusicBrainz.ReleaseGroupSearchResult
+
   describe "search_release_group/2" do
     test "it returns results with correct limit and offset" do
       results = release_group_search_results()

test/music_library/artists_test.exs

@@ -1,9 +1,10 @@
 defmodule MusicLibrary.ArtistsTest do
   use MusicLibrary.DataCase
 
-  alias MusicLibrary.Artists
   import MusicLibrary.Fixtures.Records
 
+  alias MusicLibrary.Artists
+
   describe "get_artist/1" do
     test "it returns records with essential data" do
       record = record()

test/music_library/barcode_scan/result_test.exs

@@ -1,8 +1,8 @@
 defmodule MusicLibrary.BarcodeScan.ResultTest do
   use ExUnit.Case, async: true
 
-  alias MusicLibrary.BarcodeScan.Result
   alias MusicBrainz.{ReleaseGroupSearchResult, ReleaseSearchResult}
+  alias MusicLibrary.BarcodeScan.Result
 
   @release_group %ReleaseGroupSearchResult{
     id: "123",

test/music_library/collection_test.exs

@@ -1,9 +1,10 @@
 defmodule MusicLibrary.CollectionTest do
   use MusicLibrary.DataCase
 
-  alias MusicLibrary.Collection
   import MusicLibrary.Fixtures.Records
 
+  alias MusicLibrary.Collection
+
   defp fill_collection(_) do
     # Purchased dates are in ascending order
     records = [

test/music_library/records/cover_test.exs

@@ -2,6 +2,7 @@ defmodule MusicLibrary.Records.CoverTest do
   use ExUnit.Case, async: true
 
   import MusicLibrary.Fixtures.Records
+
   alias MusicLibrary.Records.Cover
 
   describe "resize/1" do

test/music_library/records/record_test.exs

@@ -2,6 +2,7 @@ defmodule MusicLibrary.Records.RecordTest do
   use ExUnit.Case, async: true
 
   import MusicBrainz.Fixtures.ReleaseGroup
+
   alias MusicLibrary.Records.Record
 
   doctest Record

test/music_library/records_test.exs

@@ -1,11 +1,12 @@
 defmodule MusicLibrary.RecordsTest do
   use MusicLibrary.DataCase
 
+  import MusicBrainz.Fixtures.Release
+  import MusicBrainz.Fixtures.ReleaseGroup
+  import MusicLibrary.Fixtures.Records
+
   alias MusicLibrary.Records
   alias MusicLibrary.Records.SearchIndex
-  import MusicLibrary.Fixtures.Records
-  import MusicBrainz.Fixtures.ReleaseGroup
-  import MusicBrainz.Fixtures.Release
 
   defp create_records(_) do
     records = [

test/music_library/wishlist_test.exs

@@ -1,9 +1,10 @@
 defmodule MusicLibrary.WishlistTest do
   use MusicLibrary.DataCase
 
-  alias MusicLibrary.Wishlist
   import MusicLibrary.Fixtures.Records
 
+  alias MusicLibrary.Wishlist
+
   defp fill_wishlist(_) do
     records = [
       record(%{purchased_at: nil, title: "Brave"}),

test/music_library_web/controllers/cover_controller_test.exs

@@ -2,6 +2,7 @@ defmodule MusicLibraryWeb.CoverControllerTest do
   use MusicLibraryWeb.ConnCase
 
   import MusicLibrary.Fixtures.Records
+
   alias MusicLibrary.Records.Cover
 
   defp create_record(_) do

test/music_library_web/live/artist_live/show_test.exs

@@ -2,6 +2,7 @@ defmodule MusicLibraryWeb.ArtistLive.ShowTest do
   use MusicLibraryWeb.ConnCase
 
   import MusicLibrary.Fixtures.Records
+
   alias LastFm.Fixtures
 
   defp fill_collection(_config) do

test/music_library_web/live/collection_live/index_test.exs

@@ -1,10 +1,11 @@
 defmodule MusicLibraryWeb.CollectionLive.IndexTest do
   use MusicLibraryWeb.ConnCase
 
-  import MusicLibrary.Fixtures.Records
-  import MusicBrainz.Fixtures.ReleaseGroup
   import MusicBrainz.Fixtures.Release
+  import MusicBrainz.Fixtures.ReleaseGroup
+  import MusicLibrary.Fixtures.Records
   import MusicLibraryWeb.RecordComponents, only: [format_label: 1, type_label: 1]
+
   alias MusicBrainz.ReleaseGroupSearchResult
   alias MusicLibrary.Records.{Cover, Record}
 

test/music_library_web/live/collection_live/show_test.exs

@@ -2,7 +2,6 @@ defmodule MusicLibraryWeb.CollectionLive.ShowTest do
   use MusicLibraryWeb.ConnCase
 
   import MusicLibrary.Fixtures.Records
-
   import MusicLibraryWeb.RecordComponents, only: [format_label: 1, type_label: 1]
 
   alias MusicBrainz.Fixtures

test/music_library_web/live/stats_live/index_test.exs

@@ -1,11 +1,12 @@
 defmodule MusicLibraryWeb.StatsLive.IndexTest do
   use MusicLibraryWeb.ConnCase
 
-  alias MusicLibrary.{Records, Repo, Wishlist}
-  import MusicLibraryWeb.RecordComponents, only: [format_label: 1, type_label: 1]
-  import MusicLibrary.Fixtures.Records
-  import MusicBrainz.Fixtures.ReleaseGroup
   import MusicBrainz.Fixtures.Release
+  import MusicBrainz.Fixtures.ReleaseGroup
+  import MusicLibrary.Fixtures.Records
+  import MusicLibraryWeb.RecordComponents, only: [format_label: 1, type_label: 1]
+
+  alias MusicLibrary.{Records, Repo, Wishlist}
 
   defp fill_collection(_) do
     current_time = DateTime.utc_now()

test/music_library_web/live/wishlist_live/show_test.exs

@@ -2,7 +2,6 @@ defmodule MusicLibraryWeb.WishlistLive.ShowTest do
   use MusicLibraryWeb.ConnCase
 
   import MusicLibrary.Fixtures.Records
-
   import MusicLibraryWeb.RecordComponents, only: [format_label: 1, type_label: 1]
 
   alias MusicLibrary.Records.Record

test/music_library_web/plug/auth_test.exs

@@ -1,5 +1,6 @@
 defmodule MusicLibraryWeb.AuthTest do
   use ExUnit.Case, async: true
+
   import Plug.Test
 
   alias MusicLibraryWeb.Auth

test/obsidian/parser_test.exs

@@ -1,5 +1,6 @@
 defmodule Obsidian.ParserTest do
   use ExUnit.Case, async: true
+
   alias Obsidian.{Entry, Parser}
 
   @marbles_entry_path Path.expand("../support/fixtures/obsidian/marillion-marbles.md", __DIR__)

test/support/conn_case.ex

@@ -19,21 +19,22 @@ defmodule MusicLibraryWeb.ConnCase do
 
   using do
     quote do
-      # The default endpoint for testing
-      @endpoint MusicLibraryWeb.Endpoint
-
       use MusicLibraryWeb, :verified_routes
 
-      # Import conveniences for testing with connections
-      import Plug.Conn
-      import Phoenix.ConnTest
       import MusicLibraryWeb.ConnCase
-      import PhoenixTest
+      import MusicLibraryWeb.LiveTestHelpers
+      import Phoenix.ConnTest
 
       import Phoenix.LiveViewTest,
+        # The default endpoint for testing
         only: [render: 1, render_async: 1, render_hook: 2, render_hook: 3, element: 2, element: 3]
 
-      import MusicLibraryWeb.LiveTestHelpers
+      import PhoenixTest
+      import Plug.Conn
+
+      @endpoint MusicLibraryWeb.Endpoint
+
+      # Import conveniences for testing with connections
     end
   end
 

test/support/data_case.ex

@@ -22,12 +22,12 @@ defmodule MusicLibrary.DataCase do
     quote do
       use Oban.Testing, repo: MusicLibrary.BackgroundRepo
 
-      alias MusicLibrary.Repo
-
       import Ecto
       import Ecto.Changeset
       import Ecto.Query
       import MusicLibrary.DataCase
+
+      alias MusicLibrary.Repo
     end
   end