Skip to content
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

Perl test cleanup #298

Closed
wants to merge 6 commits into from
Closed

Perl test cleanup #298

wants to merge 6 commits into from

Conversation

wingolab
Copy link
Collaborator

An approach to tidying up tests by standardizing imports, using temporary directories, and using a testing library for common test components.

The intention is to illustrate an approach to tidying tests so that they are easier to run - i.e., do not produce artifacts because they add or update files under version control - and maintain.

Thomas Wingo added 6 commits October 16, 2023 20:55
- change cpanm to cpm for faster building
- minor tweaks to dist and cpanfile
- use tempdir for writing database
- remove unused packages
- organize package imports
- Use temporary directory for database
- Use test helper library t/lib for common functions
- Remove testing artifacts (i.e., test dbs)
- Style imports to improve clarity
- Remove some unused imports
@akotlar
Copy link
Collaborator

akotlar commented Oct 17, 2023

I think this is a good change. Ready to review when out of Draft

@akotlar akotlar mentioned this pull request Oct 17, 2023
@wingolab
Copy link
Collaborator Author

This was submitted as PR #303.

@wingolab wingolab closed this Oct 18, 2023
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