[checkmk-commits] [Checkmk/checkmk] 3851b9: Split parametrized test into more descriptive indi...

Branch: refs/heads/2.3.0 Home: https://github.com/Checkmk/checkmk Commit: 3851b984537a5ef94c6fbcca02cd0098f87f75e3 https://github.com/Checkmk/checkmk/commit/3851b984537a5ef94c6fbcca02cd0098f… Author: René Slowenski <rene.slowenski(a)checkmk.com> Date: 2024-05-31 (Fri, 31 May 2024) Changed paths: M tests/unit/cmk/gui/test_sites.py Log Message: ----------- Split parametrized test into more descriptive individual tests We should not have single-line tests that have a huge amount of parameters as this is really hard to read and maintain. Additionally we need to be careful when parametrizing tests with dynamic data as this may unintentionally lead to dynamic test names as well. Change-Id: Idd365f8bfdcdbe4fea35f98fb87096c16e89cd16 (cherry picked from commit 6191f3353900acdbc87a8507c77a86b8a9df172d) Commit: c42e225bfbbfdbe698e4c2a43977265df934ec08 https://github.com/Checkmk/checkmk/commit/c42e225bfbbfdbe698e4c2a43977265df… Author: René Slowenski <rene.slowenski(a)checkmk.com> Date: 2024-05-31 (Fri, 31 May 2024) Changed paths: M tests/unit/cmk/base/test_diagnostics.py Log Message: ----------- Set static test name for parametrized test We should be careful when using dynamic data in a parametrization as it is rarely desired to generate dynamic test case names. Change-Id: Ic2ce224a59bbbb3ffeb5e6f5c92c526dbb2976ee (cherry picked from commit 379cad41a66e4754150c5f3136a97cbe26a078f7) Compare: https://github.com/Checkmk/checkmk/compare/ec422ec579e3...c42e225bfbbf To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications