Pull contributions from ma4nn/fritzbox-munin-fast #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

ma4nn wants to merge 54 commits into RenWal:master from ma4nn:master

ma4nn commented Sep 28, 2023

I know this is a rather huge change but I did several improvements to the code base, so I thought you could also be interested. Unfortunately it was not really feasible to do this in single pull requests. In case it doesn't fit, simply close this PR :-)

Highlights

Added unit tests that are automatically executing via Github Actions (basically compare outputs to a fixed set of expectations)
Fixed DSL plugin for latest FritzOS version to also use POST to data.lua
Fixed some code styles to comply with pylint
Raised minimum Python version to 3.10

ma4nn and others added 30 commits

October 6, 2021 13:09


          Merge pull request #13 from RenWal/master

cd8b71e

Compatibility with FritzOS 7.27


          run pylint check also for pull requests

2966bf9

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          improved code styles

0b879fb

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          added unit test for connection uptime module, fixed code styles for c…

4fe0603

…onnection uptime module

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          added unit test for dsl module

fec1417

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          added unit test for ecostat module

9fd7fb6

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          added unit test for energy module

bc296ff

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          added unit test for traffic module

975dbf1

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          made unit test suite work and moved all tests into subdirectory

bb5c247

Signed-off-by: Christoph Massmann <c.massmann@vianetz.com>


          Merge branch 'RenWal:master' into master

669e814


          Merge pull request #14 from RenWal/master

a29ddf4

Update to FritzOS 7.50 API


          improved some code styles

432b03f

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Merge remote-tracking branch 'origin/master'

e5d4b20


          switched to pytest for test suite (much easier parametrization), adap…

280f5d7

…ted dsl plugin to new FritzOS 7.57, added pytest to Github Actions

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          renamed test base class to contain test_ prefix in order for pytest t…

b3dc545

…o work correctly with assert_stdout helper method

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          renamed github action a bit

a068d17

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          renamed camelCased files according to pylint

f797a7e

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          minor refactorings

8fcd80c

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          fixed error in github action

cf204c2

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added timeout to requests, minor code improvements

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          refactored tests to use pytest output assertions, default certificate…

6c74cca

… file is now empty, added multiple python versions to github action

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          fixed typo in github actions

d5a34c8

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          removed old python version 3.9 (added only for testing reasons) in gi…

3b0b4cd

…thub actions, added code coverage report

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Create dependabot.yml

53cf590


          replaced test base class by using pytest conftest.py, raised pylint m…

d2ec6f9

…inimum level to 9

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          renamed file to better reflect its purpose

9a1a4ab

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Merge remote-tracking branch 'origin/master'


          renamed file to better reflect its purpose, used MagicMock instead of…

eefffdc

… Mock, replaced deprecated uptime call

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          implemented munin interface class and unified print_* methods

7821e34

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          unified main_handler

Signed-off-by: Christoph Massmann <cm@vianetz.com>

ma4nn and others added 24 commits

September 29, 2023 20:26


          some minor refactorings and content updates

943e735

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Update fritzbox_file_session.py

9e5b3c6

Avoids a runtime error with Python versions < 3.10


          Update and rename fritzbox_smart_home_temperature.py to fritzbox_smar…

e9aaef5

…t_home.py

Extended Smart Home from reading temperature sensors only to temperature (where sensor value is valid), switch states, power and energy consumption.


          Update README.md

d76642f


          made scripts compatible with Python 3.9 (this is the default in Raspb…

94c88f2

…erry Pi OS)

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          improved code style a bit

5373b85

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Merge pull request #17 from ma4nn/python39_compatibility

77ed926

Python 3.9 compatibility


          Merge branch 'master' into wvogel/master

015eb20


          minor code refactoring to remove pylint warnings

46d4998

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added unit tests for smart home plugin

970ef63

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Merge branch 'add_unit_tests_for_smart_home' into wvogel/master

87ebca6


          updated unit tests

49401bf

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          used new FritzHomeAutomation object

1d4ff22

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          Merge pull request #18 from ma4nn/add_unit_tests_for_smart_home

added unit tests for smart home plugin


          Merge branch 'wvogel/master' into refactor_link_saturation

1d42cff


          refactored link saturation module

43fc243

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added 7530ax as fixture and supported fritzbox model, started refacto…

3068f97

…ring of dsl module to use fritzconnection, removed warning values for dsl as they make no sense, added more information for debug mode

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added python 3.12 to github action, disabled tls by default to avoid …

0f99ff1

…issues with missing certificates, added setting traffic_remove_max

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          updated deprecated github actions

dd4386a

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          updated deprecated github actions

59ed1cf

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          minor readme update

5cd097d

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          updated readme to include latest fritzos update

d31ef14

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added factory methods to simplify handling with Fritz* classes

f05ec64

Signed-off-by: Christoph Massmann <cm@vianetz.com>


          added python 3.13

55f1d22

Signed-off-by: Christoph Massmann <cm@vianetz.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet