Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(82)

Side by Side Diff: pkg/analyzer/test/generated/scanner_test.dart

Issue 2910583002: improve fasta interpolation recovery (Closed)
Patch Set: remove unnecessary label Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library analyzer.test.generated.scanner_test; 5 library analyzer.test.generated.scanner_test;
6 6
7 import 'package:analyzer/dart/ast/token.dart'; 7 import 'package:analyzer/dart/ast/token.dart';
8 import 'package:analyzer/src/dart/ast/token.dart'; 8 import 'package:analyzer/src/dart/ast/token.dart';
9 import 'package:analyzer/src/dart/scanner/reader.dart'; 9 import 'package:analyzer/src/dart/scanner/reader.dart';
10 import 'package:analyzer/src/dart/scanner/scanner.dart'; 10 import 'package:analyzer/src/dart/scanner/scanner.dart';
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 CharSequenceReader baseReader = new CharSequenceReader("xyzzy"); 72 CharSequenceReader baseReader = new CharSequenceReader("xyzzy");
73 CharacterRangeReader reader = new CharacterRangeReader(baseReader, 1, 4); 73 CharacterRangeReader reader = new CharacterRangeReader(baseReader, 1, 4);
74 reader.offset = 2; 74 reader.offset = 2;
75 expect(reader.offset, 2); 75 expect(reader.offset, 2);
76 } 76 }
77 } 77 }
78 78
79 @reflectiveTest 79 @reflectiveTest
80 class LineInfoTest extends EngineTestCase { 80 class LineInfoTest extends EngineTestCase {
81 void test_lineInfo_multilineComment() { 81 void test_lineInfo_multilineComment() {
82 String source = "/*\r *\r */"; 82 String source = "/*\r\n *\r\n */";
ahe 2017/05/31 13:15:48 Perhaps Fasta should support \r line endings?
danrubel 2017/06/01 19:02:46 I don't think we need to do so given that most (al
83 _assertLineInfo(source, [ 83 _assertLineInfo(source, [
84 new ScannerTest_ExpectedLocation(0, 1, 1), 84 new ScannerTest_ExpectedLocation(0, 1, 1),
85 new ScannerTest_ExpectedLocation(4, 2, 2), 85 new ScannerTest_ExpectedLocation(5, 2, 2),
86 new ScannerTest_ExpectedLocation(source.length - 1, 3, 3) 86 new ScannerTest_ExpectedLocation(source.length - 1, 3, 3)
87 ]); 87 ]);
88 } 88 }
89 89
90 void test_lineInfo_multilineString() { 90 void test_lineInfo_multilineString() {
91 String source = "'''a\r\nbc\r\nd'''"; 91 String source = "'''a\r\nbc\r\nd'''";
92 _assertLineInfo(source, [ 92 _assertLineInfo(source, [
93 new ScannerTest_ExpectedLocation(0, 1, 1), 93 new ScannerTest_ExpectedLocation(0, 1, 1),
94 new ScannerTest_ExpectedLocation(7, 2, 2), 94 new ScannerTest_ExpectedLocation(7, 2, 2),
95 new ScannerTest_ExpectedLocation(source.length - 1, 3, 4) 95 new ScannerTest_ExpectedLocation(source.length - 1, 3, 4)
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
227 buffer.write(token.type); 227 buffer.write(token.type);
228 buffer.write(", '"); 228 buffer.write(", '");
229 buffer.write(token.lexeme); 229 buffer.write(token.lexeme);
230 buffer.write("', "); 230 buffer.write("', ");
231 buffer.write(token.offset); 231 buffer.write(token.offset);
232 buffer.write(", "); 232 buffer.write(", ");
233 buffer.write(token.length); 233 buffer.write(token.length);
234 buffer.write("]"); 234 buffer.write("]");
235 } 235 }
236 } 236 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698