Skip to content

Commit 2a2e62d

Browse files
author
Philip Sampaio
committed
Simplify CLI tests
1 parent b2181a4 commit 2a2e62d

File tree

1 file changed

+11
-21
lines changed

1 file changed

+11
-21
lines changed

lib/elixir/test/elixir/kernel/cli_test.exs

Lines changed: 11 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,7 @@ defmodule Retry do
3333
end
3434

3535
defmodule Kernel.CLITest do
36-
use ExUnit.Case,
37-
async: true,
38-
parameterize:
39-
if(PathHelpers.windows?(),
40-
do: [%{cli_extension: ".bat"}, %{cli_extension: ".ps1"}],
41-
else: [%{cli_extension: ""}, %{cli_extension: ".ps1"}]
42-
)
36+
use ExUnit.Case, async: true
4337

4438
import ExUnit.CaptureIO
4539

@@ -67,16 +61,18 @@ defmodule Kernel.CLITest do
6761
end
6862
end
6963

64+
test_parameters =
65+
if(PathHelpers.windows?(),
66+
do: [%{cli_extension: ".bat"}, %{cli_extension: ".ps1"}],
67+
else:
68+
[%{cli_extension: ""}] ++
69+
if(System.find_executable("pwsh"), do: [%{cli_extension: ".ps1"}], else: [])
70+
)
71+
7072
defmodule Kernel.CLI.ExecutableTest do
7173
use ExUnit.Case,
7274
async: true,
73-
parameterize:
74-
if(PathHelpers.windows?(),
75-
do: [%{cli_extension: ".bat"}, %{cli_extension: ".ps1"}],
76-
else:
77-
[%{cli_extension: ""}] ++
78-
if(System.find_executable("pwsh"), do: [%{cli_extension: ".ps1"}], else: [])
79-
)
75+
parameterize: test_parameters
8076

8177
import Retry
8278

@@ -271,13 +267,7 @@ end
271267
defmodule Kernel.CLI.CompileTest do
272268
use ExUnit.Case,
273269
async: true,
274-
parameterize:
275-
if(PathHelpers.windows?(),
276-
do: [%{cli_extension: ".bat"}, %{cli_extension: ".ps1"}],
277-
else:
278-
[%{cli_extension: ""}] ++
279-
if(System.find_executable("pwsh"), do: [%{cli_extension: ".ps1"}], else: [])
280-
)
270+
parameterize: test_parameters
281271

282272
import Retry
283273
@moduletag :tmp_dir

0 commit comments

Comments
 (0)