Skip to content

added --rp-launch-attributes and --rp-tests-attributes CLI args#411

Open
fahadnaeemkhan wants to merge 1 commit intoreportportal:developfrom
fahadnaeemkhan:add_rp_attrs
Open

added --rp-launch-attributes and --rp-tests-attributes CLI args#411
fahadnaeemkhan wants to merge 1 commit intoreportportal:developfrom
fahadnaeemkhan:add_rp_attrs

Conversation

@fahadnaeemkhan
Copy link

@fahadnaeemkhan fahadnaeemkhan commented Feb 23, 2026

When triggering pytest from some service, its lot convenient to pass CLI args instead of generating pytest.ini

Summary by CodeRabbit

New Features

  • Added command-line options --rp-launch-attributes and --rp-tests-attributes to allow users to specify launch and test attributes directly from the command line, supporting multiple values in addition to INI file configuration.

@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5b84bff and 72129cc.

📒 Files selected for processing (2)
  • pytest_reportportal/plugin.py
  • tests/unit/test_plugin.py

Walkthrough

The pull request adds two new command-line options (--rp-launch-attributes and --rp-tests-attributes) to the pytest ReportPortal plugin, replacing static INI definitions with explicit help strings and CLI wiring. Test expectations are updated to account for these new options.

Changes

Cohort / File(s) Summary
CLI Options Addition
pytest_reportportal/plugin.py
Introduces explicit help text variables and adds --rp-launch-attributes and --rp-tests-attributes CLI options with nargs="+" to accept list inputs, while maintaining INI configuration support.
Test Expectation Update
tests/unit/test_plugin.py
Updates the expected_argument_names tuple to include the two newly added command-line options in the pytest_addoption verification test.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Two attributes now hoppingly free,
On CLI they dance with such glee,
No more INI bounds,
Commands leap and bound,
Configuration flexibility for thee!

🚥 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 clearly and specifically describes the main change: adding two new CLI arguments (--rp-launch-attributes and --rp-tests-attributes), which aligns with the changeset that adds these command-line options.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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.

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