RDBTC-221 Migrate Technical Guides: Writing unit tests with RavenDB .NET Test Driver#2514
Open
Netzach-Nyss wants to merge 1 commit into
Open
RDBTC-221 Migrate Technical Guides: Writing unit tests with RavenDB .NET Test Driver#2514Netzach-Nyss wants to merge 1 commit into
Netzach-Nyss wants to merge 1 commit into
Conversation
Contributor
|
Please fix P1-P3 |
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
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.
Issue link
RDBTC-221 Migrate technical guides and how-tos from ravend.net/articles to docs.ravendb.net/guides
Additional description
Migrating Article "Writing unit tests with RavenDB .NET Test Driver" and it's image.
Content Quality Score: 71/100 | AI Citation Readiness: 58/100
E-E-A-T Breakdown
usingdirectives in samples, disposal not explainedIssues Found
P1 (High) —
Contractorclass used in Steps 3 & 4 but never definedContractorwithFirstName/LastNameis stored in Steps 3 and 4 but has no class definition in the guide.CustomerandLocationare defined before Step 5. Beginners have nothing to reference.P2 (High) — No
usingdirectives in any code sampleusing Raven.TestDriver;,using Raven.Client.Documents;,using Xunit;missing everywhere. A beginner copying the first snippet getsCS0246immediately.P3 (High) — Conclusion is marketing boilerplate
"Elevate your testing practices to new heights" / "Embrace the power..." — flagged as low-quality AI content markers per Sept 2025 QRG.
P4 (Medium) — No version anchor in prose
NuGet install command unpinned, body text never states RavenDB 7.x target.
P5 (Medium) —
WaitForUserToContinueTheTestcaveat incompleteSays it only works with debugger attached but doesn't say it's silently skipped in CI — a real gotcha.
P6 (Medium) — License section has no code example
Tells readers to set
RAVEN_LICENSEbut gives no shell snippet or xUnit injection pattern.P7 (Low) — Step 2 sub-heading redundancy
#### Step 2: Creating a test case...duplicates the parent### 2.number label.Type of change
/templatesor readme)Changes in docs URLs
/scripts/redirects.jsonfile, setDocuments MovedPR label)Changes in UX/UI