Skip to content

Commit 36d6045

Browse files
aeschliaeschli
authored
remove special dot character from prompt test names (microsoft#251634)
1 parent 335ad54 commit 36d6045

File tree

3 files changed

+29
-29
lines changed

3 files changed

+29
-29
lines changed

‎src/vs/workbench/contrib/chat/test/common/promptSyntax/codecs/base/markdownDecoder.test.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,7 @@ suite('MarkdownDecoder', () => {
242242
'The "characterName" must be set, got "empty line".',
243243
);
244244

245-
test(`• stop character - "${characterName}"`, async () => {
245+
test(`stop character - "${characterName}"`, async () => {
246246
const test = testDisposables.add(
247247
new TestMarkdownDecoder(),
248248
);
@@ -334,7 +334,7 @@ suite('MarkdownDecoder', () => {
334334
'The "characterName" must be set, got "empty line".',
335335
);
336336

337-
test(`• stop character - "${characterName}"`, async () => {
337+
test(`stop character - "${characterName}"`, async () => {
338338
const test = testDisposables.add(
339339
new TestMarkdownDecoder(),
340340
);
@@ -583,7 +583,7 @@ suite('MarkdownDecoder', () => {
583583
'The "characterName" must be set, got "empty line".',
584584
);
585585

586-
test(`• stop character - "${characterName}"`, async () => {
586+
test(`stop character - "${characterName}"`, async () => {
587587
const test = testDisposables.add(
588588
new TestMarkdownDecoder(),
589589
);
@@ -678,7 +678,7 @@ suite('MarkdownDecoder', () => {
678678
'The "characterName" must be set, got "empty line".',
679679
);
680680

681-
test(`• stop character - "${characterName}"`, async () => {
681+
test(`stop character - "${characterName}"`, async () => {
682682
const test = testDisposables.add(
683683
new TestMarkdownDecoder(),
684684
);

‎src/vs/workbench/contrib/chat/test/common/promptSyntax/parsers/textModelPromptParser.test.ts‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -290,9 +290,9 @@ suite('TextModelPromptParser', () => {
290290
});
291291

292292
suite('header', () => {
293-
suite(' • metadata', () => {
294-
suite(' • instructions', () => {
295-
test(`• empty header`, async () => {
293+
suite('metadata', () => {
294+
suite('instructions', () => {
295+
test(`empty header`, async () => {
296296
const test = createTest(
297297
URI.file('/absolute/folder/and/a/filename.txt'),
298298
[
@@ -335,7 +335,7 @@ suite('TextModelPromptParser', () => {
335335
);
336336
});
337337

338-
test(`• has correct 'instructions' metadata`, async () => {
338+
test(`has correct 'instructions' metadata`, async () => {
339339
const test = createTest(
340340
URI.file('/absolute/folder/and/a/filename.instructions.md'),
341341
[
@@ -392,8 +392,8 @@ suite('TextModelPromptParser', () => {
392392
});
393393
});
394394

395-
suite(' • prompts', () => {
396-
test(`• empty header`, async () => {
395+
suite('prompts', () => {
396+
test(`empty header`, async () => {
397397
const test = createTest(
398398
URI.file('/absolute/folder/and/a/filename.txt'),
399399
[
@@ -436,7 +436,7 @@ suite('TextModelPromptParser', () => {
436436
);
437437
});
438438

439-
test(`• has correct 'prompt' metadata`, async () => {
439+
test(`has correct 'prompt' metadata`, async () => {
440440
const test = createTest(
441441
URI.file('/absolute/folder/and/a/filename.txt'),
442442
[
@@ -494,8 +494,8 @@ suite('TextModelPromptParser', () => {
494494
});
495495
});
496496

497-
suite(' • modes', () => {
498-
test(`• empty header`, async () => {
497+
suite('modes', () => {
498+
test(`empty header`, async () => {
499499
const test = createTest(
500500
URI.file('/absolute/folder/and/a/filename.txt'),
501501
[
@@ -538,7 +538,7 @@ suite('TextModelPromptParser', () => {
538538
);
539539
});
540540

541-
test(`• has correct metadata`, async () => {
541+
test(`has correct metadata`, async () => {
542542
const test = createTest(
543543
URI.file('/absolute/folder/and/a/filename.txt'),
544544
[

‎src/vs/workbench/contrib/chat/test/common/promptSyntax/utils/promptFilesLocator.test.ts‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ suite('PromptFilesLocator', () => {
292292
];
293293

294294
for (const setting of settings) {
295-
test(`• '${setting}'`, async () => {
295+
test(`'${setting}'`, async () => {
296296
const locator = await createPromptsLocator(
297297
{ [setting]: true },
298298
EMPTY_WORKSPACE,
@@ -350,7 +350,7 @@ suite('PromptFilesLocator', () => {
350350
}
351351
});
352352

353-
suite(`• specific`, () => {
353+
suite(`specific`, () => {
354354
const testSettings = [
355355
[
356356
'/Users/legomushroom/repos/vscode/**/*specific*',
@@ -443,7 +443,7 @@ suite('PromptFilesLocator', () => {
443443
];
444444

445445
for (const settings of testSettings) {
446-
test(`• '${JSON.stringify(settings)}'`, async () => {
446+
test(`'${JSON.stringify(settings)}'`, async () => {
447447
const vscodeSettings: Record<string, boolean> = {};
448448
for (const setting of settings) {
449449
vscodeSettings[setting] = true;
@@ -536,7 +536,7 @@ suite('PromptFilesLocator', () => {
536536
];
537537

538538
for (const setting of testSettings) {
539-
test(`• '${setting}'`, async () => {
539+
test(`'${setting}'`, async () => {
540540
const locator = await createPromptsLocator(
541541
{ [setting]: true },
542542
['/Users/legomushroom/repos/vscode'],
@@ -594,7 +594,7 @@ suite('PromptFilesLocator', () => {
594594
}
595595
});
596596

597-
suite(`• specific`, () => {
597+
suite(`specific`, () => {
598598
const testSettings = [
599599
[
600600
'**/*specific*',
@@ -687,7 +687,7 @@ suite('PromptFilesLocator', () => {
687687
];
688688

689689
for (const settings of testSettings) {
690-
test(`• '${JSON.stringify(settings)}'`, async () => {
690+
test(`'${JSON.stringify(settings)}'`, async () => {
691691
const vscodeSettings: Record<string, boolean> = {};
692692
for (const setting of settings) {
693693
vscodeSettings[setting] = true;
@@ -776,7 +776,7 @@ suite('PromptFilesLocator', () => {
776776
];
777777

778778
for (const setting of settings) {
779-
test(`• '${setting}'`, async () => {
779+
test(`'${setting}'`, async () => {
780780
const locator = await createPromptsLocator(
781781
{ [setting]: true },
782782
['/Users/legomushroom/repos/vscode'],
@@ -834,7 +834,7 @@ suite('PromptFilesLocator', () => {
834834
}
835835
});
836836

837-
suite(`• specific`, () => {
837+
suite(`specific`, () => {
838838
const testSettings = [
839839
[
840840
'/Users/legomushroom/repos/vscode/**/*specific*',
@@ -927,7 +927,7 @@ suite('PromptFilesLocator', () => {
927927
];
928928

929929
for (const settings of testSettings) {
930-
test(`• '${JSON.stringify(settings)}'`, async () => {
930+
test(`'${JSON.stringify(settings)}'`, async () => {
931931
const vscodeSettings: Record<string, boolean> = {};
932932
for (const setting of settings) {
933933
vscodeSettings[setting] = true;
@@ -1687,7 +1687,7 @@ suite('PromptFilesLocator', () => {
16871687
];
16881688

16891689
for (const setting of testSettings) {
1690-
test(`• '${setting}'`, async () => {
1690+
test(`'${setting}'`, async () => {
16911691
const locator = await createPromptsLocator(
16921692
{ [setting]: true },
16931693
[
@@ -1773,7 +1773,7 @@ suite('PromptFilesLocator', () => {
17731773
}
17741774
});
17751775

1776-
suite(`• specific`, () => {
1776+
suite(`specific`, () => {
17771777
const testSettings = [
17781778
[
17791779
'**/my.prompt.md',
@@ -1888,7 +1888,7 @@ suite('PromptFilesLocator', () => {
18881888
];
18891889

18901890
for (const settings of testSettings) {
1891-
test(`• '${JSON.stringify(settings)}'`, async () => {
1891+
test(`'${JSON.stringify(settings)}'`, async () => {
18921892
const vscodeSettings: Record<string, boolean> = {};
18931893
for (const setting of settings) {
18941894
vscodeSettings[setting] = true;
@@ -2018,7 +2018,7 @@ suite('PromptFilesLocator', () => {
20182018
];
20192019

20202020
for (const setting of testSettings) {
2021-
test(`• '${setting}'`, async () => {
2021+
test(`'${setting}'`, async () => {
20222022
const locator = await createPromptsLocator(
20232023
{ [setting]: true },
20242024
[
@@ -2104,7 +2104,7 @@ suite('PromptFilesLocator', () => {
21042104
}
21052105
});
21062106

2107-
suite(`• specific`, () => {
2107+
suite(`specific`, () => {
21082108
const testSettings = [
21092109
[
21102110
'/Users/legomushroom/repos/**/my.prompt.md',
@@ -2249,7 +2249,7 @@ suite('PromptFilesLocator', () => {
22492249
];
22502250

22512251
for (const settings of testSettings) {
2252-
test(`• '${JSON.stringify(settings)}'`, async () => {
2252+
test(`'${JSON.stringify(settings)}'`, async () => {
22532253
const vscodeSettings: Record<string, boolean> = {};
22542254
for (const setting of settings) {
22552255
vscodeSettings[setting] = true;

0 commit comments

Comments
 (0)