test: add middle-card navigation test for DeckService#2847
Merged
sydseter merged 2 commits intoOWASP:masterfrom Apr 20, 2026
Merged
test: add middle-card navigation test for DeckService#2847sydseter merged 2 commits intoOWASP:masterfrom
sydseter merged 2 commits intoOWASP:masterfrom
Conversation
Contributor
Author
|
Hi! I noticed the COPI test job is still failing due to the global coverage threshold (95%), currently around ~94%. From what I can see, this doesn’t seem related to this PR, since the changes here are limited to test additions on the website side. Happy to take a closer look at COPI coverage separately if needed. |
sydseter
approved these changes
Apr 20, 2026
Collaborator
|
@theanand108 Thank you for your time! |
Contributor
Author
|
Thanks! Really appreciate your guidance and support — it means a lot. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds a test case to cover navigation logic for a middle card within the same suit.
Changes
Motivation
Existing tests cover navigation for first and last cards, but not the intra-suit navigation case.
This test ensures coverage of that branch.
No production code changes.
AI Disclosure
[x] My contribution includes AI-generated content
AI Tools: ChatGPT
LLMs: GPT-5 class model
Prompts: Used for identifying uncovered branch and structuring test case
Affirmation
[x] My code follows the CONTRIBUTING.md guidelines