[checkmk-commits] [tribe29/checkmk] 22a3b5: refactoring: clear up site mocks in tests

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 22a3b5618dfa665cd6785245ac0c7dd9bfd589fa https://github.com/tribe29/checkmk/commit/22a3b5618dfa665cd6785245ac0c7dd9b… Author: Christoph Rauch <christoph.rauch(a)tribe29.com> Date: 2021-05-10 (Mon, 10 May 2021) Changed paths: M tests/unit/cmk/ec/test_rule_matching.py Log Message: ----------- refactoring: clear up site mocks in tests * we already have an autouse site mocking fixture, which should be used everywhere. Change-Id: Ie2dbeb7053b6f3b022e6b36d70bd3e84e30a8d7e Commit: 0ca1b6944df7cc23fdeae59bb846bc127322bd14 https://github.com/tribe29/checkmk/commit/0ca1b6944df7cc23fdeae59bb846bc127… Author: Christoph Rauch <christoph.rauch(a)tribe29.com> Date: 2021-05-10 (Mon, 10 May 2021) Changed paths: M .pre-commit-config.yaml Log Message: ----------- pre-commit: ignore missing trailing newline in werks Change-Id: I2c33d547dd72d9acc79b1dca2cc237b8bea4961d Commit: 42ed2c16c940cbaf790c3dbe083d2177d88b9591 https://github.com/tribe29/checkmk/commit/42ed2c16c940cbaf790c3dbe083d2177d… Author: Christoph Rauch <christoph.rauch(a)tribe29.com> Date: 2021-05-10 (Mon, 10 May 2021) Changed paths: M werk Log Message: ----------- werk: improve "werk pick" * if pre-commit hooks fail, the "werk pick" can now be continued with "git cherry-pick --continue" * if the sha given to "werk pick" isn't a werk, it will work like "git cherry-pick", but it doesn't support the same command line parameters. Change-Id: I32ae2032b0ba8298706d537e33573f35065426d7 Commit: 1a3c48b9cf0de1a2c62b14b7efa43f966d93dbfd https://github.com/tribe29/checkmk/commit/1a3c48b9cf0de1a2c62b14b7efa43f966… Author: Óscar Nájera <oscar.najera(a)tribe29.com> Date: 2021-05-10 (Mon, 10 May 2021) Changed paths: A .werks/12491 Log Message: ----------- 12491 No autorefresh in Custom Graph editor The custom graph editor in a pursuit of convenience would save changes and refresh the page on every change. This convenience resulted in a high performance penalty when editing graphs with multiple metrics, as the entire page would need to be rendered again, and specially recollecting data from the RRD files. With this Werk, the custom graph editor does not execute the page reload upon changes. The user is now responsible for this interaction. Following our UX project, now there is an "Apply" button on the top left corner of the editor to save and apply the changes to the graph. This allows users to place multiple changes at once, without incurring in a performance penalty, due to the constant refreshing of the editor. Users do need to press the "Apply" button to save changes, it is now not enough to just do changes and leave the editor. Change-Id: Ibd3e8f44859300fe9e2de895884348621c877245 Compare: https://github.com/tribe29/checkmk/compare/a19552902837...1a3c48b9cf0d