Skip to content

Update Test_Plan_Full_search.md#719

Open
jnweiger wants to merge 2 commits into
masterfrom
jnweiger-more-reindex-tests
Open

Update Test_Plan_Full_search.md#719
jnweiger wants to merge 2 commits into
masterfrom
jnweiger-more-reindex-tests

Conversation

@jnweiger

@jnweiger jnweiger commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

more re-index tests

more re-index tests
@jnweiger
jnweiger requested review from cdamken and hodyroff February 1, 2022 18:11

@dj4oC dj4oC left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds a TODO list of untested full-search re-indexing scenarios (share reject/accept, user removal/re-add, occ search:scan:rebuild) to the Full Search test plan, plus a trailing-newline fix.

Findings

Nothing blocking. Documentation-only change to a manual test plan; no executable test code and no drift/vacuous-assertion risk since these are TODO placeholders rather than claimed-passing scenarios.

Verdict

Approved

🤖 Automated review by Claude Code


Generated by Claude Code

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.

2 participants