LCORE-987: Minor e2e tests fixes

[tisnik]

Description

LCORE-987: Minor e2e tests fixes

Type of change

• Refactor
• New feature
• Bug fix
• CVE fix
• Optimization
• Documentation Update
• Configuration Update
• Bump-up service version
• Bump-up dependent library
• Bump-up library or tool used for development (does not change the final image)
• CI configuration change
• Konflux configuration change
• Unit tests improvement
• Integration tests improvement
• End to end tests improvement

Related Tickets & Documents

• Related Issue #LCORE-987

Summary by CodeRabbit

• Documentation
  • Corrected HTTP method descriptions in test documentation.
  • Fixed typo in test step definition documentation.

[coderabbitai]

Walkthrough

This PR corrects documentation inaccuracies in test step definitions: updating docstrings for conversation endpoints to correctly describe DELETE requests instead of GET, and fixing a typo in an LLM query response comparison docstring.

Changes

Cohort / File(s)	Summary
Docstring corrections in test step definitions tests/e2e/features/steps/conversation.py	Updated two step definition docstrings to correctly describe DELETE HTTP requests instead of GET for access_conversation_endpoint_delete and access_conversation_endpoint_delete_specific
Typo fix in test step definitions tests/e2e/features/steps/llm_query_response.py	Corrected spelling error in compare_streamed_responses docstring: "reponse" → "response"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• LCORE-492: added tests for v1/conversations endpoints #646: Corrects HTTP method documentation (GET → DELETE) in the same conversation test step file, addressing related docstring accuracy.
• LCORE-987: proper HTTP method in conversation endpoint e2e tests #800: Addresses HTTP method corrections for conversation endpoint e2e tests (GET → DELETE), closely aligned with this PR's conversation endpoint fixes.

Pre-merge checks and finishing touches

✅ 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 concisely describes the main changes: minor fixes to end-to-end tests, specifically docstring corrections in test step definitions.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

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

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 437f6bc and 9123845.

📒 Files selected for processing (2)

• tests/e2e/features/steps/conversation.py (2 hunks)
• tests/e2e/features/steps/llm_query_response.py (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: build-pr
• GitHub Check: Konflux kflux-prd-rh02 / lightspeed-stack-on-pull-request
• GitHub Check: e2e_tests (ci)
• GitHub Check: e2e_tests (azure)

🔇 Additional comments (3)

tests/e2e/features/steps/llm_query_response.py (1)

146-152: Docstring typo corrected.

The docstring has been corrected to fix the spelling of "response". This improves documentation accuracy without affecting function logic or behavior.

tests/e2e/features/steps/conversation.py (2)

57-73: Docstring corrected to reflect actual HTTP method.

The docstring for access_conversation_endpoint_delete has been updated from "Send GET HTTP request" to "Send DELETE HTTP request", which now accurately describes the requests.delete() call on line 73. Documentation now matches implementation.

---

79-92: Docstring corrected to reflect actual HTTP method.

The docstring for access_conversation_endpoint_delete_specific has been updated from "Send GET HTTP request" to "Send DELETE HTTP request", which now accurately describes the requests.delete() call on line 92. Documentation now matches implementation.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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