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

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 6191f3353900acdbc87a8507c77a86b8a9df172d https://github.com/Checkmk/checkmk/commit/6191f3353900acdbc87a8507c77a86b8a… Author: René Slowenski <rene.slowenski(a)checkmk.com> Date: 2024-05-30 (Thu, 30 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 Commit: 379cad41a66e4754150c5f3136a97cbe26a078f7 https://github.com/Checkmk/checkmk/commit/379cad41a66e4754150c5f3136a97cbe2… Author: René Slowenski <rene.slowenski(a)checkmk.com> Date: 2024-05-30 (Thu, 30 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 Compare: https://github.com/Checkmk/checkmk/compare/c448a3d0769f...379cad41a66e To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications