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

Side by Side Diff: pkg/analyzer/test/services/formatter_test.dart

Issue 153203002: Indent fix for trailing comments (dartbug.com/16383). (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 6 years, 10 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 | Annotate | Revision Log
« no previous file with comments | « pkg/analyzer/test/services/data/stmt_tests.data ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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 import 'dart:io'; 5 import 'dart:io';
6 6
7 import 'package:path/path.dart'; 7 import 'package:path/path.dart';
8 import 'package:unittest/unittest.dart'; 8 import 'package:unittest/unittest.dart';
9 9
10 import 'package:analyzer/src/generated/java_core.dart' show CharSequence; 10 import 'package:analyzer/src/generated/java_core.dart' show CharSequence;
(...skipping 1125 matching lines...) Expand 10 before | Expand all | Expand 10 after
1136 expectStreamsNotEqual(t1, t2); 1136 expectStreamsNotEqual(t1, t2);
1137 }); 1137 });
1138 1138
1139 }); 1139 });
1140 1140
1141 1141
1142 /// Line tests 1142 /// Line tests
1143 group('line', () { 1143 group('line', () {
1144 1144
1145 test('space', () { 1145 test('space', () {
1146 var line = new Line(indent: 0); 1146 var line = new Line(indentLevel: 0);
1147 line.addSpaces(2); 1147 line.addSpaces(2);
1148 expect(line.toString(), equals(' ')); 1148 expect(line.toString(), equals(' '));
1149 }); 1149 });
1150 1150
1151 test('initial indent', () { 1151 test('initial indent', () {
1152 var line = new Line(indent: 2); 1152 var line = new Line(indentLevel: 2);
1153 expect(line.toString(), equals(' ')); 1153 expect(line.toString(), equals(' '));
1154 }); 1154 });
1155 1155
1156 test('initial indent (tabbed)', () { 1156 test('initial indent (tabbed)', () {
1157 var line = new Line(indent:1, useTabs: true); 1157 var line = new Line(indentLevel:1, useTabs: true);
1158 expect(line.toString(), equals('\t')); 1158 expect(line.toString(), equals('\t'));
1159 }); 1159 });
1160 1160
1161 test('addToken', () { 1161 test('addToken', () {
1162 var line = new Line(); 1162 var line = new Line();
1163 line.addToken(new LineToken('foo')); 1163 line.addToken(new LineToken('foo'));
1164 expect(line.toString(), equals('foo')); 1164 expect(line.toString(), equals('foo'));
1165 }); 1165 });
1166 1166
1167 test('addToken (2)', () { 1167 test('addToken (2)', () {
1168 var line = new Line(indent: 1); 1168 var line = new Line(indentLevel: 1);
1169 line.addToken(new LineToken('foo')); 1169 line.addToken(new LineToken('foo'));
1170 expect(line.toString(), equals(' foo')); 1170 expect(line.toString(), equals(' foo'));
1171 }); 1171 });
1172 1172
1173 test('isWhitespace', () { 1173 test('isWhitespace', () {
1174 var line = new Line(indent: 1); 1174 var line = new Line(indentLevel: 1);
1175 expect(line.isWhitespace(), isTrue); 1175 expect(line.isWhitespace(), isTrue);
1176 }); 1176 });
1177 1177
1178 }); 1178 });
1179 1179
1180 /// Writer tests 1180 /// Writer tests
1181 group('writer', () { 1181 group('writer', () {
1182 1182
1183 test('basic print', () { 1183 test('basic print', () {
1184 var writer = new SourceWriter(); 1184 var writer = new SourceWriter();
1185 writer.print('foo'); 1185 writer.write('foo');
1186 writer.print(' '); 1186 writer.write(' ');
1187 writer.print('bar'); 1187 writer.write('bar');
1188 expect(writer.toString(), equals('foo bar')); 1188 expect(writer.toString(), equals('foo bar'));
1189 }); 1189 });
1190 1190
1191 test('newline', () { 1191 test('newline', () {
1192 var writer = new SourceWriter(); 1192 var writer = new SourceWriter();
1193 writer.print('foo'); 1193 writer.write('foo');
1194 writer.newline(); 1194 writer.newline();
1195 expect(writer.toString(), equals('foo\n')); 1195 expect(writer.toString(), equals('foo\n'));
1196 }); 1196 });
1197 1197
1198 test('newline trims whitespace', () { 1198 test('newline trims whitespace', () {
1199 var writer = new SourceWriter(indentCount:2); 1199 var writer = new SourceWriter(indentCount:2);
1200 writer.newline(); 1200 writer.newline();
1201 expect(writer.toString(), equals('\n')); 1201 expect(writer.toString(), equals('\n'));
1202 }); 1202 });
1203 1203
1204 test('basic print (with indents)', () { 1204 test('basic print (with indents)', () {
1205 var writer = new SourceWriter(); 1205 var writer = new SourceWriter();
1206 writer.print('foo'); 1206 writer.write('foo');
1207 writer.indent(); 1207 writer.indent();
1208 writer.newline(); 1208 writer.newline();
1209 writer.print('bar'); 1209 writer.write('bar');
1210 writer.unindent(); 1210 writer.unindent();
1211 writer.newline(); 1211 writer.newline();
1212 writer.print('baz'); 1212 writer.write('baz');
1213 expect(writer.toString(), equals('foo\n bar\nbaz')); 1213 expect(writer.toString(), equals('foo\n bar\nbaz'));
1214 }); 1214 });
1215 1215
1216 }); 1216 });
1217 1217
1218 1218
1219 1219
1220 /// Line breaker tests 1220 /// Line breaker tests
1221 group('linebreaker', () { 1221 group('linebreaker', () {
1222 1222
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
1448 input += lines[i++] + '\n'; 1448 input += lines[i++] + '\n';
1449 } 1449 }
1450 while(++i < lines.length && !lines[i].startsWith('>>>')) { 1450 while(++i < lines.length && !lines[i].startsWith('>>>')) {
1451 expectedOutput += lines[i] + '\n'; 1451 expectedOutput += lines[i] + '\n';
1452 } 1452 }
1453 test('test - (${testIndex++})', () { 1453 test('test - (${testIndex++})', () {
1454 expectClause(input, expectedOutput); 1454 expectClause(input, expectedOutput);
1455 }); 1455 });
1456 } 1456 }
1457 } 1457 }
OLDNEW
« no previous file with comments | « pkg/analyzer/test/services/data/stmt_tests.data ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698