Merge pull request #1372 from absinthe-graphql/cschiewek/elixir-1.19

Updates for Elixir 1.19

Author: cschiewek

.github/workflows/elixir.yml

@@ -18,13 +18,16 @@ jobs:
           - "1.16"
           - "1.17"
           - "1.18"
+          - "1.19"
         otp:
           - "25"
           - "26"
           - "27"
           - "28"
         # see https://hexdocs.pm/elixir/compatibility-and-deprecations.html#between-elixir-and-erlang-otp
         exclude:
+          - elixir: 1.19
+            otp: 25
           - elixir: 1.17
             otp: 28
           - elixir: 1.16

lib/absinthe/language/render.ex

@@ -416,10 +416,6 @@ defmodule Absinthe.Language.Render do
     do_block(docs)
   end
 
-  defp block(:doc_nil, docs) do
-    do_block(docs)
-  end
-
   defp block(name, docs) do
     glue(
       name,

mix.exs

@@ -8,15 +8,12 @@ defmodule Absinthe.Mixfile do
     [
       app: :absinthe,
       version: @version,
-      elixir: "~> 1.11",
+      elixir: "~> 1.15",
       elixirc_paths: elixirc_paths(Mix.env()),
       build_embedded: Mix.env() == :prod,
       start_permanent: Mix.env() == :prod,
       package: package(),
       source_url: @source_url,
-      preferred_cli_env: [
-        dialyzer: :test
-      ],
       docs: [
         source_ref: "v#{@version}",
         main: "overview",
@@ -39,6 +36,10 @@ defmodule Absinthe.Mixfile do
     ]
   end
 
+  def cli do
+    [preferred_envs: [dialyzer: :test]]
+  end
+
   defp package do
     [
       description: "GraphQL for Elixir",
@@ -82,7 +83,6 @@ defmodule Absinthe.Mixfile do
       {:ex_doc, "~> 0.22", only: :dev},
       {:benchee, ">= 1.0.0", only: :dev},
       {:dialyxir, "~> 1.1", only: [:dev, :test], runtime: false},
-      {:mix_test_watch, "~> 1.0", only: :dev, runtime: false},
       {:makeup_graphql, "~> 0.1.0", only: :dev}
     ]
   end

test/absinthe/subscription/pipeline_serializer_test.exs

@@ -28,15 +28,12 @@ defmodule Absinthe.Subscription.PipelineSerializerTest do
         {Phase4, [option1: :value1]}
       ]
 
-      assert {:packed,
-              [
-                {Phase1, [:pack | 0]},
-                Phase2,
-                {Phase3, [:pack | 1]},
-                {Phase4, [:pack | 0]}
-              ],
-              %{0 => [option1: :value1], 1 => [option2: :value2]}} =
-               PipelineSerializer.pack(pipeline)
+      assert {:packed, config, options} = PipelineSerializer.pack(pipeline)
+
+      assert [{Phase1, [:pack | 0]}, Phase2, {Phase3, [:pack | 1]}, {Phase4, [:pack | 0]}] =
+               config
+
+      assert %{0 => [option1: :value1], 1 => [option2: :value2]} = options
     end
 
     test "packs variables and contexts in options" do