Make source map location information more readable.

pkg/source_maps/test/span_test.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library test.span_test;
 
 import 'package:unittest/unittest.dart';
 import 'package:source_maps/span.dart';
 
 const String TEST_FILE = '''
@@ skipping 54 matching lines @@
 
   test('get text', () {
     // fifth line (including 4 new lines), columns 2 .. 11
     var line = 10 + 80 + 31 + 27 + 4;
     expect(file.getText(line + 2, line + 11), '34+6789_1');
   });
 
   group('location message', () {
     test('first line', () {
       expect(file.getLocationMessage('the message', 1, 3),
-          'file:1:2: the message\n'
+          'line 1, column 2 of file: the message\n'
           '+23456789_\n'
           ' ^^');
     });
 
     test('in the middle of the file', () {
       // fifth line (including 4 new lines), columns 2 .. 11
       var line = 10 + 80 + 31 + 27 + 4;
       expect(file.getLocationMessage('the message', line + 2, line + 11),
-          'file:5:3: the message\n'
+          'line 5, column 3 of file: the message\n'
           '1234+6789_1234\n'
           '  ^^^^^^^^^');
     });
 
     test('no file url', () {
       var line = 10 + 80 + 31 + 27 + 4;
       expect(new SourceFile.text(null, TEST_FILE).getLocationMessage(
           'the message', line + 2, line + 11),
-          ':5:3: the message\n'
+          'line 5, column 3: the message\n'
           '1234+6789_1234\n'
           '  ^^^^^^^^^');
     });
 
     test('penultimate line', () {
       // We search '\n' backwards twice because last line is \n terminated:
       int index = TEST_FILE.lastIndexOf('\n');
       var start = TEST_FILE.lastIndexOf('\n', index - 1) - 3;
       expect(file.getLocationMessage('the message', start, start + 2),
-          'file:11:41: the message\n'
+          'line 11, column 41 of file: the message\n'
           '123456789_+23456789_123456789_123456789_123\n'
           '                                        ^^');
     });
 
     test('last line', () {
       var start = TEST_FILE.lastIndexOf('\n') - 2;
       expect(file.getLocationMessage('the message', start, start + 1),
-          'file:12:28: the message\n'
+          'line 12, column 28 of file: the message\n'
           '123456789_1+3456789_123456789\n'
           '                           ^');
     });
 
     group('no trailing empty-line at the end -', () {
       var text = TEST_FILE.substring(0, TEST_FILE.length - 1);
       var file2 = new SourceFile.text('file', text);
 
       test('penultimate line', () {
         var start = text.lastIndexOf('\n') - 3;
         expect(file2.getLocationMessage('the message', start, start + 2),
-            'file:11:41: the message\n'
+            'line 11, column 41 of file: the message\n'
             '123456789_+23456789_123456789_123456789_123\n'
             '                                        ^^');
       });
 
       test('last line', () {
         var start = text.length - 2;
         expect(file2.getLocationMessage('the message', start, start + 1),
-            'file:12:28: the message\n'
+            'line 12, column 28 of file: the message\n'
             '123456789_1+3456789_123456789\n'
             '                           ^');
       });
     });
 
     test('single line', () {
       var text = "this is a single line";
       int start = text.indexOf(' ') + 1;
       var file2 = new SourceFile.text('file', text);
       expect(file2.getLocationMessage('the message', start, start + 2),
-            'file:1:${start + 1}: the message\n'
+            'line 1, column ${start + 1} of file: the message\n'
             'this is a single line\n'
             '     ^^');
     });
   });
 
   test('location getters', () {
     expect(loc(8).line, 0);
     expect(loc(8).column, 8);
     expect(loc(9).line, 0);
     expect(loc(9).column, 9);
@@ skipping 18 matching lines @@
     expect(span(8, 9).start.line, 0);
     expect(span(8, 9).start.column, 8);
     expect(span(8, 9).end.line, 0);
     expect(span(8, 9).end.column, 9);
     expect(span(8, 9).text, '9');
     expect(span(8, 9).isIdentifier, false);
     expect(span(8, 9).formatLocation, 'file:1:9');
 
     var line = 10 + 80 + 31 + 27 + 4;
     expect(span(line + 2, line + 11).getLocationMessage('the message'),
-        'file:5:3: the message\n'
+        'line 5, column 3 of file: the message\n'
         '1234+6789_1234\n'
         '  ^^^^^^^^^');
 
     expect(span(12, 95).start.line, 1);
     expect(span(12, 95).start.column, 1);
     expect(span(12, 95).end.line, 2);
     expect(span(12, 95).end.column, 3);
     expect(span(12, 95).text,
         '+       _123456789_123456789_123456789_123456789_123456789_1234567'
         '89_123456789_\n  +');
@@ skipping 27 matching lines @@
         expect(span.end.offset, greaterThanOrEqualTo(lastEnd));
       }
       lastStart = span.start.offset;
       lastEnd = span.end.offset;
     }
   });
 
   test('range check for large offsets', () {
     var start = TEST_FILE.length;
     expect(file.getLocationMessage('the message', start, start + 9),
-        'file:13:1: the message\n');
+        'line 13, column 1 of file: the message\n');
   });
 
   group('file segment', () {
     var baseOffset = 123;
     var segmentText = TEST_FILE.substring(baseOffset, TEST_FILE.length - 100);
     var segment = new SourceFileSegment('file', segmentText, loc(baseOffset));
     sline(int n) => segment.getLine(n);
     scol(int n) => segment.getColumn(segment.getLine(n), n);
     line(int n) => file.getLine(n);
     col(int n) => file.getColumn(file.getLine(n), n);
@@ skipping 49 matching lines @@
             file.getLocationMessage('the message', start, start + 2));
 
         start = segmentText.length + 20;
         expect(segment.getLocationMessage('the message', start, start + 2),
             file.getLocationMessage('the message', start, start + 2));
       });
 
       test('past segment, past its line', () {
         var start = TEST_FILE.length - 2;
         expect(file.getLocationMessage('the message', start, start + 1),
-          'file:12:29: the message\n'
+          'line 12, column 29 of file: the message\n'
           '123456789_1+3456789_123456789\n'
           '                            ^');
 
         // The answer below is different because the segment parsing only knows
         // about the 10 lines it has (and nothing about the possible extra lines
         // afterwards)
         expect(segment.getLocationMessage('the message', start, start + 1),
-          'file:11:1: the message\n');
+          'line 11, column 1 of file: the message\n');
       });
     });
   });
 
   test('span isIdentifier defaults to false', () {
     var start = new TestLocation(0);
     var end = new TestLocation(1);
     expect(new TestSpan(start, end).isIdentifier, false);
     expect(file.span(8, 9, null).isIdentifier, false);
     expect(new FixedSpan('', 8, 1, 8, isIdentifier: null).isIdentifier, false);
@@ skipping 14 matching lines @@
   TestSpan(Location start, Location end) : super(start, end, null);
   get text => null;
 }
 
 class TestLocation extends Location {
   String get sourceUrl => '';
   TestLocation(int offset) : super(offset);
   get line => 0;
   get column => 0;
 }