Skip to content

Add automatic testing to InOutModule#3

Merged
FelixCAAuer merged 2 commits intomainfrom
feature/automaticTesting
Jun 16, 2025
Merged

Add automatic testing to InOutModule#3
FelixCAAuer merged 2 commits intomainfrom
feature/automaticTesting

Conversation

@FelixCAAuer
Copy link
Copy Markdown
Contributor

No description provided.

Rename "example_folder" to "data_folder"
@FelixCAAuer FelixCAAuer requested a review from Copilot June 16, 2025 14:07
@FelixCAAuer FelixCAAuer self-assigned this Jun 16, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 automatic testing for the InOutModule by introducing a new test suite for ExcelReader/ExcelWriter, updating import statements to use relative paths instead of the InOutModule package prefix, and adding a CI workflow to run tests across multiple OS environments.

  • Introduces tests for reading and writing Excel files.
  • Updates module import paths to use relative imports.
  • Adds a CI pipeline configuration.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/test_ExcelReaderWriter.py Adds tests for ExcelReader and ExcelWriter functionality.
printer.py Moves a pyomo import to an internal scope to prevent circular imports.
environment.yml Provides environment configuration for dependencies.
ExcelWriter.py Updates import references and the cell style method signature.
ExcelReader.py Updates import references for Printer.
CaseStudy.py Refactors parameter name from example_folder to data_folder.
.github/workflows/ci.yaml Adds CI workflow to run tests on multiple OS platforms.
Comments suppressed due to low confidence (2)

CaseStudy.py:13

  • The parameter was renamed from 'example_folder' to 'data_folder' in the signature, but corresponding docstrings or inline comments might not reflect this update. Consider reviewing and updating any documentation or comments to maintain consistency.
def __init__(self, data_folder: str, do_not_merge_single_node_buses: bool = False,

tests/test_ExcelReaderWriter.py:10

  • [nitpick] The variable name 'ew' could be made more descriptive (e.g., 'excel_writer') to improve readability in the test code.
ew = ExcelWriter()

@FelixCAAuer FelixCAAuer requested a review from RobertGaugl June 16, 2025 14:12
@FelixCAAuer FelixCAAuer merged commit 89b1722 into main Jun 16, 2025
6 checks passed
@FelixCAAuer FelixCAAuer deleted the feature/automaticTesting branch June 16, 2025 14:18
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