Skip to content

Conversation

@sabiwara
Copy link
Contributor

@sabiwara sabiwara commented Jul 8, 2025

The issue is introduced in 99be673

The code is a bit complex because require_test_helper which might call ExUnit.configure is wrapped in the try/catch, which prevents it from assigning it to opts once in for all and pass it around.

https://github.com/elixir-lang/elixir/blob/main/lib/mix/lib/mix/tasks/test.ex#L663-L670

These two both now work:

# test/test_helper.exs
ExUnit.start(failures_manifest_path: "failed.txt")
# test/test_helper.exs
ExUnit.start(failures_manifest_path: "failed.txt")
raise "test the try/catch"

@josevalim josevalim merged commit 0be3e70 into elixir-lang:main Jul 8, 2025
13 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@sabiwara sabiwara deleted the fix-failures-manifest-path branch July 8, 2025 07:29
ggVGc pushed a commit to ggVGc/elixir-verbatim that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants