Skip to content

[test] Add unit tests for formatting utility functions#177

Open
adity1raut wants to merge 1 commit into52North:mainfrom
adity1raut:tests/add-formatting-utils-tests
Open

[test] Add unit tests for formatting utility functions#177
adity1raut wants to merge 1 commit into52North:mainfrom
adity1raut:tests/add-formatting-utils-tests

Conversation

@adity1raut
Copy link
Copy Markdown

@adity1raut adity1raut commented Mar 18, 2026

Related Issue / Discussion:

Fixes Issue #XX

image

Changes:

  • New tests/test_formatting.py

Summary:

Before: WeatherRoutingTool/utils/formatting.py had zero test coverage.
After: 13 unit tests covering get_line_string, get_log_step,
get_point_from_string, and get_bbox_from_string, grouped into
pytest classes. Tests are pure unit tests with no I/O or fixtures needed.

Dependencies:

None.

PR Checklist:

In the context of this PR, I:

Please consider that PRs which do not meet the requirements specified in the checklist will not be evaluated. Also, PRs with no activities will be closed after a reasonable amount of time.

Signed-off-by: Aditya Raut <araut7798@gmail.com>
@kdemmich
Copy link
Copy Markdown
Collaborator

Hi @adity1raut, thank you for your PR. We appreciate the time and effort that was put into it. In the PR template checklist, please ensure that your changes follow the WRT guidelines for contributors. Then, we will review your PR.

@kdemmich kdemmich added the question Further information is requested label Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants