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

Side by Side Diff: pkg/barback/test/utils.dart

Issue 196473016: Consider errors logged by a transformer equivalent to thrown errors. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
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 library barback.test.utils; 5 library barback.test.utils;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:collection'; 8 import 'dart:collection';
9 import 'dart:convert' show Encoding; 9 import 'dart:convert' show Encoding;
10 10
11 import 'package:barback/barback.dart'; 11 import 'package:barback/barback.dart';
12 import 'package:barback/src/cancelable_future.dart'; 12 import 'package:barback/src/cancelable_future.dart';
13 import 'package:barback/src/utils.dart'; 13 import 'package:barback/src/utils.dart';
14 import 'package:path/path.dart' as pathos; 14 import 'package:path/path.dart' as pathos;
15 import 'package:scheduled_test/scheduled_test.dart'; 15 import 'package:scheduled_test/scheduled_test.dart';
16 import 'package:stack_trace/stack_trace.dart'; 16 import 'package:stack_trace/stack_trace.dart';
17 import 'package:unittest/compact_vm_config.dart'; 17 import 'package:unittest/compact_vm_config.dart';
18 18
19 export 'transformer/bad.dart'; 19 export 'transformer/bad.dart';
20 export 'transformer/bad_log.dart';
20 export 'transformer/check_content.dart'; 21 export 'transformer/check_content.dart';
21 export 'transformer/check_content_and_rename.dart'; 22 export 'transformer/check_content_and_rename.dart';
22 export 'transformer/conditionally_consume_primary.dart'; 23 export 'transformer/conditionally_consume_primary.dart';
23 export 'transformer/create_asset.dart'; 24 export 'transformer/create_asset.dart';
24 export 'transformer/emit_nothing.dart'; 25 export 'transformer/emit_nothing.dart';
25 export 'transformer/lazy_bad.dart'; 26 export 'transformer/lazy_bad.dart';
26 export 'transformer/lazy_many_to_one.dart'; 27 export 'transformer/lazy_many_to_one.dart';
27 export 'transformer/lazy_rewrite.dart'; 28 export 'transformer/lazy_rewrite.dart';
28 export 'transformer/many_to_one.dart'; 29 export 'transformer/many_to_one.dart';
29 export 'transformer/mock.dart'; 30 export 'transformer/mock.dart';
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 (package, assets) => new AssetSet.from(assets)); 91 (package, assets) => new AssetSet.from(assets));
91 92
92 // Make sure that packages that have transformers but no assets are considered 93 // Make sure that packages that have transformers but no assets are considered
93 // by MockProvider to exist. 94 // by MockProvider to exist.
94 for (var package in transformers.keys) { 95 for (var package in transformers.keys) {
95 assetMap.putIfAbsent(package, () => new AssetSet()); 96 assetMap.putIfAbsent(package, () => new AssetSet());
96 } 97 }
97 98
98 _provider = new MockProvider(assetMap); 99 _provider = new MockProvider(assetMap);
99 _barback = new Barback(_provider); 100 _barback = new Barback(_provider);
101 // Add a dummy listener to the log so it doesn't print to stdout.
102 _barback.log.listen((_) {});
100 _nextBuildResult = 0; 103 _nextBuildResult = 0;
101 _nextLog = 0; 104 _nextLog = 0;
102 105
103 schedule(() => transformers.forEach(_barback.updateTransformers)); 106 schedule(() => transformers.forEach(_barback.updateTransformers));
104 107
105 // There should be one successful build after adding all the transformers but 108 // There should be one successful build after adding all the transformers but
106 // before adding any sources. 109 // before adding any sources.
107 if (!transformers.isEmpty) buildShouldSucceed(); 110 if (!transformers.isEmpty) buildShouldSucceed();
108 } 111 }
109 112
(...skipping 457 matching lines...) Expand 10 before | Expand all | Expand 10 after
567 570
568 _MockAsset(this.id, this.contents); 571 _MockAsset(this.id, this.contents);
569 572
570 Future<String> readAsString({Encoding encoding}) => 573 Future<String> readAsString({Encoding encoding}) =>
571 new Future.value(contents); 574 new Future.value(contents);
572 575
573 Stream<List<int>> read() => throw new UnimplementedError(); 576 Stream<List<int>> read() => throw new UnimplementedError();
574 577
575 String toString() => "MockAsset $id $contents"; 578 String toString() => "MockAsset $id $contents";
576 } 579 }
OLDNEW
« pkg/barback/lib/src/transform_node.dart ('K') | « pkg/barback/test/transformer/bad_log.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698