Closed jjjermiah closed 1 month ago
Thansk @jjjermiah ! You probably want to remove the __pycache__
stuff.
thanks for pointing that out @vsoch I also tried formatting and fixing the test suite
This update enhances the GitHub Actions workflow for testing, improves handling of Google Cloud Storage in the snakemake_storage_plugin_gcs
module with new methods, and refines test cases for better clarity and functionality. Key changes include increased verbosity in test outputs, a new .gitignore
file to exclude unnecessary files, and adjustments to ensure comprehensive testing of both files and directories.
Files | Change Summary |
---|---|
.github/workflows/ci.yml |
Updated pytest command to include more verbosity and display print outputs during tests. |
.gitignore |
New file created to prevent tracking of autogenerated files such as .coverage and __pycache__ . |
snakemake_storage_plugin_gcs/__init__.py |
Added ensure_bucket_exists and upload_directory methods; refactored store_object and remove methods for improved functionality and clarity. |
tests/tests.py |
Updated TestStorage class to prioritize file operations, added tests for non-empty directories, and improved debug output and cleanup. |
sequenceDiagram
participant User
participant CI
participant GCS
participant StoragePlugin
User->>CI: Trigger Test Workflow
CI->>StoragePlugin: execute tests
StoragePlugin->>GCS: ensure_bucket_exists()
GCS-->>StoragePlugin: bucket status
StoragePlugin->>StoragePlugin: upload_directory()
StoragePlugin->>GCS: upload files
GCS-->>StoragePlugin: confirm upload
StoragePlugin->>CI: return test results
CI->>User: display results
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Thanks a lot for the fixes!
and sorry for the slow reply. This somehow slipped my attention. Feel free to contact me via discord or email if something like that happens again in the future.
first contribution here. Think I found the error.
This works for my test Snakefile:
I also tried to add another test case which seems to work. I see that the test suites in the storage plugins use
testClasses
. Would adding actual snakefiles and a script to run them also help or does that not follow the plugin standards?Summary by CodeRabbit
New Features
ensure_bucket_exists
andupload_directory
.Bug Fixes
Chores
.gitignore
file to improve repository cleanliness by excluding unnecessary files.Tests