Skip to content

Enhance the Tests section with best practices for clarity and documentation #509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
akhilkrishnar0 opened this issue May 20, 2025 · 4 comments

Comments

@akhilkrishnar0
Copy link

akhilkrishnar0 commented May 20, 2025

Hi everyone,

I'd like to propose adding a new “Tests” section to the packaging guide that outlines best practices for writing clear, well-documented tests. This idea came up in a recent conversation pyOpenSci/software-peer-review#201, where we discussed the value of making tests easier for reviewers and contributors to understand.

Here’s what the new section could include:

  • Use of meaningful test names that reflect what is being tested
  • Docstrings or comments in test functions/modules that describe the purpose of the test
  • Clear test structure and organisation
  • Simple examples of good and bad test documentation practices

This would help both reviewers (who need to understand what’s being tested) and new contributors (who may want to add tests or debug).
Thanks to @ucodery for suggesting that I open this issue to get the conversation started.
Any comments or contributions are invited.

@akhilkrishnar0 akhilkrishnar0 changed the title Add a “Tests” section to the packaging guide with best practices for test clarity and documentation Enhance the Tests section with best practices for clarity and documentation May 21, 2025
@akhilkrishnar0
Copy link
Author

I found that tests are already covered in this issue: #59 (comment)

I'm realising now that the current "Tests" section already does a solid job of covering the basics. As a beginner, I don’t feel I can meaningfully contribute more at this stage, so I’ll step back and leave this issue open in case someone with more experience wants to build on it. I'm happy to revisit this in the future as I gain more insight.

@lwasser
Copy link
Member

lwasser commented May 29, 2025

@all-contributors add @akhilkrishnar0 for ideas, review

Copy link
Contributor

@lwasser

I've put up a pull request to add @akhilkrishnar0! 🎉

@lwasser
Copy link
Member

lwasser commented May 29, 2025

hey @akhilkrishnar0 I can add this issue to our help-wanted board. Could you provide some more details around what you'd like to see added to that section? I'm asking for this because it will make it easier for someone else to pick things up if we know what is missing!! Many thanks!

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

No branches or pull requests

2 participants