Consistently use mut marker for tests that write to the database

[PGijsbers]

Even within a transaction, as some requests may stall until transactions with write requests are committed/aborted.

[coderabbitai]

Walkthrough

This change updates pytest markers across several test modules. The @pytest.mark.mut marker was added to tests in:

• tests/routers/openml/dataset_tag_test.py (3 tests)
• tests/routers/openml/datasets_test.py (1 test)
• tests/routers/openml/migration/datasets_migration_test.py (1 test)
• tests/routers/openml/migration/setups_migration_test.py (3 tests)
• tests/routers/openml/qualities_test.py (1 test)
• tests/routers/openml/study_test.py (5 tests)

The @pytest.mark.mut marker was removed from one test in tests/routers/openml/migration/flows_migration_test.py. No test logic, assertions, parameterization, or public/exported entities were modified. Total changes are additive/removal of decorators only.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 5.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding mut markers to tests that write to the database for consistent handling across the test suite.
Description check	✅ Passed	The description explains the rationale for the change (preventing request stalls in transactions with write operations), which relates to the changeset's purpose of marking database-write tests.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch mut-marker

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@tests/routers/openml/study_test.py`:
- Line 461: The test module uses the `@pytest.mark.mut` decorator but never
imports pytest, causing NameError during collection; add a top-level import for
pytest (i.e., import pytest) in the same test file that contains the
`@pytest.mark.mut` usages so the decorator resolves (search for occurrences of
`@pytest.mark.mut` to locate the affected module).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 71e844be-d238-43b4-b13c-4ee032635c4b

📥 Commits

Reviewing files that changed from the base of the PR and between f77c4a8 and 00675ab.

📒 Files selected for processing (7)

• tests/routers/openml/dataset_tag_test.py
• tests/routers/openml/datasets_test.py
• tests/routers/openml/migration/datasets_migration_test.py
• tests/routers/openml/migration/flows_migration_test.py
• tests/routers/openml/migration/setups_migration_test.py
• tests/routers/openml/qualities_test.py
• tests/routers/openml/study_test.py

💤 Files with no reviewable changes (1)

• tests/routers/openml/migration/flows_migration_test.py