Skip to content

Increate test coverage - #12

Merged
ammancilla merged 3 commits into
mainfrom
task/excoveralls
Jul 14, 2026
Merged

ammancilla merged 3 commits into
mainfrom
task/excoveralls

Conversation

@ammancilla

Copy link
Copy Markdown
Contributor

No description provided.

@ammancilla ammancilla self-assigned this Jul 8, 2026
@ammancilla
ammancilla force-pushed the task/excoveralls branch 6 times, most recently from 596c55b to fa40ce7 Compare July 8, 2026 13:39
@ammancilla
ammancilla marked this pull request as ready for review July 8, 2026 13:54
@ammancilla
ammancilla requested a review from joelpaulkoch July 8, 2026 13:54
Comment thread test/support/tmp_file_fixtures.ex Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We could replace the general tmp helpers with the @tmp_dir tag from ex_unit 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done ✅

@joelpaulkoch joelpaulkoch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great test helpers and good to have better coverage. We could think about using the @tmp_dir tag from ex_unit but I'll already approve

@ammancilla
ammancilla merged commit 1fc2797 into main Jul 14, 2026
1 check passed
@ammancilla
ammancilla deleted the task/excoveralls branch July 14, 2026 08:32
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