Skip to content

Add tests for start-from-nothing cases of usethis tool requirements.txt based on backend#1255

Merged
nathanjmcdougall merged 2 commits into
mainfrom
894-dont-create-pyprojecttoml-unnecessarily-when-creating-requirementstxt
Dec 30, 2025
Merged

Add tests for start-from-nothing cases of usethis tool requirements.txt based on backend#1255
nathanjmcdougall merged 2 commits into
mainfrom
894-dont-create-pyprojecttoml-unnecessarily-when-creating-requirementstxt

Conversation

@nathanjmcdougall

Copy link
Copy Markdown
Collaborator

Also bump the uv fallback version

…txt` based on backend

also bump the uv fallback version

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds test coverage for the requirements.txt tool when starting from scratch with different backend configurations, and updates the fallback uv version.

  • Adds test for none backend configuration when creating requirements.txt from nothing
  • Renames existing test to clarify it tests the uv backend case
  • Updates fallback uv version from 0.9.18 to 0.9.20

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/usethis/_core/test_core_tool.py Adds test for none backend and renames existing test to indicate uv backend
src/usethis/_integrations/backend/uv/version.py Updates fallback uv version constant

@codecov

codecov Bot commented Dec 30, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.44444% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/usethis/_core/tool.py 88.88% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@codspeed-hq

codspeed-hq Bot commented Dec 30, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #1255 will not alter performance

Comparing 894-dont-create-pyprojecttoml-unnecessarily-when-creating-requirementstxt (dbb5824) with main (d30f920)

Summary

✅ 2 untouched

@nathanjmcdougall nathanjmcdougall merged commit a22f939 into main Dec 30, 2025
19 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 894-dont-create-pyprojecttoml-unnecessarily-when-creating-requirementstxt branch December 30, 2025 09:48
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.

Don't create pyproject.toml unnecessarily when creating requirements.txt

2 participants