test_runner: skip more tests using decorator instead of pytest.skip

[bayandin]

Problem

Running pytest.skip(...) in a test body instead of marking the test with @pytest.mark.skipif(...) makes all fixtures to be initialised, which is not necessary if the test is going to be skipped anyway.

Also, some tests are unnecessarily skipped (e.g. test_layer_bloating on Postgres 17, or test_idle_reconnections at all) or run (e.g. test_parse_project_git_version_output_positive more than on once configuration) according to comments.

Summary of changes

• Move skip_on_postgres / xfail_on_postgres / run_only_on_default_postgres decorators to fixture.utils
• Add new skip_in_debug_build and skip_on_ci decorators
• Replace pytest.skip(...) calls with decorators where possible

Checklist before requesting a review

• I have performed a self-review of my code.
• If it is a core feature, I have added thorough tests.
• Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
• If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

• Do not forget to reformat commit message to not include the above checklist

[github-actions]

5346 tests run: 5126 passed, 0 failed, 220 skipped (full report)

---

Flaky tests (1)

Postgres 17

• test_deletion_queue_recovery[no-validate-keep]: debug-x86-64

Code coverage* (full report)

• functions: 31.7% (7869 of 24806 functions)
• lines: 49.4% (62257 of 126036 lines)

* collected from Rust tests only

---

_{The comment gets automatically updated with the latest test results
d190e98 at 2024-11-11T18:05:53.027Z :recycle:}