[ste1hi]

Inprove random.sample test. Add tests for when the items in the list are not integers.
Add test for os.rename function.

[bedevere-app]

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

[picnixz]

I don't think the tests for random are relevant. It's a garbage-in/garbage-out case. For os.renames, it makes sense (and I'm surprised we didn't have a test!)

[python-cla-bot]

All commit authors signed the Contributor License Agreement.

[ste1hi]

I removed the test for random.sample

[bedevere-app]

Thank you for your interest in contributing to Python. This is a community-driven project, and we want it to be a welcoming place to learn and collaborate.

Contributors are responsible for reviewing and understanding what they submit, checking its claims, and being able to explain changes in their own words. This helps reviewers focus on the contribution and leaves more time to support other contributors.

This contribution appears to have been authored with AI assistance. Please explain your contribution in your own words and describe how you checked its correctness.

Please read our guidelines for using AI tools.

[picnixz]

@StanFromIreland This PR doesn't look like AI I guess?

[StanFromIreland]

I suppose, it's hard to tell sometimes.

[ste1hi]

I really did NOT use any AI assistant. Btw, I think there is no need to use AI for this simple case.

Explanation

I made a file named test in old_dir directory. Then I used os.renames to move this file to new_dir/test. Finally, I checked if the old_dir and old file test had been removed and new directory had been created.

Just curious, I want to know why you think I used AI assistant and how can I prove that I have not used AI assitant.

[StanFromIreland]

That's fine, I'm sorry I didn't intend it as an insult or such thing. What I am curious about however, is how you checked coverage? What are the statistics before/after?

[picnixz]

I think the coverage checking is simple using coverage.py. FTR, we don't even test os.renames at all in the file (i did a Crtl+F of os.renames itself)

[ste1hi]

before is 51%, and after is 52%
As you can see, it's really a simple and small change.