From 2dacd0df81d8b4e61a132f82b088ea252f35d53a Mon Sep 17 00:00:00 2001 From: Dave Lucia Date: Thu, 17 Oct 2024 06:37:38 -0400 Subject: [PATCH] Use ranges with step in tests StreamData depends on Elixir 1.12+ now, so we can clean these up --- test/stdlib/enum_test.exs | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/test/stdlib/enum_test.exs b/test/stdlib/enum_test.exs index 8bb9da6..77d20a5 100644 --- a/test/stdlib/enum_test.exs +++ b/test/stdlib/enum_test.exs @@ -14,13 +14,7 @@ defmodule StreamData.EnumTest do ) do length = length(list) - # TODO: When we depend on 1.12+, rewrite as: - # negative_range = (range.first - length)..(range.last - length)//1 - negative_range = %Range{ - first: range.first - length, - last: range.last - length, - step: 1 - } + negative_range = (range.first - length)..(range.last - length)//1 assert Enum.slide(list, negative_range, insertion_idx) == Enum.slide(list, range, insertion_idx) @@ -92,9 +86,7 @@ defmodule StreamData.EnumTest do first <- integer(), last <- integer(), step <- positive_integer() do - # TODO: When we depend on 1.12+, rewrite as: - # first..last//step - range = %Range{first: first, last: last, step: step} + range = first..last//step enum_consistency_spec(count, fn enumerable -> Enum.slice(enumerable, range) @@ -131,9 +123,7 @@ defmodule StreamData.EnumTest do end defp enum_consistency_spec(count, fun) do - # TODO: When we depend on 1.12+, rewrite as: - # 1..count//1 - range = %Range{first: 1, last: count, step: 1} + range = 1..count//1 list = Enum.to_list(range) stream = Stream.map(range, & &1)