docs(kaneai): correct what several configurations do to Adaptive Heal and Dynamic Test - #3516
Open
anubhavsinghmaar wants to merge 1 commit into
Open
anubhavsinghmaar wants to merge 1 commit into
anubhavsinghmaar wants to merge 1 commit into
Conversation
… and Dynamic Test The page said only the first test instance picked up for execution is eligible, that the rest replay their recorded steps without being repaired, and that a test case across several Chrome or operating system versions is not supported. That is not the behaviour, and it sent a reader to the wrong conclusion twice: that most of their instances would be skipped, and that they should split a run to get coverage. What actually happens concerns the draft, not eligibility. A test run can include the same test case under several configurations, and it produces a single draft, because it is one test case. The claim is removed from When these strategies do not apply, where it did not belong, since this is not a case of the strategies failing to apply. The correct behaviour is stated as a note in Reviewing a change, beside the version it is about. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Corrects a factual error introduced in #3492 and carried into #3515, both now merged.
What the page says today
Why that is wrong
It describes eligibility, and the real behaviour concerns the draft. As written it leads a reader to two wrong conclusions: that most of their test instances will be silently skipped, and that they should split one run into several to get coverage.
What it says now
The claim is removed from When these strategies do not apply, where it did not belong, because this is not a case of the strategies failing to apply.
The correct behaviour is stated as a note in Reviewing a change, beside the version it is actually about:
Verification
stagebefore applying, so nothing merged since docs(kaneai): scope, cost and screenshots for Adaptive Heal and Dynamic Test #3515 is clobbered🤖 Generated with Claude Code