improve version checks

Author: lukaszsamson

apps/elixir_ls_utils/test/complete_test.exs

@@ -1029,7 +1029,7 @@ defmodule ElixirLS.Utils.CompletionEngineTest do
 
     # local call on var
 
-    if Version.match?(System.version(), "< 1.16.0-dev") do
+    if Version.match?(System.version(), "< 1.16.0") do
       assert [] == expand(~c"asd.(")
       assert [] == expand(~c"@asd.(")
     else

apps/language_server/lib/language_server/providers/plugins/phoenix.ex

@@ -24,7 +24,7 @@ defmodule ElixirLS.LanguageServer.Plugins.Phoenix do
     ModuleStore.ensure_compiled(context, Phoenix.Router)
   end
 
-  if Version.match?(System.version(), ">= 1.14.0") do
+  if Version.match?(System.version(), ">= 1.14.0-dev") do
     @impl true
     def suggestions(hint, {Phoenix.Router, func, 1, _info}, _list, opts)
         when func in @phoenix_route_funcs do

apps/language_server/test/diagnostics_test.exs

@@ -2,7 +2,7 @@ defmodule ElixirLS.LanguageServer.DiagnosticsTest do
   alias ElixirLS.LanguageServer.Diagnostics
   use ExUnit.Case
 
-  if Version.match?(System.version(), "< 1.16.0-dev") do
+  if Version.match?(System.version(), "< 1.16.0") do
     describe "pre 1.16 Mix.Task.Compiler.Diagnostic normalization" do
       test "extract the stacktrace from the message and format it" do
         root_path = Path.join(__DIR__, "fixtures/build_errors")

apps/language_server/test/providers/references/locator_test.exs

@@ -411,7 +411,7 @@ defmodule ElixirLS.LanguageServer.Providers.References.LocatorTest do
              }
            ] = references
 
-    if Version.match?(System.version(), ">= 1.14.0-rc.0") do
+    if Version.match?(System.version(), ">= 1.14.0") do
       # before 1.14 tracer reports invalid positions for captures
       # https://github.com/elixir-lang/elixir/issues/12023
       assert range == %{start: %{line: 55, column: 72}, end: %{line: 55, column: 83}}

apps/language_server/test/providers/selection_ranges_test.exs

@@ -228,7 +228,7 @@ defmodule ElixirLS.LanguageServer.Providers.SelectionRangesTest do
       assert_range(ranges, range(1, 0, 2, 4))
     end
 
-    if Version.match?(System.version(), ">= 1.14.0-dev") do
+    if Version.match?(System.version(), ">= 1.14.0") do
       test "left from do" do
         text = """
         do
@@ -262,7 +262,7 @@ defmodule ElixirLS.LanguageServer.Providers.SelectionRangesTest do
       assert_range(ranges, range(0, 0, 3, 3))
     end
 
-    if Version.match?(System.version(), ">= 1.14.0-dev") do
+    if Version.match?(System.version(), ">= 1.14.0") do
       test "left from end" do
         text = """
         do
@@ -1030,7 +1030,7 @@ defmodule ElixirLS.LanguageServer.Providers.SelectionRangesTest do
       assert_range(ranges, range(0, 8, 0, 12))
     end
 
-    if Version.match?(System.version(), ">= 1.14.0-dev") do
+    if Version.match?(System.version(), ">= 1.14.0") do
       test "left side of | near" do
         text = """
         %{state | 1 => 1, counter: counter + to_dispatch, demand: demand - to_dispatch}

apps/language_server/test/support/server_test_helpers.ex

@@ -27,7 +27,7 @@ defmodule ElixirLS.LanguageServer.Test.ServerTestHelpers do
   def replace_logger(packet_capture) do
     # :logger application is already started
     # replace console logger with LSP
-    if Version.match?(System.version(), ">= 1.15.0-dev") do
+    if Version.match?(System.version(), ">= 1.15.0") do
       configs =
         for handler_id <- :logger.get_handler_ids() do
           {:ok, config} = :logger.get_handler_config(handler_id)