[checkmk-commits] [tribe29/checkmk] 78d4c1: snmplib test fixture update: Printing error messag...

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 78d4c145296b3eddea50dedf1019216a15fbc563 https://github.com/tribe29/checkmk/commit/78d4c145296b3eddea50dedf1019216a1… Author: René Slowenski <rene.slowenski(a)tribe29.com> Date: 2022-10-21 (Fri, 21 Oct 2022) Changed paths: M tests/integration/cmk/lib/snmplib/conftest.py Log Message: ----------- snmplib test fixture update: Printing error message in case snmpsimd dies Change-Id: I349cbf5e3c182e56e402ecd585ab244298db0443 Commit: 9da9aabe6c1a145e6f243e152743a79604e7f6ca https://github.com/tribe29/checkmk/commit/9da9aabe6c1a145e6f243e152743a7960… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2022-10-21 (Fri, 21 Oct 2022) Changed paths: M cmk/base/sources/__init__.py R cmk/base/sources/fetcher_configuration.py Log Message: ----------- Move fetcher config serialization to CEE It is only useful with the CMC, so => enterprise Change-Id: I2ed8abdc5b7ec9ffb5aab69bfebfb95e3a3035cd Commit: 53fa3dec8c6d522519a99be1cd4f132f1ff84402 https://github.com/tribe29/checkmk/commit/53fa3dec8c6d522519a99be1cd4f132f1… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2022-10-21 (Fri, 21 Oct 2022) Changed paths: A cmk/base/sources.py R cmk/base/sources/__init__.py R cmk/base/sources/_checkers.py M tests/unit/cmk/base/sources/test_cmd_caching.py Log Message: ----------- Move cmk.base.sources._checkers -> cmk.base.source Package with a single file -> module Change-Id: Ic838b287cde13f0fe270baeebe85e696139b019d Commit: 2afffc4bae5737b3e59b4e906c1602590aa3147f https://github.com/tribe29/checkmk/commit/2afffc4bae5737b3e59b4e906c1602590… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2022-10-21 (Fri, 21 Oct 2022) Changed paths: R tests/unit/cmk/base/sources/test_cmd_caching.py Log Message: ----------- Remove unintelligible tests They seem to test some of the modes, which would belong in some component/integration tests. They may also check that globals are set properly, which is more or less testing Python and not that much our own code. In any case, the amount of monkeypatching and the assertions in the fixture make these patch extremely brittle and unintelligible, we are better off without them. 🚮 Change-Id: I914af99288415e9b86f6ef98c6679fbd3cc38744 Compare: https://github.com/tribe29/checkmk/compare/64af4df887fe...2afffc4bae57