Skip to content

Conversation

@antedeguemon
Copy link
Contributor

Description

Hey there!

This pull request fixes the Elixir tests paths in CircleCI guides to test/**/*_test.exs instead of lib/**/*_test.exs. It is a small mistake, but might save somelate-night debugging of anyone else out there. 🦉

Reasons

By default, ExUnit expects tests to be located at test/**/*_test.exs unless purposefully changed by developer.

If the incorrect path is used, then CircleCI pipelines won't find the project's tests. Depending on the setup, the pipeline might be wrongfully marked as passing even if the tests would fail if executed.

mix test

(..) loads up test/test_helper.exs and then, requires all files matching the test/**/*_test.exs pattern in parallel.

Source: Official Elixir docs

Additionally, the correct path can be confirmed by running mix new (..) to initialize an empty project boilerplate:

❯ mix new sanity_check
# ...
* creating lib
* creating lib/sanity_check.ex
* creating test
* creating test/test_helper.exs
* creating test/sanity_check_test.exs

Content Checklist

Please follow our style when contributing to CircleCI docs. Our style guide is here: https://circleci.com/docs/style/style-guide-overview.

Please take a moment to check through the following items when submitting your PR (this is just a guide so will not be relevant for all PRs):

  • Break up walls of text by adding paragraph breaks.
  • Consider if the content could benefit from more structure, such as lists or tables, to make it easier to consume.
  • Keep the title between 20 and 70 characters.
  • Consider whether the content would benefit from more subsections (h2-h6 headings) to make it easier to consume.
  • Check all headings h1-h6 are in sentence case (only first letter is capitalized).
  • Include relevant backlinks to other CircleCI docs/pages.

@antedeguemon antedeguemon requested review from a team as code owners November 13, 2025 05:21
@rosieyohannan
Copy link
Contributor

Thank you so much for fixing this for us @antedeguemon!

Copy link
Contributor

@rosieyohannan rosieyohannan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@rosieyohannan rosieyohannan merged commit 1093b49 into circleci:main Nov 13, 2025
2 of 3 checks passed
@antedeguemon antedeguemon deleted the antedeguemon-fix-elixir-paths branch November 13, 2025 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants