[WIP] [test for ai] Simplified the python version detection logic from an if else statement

[PavolSloboda]

Summary by CodeRabbit

• Chores
  • Adjusted Python version detection in test runner scripts to make test environment handling more consistent across Python versions.
  • One script includes a formatting-only tweak; another updates the exit-status evaluation that affects which test-run path is selected around the Python 3.13 threshold. No product-facing behavior or public interfaces changed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b3108a2 and 93f1a13.

📒 Files selected for processing (2)

• test/run-openshift-pytest
• test/run-pytest

---

📝 Walkthrough

Walkthrough

Two test runner scripts were edited: test/run-pytest now uses int(sys.version_info >= (3,13)) which flips the exit-status interpretation and thus which branch sets PYTHON_VERSION; test/run-openshift-pytest received a minor formatting change inside the -c string without changing logic.

Changes

Cohort / File(s)	Summary
Pytest runner logic test/run-pytest	Replaced previous version-check command with python3 -c 'import sys;sys.exit(int(sys.version_info >= (3,13)))', changing the exit-status mapping and therefore inverting which branch assigns PYTHON_VERSION.
Openshift pytest formatting test/run-openshift-pytest	Adjusted the inline python3 -c string formatting (removed a space after semicolon). No change to exit-status logic or control flow.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I nibbled at code, found a tiny twist,
A boolean hop, a semicolon missed,
One flip of a check, one tidy-up cue,
Quiet little changes — a rabbit's small chew. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title claims to simplify Python version detection logic, but the raw summary shows conflicting changes: one file maintains unchanged functionality while the other inverts the branch logic, changing the resulting PYTHON_VERSION assignments.	Clarify whether the changes are true simplifications or if they introduce behavioral differences. Update the title to accurately reflect the actual modifications made to each file.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[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 `@test/run-pytest`:
- Around line 13-16: The version check in the python3 -c condition is inverted
causing PYTHON_VERSION to be set incorrectly; update the inline Python
expression in the conditional (the python3 -c 'import sys;sys.exit(int(...))'
check) to use sys.version_info >= (3,13) instead of sys.version_info < (3,13) so
the if/else branches assign PYTHON_VERSION ("3.12" vs "3") in the correct cases.

---

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b3108a2 and d956e0c.

📒 Files selected for processing (2)

• test/run-openshift-pytest
• test/run-pytest

[github-actions]

Pull Request validation

Failed

🔴 Review - Missing review from a member (1 required)

Success

🟢 CI - All checks have passed

[PavolSloboda]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[PavolSloboda]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.