Skip to content

[TEST] Added some tests#25

Merged
eleanorfrajka merged 1 commit into
mainfrom
eleanor-tests-1
Sep 20, 2025
Merged

[TEST] Added some tests#25
eleanorfrajka merged 1 commit into
mainfrom
eleanor-tests-1

Conversation

@eleanorfrajka

@eleanorfrajka eleanorfrajka commented Sep 20, 2025

Copy link
Copy Markdown
Contributor

Description

Initial add of tests for plotters, merge_datasets.py and tools.py.

Checklist

  • I have updated or added tests if needed
  • I have updated the documentation if needed
  • I have run the code and verified it works
  • I have run pre-commit run --all-files (if pre-commit is installed)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds initial test coverage for three key modules in the WBTSdata package: plotters, merge_datasets, and tools. The tests include unit tests, edge case handling, and integration test placeholders to establish a testing foundation for the codebase.

  • Added comprehensive test suites for tools.py (config loading and unit conversion functions)
  • Added test coverage for plotters.py (plotting functions with matplotlib testing patterns)
  • Added test structure for merge_datasets.py (directory listing and dataset merging functions)

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/test_tools.py Unit tests for config loading and unit conversion functionality
tests/test_plotters.py Tests for plotting functions including edge cases and visual tests
tests/test_merge_datasets.py Tests for dataset merging and directory discovery functions
tests/conftest.py Pytest fixtures providing sample datasets and configuration for testing
requirements-dev.txt Added pytest dependencies for testing infrastructure
pyproject.toml Added pytest configuration with markers and test options
WBTSdata/plotters.py Minor fix to ensure y-tick positions are set before labels

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread tests/test_tools.py
Comment thread tests/test_plotters.py
Comment thread tests/test_plotters.py
@eleanorfrajka eleanorfrajka merged commit 3722297 into main Sep 20, 2025
1 check passed
@eleanorfrajka eleanorfrajka deleted the eleanor-tests-1 branch September 20, 2025 16:33
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