Skip to content

Commit

Permalink
Updated build_web_compilers and added tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Anna Gringauze committed Jul 7, 2023
1 parent 53c7da0 commit dce82fb
Show file tree
Hide file tree
Showing 16 changed files with 25 additions and 18 deletions.
1 change: 1 addition & 0 deletions dwds/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

- Require clients to specify the `basePath` on `AssetReader`. - [#2160](https://github.com/dart-lang/webdev/pull/2160)
- Update SDK constraint to `>=3.1.0-254.0.dev <4.0.0`. - [#2169](https://github.com/dart-lang/webdev/pull/2169)
- Require min `build_web_compilers` version `4.0.4` - [#]()

## 19.0.2

Expand Down
2 changes: 1 addition & 1 deletion dwds/debug_extension/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ dependencies:

dev_dependencies:
build: ^2.0.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
build_runner: ^2.4.0
built_collection: ^5.0.0
dwds: ^11.0.0
Expand Down
2 changes: 1 addition & 1 deletion dwds/debug_extension_mv3/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ dev_dependencies:
build_runner: ^2.4.0
built_collection: ^5.0.0
built_value_generator: ^8.3.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
dwds: ^16.0.0
sse: ^4.1.2
web_socket_channel: ^2.2.0
Expand Down
2 changes: 1 addition & 1 deletion dwds/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ dev_dependencies:
build_daemon: ^4.0.0
build_runner: ^2.4.0
build_version: ^2.1.1
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
built_value_generator: ^8.3.0
dart_code_metrics: ^5.5.0
graphs: ^2.1.0
Expand Down
12 changes: 7 additions & 5 deletions dwds/test/instances/instance_canary_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,13 @@ void main() {
// Enable verbose logging for debugging.
final debug = false;

_runAllTests(
canaryFeatures: true,
compilationMode: CompilationMode.frontendServer,
debug: debug,
);
for (var compilationMode in CompilationMode.values) {
_runAllTests(
canaryFeatures: true,
compilationMode: compilationMode,
debug: debug,
);
}
}

void _runAllTests({
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/instance_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ void main() {
final provider = TestSdkConfigurationProvider(verbose: debug);
tearDownAll(provider.dispose);

for (var compilationMode in [CompilationMode.frontendServer]) {
for (var compilationMode in CompilationMode.values) {
_runTests(
provider: provider,
compilationMode: compilationMode,
Expand Down
2 changes: 1 addition & 1 deletion example/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ environment:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
2 changes: 1 addition & 1 deletion fixtures/_experimentSound/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,4 @@ dependencies:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
2 changes: 1 addition & 1 deletion fixtures/_testCircular1Sound/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,5 @@ dependencies:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4

2 changes: 1 addition & 1 deletion fixtures/_testCircular2Sound/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ dependencies:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4

2 changes: 1 addition & 1 deletion fixtures/_testPackageSound/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ dependencies:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4

2 changes: 1 addition & 1 deletion fixtures/_testSound/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ dependencies:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4

2 changes: 1 addition & 1 deletion fixtures/_webdevSoundSmoke/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ environment:

dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
4 changes: 4 additions & 0 deletions webdev/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 3.0.7-wip

- Update `build_web_compilers` constraint to `^4.0.4`.

## 3.0.6

- Update `dwds` constraint to `19.0.2`.
Expand Down
2 changes: 1 addition & 1 deletion webdev/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ similar to:
...
dev_dependencies:
build_runner: ^2.4.0
build_web_compilers: ^4.0.0
build_web_compilers: ^4.0.4
...
```

Expand Down
2 changes: 1 addition & 1 deletion webdev/lib/src/pubspec.dart
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ Future<List<PackageExceptionDetails>> _validateBuildDaemonVersion(
}

final buildRunnerConstraint = VersionConstraint.parse('^2.4.0');
final buildWebCompilersConstraint = VersionConstraint.parse('^4.0.0');
final buildWebCompilersConstraint = VersionConstraint.parse('^4.0.4');

// Note the minimum versions should never be dev versions as users will not
// get them by default.
Expand Down

0 comments on commit dce82fb

Please sign in to comment.