Skip to content

Commit cec2039

Browse files
jriekenjrieken
authored
chore - Use simple suite- and test-names and leave styling up to the reporter (microsoft#250811)
1 parent 478c7cf commit cec2039

27 files changed

+352
-352
lines changed

‎src/vs/base/test/common/logTime.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ suite('logTime', () => {
3434
.replaceAll(/\s\d+.\d{2}\sms$/gi, ' 100.50 ms');
3535
};
3636

37-
suite('• decorator', () => {
38-
test('• async method', async () => {
37+
suite('decorator', () => {
38+
test('async method', async () => {
3939
const logSpy = sinon.spy();
4040
class TestClass {
4141
public logTime = logSpy;
@@ -89,7 +89,7 @@ suite('logTime', () => {
8989
});
9090
});
9191

92-
test('• sync method', async () => {
92+
test('sync method', async () => {
9393
const logSpy = sinon.spy();
9494

9595
class TestClass {
@@ -134,7 +134,7 @@ suite('logTime', () => {
134134
);
135135
});
136136

137-
test('• uses \'trace\' level by default', async () => {
137+
test('uses \'trace\' level by default', async () => {
138138
const logSpy = sinon.spy();
139139

140140
class TestClass {
@@ -188,8 +188,8 @@ suite('logTime', () => {
188188
);
189189
});
190190

191-
suite('• logExecutionTime helper', () => {
192-
test('• async function', async () => {
191+
suite('logExecutionTime helper', () => {
192+
test('async function', async () => {
193193
const logSpy = sinon.spy();
194194

195195
const expectedReturnValue = randomInt(1000);
@@ -234,7 +234,7 @@ suite('logTime', () => {
234234
);
235235
});
236236

237-
test('• sync function', () => {
237+
test('sync function', () => {
238238
const logSpy = sinon.spy();
239239

240240
const expectedReturnValue = randomInt(1000);

‎src/vs/base/test/common/observableDisposable.test.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { assertNotDisposed, ObservableDisposable } from '../../common/observable
1313
suite('ObservableDisposable', () => {
1414
const disposables = ensureNoDisposablesAreLeakedInTestSuite();
1515

16-
test('• tracks `disposed` state', () => {
16+
test('tracks `disposed` state', () => {
1717
// this is an abstract class, so we have to create
1818
// an anonymous class that extends it
1919
const object = new class extends ObservableDisposable { }();
@@ -42,8 +42,8 @@ suite('ObservableDisposable', () => {
4242
);
4343
});
4444

45-
suite('• onDispose()', () => {
46-
test('• fires the event on dispose', async () => {
45+
suite('onDispose()', () => {
46+
test('fires the event on dispose', async () => {
4747
// this is an abstract class, so we have to create
4848
// an anonymous class that extends it
4949
const object = new class extends ObservableDisposable { }();
@@ -107,7 +107,7 @@ suite('ObservableDisposable', () => {
107107
);
108108
});
109109

110-
test('• executes callback immediately if already disposed', async () => {
110+
test('executes callback immediately if already disposed', async () => {
111111
// this is an abstract class, so we have to create
112112
// an anonymous class that extends it
113113
const object = new class extends ObservableDisposable { }();
@@ -149,8 +149,8 @@ suite('ObservableDisposable', () => {
149149
});
150150
});
151151

152-
suite('• addDisposable()', () => {
153-
test('• disposes provided object with itself', async () => {
152+
suite('addDisposable()', () => {
153+
test('disposes provided object with itself', async () => {
154154
class TestDisposable implements IDisposable {
155155
private _disposed = false;
156156
public get disposed() {
@@ -208,7 +208,7 @@ suite('ObservableDisposable', () => {
208208
);
209209
});
210210

211-
test('• disposes the entire tree of disposables', async () => {
211+
test('disposes the entire tree of disposables', async () => {
212212
class TestDisposable extends ObservableDisposable { }
213213

214214
/**
@@ -290,8 +290,8 @@ suite('ObservableDisposable', () => {
290290
});
291291
});
292292

293-
suite('• asserts', () => {
294-
test('• not disposed (method)', async () => {
293+
suite('asserts', () => {
294+
test('not disposed (method)', async () => {
295295
// this is an abstract class, so we have to create
296296
// an anonymous class that extends it
297297
const object: ObservableDisposable = new class extends ObservableDisposable { }();
@@ -322,7 +322,7 @@ suite('ObservableDisposable', () => {
322322
});
323323
});
324324

325-
test('• not disposed (function)', async () => {
325+
test('not disposed (function)', async () => {
326326
// this is an abstract class, so we have to create
327327
// an anonymous class that extends it
328328
const object: ObservableDisposable = new class extends ObservableDisposable { }();

‎src/vs/editor/test/common/codecs/frontMatterDecoder/frontMatterBoolean.test.ts

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ import { FrontMatterSequence } from '../../../../common/codecs/frontMatterCodec/
1414
suite('FrontMatterBoolean', () => {
1515
ensureNoDisposablesAreLeakedInTestSuite();
1616

17-
suite('• equals()', () => {
18-
suite('• base case', () => {
19-
test('• true', () => {
17+
suite('equals()', () => {
18+
suite('base case', () => {
19+
test('true', () => {
2020
// both values should yield the same result
2121
const booleanText = (randomBoolean())
2222
? 'true'
@@ -48,7 +48,7 @@ suite('FrontMatterBoolean', () => {
4848
);
4949
});
5050

51-
test('• false', () => {
51+
test('false', () => {
5252
// both values should yield the same result
5353
const booleanText = (randomBoolean())
5454
? 'false'
@@ -81,9 +81,9 @@ suite('FrontMatterBoolean', () => {
8181
});
8282
});
8383

84-
suite('• non-boolean token', () => {
85-
suite('• word token', () => {
86-
test('• true', () => {
84+
suite('non-boolean token', () => {
85+
suite('word token', () => {
86+
test('true', () => {
8787
// both values should yield the same result
8888
const booleanText = (randomBoolean())
8989
? 'true'
@@ -107,7 +107,7 @@ suite('FrontMatterBoolean', () => {
107107
);
108108
});
109109

110-
test('• false', () => {
110+
test('false', () => {
111111
// both values should yield the same result
112112
const booleanText = (randomBoolean())
113113
? 'false'
@@ -132,8 +132,8 @@ suite('FrontMatterBoolean', () => {
132132
});
133133
});
134134

135-
suite('• sequence token', () => {
136-
test('• true', () => {
135+
suite('sequence token', () => {
136+
test('true', () => {
137137
// both values should yield the same result
138138
const booleanText = (randomBoolean())
139139
? 'true'
@@ -159,7 +159,7 @@ suite('FrontMatterBoolean', () => {
159159
);
160160
});
161161

162-
test('• false', () => {
162+
test('false', () => {
163163
// both values should yield the same result
164164
const booleanText = (randomBoolean())
165165
? 'false'
@@ -187,8 +187,8 @@ suite('FrontMatterBoolean', () => {
187187
});
188188
});
189189

190-
suite('• different range', () => {
191-
test('• true', () => {
190+
suite('different range', () => {
191+
test('true', () => {
192192
// both values should yield the same result
193193
const booleanText = (randomBoolean())
194194
? 'true'
@@ -214,7 +214,7 @@ suite('FrontMatterBoolean', () => {
214214
);
215215
});
216216

217-
test('• false', () => {
217+
test('false', () => {
218218
// both values should yield the same result
219219
const booleanText = (randomBoolean())
220220
? 'false'
@@ -241,8 +241,8 @@ suite('FrontMatterBoolean', () => {
241241
});
242242
});
243243

244-
suite('• different text', () => {
245-
test('• true', () => {
244+
suite('different text', () => {
245+
test('true', () => {
246246
const boolean = new FrontMatterBoolean(
247247
new Word(
248248
new Range(1, 1, 1, 5),
@@ -263,7 +263,7 @@ suite('FrontMatterBoolean', () => {
263263
);
264264
});
265265

266-
test('• false', () => {
266+
test('false', () => {
267267
const boolean = new FrontMatterBoolean(
268268
new Word(
269269
new Range(5, 15, 5, 15 + 6),
@@ -285,7 +285,7 @@ suite('FrontMatterBoolean', () => {
285285
});
286286
});
287287

288-
test('• throws if cannot be converted to a boolean', () => {
288+
test('throws if cannot be converted to a boolean', () => {
289289
assert.throws(() => {
290290
new FrontMatterBoolean(
291291
new Word(

‎src/vs/editor/test/common/codecs/frontMatterDecoder/frontMatterDecoder.test.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export class TestFrontMatterDecoder extends TestDecoder<TSimpleDecoderToken, Fro
3333
suite('FrontMatterDecoder', () => {
3434
const disposables = ensureNoDisposablesAreLeakedInTestSuite();
3535

36-
test('• produces expected tokens', async () => {
36+
test('produces expected tokens', async () => {
3737
const test = disposables.add(new TestFrontMatterDecoder());
3838

3939
await test.run(
@@ -115,9 +115,9 @@ suite('FrontMatterDecoder', () => {
115115
]);
116116
});
117117

118-
suite('• record', () => {
119-
suite('• values', () => {
120-
test('• unquoted string', async () => {
118+
suite('record', () => {
119+
suite('values', () => {
120+
test('unquoted string', async () => {
121121
const test = disposables.add(new TestFrontMatterDecoder());
122122

123123
await test.run(
@@ -164,7 +164,7 @@ suite('FrontMatterDecoder', () => {
164164
]);
165165
});
166166

167-
test('• quoted string', async () => {
167+
test('quoted string', async () => {
168168
const test = disposables.add(new TestFrontMatterDecoder());
169169

170170
await test.run(
@@ -219,7 +219,7 @@ suite('FrontMatterDecoder', () => {
219219
]);
220220
});
221221

222-
test('• boolean', async () => {
222+
test('boolean', async () => {
223223
const test = disposables.add(new TestFrontMatterDecoder());
224224

225225
await test.run(
@@ -269,8 +269,8 @@ suite('FrontMatterDecoder', () => {
269269
]);
270270
});
271271

272-
suite('• array', () => {
273-
test('• empty', async () => {
272+
suite('array', () => {
273+
test('empty', async () => {
274274
const test = disposables.add(new TestFrontMatterDecoder());
275275

276276
await test.run(
@@ -314,7 +314,7 @@ suite('FrontMatterDecoder', () => {
314314
]);
315315
});
316316

317-
test('• mixed values', async () => {
317+
test('mixed values', async () => {
318318
const test = disposables.add(new TestFrontMatterDecoder());
319319

320320
await test.run(
@@ -358,7 +358,7 @@ suite('FrontMatterDecoder', () => {
358358
]);
359359
});
360360

361-
test('• redundant commas', async () => {
361+
test('redundant commas', async () => {
362362
const test = disposables.add(new TestFrontMatterDecoder());
363363

364364
await test.run(
@@ -405,7 +405,7 @@ suite('FrontMatterDecoder', () => {
405405
});
406406
});
407407

408-
test('• empty', async () => {
408+
test('empty', async () => {
409409
const test = disposables.add(
410410
new TestFrontMatterDecoder(),
411411
);

‎src/vs/editor/test/common/codecs/frontMatterDecoder/frontMatterRecord.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ import { FrontMatterArray, FrontMatterBoolean, FrontMatterRecord, FrontMatterRec
1313
suite('FrontMatterBoolean', () => {
1414
ensureNoDisposablesAreLeakedInTestSuite();
1515

16-
suite('• trimValueEnd()', () => {
17-
test('• trims space tokens at the end of record\'s value', () => {
16+
suite('trimValueEnd()', () => {
17+
test('trims space tokens at the end of record\'s value', () => {
1818
const recordName = new FrontMatterRecordName([
1919
new Word(
2020
new Range(4, 10, 4, 10 + 3),
@@ -59,8 +59,8 @@ suite('FrontMatterBoolean', () => {
5959
);
6060
});
6161

62-
suite('• does not trim non-sequence value tokens', () => {
63-
test('• boolean', () => {
62+
suite('does not trim non-sequence value tokens', () => {
63+
test('boolean', () => {
6464
const recordName = new FrontMatterRecordName([
6565
new Word(
6666
new Range(4, 10, 4, 10 + 3),
@@ -96,7 +96,7 @@ suite('FrontMatterBoolean', () => {
9696
);
9797
});
9898

99-
test('• quoted string', () => {
99+
test('quoted string', () => {
100100
const recordName = new FrontMatterRecordName([
101101
new Word(
102102
new Range(4, 10, 4, 10 + 3),
@@ -134,7 +134,7 @@ suite('FrontMatterBoolean', () => {
134134
);
135135
});
136136

137-
test('• array', () => {
137+
test('array', () => {
138138
const recordName = new FrontMatterRecordName([
139139
new Word(
140140
new Range(4, 10, 4, 10 + 3),

‎src/vs/editor/test/common/codecs/frontMatterDecoder/frontMatterSequence.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { FrontMatterSequence } from '../../../../common/codecs/frontMatterCodec/
1313
suite('FrontMatterSequence', () => {
1414
ensureNoDisposablesAreLeakedInTestSuite();
1515

16-
test('• extends \'FrontMatterValueToken\'', () => {
16+
test('extends \'FrontMatterValueToken\'', () => {
1717
const sequence = new FrontMatterSequence([
1818
new Word(
1919
new Range(1, 1, 1, 5),
@@ -27,8 +27,8 @@ suite('FrontMatterSequence', () => {
2727
);
2828
});
2929

30-
suite('• trimEnd()', () => {
31-
test('• trims space tokens at the end of the sequence', () => {
30+
suite('trimEnd()', () => {
31+
test('trims space tokens at the end of the sequence', () => {
3232
const sequence = new FrontMatterSequence([
3333
new Word(new Range(4, 18, 4, 18 + 10), 'some-value'),
3434
new Space(new Range(4, 28, 4, 29)),
@@ -59,7 +59,7 @@ suite('FrontMatterSequence', () => {
5959
);
6060
});
6161

62-
test('• remains functional if only spacing tokens were present', () => {
62+
test('remains functional if only spacing tokens were present', () => {
6363
const sequence = new FrontMatterSequence([
6464
new Space(new Range(4, 28, 4, 29)),
6565
new Space(new Range(4, 29, 4, 30)),
@@ -98,7 +98,7 @@ suite('FrontMatterSequence', () => {
9898
});
9999
});
100100

101-
test('• throws if no tokens provided', () => {
101+
test('throws if no tokens provided', () => {
102102
assert.throws(() => {
103103
new FrontMatterSequence([]);
104104
});

0 commit comments

Comments
 (0)