Stabilize descending threaded-comments integration assertions

[Copilot]

The integration suite was failing intermittently in the threaded comments descending-order test. The failure came from validating top-level ordering against the full threaded list, which includes interleaved child comments.

• Root cause

  • CommentsThreaded_Sort_Extension_Desc filtered top-level comments into firstLvl, but its final ordering assertions still indexed into threaded.
  • That made the test compare unrelated child comments and fail depending on comment timestamps.

• Change

  • Updated the descending-order assertions to use firstLvl consistently for the top-level ordering check.
  • Removed the extra ID-based assertion from that loop so the test verifies the behavior that actually matters: descending date order among root comments.

• Regression coverage

  • Added a focused unit test for ThreadedCommentsHelper.ToThreaded(true) that constructs a root/child/root comment shape and asserts that:
    • children remain attached to their parent in threaded order
    • top-level comments remain ordered by descending date

List<CommentThreaded>? threaded = comments.ToThreaded(true);
List<CommentThreaded> firstLevel = threaded.Where(comment => comment.Depth == 0).ToList();

CollectionAssert.AreEqual(new[] { 1, 3 }, firstLevel.Select(comment => comment.Id).ToList());
Assert.IsTrue(firstLevel[0].Date >= firstLevel[1].Date);

[Copilot]

Pull request overview

Stabilizes the self-hosted integration test coverage for threaded comments in descending order by ensuring top-level ordering assertions are applied only to top-level comments (not the full interleaved threaded list), and adds a focused regression test for ThreadedCommentsHelper.ToThreaded(true).

Changes:

• Fix descending-order assertions in CommentsThreaded_Sort_Extension_Desc to consistently validate ordering using the top-level (Depth == 0) subset.
• Remove the ID-based ordering assertion that was coupling the test to unrelated threaded/child ordering.
• Add a new self-hosted utility test that verifies top-level descending date order while keeping children attached to their parent in the threaded output.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
WordPressPCL.Tests.Selfhosted/CommentsThreaded_Tests.cs	Updates descending-order assertions to use the filtered top-level list (firstLvl) for ordering checks.
WordPressPCL.Tests.Selfhosted/Utility/ThreadedCommentsHelper_Tests.cs	Adds a regression test validating ToThreaded(true) keeps root comments in descending date order and preserves parent/child adjacency.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.