Skip to content

Commit fd62885

Browse files
alexdimaalexdima
committed
Fix \t as BREAK_AFTER in tests
1 parent ce3a963 commit fd62885

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

‎src/vs/editor/test/common/viewModel/characterHardWrappingLineMapper.test.ts‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ function assertLineMapping(factory: ILineMapperFactory, tabSize: number, breakAf
5151
suite('Editor ViewModel - CharacterHardWrappingLineMapper', () => {
5252
test('CharacterHardWrappingLineMapper', () => {
5353

54-
let factory = new CharacterHardWrappingLineMapperFactory('(', ').');
54+
let factory = new CharacterHardWrappingLineMapperFactory('(', '\t).');
5555

5656
// Empty string
5757
assertLineMapping(factory, 4, 5, '');
@@ -73,10 +73,10 @@ suite('Editor ViewModel - CharacterHardWrappingLineMapper', () => {
7373

7474
// Honors tabs when computing wrapping position
7575
assertLineMapping(factory, 4, 5, '\t');
76-
assertLineMapping(factory, 4, 5, '\ta|aa');
77-
assertLineMapping(factory, 4, 5, '\ta|\ta|a');
76+
assertLineMapping(factory, 4, 5, '\t|aaa');
77+
assertLineMapping(factory, 4, 5, '\t|a\t|aa');
7878
assertLineMapping(factory, 4, 5, 'aa\ta');
79-
assertLineMapping(factory, 4, 5, 'aa\ta|a');
79+
assertLineMapping(factory, 4, 5, 'aa\t|aa');
8080

8181
// Honors wrapping before characters (& gives it priority)
8282
assertLineMapping(factory, 4, 5, 'aaa.|aa');
@@ -92,7 +92,7 @@ suite('Editor ViewModel - CharacterHardWrappingLineMapper', () => {
9292
});
9393

9494
test('CharacterHardWrappingLineMapper - CJK and Kinsoku Shori', () => {
95-
let factory = new CharacterHardWrappingLineMapperFactory('(', ')');
95+
let factory = new CharacterHardWrappingLineMapperFactory('(', '\t)');
9696
assertLineMapping(factory, 4, 5, 'aa \u5b89|\u5b89');
9797
assertLineMapping(factory, 4, 5, '\u3042 \u5b89|\u5b89');
9898
assertLineMapping(factory, 4, 5, '\u3042\u3042|\u5b89\u5b89');
@@ -102,28 +102,28 @@ suite('Editor ViewModel - CharacterHardWrappingLineMapper', () => {
102102
});
103103

104104
test('CharacterHardWrappingLineMapper - WrappingIndent.Same', () => {
105-
let factory = new CharacterHardWrappingLineMapperFactory('', ' ');
105+
let factory = new CharacterHardWrappingLineMapperFactory('', '\t ');
106106
assertLineMapping(factory, 4, 38, ' *123456789012345678901234567890123456|7890', WrappingIndent.Same);
107107
});
108108

109109
test('issue #16332: Scroll bar overlaying on top of text', () => {
110-
let factory = new CharacterHardWrappingLineMapperFactory('', ' ');
110+
let factory = new CharacterHardWrappingLineMapperFactory('', '\t ');
111111
assertLineMapping(factory, 4, 24, 'a/ very/long/line/of/tex|t/that/expands/beyon|d/your/typical/line/|of/code/', WrappingIndent.Indent);
112112
});
113113

114114
test('issue #35162: wrappingIndent not consistently working', () => {
115-
let factory = new CharacterHardWrappingLineMapperFactory('', ' ');
115+
let factory = new CharacterHardWrappingLineMapperFactory('', '\t ');
116116
let mapper = assertLineMapping(factory, 4, 24, ' t h i s |i s |a l |o n |g l |i n |e', WrappingIndent.Indent);
117117
assert.equal(mapper!.getWrappedLinesIndent(), ' \t');
118118
});
119119

120120
test('issue #75494: surrogate pairs', () => {
121-
let factory = new CharacterHardWrappingLineMapperFactory('', ' ');
121+
let factory = new CharacterHardWrappingLineMapperFactory('\t', ' ');
122122
assertLineMapping(factory, 4, 49, '🐇👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇|👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇👬🌖🌞🏇🍼🐇👬', WrappingIndent.Same);
123123
});
124124

125125
test('CharacterHardWrappingLineMapper - WrappingIndent.DeepIndent', () => {
126-
let factory = new CharacterHardWrappingLineMapperFactory('', ' ');
126+
let factory = new CharacterHardWrappingLineMapperFactory('', '\t ');
127127
let mapper = assertLineMapping(factory, 4, 26, ' W e A r e T e s t |i n g D e |e p I n d |e n t a t |i o n', WrappingIndent.DeepIndent);
128128
assert.equal(mapper!.getWrappedLinesIndent(), ' \t\t');
129129
});

0 commit comments

Comments
 (0)