Skip to content

Add tests for combine and koyo tag import tools - #22

Merged
IliTheButterfly merged 2 commits into
better-toolsfrom
codex/add-tests-for-combine-tools
Apr 21, 2026
Merged

Add tests for combine and koyo tag import tools#22
IliTheButterfly merged 2 commits into
better-toolsfrom
codex/add-tests-for-combine-tools

Conversation

@IliTheButterfly

Copy link
Copy Markdown
Owner

Motivation

  • Provide automated coverage for the two CLI tools that merge/convert EasyBuilder tags to prevent regressions and ensure deterministic behavior.
  • Verify merging, conversion and append/merge semantics across representative test fixtures (hmi_tags0/1/2.csv, koyo_tags.csv).

Description

  • Add a new test module tests/test_combine_tools.py which exercises the CLI entrypoints via their main() functions (combine_tags_main, koyo_tags_import_main) using controlled sys.argv values.
  • Implement tests for: combining hmi_tags0.csv with hmi_tags1.csv, converting koyo_tags.csv into an EasyBuilder CSV, and importing/merging koyo_tags.csv into hmi_tags0.csv, hmi_tags1.csv, and hmi_tags2.csv.
  • All CLI-style tests use --force --strategy skip so runs are deterministic and non-interactive, and assertions check final list sizes and representative tag address/type outcomes (including an expected absent/skipped tag case).
  • New file: tests/test_combine_tools.py (adds the five tests and small helper runners that set sys.argv).

Testing

  • Ran pytest -q tests/test_combine_tools.py and the new module passed (5 passed).
  • Ran the full test suite with pytest -q and all tests passed (23 passed).

Codex Task

@IliTheButterfly
IliTheButterfly merged commit 5646a47 into better-tools Apr 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant