Skip to content

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Jan 16, 2026

This pull request contains improvements for user scripts. These include:

  • More shared code
  • Keyboard control and improvement of the display of watch scripts
  • Partial pytest for scripts
  • Reusable fake objects (move code into separate files)
  • Added watch-cfi and dump-cfi commands to CLI

relates to #221

dump-cfi / watch-cfi commands are now available in CLI
This allows the fake classes to be used in multiple unit tests.
No productive code change!
@github-actions
Copy link

github-actions bot commented Jan 16, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __main__.py25250%3–66
   data_vector.py115199%42
   datatypes.py399299%57, 127
   discover.py58198%63
luxtronik/cfi
   interface.py1491391%79–92, 241–242, 247
   parameters.py31197%49
luxtronik/scripts
   __init__.py35586%7–16, 51
   discover_shi.py60600%8–98
   dump_cfi.py14657%18–25, 29
   dump_shi.py15660%21–28, 32
   performance_shi.py63630%7–89
   update_screen.py1058222%6, 20–41, 46–48, 51–61, 64, 70–75, 78, 81–85, 88–92, 95–97, 100–103, 106–111, 115–116, 119–126, 129–136
   watch_cfi.py321650%36–47, 50–56, 60
   watch_shi.py331652%38–49, 52–58, 62
luxtronik/shi
   contiguous.py136299%265–266
TOTAL239929988% 

Tests Skipped Failures Errors Time
380 0 💤 0 ❌ 0 🔥 29.828s ⏱️

@@ -1,52 +1,29 @@
#! /usr/bin/env python3

# pylint: disable=invalid-name
"""
Script to dump all available config interface values of the Luxtronik controller
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this file / script be renamed (dump_cfi.py) to stay consistent with the dump_shi.py one?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@kbabioch kbabioch merged commit c5054c8 into Bouni:main Jan 18, 2026
4 checks passed
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