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.

Sign up for GitHub

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

Jump to bottom

Unit test setup #21

Merged
merged 15 commits into from
Aug 29, 2024
Merged

Unit test setup #21

merged 15 commits into from
Aug 29, 2024

Conversation

timbernat
Copy link
Owner

@timbernat timbernat commented Aug 29, 2024 •
edited
Loading

Description

Set up polymerist.tests to enable pytest compatibility and give a home to unit tests for future PRs

Status

  • Add data directory to tests
    • Make data accessible in build via pyproject.toml
  • Set up filetree in tests which mirrors the package structure of polymerist code
  • Moved existing DISCERNMENT unit tests from .genutils.sequnces.discernment to .tests proper
  • Add module for package resource import and inspection
    • Write unit tests for this module

timbernat added 15 commits August 16, 2024 17:34
@timbernat
Generated directory + module tree for unit tests
986a859
@timbernat
Removed "test_" prefixes from test filetree names (will reserve this …
0a45a51 
…for leaf module-level files)
@timbernat
Added sample data file to test whether unit test data is shipped/acce…
4b93b0e 
…ssed properly
@timbernat
Added namespace header __init__.py files to data dirs to allow pyproj…
2c7109b 
…ect.toml builds with data
@timbernat
Updated pyproject.toml to include data bundled with polymerist.tests …
89e3e0e 
…(commented out polymerist.data for now)
@timbernat
Added module for low-level Python package inspection
4e25010
@timbernat
Moved discernment unit tests to tests submodule, set up basic pytest …
61e34e0 
…fixtures for correct solution read
@timbernat
Broke monolithic DISCERNMENT test into test class with mutliple tests…
31f379e 
… in a crude manner
@timbernat
Added unit tests location for pkginspect
84d8089
@timbernat
Renamed "fake.dat" to "sample.dat" to avoid confusion, added dummy te…
161f84a 
…xt data
@timbernat
Added expected-fail tests to exnsure obviously non-module types fail …
ca907fe 
…when passed to is_module and _is_package
@timbernat
Added test for get_resource_path_within_package()
f5472c8
@timbernat
Added test for get_file_path_within_package()
5ef9833
@timbernat
Added test for get_dir_path_within_package()
8208c97
@timbernat
Made all xfails strict, so that an xpass in a package resource read i…
5dafd0e 
…s recorded as a failure
@timbernat timbernat self-assigned this Aug 29, 2024
timbernat
timbernat commented Aug 29, 2024
View reviewed changes
Copy link
Owner Author

@timbernat timbernat left a comment

Choose a reason for hiding this comment

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

Changes don't touch existing code, and are sorely needed for future feature PRs

@timbernat timbernat merged commit c7f191a into main Aug 29, 2024
0 of 6 checks passed
@timbernat timbernat deleted the unit-test-setup branch August 29, 2024 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@timbernat timbernat

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@timbernat