Test on the latest Erlang/Elixir in CI (#841)

Author: whatyouhide

.github/workflows/main.yml

@@ -14,26 +14,29 @@ env:
 jobs:
   test:
     name: Test (Elixir ${{ matrix.elixir }}, OTP ${{ matrix.otp }})
-    runs-on: ubuntu-20.04
+    runs-on: ${{ matrix.os }}
     timeout-minutes: 10
     strategy:
       fail-fast: false
       matrix:
         # https://hexdocs.pm/elixir/compatibility-and-deprecations.html#compatibility-between-elixir-and-erlang-otp
         include:
           # Newest supported Elixir/Erlang pair.
-          - elixir: '1.17'
-            otp: '27.1'
+          - elixir: '1.18'
+            otp: '27.2'
             lint: true
             dialyzer: true
+            os: ubuntu-latest
 
           # One version before the last supported one.
-          - elixir: '1.16'
+          - elixir: '1.17'
             otp: '26.2'
+            os: ubuntu-22.04
 
           # Oldest supported Elixir/Erlang pair.
           - elixir: '1.13.4-otp-22'
             otp: '22.3.4'
+            os: ubuntu-20.04
 
     steps:
     - name: Check out this repository

test/envelope_test.exs

@@ -144,8 +144,5 @@ defmodule Sentry.EnvelopeTest do
              event_id: Sentry.UUID.uuid4_hex(),
              timestamp: "2024-10-12T13:21:13"
            }) == "error"
-
-    assert_raise FunctionClauseError,
-                 fn -> Envelope.get_data_category(:completely_banana_value) end
   end
 end

test/sentry/client_test.exs

@@ -234,7 +234,12 @@ defmodule Sentry.ClientTest do
       put_test_config(before_send: {CallbackModuleArithmeticError, :before_send})
 
       try do
-        :rand.uniform() + "1"
+        # This is the equivalent of
+        #
+        #     :rand.uniform() + "1"
+        #
+        # but made dynamic to avoid type warnings.
+        apply(Kernel, :+, [:rand.uniform(), "1"])
       rescue
         exception ->
           event = Event.transform_exception(exception, _opts = [])

test/sentry/logger_handler_test.exs

@@ -635,7 +635,8 @@ defmodule Sentry.LoggerHandlerTest do
   end
 
   defp invalid_function do
-    NaiveDateTime.from_erl({}, {}, {})
+    # This needs to be dynamic in order to not warn with Elixir's type system
+    apply(NaiveDateTime, :from_erl, [{}, {}, {}])
   end
 
   defp enable_sasl_reports do