Skip to content

fix: update test_e2e.py path in SQL test workflows#12

Merged
rophy merged 1 commit intomasterfrom
fix/ci-workflow-paths
Mar 27, 2026
Merged

fix: update test_e2e.py path in SQL test workflows#12
rophy merged 1 commit intomasterfrom
fix/ci-workflow-paths

Conversation

@rophy
Copy link
Copy Markdown
Owner

@rophy rophy commented Mar 27, 2026

Summary

Test plan

  • Path matches actual file location at tests/sql/test_e2e.py

Summary by CodeRabbit

  • Chores
    • Updated automated testing workflow configurations to adjust test discovery paths across Oracle Database environments, improving test execution reliability.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6658b71c-f0f8-4b98-a2d9-198faf0edd7b

📥 Commits

Reviewing files that changed from the base of the PR and between 71dd09d and 12d955f.

📒 Files selected for processing (2)
  • .github/workflows/sql-tests-free23.yaml
  • .github/workflows/sql-tests-xe21.yaml

📝 Walkthrough

Walkthrough

Two GitHub Actions workflows updated to execute pytest against sql/test_e2e.py instead of the previous test discovery path. The change reflects a relocation of test file location while preserving all pytest configuration flags and environment parameters.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Test Path Updates
.github/workflows/sql-tests-free23.yaml, .github/workflows/sql-tests-xe21.yaml
Updated pytest test execution path from tests/test_e2e.py to sql/test_e2e.py in the "Generate all fixtures" step, maintaining all pytest flags and oracle environment parameters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 Tests hop to their new home,
sql/test_e2e.py blooms where they roam,
Workflows adapt with a gentle shift,
Paths realign—a structural gift!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating test file paths in SQL test workflows from tests/ root to sql/ subdirectory.
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
  • Commit unit tests in branch fix/ci-workflow-paths

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

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

@rophy rophy merged commit 6ec5cdb into master Mar 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant