test: Comment Threading & States#2
Conversation
| @@ -0,0 +1,11 @@ | |||
| // Line 1: context | |||
| // Line 2: added - CT-01 active thread here | |||
There was a problem hiding this comment.
[CT-01] Active thread - status 1 (open discussion)
| // Line 1: context | ||
| // Line 2: added - CT-01 active thread here | ||
| // Line 3: added | ||
| // Line 4: added - CT-02 resolved thread here |
There was a problem hiding this comment.
[CT-02] This thread has been RESOLVED (status 2 - Fixed)
| // Line 3: added | ||
| // Line 4: added - CT-02 resolved thread here | ||
| // Line 5: added | ||
| // Line 6: added - CT-07 3-level reply chain here |
There was a problem hiding this comment.
[CT-07] ROOT COMMENT - Starting discussion
There was a problem hiding this comment.
[CT-07] LEVEL 2 REPLY - Responding to root
There was a problem hiding this comment.
[CT-07] LEVEL 3 REPLY - Responding to level 2
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads |
There was a problem hiding this comment.
[CT-08] Thread 1 on line 10 - First parallel thread
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads |
There was a problem hiding this comment.
[CT-08] Thread 2 on line 10 - Second parallel thread
There was a problem hiding this comment.
Pull request overview
This PR introduces a test file for validating comment threading functionality and state transitions. The file serves as a fixture to test various thread scenarios including active threads, resolved threads, multi-level reply chains, and parallel thread handling.
Key Changes:
- Addition of a new test fixture file with annotated lines marking different thread test scenarios
- References to test cases CT-01, CT-02, CT-07, and CT-08 for tracking different threading behaviors
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Line 1: context | ||
| // Line 2: added - CT-01 active thread here | ||
| // Line 3: added | ||
| // Line 4: added - CT-02 resolved thread here | ||
| // Line 5: added | ||
| // Line 6: added - CT-07 3-level reply chain here | ||
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads | ||
| // Original line 5 |
There was a problem hiding this comment.
The file has a TypeScript extension (.ts) but contains only comments without any executable code. Consider either adding actual TypeScript code/tests, renaming the file with a more appropriate extension (e.g., .txt or .md), or adding a comment at the top explaining that this is a test fixture file for comment threading functionality.
| // Line 6: added - CT-07 3-level reply chain here | ||
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads |
There was a problem hiding this comment.
The test case IDs skip from CT-02 to CT-07, omitting CT-03 through CT-06. If this is intentional (e.g., those test cases exist elsewhere), consider adding a comment explaining the gap. Otherwise, ensure the numbering is sequential or corrected to match the actual test case structure.
| // Line 6: added - CT-07 3-level reply chain here | |
| // Line 7: added | |
| // Line 8: added | |
| // Line 9: added | |
| // Line 10: added - CT-08 multiple parallel threads | |
| // Line 6: added - CT-03 3-level reply chain here | |
| // Line 7: added | |
| // Line 8: added | |
| // Line 9: added | |
| // Line 10: added - CT-04 multiple parallel threads |
pedropaulovc
force-pushed
the
test/comment-threading
branch
from
291e63c to
e80a4d4
Compare
| @@ -0,0 +1,11 @@ | |||
| // Line 1: context | |||
| // Line 2: added - CT-01 active thread here | |||
There was a problem hiding this comment.
[CT-01] Active thread - status 1 (open discussion)
| // Line 1: context | ||
| // Line 2: added - CT-01 active thread here | ||
| // Line 3: added | ||
| // Line 4: added - CT-02 resolved thread here |
There was a problem hiding this comment.
[CT-02] This thread has been RESOLVED (status 2 - Fixed)
| // Line 3: added | ||
| // Line 4: added - CT-02 resolved thread here | ||
| // Line 5: added | ||
| // Line 6: added - CT-07 3-level reply chain here |
There was a problem hiding this comment.
[CT-07] ROOT COMMENT - Starting discussion
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads |
There was a problem hiding this comment.
[CT-08] Thread 1 on line 10 - First parallel thread
| // Line 7: added | ||
| // Line 8: added | ||
| // Line 9: added | ||
| // Line 10: added - CT-08 multiple parallel threads |
There was a problem hiding this comment.
[CT-08] Thread 2 on line 10 - Second parallel thread
Test PR for thread status transitions and reply chains