Skip to content

Commit da6cf0f

Browse files
Copilotsapphi-red
andauthored
test: update conformance snapshots for merge conflict diagnostics (#15462)
Updates test snapshots to reflect the improved merge conflict diagnostic messages implemented in previous commits. ## Changes - Updated `parser_misc.snap` and `parser_typescript.snap` with capitalized error titles and properly formatted help text - Snapshots now match the current diagnostic output: "Encountered diff marker" with sentence-case help messages The snapshots were updated by running `just conformance` after the diagnostic message improvements were merged. <!-- START COPILOT CODING AGENT TIPS --> --- 💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more [Copilot coding agent tips](https://gh.io/copilot-coding-agent-tips) in the docs. --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sapphi-red <49056869+sapphi-red@users.noreply.github.com>
1 parent 8b1f333 commit da6cf0f

2 files changed

Lines changed: 33 additions & 33 deletions

File tree

tasks/coverage/snapshots/parser_misc.snap

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ Negative Passed: 108/108 (100.00%)
5050
8
5151
╰────
5252

53-
× encountered diff marker
53+
× Encountered diff marker
5454
╭─[misc/fail/diff-markers.js:10:1]
5555
9function test() {
5656
10<<<<<<< HEAD
@@ -66,11 +66,11 @@ Negative Passed: 108/108 (100.00%)
6666
· ╰── this marker concludes the conflict region
6767
15return x;
6868
╰────
69-
help: conflict markers indicate that a merge was started but could not be completed due to merge conflicts
70-
to resolve a conflict, keep only the code you want and then delete the lines containing conflict markers
71-
if you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code
72-
if you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased
73-
if you have nested conflicts, resolve the outermost conflict first
69+
help: Conflict markers indicate that a merge was started but could not be completed due to merge conflicts.
70+
To resolve a conflict, keep only the code you want and then delete the lines containing conflict markers.
71+
If you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code.
72+
If you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased.
73+
If you have nested conflicts, resolve the outermost conflict first.
7474

7575
× Expected `,` or `]` but found `const`
7676
╭─[misc/fail/imbalanced-array-expr.js:2:1]

tasks/coverage/snapshots/parser_typescript.snap

Lines changed: 27 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -6329,7 +6329,7 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
63296329
╰────
63306330
help: Remove these parameters here
63316331

6332-
× encountered diff marker
6332+
× Encountered diff marker
63336333
╭─[typescript/tests/cases/compiler/conflictMarkerDiff3Trivia1.ts:2:1]
63346334
1 │ class C {
63356335
2 │ <<<<<<< HEAD
@@ -6348,13 +6348,13 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
63486348
· ╰── this marker concludes the conflict region
63496349
9 │ }
63506350
╰────
6351-
help: conflict markers indicate that a merge was started but could not be completed due to merge conflicts
6352-
to resolve a conflict, keep only the code you want and then delete the lines containing conflict markers
6353-
if you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code
6354-
if you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased
6355-
if you have nested conflicts, resolve the outermost conflict first
6351+
help: Conflict markers indicate that a merge was started but could not be completed due to merge conflicts.
6352+
To resolve a conflict, keep only the code you want and then delete the lines containing conflict markers.
6353+
If you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code.
6354+
If you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased.
6355+
If you have nested conflicts, resolve the outermost conflict first.
63566356

6357-
× encountered diff marker
6357+
× Encountered diff marker
63586358
╭─[typescript/tests/cases/compiler/conflictMarkerDiff3Trivia2.ts:3:1]
63596359
2 │ foo() {
63606360
3 │ <<<<<<< B
@@ -6374,13 +6374,13 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
63746374
· ╰── this marker concludes the conflict region
63756375
13 │
63766376
╰────
6377-
help: conflict markers indicate that a merge was started but could not be completed due to merge conflicts
6378-
to resolve a conflict, keep only the code you want and then delete the lines containing conflict markers
6379-
if you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code
6380-
if you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased
6381-
if you have nested conflicts, resolve the outermost conflict first
6377+
help: Conflict markers indicate that a merge was started but could not be completed due to merge conflicts.
6378+
To resolve a conflict, keep only the code you want and then delete the lines containing conflict markers.
6379+
If you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code.
6380+
If you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased.
6381+
If you have nested conflicts, resolve the outermost conflict first.
63826382

6383-
× encountered diff marker
6383+
× Encountered diff marker
63846384
╭─[typescript/tests/cases/compiler/conflictMarkerTrivia1.ts:2:1]
63856385
1 │ class C {
63866386
2 │ <<<<<<< HEAD
@@ -6396,13 +6396,13 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
63966396
· ╰── this marker concludes the conflict region
63976397
7 │ }
63986398
╰────
6399-
help: conflict markers indicate that a merge was started but could not be completed due to merge conflicts
6400-
to resolve a conflict, keep only the code you want and then delete the lines containing conflict markers
6401-
if you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code
6402-
if you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased
6403-
if you have nested conflicts, resolve the outermost conflict first
6399+
help: Conflict markers indicate that a merge was started but could not be completed due to merge conflicts.
6400+
To resolve a conflict, keep only the code you want and then delete the lines containing conflict markers.
6401+
If you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code.
6402+
If you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased.
6403+
If you have nested conflicts, resolve the outermost conflict first.
64046404

6405-
× encountered diff marker
6405+
× Encountered diff marker
64066406
╭─[typescript/tests/cases/compiler/conflictMarkerTrivia2.ts:3:1]
64076407
2 │ foo() {
64086408
3 │ <<<<<<< B
@@ -6420,11 +6420,11 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
64206420
· ╰── this marker concludes the conflict region
64216421
10 │
64226422
╰────
6423-
help: conflict markers indicate that a merge was started but could not be completed due to merge conflicts
6424-
to resolve a conflict, keep only the code you want and then delete the lines containing conflict markers
6425-
if you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code
6426-
if you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased
6427-
if you have nested conflicts, resolve the outermost conflict first
6423+
help: Conflict markers indicate that a merge was started but could not be completed due to merge conflicts.
6424+
To resolve a conflict, keep only the code you want and then delete the lines containing conflict markers.
6425+
If you're having merge conflicts after pulling new code, the top section is the code you already had and the bottom section is the remote code.
6426+
If you're in the middle of a rebase, the top section is the code being rebased onto and the bottom section is the code coming from the current commit being rebased.
6427+
If you have nested conflicts, resolve the outermost conflict first.
64286428

64296429
× Unexpected token
64306430
╭─[typescript/tests/cases/compiler/conflictMarkerTrivia3.tsx:2:1]
@@ -6433,15 +6433,15 @@ Expect to Parse: tasks/coverage/typescript/tests/cases/conformance/parser/ecmasc
64336433
· ──
64346434
╰────
64356435

6436-
× encountered diff marker
6436+
× Encountered diff marker
64376437
╭─[typescript/tests/cases/compiler/conflictMarkerTrivia4.ts:2:1]
64386438
1 │ const x = <div>
64396439
2 │ <<<<<<< HEAD
64406440
· ───┬───
64416441
· ╰── between this marker and `=======` is the code that we're merging into
64426442
╰────
6443-
help: this conflict marker appears to be incomplete (missing `=======` or `>>>>>>>`)
6444-
check if the conflict markers were accidentally modified or partially deleted
6443+
help: This conflict marker appears to be incomplete (missing `=======` or `>>>>>>>`).
6444+
Check if the conflict markers were accidentally modified or partially deleted.
64456445

64466446
× Identifier `foo` has already been declared
64476447
╭─[typescript/tests/cases/compiler/conflictingTypeAnnotatedVar.ts:1:5]

0 commit comments

Comments
 (0)