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

Unified Diff: pkg/scheduled_test/lib/src/descriptor/file.dart

Issue 12853005: Change the way Patterns work in scheduled_test/descriptor. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes. Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: pkg/scheduled_test/lib/src/descriptor/file.dart
diff --git a/pkg/scheduled_test/lib/src/descriptor/file.dart b/pkg/scheduled_test/lib/src/descriptor/file.dart
deleted file mode 100644
index 15b9ef46f3f0d665dcafc2b7031cdfd210eb587f..0000000000000000000000000000000000000000
--- a/pkg/scheduled_test/lib/src/descriptor/file.dart
+++ /dev/null
@@ -1,107 +0,0 @@
-// 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 descriptor.file;
-
-import 'dart:async';
-import 'dart:io' as io;
-import 'dart:math' as math;
-import 'dart:utf';
-
-import '../../../../../pkg/pathos/lib/path.dart' as path;
-
-import '../../descriptor.dart' as descriptor;
-import '../../scheduled_test.dart';
-import '../utils.dart';
-import 'utils.dart';
-
-/// A descriptor describing a single file.
-class File extends descriptor.Entry {
- /// Whether this descriptor describes a binary file. This is only used when
- /// displaying error messages.
- final bool isBinary;
-
- /// The contents of the file, in bytes.
- final List<int> contents;
-
- /// The contents of the file as a String. Assumes UTF-8 encoding.
- String get textContents => new String.fromCharCodes(contents);
-
- File.binary(Pattern name, List<int> contents)
- : this._(name, contents, true);
-
- File(Pattern name, String contents)
- : this._(name, encodeUtf8(contents), false);
-
- File._(Pattern name, this.contents, this.isBinary)
- : super(name);
-
- Future create([String parent]) => schedule(() {
- if (parent == null) parent = descriptor.defaultRoot;
- return new io.File(path.join(parent, stringName)).writeAsBytes(contents);
- }, 'creating file $nameDescription');
-
- Future validate([String parent]) => schedule(() {
- if (parent == null) parent = descriptor.defaultRoot;
- var fullPath = entryMatchingPattern('File', parent, name);
- return new io.File(fullPath).readAsBytes()
- .then((actualContents) {
- if (orderedIterableEquals(contents, actualContents)) return;
- if (isBinary) {
- // TODO(nweiz): show a hex dump here if the data is small enough.
- throw "File $nameDescription didn't contain the expected binary "
- "data.";
- }
- var description = nameDescription;
- if (name is! String) {
- description = "'${path.basename(fullPath)}' (matching $description)";
- }
- throw _textMismatchMessage(description, textContents,
- new String.fromCharCodes(actualContents));;
- });
- }, 'validating file $nameDescription');
-
- Stream<List<int>> read() => new Future.immediate(contents).asStream();
-
- String describe() {
- if (name is String) return name;
- return 'file matching $nameDescription';
- }
-}
-
-String _textMismatchMessage(String description, String expected,
- String actual) {
- final expectedLines = expected.split('\n');
- final actualLines = actual.split('\n');
-
- var results = [];
-
- // Compare them line by line to see which ones match.
- var length = math.max(expectedLines.length, actualLines.length);
- for (var i = 0; i < length; i++) {
- if (i >= actualLines.length) {
- // Missing output.
- results.add('? ${expectedLines[i]}');
- } else if (i >= expectedLines.length) {
- // Unexpected extra output.
- results.add('X ${actualLines[i]}');
- } else {
- var expectedLine = expectedLines[i];
- var actualLine = actualLines[i];
-
- if (expectedLine != actualLine) {
- // Mismatched lines.
- results.add('X $actualLine');
- } else {
- // Matched lines.
- results.add('| $actualLine');
- }
- }
- }
-
- return "File $description should contain:\n"
- "${prefixLines(expected)}\n"
- "but actually contained:\n"
- "${results.join('\n')}";
-}
« no previous file with comments | « pkg/scheduled_test/lib/src/descriptor/entry.dart ('k') | pkg/scheduled_test/lib/src/descriptor/file_descriptor.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698