Skip to content

fix(scripts): pass arguments to pytest in run_tests.sh - #774

Merged
waltsims merged 2 commits into
waltsims:masterfrom
amir-rezaei:fix/run-tests-script-pytest-args
Jul 24, 2026
Merged

fix(scripts): pass arguments to pytest in run_tests.sh#774
waltsims merged 2 commits into
waltsims:masterfrom
amir-rezaei:fix/run-tests-script-pytest-args

Conversation

@amir-rezaei

@amir-rezaei amir-rezaei commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

This PR allows passing CLI arguments to pytest when running run_tests.sh.

Details

Updated script execution line (pytest $\to$ pytest "$@") to forward options (such as specific test file paths or markers).

Greptile Summary

Updates local helper scripts:

  • Forwards all command-line arguments from run_tests.sh to pytest while preserving argument boundaries.
  • Standardizes the make_docs.sh shebang and whitespace.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The pytest arguments remain individually quoted and are placed correctly after the container command, while the documentation script changes do not alter its build command.

Important Files Changed

Filename Overview
run_tests.sh Correctly forwards caller arguments to pytest inside the Docker container using quoted "$@".
make_docs.sh Standardizes the Bash shebang and removes insignificant whitespace without changing documentation-build behavior.

Reviews (1): Last reviewed commit: "fix(scripts): pass arguments to pytest i..." | Re-trigger Greptile

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.83%. Comparing base (aff369e) to head (1616240).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #774   +/-   ##
=======================================
  Coverage   75.83%   75.83%           
=======================================
  Files          58       58           
  Lines        8274     8274           
  Branches     1614     1614           
=======================================
  Hits         6275     6275           
  Misses       1373     1373           
  Partials      626      626           
Flag Coverage Δ
3.10 75.80% <ø> (ø)
3.11 75.80% <ø> (ø)
3.12 75.80% <ø> (ø)
3.13 75.80% <ø> (ø)
macos-latest 75.71% <ø> (ø)
ubuntu-latest 75.71% <ø> (ø)
windows-latest 75.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@waltsims

Copy link
Copy Markdown
Owner

Thanks for the PR!

@waltsims
waltsims merged commit 25f4164 into waltsims:master Jul 24, 2026
19 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.

3 participants