Skip to content

fix: correct dynamic URL rendering - #1069

Open
ketankurhade wants to merge 1 commit into
scanapi:mainfrom
ketankurhade:fix/1007-dynamic-url
Open

ketankurhade wants to merge 1 commit into
scanapi:mainfrom
ketankurhade:fix/1007-dynamic-url

Conversation

@ketankurhade

Copy link
Copy Markdown

Summary

Fix dynamic URL rendering when variables or Python expressions are surrounded by literal text.

Changes

  • Corrected variable and Python expression matching so surrounding literal URL text is preserved.
  • Added tests for prefixes and suffixes around dynamic variables and expressions.
  • Added request URL integration coverage.

Testing

  • uv run pytest tests/unit/evaluators/ tests/unit/tree/request_node/test_full_path_url.py
  • uv run ruff check scanapi tests
  • git diff --check

@ketankurhade
ketankurhade requested review from a team as code owners September 1, 2026 16:47
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

📌 Esta mensagem está tanto em português quanto em inglês (mais abaixo) — assim todo mundo consegue acompanhar!
📌 This message is in both Portuguese and English (further down) — so everyone can follow along!

🇧🇷 Português

👋 Olá!

Este PR está obsoleto porque ficou aberto por 30 dias sem atividade. Remova o rótulo de stale ou comente, caso contrário ele será fechado em 15 dias.

🇬🇧 English

Hey there! 👋

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 15 days.

@github-actions github-actions Bot added the Stale label Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant