Google Git
Sign in
dart / sdk.git / refs/heads/lkgr^! / .
commitc73677e606f78b15efa25971699af1fdc5824fd6[log] [tgz]
authorParker Lougheed <parlough@gmail.com>Wed Oct 09 08:38:48 2024 +0000
committerCommit Queue <dart-scoped@luci-project-accounts.iam.gserviceaccount.com>Wed Oct 09 08:38:48 2024 +0000
treef6e66c4eddcd96aa27bcd3d8207107c643775930
parenta1c805431ef3e3e62b55b7432fa2e71c72fe81ef [diff]
[tests] Minor spelling and grammar fixes

Change-Id: I18c309b9037cf94a883443ac9067f911f829516c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/388945
Reviewed-by: Lasse Nielsen <lrn@google.com>
Reviewed-by: Nate Bosch <nbosch@google.com>
Commit-Queue: Lasse Nielsen <lrn@google.com>

diff --git a/tests/corelib/date_time_far_away_dates_test.dart b/tests/corelib/date_time_far_away_dates_test.dart
index d5ab6bb..d4a08c4 100644
--- a/tests/corelib/date_time_far_away_dates_test.dart
+++ b/tests/corelib/date_time_far_away_dates_test.dart

@@ -50,7 +50,7 @@
   Expect.equals(true, dt.hour >= 0 && dt.hour < 24);
   // Not much we can test for the minute and second.
   //
-  // Historical timezones can have arbitrary offets. For example, "Pacific
+  // Historical timezones can have arbitrary offsets. For example, "Pacific
   // Standard Time" in the United States is represented by Los Angeles. Prior to
   // the adoption of "Railway Time" in the United States at noon on 18 November
   // 1883, the local time in Los Angeles had an offset of a few minutes from the

diff --git a/tests/corelib/map_test.dart b/tests/corelib/map_test.dart
index cedf513..751085f 100644
--- a/tests/corelib/map_test.dart
+++ b/tests/corelib/map_test.dart

@@ -57,7 +57,7 @@
 
   // NaN maps need to have nullable value types because the forEach method
   // cannot look up the value and therefore might find `null` instead of the
-  // actuall value. See MapMixin.forEach in dart:collection/maps.dart
+  // actual value. See MapMixin.forEach in dart:collection/maps.dart
   testNaNKeys(new Map());
   testNaNKeys(new Map<num, String?>());
   testNaNKeys(new HashMap());

diff --git a/tests/corelib/regexp/overflow_test.dart b/tests/corelib/regexp/overflow_test.dart
index 42523d0..20d0a75 100644
--- a/tests/corelib/regexp/overflow_test.dart
+++ b/tests/corelib/regexp/overflow_test.dart

@@ -27,7 +27,7 @@
 
 void main() {
   description(
-      "This test checks expressions with alternative lengths of appox. 2^31.");
+      "This test checks expressions with alternative lengths of approx. 2^31.");
 
   var regexp1 = new RegExp(r"(?:(?=g))|(?:m).{2147483648,}");
   shouldBeNull(regexp1.firstMatch(''));

diff --git a/tests/corelib/regexp/unicode-regexp-restricted-syntax_test.dart b/tests/corelib/regexp/unicode-regexp-restricted-syntax_test.dart
index d894c7b..b9f84f4 100644
--- a/tests/corelib/regexp/unicode-regexp-restricted-syntax_test.dart
+++ b/tests/corelib/regexp/unicode-regexp-restricted-syntax_test.dart

@@ -51,7 +51,7 @@
   assertThrows(() => RegExp(r"[\c]", unicode: true));
   // test262/built-ins/RegExp/unicode_restricted_identity_escape_c
   assertThrows(() => RegExp(r"[\c0]", unicode: true));
-  // test262/built-ins/RegExp/unicode_restricted_incomple_quantifier
+  // test262/built-ins/RegExp/unicode_restricted_incomplete_quantifier
   assertThrows(() => RegExp(r"a{", unicode: true));
   assertThrows(() => RegExp(r"a{1,", unicode: true));
   assertThrows(() => RegExp(r"{", unicode: true));

diff --git a/tests/corelib/stopwatch_test.dart b/tests/corelib/stopwatch_test.dart
index e6b8a97..0aa1fcb 100644
--- a/tests/corelib/stopwatch_test.dart
+++ b/tests/corelib/stopwatch_test.dart

@@ -44,7 +44,7 @@
       }
       sw2LastElapsed = sw2.elapsedTicks;
     }
-    // The test only makes sense if measureable time elapsed and elapsed time
+    // The test only makes sense if measurable time elapsed and elapsed time
     // on the stopped Stopwatch did not increase.
     Expect.isTrue(sw2.elapsedTicks > 0);
     return true;

diff --git a/tests/language/await/await_with_no_context_test.dart b/tests/language/await/await_with_no_context_test.dart
index 872f530..edfaace 100644
--- a/tests/language/await/await_with_no_context_test.dart
+++ b/tests/language/await/await_with_no_context_test.dart

@@ -2,7 +2,7 @@
 // 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.
 
-// Verify that if an `await` expression appears in a syntactic postion that
+// Verify that if an `await` expression appears in a syntactic position that
 // doesn't impose a context on it, it supplies a context of `FutureOr<_>` to its
 // operand, rather than `_` (which was the behavior prior to fixing
 // https://github.com/dart-lang/language/issues/3648).

diff --git a/tests/language/bool/has_environment_not_new_test.dart b/tests/language/bool/has_environment_not_new_test.dart
index ec4a7f0..aa6c13b9 100644
--- a/tests/language/bool/has_environment_not_new_test.dart
+++ b/tests/language/bool/has_environment_not_new_test.dart

@@ -8,7 +8,7 @@
 
 main() {
   // On non-VM targets `new bool.hasEnvironment(...)` just throws, because it
-  // is only guaranted to work with `const`. However on VM it actually works.
+  // is only guaranteed to work with `const`. However on VM it actually works.
   if (!isVM) {
     Expect.throws(() => new bool.hasEnvironment("Anything"));
   } else {

diff --git a/tests/language/class_modifiers/trans_legacy/legacy_lib.dart b/tests/language/class_modifiers/trans_legacy/legacy_lib.dart
index 3f30bdb..c71e8e5 100644
--- a/tests/language/class_modifiers/trans_legacy/legacy_lib.dart
+++ b/tests/language/class_modifiers/trans_legacy/legacy_lib.dart

@@ -8,7 +8,7 @@
 
 // Pre-feature declarations which ignore platform library restrictions.
 // Used to test that feature-enabled libraries behave correctly when
-// going througn pre-feature super-declarations.
+// going through pre-feature super-declarations.
 
 // Ignoring rule against extending, implementing, mixing in and `on`-typing
 // a final declaration.

diff --git a/tests/language/deferred/exported_main_test.dart b/tests/language/deferred/exported_main_test.dart
index 60b5473..8efa37a 100644
--- a/tests/language/deferred/exported_main_test.dart
+++ b/tests/language/deferred/exported_main_test.dart

@@ -9,6 +9,6 @@
   // The frontend will not recognize that this is the root library. Though this
   // library is not reachable from what the frontend considers the root library,
   // the entry point pragma will still cause this function to compiled by
-  // gen_snaption, so it is important that this library is assigned to a
+  // gen_snapshot, so it is important that this library is assigned to a
   // loading unit.
 }

diff --git a/tests/language/enum/enhanced_enums_basic_test.dart b/tests/language/enum/enhanced_enums_basic_test.dart
index be97c45..43deaef 100644
--- a/tests/language/enum/enhanced_enums_basic_test.dart
+++ b/tests/language/enum/enhanced_enums_basic_test.dart

@@ -236,7 +236,7 @@
   // constructors and the implicit element creation expressions.)
   // Cannot have const factory constructor, because they *must* redirect to
   // generative constructors.
-  // Cannot have `super`-constuctor invocations in initializer lists.
+  // Cannot have `super`-constructor invocations in initializer lists.
 
   // Instance members.
 

diff --git a/tests/language/inference_update_2/cascaded_field_promotion_test.dart b/tests/language/inference_update_2/cascaded_field_promotion_test.dart
index d1cc807..4f0909c 100644
--- a/tests/language/inference_update_2/cascaded_field_promotion_test.dart
+++ b/tests/language/inference_update_2/cascaded_field_promotion_test.dart

@@ -90,7 +90,7 @@
 void writeDefeatsLaterAccessesButNotCascadeTarget(C c) {
   // If a write to a variable happens during a cascade, any promotions based on
   // that variable are invalidated, but the target of the cascade remains
-  // promoted, since it's stored in an implicit temporarly variable that's
+  // promoted, since it's stored in an implicit temporary variable that's
   // unaffected by the write.
   c._field as C;
   c._field.expectStaticType<Exactly<C>>();

diff --git a/tests/language/inference_update_2/cascaded_field_promotion_unnecessary_null_aware_error_test.dart b/tests/language/inference_update_2/cascaded_field_promotion_unnecessary_null_aware_error_test.dart
index 89d1282..48fc48f 100644
--- a/tests/language/inference_update_2/cascaded_field_promotion_unnecessary_null_aware_error_test.dart
+++ b/tests/language/inference_update_2/cascaded_field_promotion_unnecessary_null_aware_error_test.dart

@@ -100,7 +100,7 @@
 void writeDefeatsLaterAccessesButNotCascadeTarget(C c) {
   // If a write to a variable happens during a cascade, any promotions based on
   // that variable are invalidated, but the target of the cascade remains
-  // promoted, since it's stored in an implicit temporarly variable that's
+  // promoted, since it's stored in an implicit temporary variable that's
   // unaffected by the write.
   c._field as C;
   c._field.expectStaticType<Exactly<C>>();

diff --git a/tests/language/inference_update_2/field_promotion_and_no_such_method_library_cycle_test.dart b/tests/language/inference_update_2/field_promotion_and_no_such_method_library_cycle_test.dart
index c1add5a..a0f4513 100644
--- a/tests/language/inference_update_2/field_promotion_and_no_such_method_library_cycle_test.dart
+++ b/tests/language/inference_update_2/field_promotion_and_no_such_method_library_cycle_test.dart

@@ -78,7 +78,7 @@
 }
 
 testImplementedFieldSeenViaOtherLib(H h) {
-  // Class `G` inherits an implmentation of `_f1` from `C` (via `B` and
+  // Class `G` inherits an implementation of `_f1` from `C` (via `B` and
   // `A`). Therefore it doesn't need a noSuchMethod forwarder for `_f1`, and
   // consequently, promotion of `H._f1` works.
   if (h._f1 != null) {
@@ -86,7 +86,7 @@
   }
 }
 
-testInterfceFieldSeenViaOtherLib(H h) {
+testInterfaceFieldSeenViaOtherLib(H h) {
   // Class `G` inherits `_f2` into its interface from `F` (via `E` and `D`). But
   // it doesn't inherit an implementation of `_f2` from anywhere. Therefore it
   // needs a noSuchMethod forwarder for `_f2`, and consequently, promotion of
@@ -99,6 +99,6 @@
 main() {
   for (var h in [H(null), H(0)]) {
     testImplementedFieldSeenViaOtherLib(h);
-    testInterfceFieldSeenViaOtherLib(h);
+    testInterfaceFieldSeenViaOtherLib(h);
   }
 }

diff --git a/tests/language/inference_update_2/field_promotion_name_conflicts_mixin_test.dart b/tests/language/inference_update_2/field_promotion_name_conflicts_mixin_test.dart
index 7c719d2..4a3477b 100644
--- a/tests/language/inference_update_2/field_promotion_name_conflicts_mixin_test.dart
+++ b/tests/language/inference_update_2/field_promotion_name_conflicts_mixin_test.dart

@@ -50,7 +50,7 @@
   int? get _nonLate3 => 0;
   int? get _late3 => 0;
   // Any noSuchMethod-forwarding getter inhibits promotion, since the
-  // implementation of noSuchMethod is assumeb to not be stable. (Requires that
+  // implementation of noSuchMethod is assumed to not be stable. (Requires that
   // the class be concrete and fail to implement a part of its interface; such a
   // class is only allowed if it contains or inherits a noSuchMethod
   // declaration).

diff --git a/tests/language/mixin/black_listed_test.dart b/tests/language/mixin/deny_listed_test.dart
similarity index 96%
rename from tests/language/mixin/black_listed_test.dart
rename to tests/language/mixin/deny_listed_test.dart
index b8a89ad..5a4e570 100644
--- a/tests/language/mixin/black_listed_test.dart
+++ b/tests/language/mixin/deny_listed_test.dart

@@ -2,7 +2,7 @@
 // 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.
 
-// Check mixin of black-listed types.
+// Check mixin of deny-listed types.
 
 import 'package:expect/expect.dart';
 

diff --git a/tests/language/mixin_legacy/black_listed_test.dart b/tests/language/mixin_legacy/deny_listed_test.dart
similarity index 96%
rename from tests/language/mixin_legacy/black_listed_test.dart
rename to tests/language/mixin_legacy/deny_listed_test.dart
index 5dc5088..5d3fe22 100644
--- a/tests/language/mixin_legacy/black_listed_test.dart
+++ b/tests/language/mixin_legacy/deny_listed_test.dart

@@ -4,7 +4,7 @@
 
 // @dart=2.19
 
-// Check mixin of black-listed types.
+// Check mixin of deny-listed types.
 
 import 'package:expect/expect.dart';
 

diff --git a/tests/language/nnbd/definite_assignment/definite_assignment_error_test.dart b/tests/language/nnbd/definite_assignment/definite_assignment_error_test.dart
index 69e40c7..6e856be 100644
--- a/tests/language/nnbd/definite_assignment/definite_assignment_error_test.dart
+++ b/tests/language/nnbd/definite_assignment/definite_assignment_error_test.dart

@@ -198,7 +198,7 @@
   }
 
   // Assignments in a branch of a switch which returns do not affect definite
-  // uassignment after the switch.
+  // assignment after the switch.
   if (b) {
     int x;
     late int y;

diff --git a/tests/language/null_aware_elements/type_inference_simple_positive_test.dart b/tests/language/null_aware_elements/type_inference_simple_positive_test.dart
index 51d7239..d26c865 100644
--- a/tests/language/null_aware_elements/type_inference_simple_positive_test.dart
+++ b/tests/language/null_aware_elements/type_inference_simple_positive_test.dart

@@ -7,7 +7,7 @@
 import '../static_type_helper.dart';
 
 // This is a helper function to suggest to the type inference that the argument
-// is a map and should be disambiguated as such, whithough affecting the type
+// is a map and should be disambiguated as such, without affecting the type
 // arguments of the map argument
 acceptsMap<X, Y>(Map<X, Y> m) {}
 

diff --git a/tests/language/operator/truncdiv_zero_test.dart b/tests/language/operator/truncdiv_zero_test.dart
index db50eb2..e35fe49 100644
--- a/tests/language/operator/truncdiv_zero_test.dart
+++ b/tests/language/operator/truncdiv_zero_test.dart

@@ -13,7 +13,7 @@
   if (!v.jsNumbers) {
     Expect.throws<UnsupportedError>(() => truncdiv_test.foo(12, 0));
   } else {
-    // Web numbers consider infinities to be large-magnitide 'int' values.
+    // Web numbers consider infinities to be large-magnitude 'int' values.
     truncdiv_test.foo(12, 0);
   }
   Expect.throws<UnsupportedError>(() => truncdiv_test.foo2(0));

diff --git a/tests/language/optimize/lists_test.dart b/tests/language/optimize/lists_test.dart
index ea4cd6d..d320c48 100644
--- a/tests/language/optimize/lists_test.dart
+++ b/tests/language/optimize/lists_test.dart

@@ -16,7 +16,7 @@
 test(n) {
   var a = []; //    Growable list.
   var b = new List<dynamic>.filled(10, null); // Fixed size list.
-  var c = const [1, 2, 3, 4]; // Constant aray.
+  var c = const [1, 2, 3, 4]; // Constant array.
   // In optimized mode the class checks will be eliminated since the
   // constructors above provide information about exact types.
   a.add(4);

diff --git a/tests/language/regress/regress52468/before_last_test.dart b/tests/language/regress/regress52468/before_last_test.dart
index 76ae962..989aef4 100644
--- a/tests/language/regress/regress52468/before_last_test.dart
+++ b/tests/language/regress/regress52468/before_last_test.dart

@@ -6,7 +6,7 @@
 //
 // Similar to simple_test.dart, but ensure the logic works also when the
 // unevaluated constant is not the last record entry (tracks properly managing
-// the internal state of unevaluated constnats on sequence of subexpressions).
+// the internal state of unevaluated constants on sequence of subexpressions).
 //
 // Regression test for https://github.com/dart-lang/sdk/issues/52468
 const unevaluated = const bool.fromEnvironment('a.b.c') ? 1 : 2;

diff --git a/tests/language/regress/regress56314_test.dart b/tests/language/regress/regress56314_test.dart
index 71ecfab..a306bea 100644
--- a/tests/language/regress/regress56314_test.dart
+++ b/tests/language/regress/regress56314_test.dart

@@ -48,7 +48,7 @@
 }
 
 class D4 extends B4 {
-  // The default value is negative, so the conversion is neccessary. The
+  // The default value is negative, so the conversion is necessary. The
   // conversion will be optimized away if the negative default value is ignored
   // in the global type analysis.
   int foo4([int a = -16]) => a | 1;

diff --git a/tests/language/stack_trace/full1_test.dart b/tests/language/stack_trace/full1_test.dart
index 35b5824..0d32ab3 100644
--- a/tests/language/stack_trace/full1_test.dart
+++ b/tests/language/stack_trace/full1_test.dart

@@ -4,7 +4,7 @@
 
 // Avoid dart2js optimizations that alter the JavaScript stack trace. (1)
 // dart2js inlines methods in the generated JavaScript which leads to missing
-// frames in the stack trace. (2) Minification obfucsates names. Both issues can
+// frames in the stack trace. (2) Minification obfuscates names. Both issues can
 // be addressed offline by post-processing the stack trace using the source map
 // file.
 //

diff --git a/tests/language/stack_trace/full2_test.dart b/tests/language/stack_trace/full2_test.dart
index ae1e936..f6075ad 100644
--- a/tests/language/stack_trace/full2_test.dart
+++ b/tests/language/stack_trace/full2_test.dart

@@ -4,7 +4,7 @@
 
 // Avoid dart2js optimizations that alter the JavaScript stack trace. (1)
 // dart2js inlines methods in the generated JavaScript which leads to missing
-// frames in the stack trace. (2) Minification obfucsates names. Both issues can
+// frames in the stack trace. (2) Minification obfuscates names. Both issues can
 // be addressed offline by post-processing the stack trace using the source map
 // file.
 //

diff --git a/tests/language/stack_trace/full3_test.dart b/tests/language/stack_trace/full3_test.dart
index 5dc16ae..f675dc8 100644
--- a/tests/language/stack_trace/full3_test.dart
+++ b/tests/language/stack_trace/full3_test.dart

@@ -4,7 +4,7 @@
 
 // Avoid dart2js optimizations that alter the JavaScript stack trace. (1)
 // dart2js inlines methods in the generated JavaScript which leads to missing
-// frames in the stack trace. (2) Minification obfucsates names. Both issues can
+// frames in the stack trace. (2) Minification obfuscates names. Both issues can
 // be addressed offline by post-processing the stack trace using the source map
 // file.
 //

diff --git a/tests/language/super/call2_test.dart b/tests/language/super/call2_test.dart
index 13ae089..a07f363 100644
--- a/tests/language/super/call2_test.dart
+++ b/tests/language/super/call2_test.dart

@@ -2,8 +2,9 @@
 // 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.
 
-// Regresion test for bug discovered in frog handling super calls: the test case
-// mixes generics, super calls, and purposely doesn't allocate the base type.
+// Regression test for bug discovered in frog handling super calls:
+// the test case mixes generics, super calls, and purposely doesn't
+// allocate the base type.
 //
 // Also is a regression test for https://github.com/dart-lang/sdk/issues/31973
 

diff --git a/tests/language/sync_star/sync_star_exception_current_test.dart b/tests/language/sync_star/sync_star_exception_current_test.dart
index 9cdbc58..29adbd5 100644
--- a/tests/language/sync_star/sync_star_exception_current_test.dart
+++ b/tests/language/sync_star/sync_star_exception_current_test.dart

@@ -47,7 +47,7 @@
   Expect.throws(() => i2.moveNext());
   Expect.isFalse(i2.moveNext());
 
-  // Slighly surprising consequence of the specified behavior.
+  // Slightly surprising consequence of the specified behavior.
   Expect.throws(() => f1().length);
   Expect.throws(() => f2().length);
 }

diff --git a/tests/language/type_object/futureor_tostring_test.dart b/tests/language/type_object/futureor_tostring_test.dart
index b43cdbf..5a1441d 100644
--- a/tests/language/type_object/futureor_tostring_test.dart
+++ b/tests/language/type_object/futureor_tostring_test.dart

@@ -22,7 +22,7 @@
 import "package:expect/expect.dart";
 
 void main() {
-  // Generally assumes that `Type.toString()` mathches the canonical
+  // Generally assumes that `Type.toString()` matches the canonical
   // format of the type, as used in the language specification,
   // with a leading name, `<...>` around type arguments,
   // and trailing `?` for the nullable union types,

diff --git a/tests/language/void/return_future_or_future_or_void_sync_test.dart b/tests/language/void/return_future_or_future_or_void_sync_test.dart
index f2254f7..1ccb54f 100644
--- a/tests/language/void/return_future_or_future_or_void_sync_test.dart
+++ b/tests/language/void/return_future_or_future_or_void_sync_test.dart

@@ -28,7 +28,7 @@
   return null as FutureOr<void>;
 }
 
-// Testing that a block bodied function may return Futu``re<void>
+// Testing that a block bodied function may return `Future<void>`
 FutureOr<FutureOr<void>> test4() {
   return Future<Future<void>>.value(Future<void>.value(null));
 }

diff --git a/tests/lib/async/future_onerror_test.dart b/tests/lib/async/future_onerror_test.dart
index 307f76e..aa69c8b 100644
--- a/tests/lib/async/future_onerror_test.dart
+++ b/tests/lib/async/future_onerror_test.dart

@@ -23,7 +23,7 @@
 
   /// Perform `onError` on [future] with arguments, and expect [result].
   ///
-  /// If [result] is [AsyncError], expecte an error result with
+  /// If [result] is [AsyncError], expect an error result with
   /// that error and stack trace.
   /// If [result.stackTrace] is [StackTrace.empty],
   /// check that the stack trace is *not* the original [stack].

diff --git a/tests/lib/async/stream_empty_test.dart b/tests/lib/async/stream_empty_test.dart
index a5c6fdd..770ee75 100644
--- a/tests/lib/async/stream_empty_test.dart
+++ b/tests/lib/async/stream_empty_test.dart

@@ -52,7 +52,7 @@
 
   StreamSubscription<int> sub =
       s.listen(unreachable, onError: unreachable, onDone: checker.check);
-  // Type parameter of subscription repspects stream.
+  // Type parameter of subscription respects stream.
   // Not a `StreamSubscription<Never>`.
   Expect.isFalse(sub is StreamSubscription<String>);
 

diff --git a/tests/lib/convert/line_splitter_performance_test.dart b/tests/lib/convert/line_splitter_performance_test.dart
index 4ccac72..7d23f8e 100644
--- a/tests/lib/convert/line_splitter_performance_test.dart
+++ b/tests/lib/convert/line_splitter_performance_test.dart

@@ -9,7 +9,7 @@
 import "package:expect/expect.dart";
 
 void main() {
-  testEfficency();
+  testEfficiency();
 }
 
 /// Regression test for https://dartbug.com/51167
@@ -17,7 +17,7 @@
 /// Had quadratic time behavior when concatenating chunks without linebreaks.
 ///
 /// Should now only use linear time/space for buffering.
-void testEfficency() {
+void testEfficiency() {
   // After fix: finishes in < 1 second on desktop.
   // Before fix, with N = 100000, took 25 seconds.
   const N = 1000000;

diff --git a/tests/lib/html/fileapi_directory_reader_test.dart b/tests/lib/html/fileapi_directory_reader_test.dart
index 93a614a..52736cd 100644
--- a/tests/lib/html/fileapi_directory_reader_test.dart
+++ b/tests/lib/html/fileapi_directory_reader_test.dart

@@ -20,7 +20,7 @@
   // test the functions that use those items.
   Future doDirSetup(String testName) async {
     final fs = await window.requestFileSystem(100);
-    // Prepend this file name to prevent collisions among tests runnning on the
+    // Prepend this file name to prevent collisions among tests running on the
     // same browser.
     const prefix = 'fileapi_directory_reader_';
     var file =

diff --git a/tests/lib/html/fileapi_directory_test.dart b/tests/lib/html/fileapi_directory_test.dart
index 9c8f701..2892d6e 100644
--- a/tests/lib/html/fileapi_directory_test.dart
+++ b/tests/lib/html/fileapi_directory_test.dart

@@ -8,7 +8,7 @@
 
 main() {
   if (!FileSystem.supported) return;
-  // Prepend this file name to prevent collisions among tests runnning on the
+  // Prepend this file name to prevent collisions among tests running on the
   // same browser.
   const prefix = 'fileapi_directory_';
 

diff --git a/tests/lib/html/fileapi_entry_test.dart b/tests/lib/html/fileapi_entry_test.dart
index 3e74130..74003fe 100644
--- a/tests/lib/html/fileapi_entry_test.dart
+++ b/tests/lib/html/fileapi_entry_test.dart

@@ -15,7 +15,7 @@
 
 main() {
   if (!FileSystem.supported) return;
-  // Prepend this file name to prevent collisions among tests runnning on the
+  // Prepend this file name to prevent collisions among tests running on the
   // same browser.
   const prefix = 'fileapi_entry_';
 

diff --git a/tests/lib/html/fileapi_file_entry_test.dart b/tests/lib/html/fileapi_file_entry_test.dart
index 1eb915b..7dc6c44 100644
--- a/tests/lib/html/fileapi_file_entry_test.dart
+++ b/tests/lib/html/fileapi_file_entry_test.dart

@@ -15,7 +15,7 @@
 
 main() {
   if (!FileSystem.supported) return;
-  // Prepend this file name to prevent collisions among tests runnning on the
+  // Prepend this file name to prevent collisions among tests running on the
   // same browser.
   const prefix = 'fileapi_file_entry_';
 

diff --git a/tests/lib/html/fileapi_file_test.dart b/tests/lib/html/fileapi_file_test.dart
index 0f706c1..3caeb0a 100644
--- a/tests/lib/html/fileapi_file_test.dart
+++ b/tests/lib/html/fileapi_file_test.dart

@@ -7,7 +7,7 @@
 
 main() {
   if (!FileSystem.supported) return;
-  // Prepend this file name to prevent collisions among tests runnning on the
+  // Prepend this file name to prevent collisions among tests running on the
   // same browser.
   const prefix = 'fileapi_file_';
 

diff --git a/tests/lib/html/htmloptionscollection_test.dart b/tests/lib/html/htmloptionscollection_test.dart
index df1daea..304296d 100644
--- a/tests/lib/html/htmloptionscollection_test.dart
+++ b/tests/lib/html/htmloptionscollection_test.dart

@@ -9,7 +9,7 @@
 main() {
   test('indexedAccessTest', () {
     // FIXME: we need some massaging to dart:html to enable HTMLOptionsCollection.add
-    // and hence programatic building of collection.
+    // and hence programmatic building of collection.
     SelectElement selectElement = new Element.html('''
       <select>
         <option value="0">Option0</option>

diff --git a/tests/lib/html/js_typed_interop_lazy_test.dart b/tests/lib/html/js_typed_interop_lazy_test.dart
index dac890b..cff72c8 100644
--- a/tests/lib/html/js_typed_interop_lazy_test.dart
+++ b/tests/lib/html/js_typed_interop_lazy_test.dart

@@ -101,7 +101,7 @@
   group('lazy class', () {
     test('type literal', () {
       // Fine because we can determine the class literals are equal without
-      // having to determine what (non-existant) JS type they correspond to.
+      // having to determine what (non-existent) JS type they correspond to.
       var x = LazyClass;
       var y = LazyClass;
       expect(x == y, isTrue);
@@ -191,7 +191,7 @@
   group('nested lazy class', () {
     test('type literal', () {
       // Fine because we can determine the class literals are equal without
-      // having to determine what (non-existant) JS type they correspond to.
+      // having to determine what (non-existent) JS type they correspond to.
       var x = NestedLazyClass;
       var y = NestedLazyClass;
       expect(x == y, isTrue);

diff --git a/tests/lib/isolate/transferable_failed_to_send_test.dart b/tests/lib/isolate/transferable_failed_to_send_test.dart
index f399576..c55ae3c 100644
--- a/tests/lib/isolate/transferable_failed_to_send_test.dart
+++ b/tests/lib/isolate/transferable_failed_to_send_test.dart

@@ -28,8 +28,8 @@
     // Once TransferableTypedData was sent even if attempt failed, it can't be
     // materialized.
     // This need to be changed so that on failed send we should not detach the
-    // buffer form the transferrable. The order should not matter (i.e. if the
-    // error happens before or after the serializer hits a transferrable object)
+    // buffer form the transferable. The order should not matter (i.e. if the
+    // error happens before or after the serializer hits a transferable object)
 
     final data1 = transferableFirst.materialize().asUint8List();
     Expect.equals(x.lengthInBytes, data1.length);
@@ -44,8 +44,8 @@
     // Once TransferableTypedData was sent even if attempt failed, it can't be
     // materialized.
     // This need to be changed so that on failed send we should not detach the
-    // buffer form the transferrable. The order should not matter (i.e. if the
-    // error happens before or after the serializer hits a transferrable object)
+    // buffer form the transferable. The order should not matter (i.e. if the
+    // error happens before or after the serializer hits a transferable object)
 
     final data1 = transferableFirst.materialize().asUint8List();
     Expect.equals(x.lengthInBytes, data1.length);

diff --git a/tests/lib/js/call_field_test.dart b/tests/lib/js/call_field_test.dart
index b0cd0ce..5cf5ece 100644
--- a/tests/lib/js/call_field_test.dart
+++ b/tests/lib/js/call_field_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 import 'package:js/js.dart';

diff --git a/tests/lib/js/call_getter_test.dart b/tests/lib/js/call_getter_test.dart
index d52853a..cb6abf0 100644
--- a/tests/lib/js/call_getter_test.dart
+++ b/tests/lib/js/call_getter_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 import 'package:js/js.dart';

diff --git a/tests/lib/js/call_method_test.dart b/tests/lib/js/call_method_test.dart
index cf31eaf..d3df969 100644
--- a/tests/lib/js/call_method_test.dart
+++ b/tests/lib/js/call_method_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 import 'package:js/js.dart';

diff --git a/tests/lib/js/static_interop_test/call_getter_test.dart b/tests/lib/js/static_interop_test/call_getter_test.dart
index 1150e46..c1007f2 100644
--- a/tests/lib/js/static_interop_test/call_getter_test.dart
+++ b/tests/lib/js/static_interop_test/call_getter_test.dart

@@ -21,7 +21,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'dart:js_interop';
 

diff --git a/tests/lib/js/static_interop_test/call_method_test.dart b/tests/lib/js/static_interop_test/call_method_test.dart
index 66ac758..0411237 100644
--- a/tests/lib/js/static_interop_test/call_method_test.dart
+++ b/tests/lib/js/static_interop_test/call_method_test.dart

@@ -21,7 +21,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'dart:js_interop';
 

diff --git a/tests/standalone/deny_listed_test.dart b/tests/standalone/deny_listed_test.dart
index 2516b20..e802c6d 100644
--- a/tests/standalone/deny_listed_test.dart
+++ b/tests/standalone/deny_listed_test.dart

@@ -6,8 +6,6 @@
 // Test that certain interfaces/classes are denylisted from being
 // implemented or extended (VM corelib only).
 
-library BlackListedTest;
-
 class MyBool extends Bool {} // //# 01: compile-time error
 
 class MyDouble extends Double {} // //# 02: compile-time error

diff --git a/tests/web/call_field_test.dart b/tests/web/call_field_test.dart
index 486c279..04caea3 100644
--- a/tests/web/call_field_test.dart
+++ b/tests/web/call_field_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 

diff --git a/tests/web/call_getter_test.dart b/tests/web/call_getter_test.dart
index 9b71303..b1e23ae 100644
--- a/tests/web/call_getter_test.dart
+++ b/tests/web/call_getter_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 

diff --git a/tests/web/call_method_test.dart b/tests/web/call_method_test.dart
index 0bc9656..caf6a25 100644
--- a/tests/web/call_method_test.dart
+++ b/tests/web/call_method_test.dart

@@ -20,7 +20,7 @@
 /// compilers differ.
 ///
 /// If your change causes an expectation to fail you should decide if the
-/// new result is desireable and update the expectation accordingly.
+/// new result is desirable and update the expectation accordingly.
 
 import 'package:expect/expect.dart';
 

diff --git a/tests/web/native/fixup_get_tag_test.dart b/tests/web/native/fixup_get_tag_test.dart
index a904982..8c3040d 100644
--- a/tests/web/native/fixup_get_tag_test.dart
+++ b/tests/web/native/fixup_get_tag_test.dart

@@ -54,7 +54,7 @@
   Expect.equals('isA', a.token());
 
   // This call succeeds because the fixed-up 'getTag' method returns Foo's
-  // dispatch tag, and B is a faithful polyfil for Foo/A.
+  // dispatch tag, and B is a faithful polyfill for Foo/A.
   Expect.equals('isB', b.token());
 
   Expect.isTrue(a is Foo);

diff --git a/tests/web/native/js_constant_test.dart b/tests/web/native/js_constant_test.dart
index 40ce27c..d76cf97 100644
--- a/tests/web/native/js_constant_test.dart
+++ b/tests/web/native/js_constant_test.dart

@@ -13,7 +13,7 @@
 //
 //     -5 .toString()
 //
-// The unparethesized version is `-(5 .toString())`, which creates the string
+// The unparenthesized version is `-(5 .toString())`, which creates the string
 // `"5"`, then converts it to a number for negation, giving a number result
 // instead of a string result.
 

diff --git a/tests/web/non_jsinterop_test.dart b/tests/web/non_jsinterop_test.dart
index 6173cd9..da76f13 100644
--- a/tests/web/non_jsinterop_test.dart
+++ b/tests/web/non_jsinterop_test.dart

@@ -5,7 +5,7 @@
 // TODO(johnniwinther): Share this test with ddc.
 
 // Test for positive and negative uses of js-interop declarations in a library
-// _without_ a @JS() anntaotion. This file is also used in
+// _without_ a @JS() annotation. This file is also used in
 // tests/compiler/dart2js/model/native_test.dart.
 
 library lib;

diff --git a/tests/web/string_escape_test.dart b/tests/web/string_escape_test.dart
index 3bd1b99..2b74999 100644
--- a/tests/web/string_escape_test.dart
+++ b/tests/web/string_escape_test.dart

@@ -139,7 +139,7 @@
   testUEscapes();
   // Test \b, \f, \n, \r, \t, \v.
   testSingleCharacterEscapes();
-  // Test all other single character (identity) escaeps.
+  // Test all other single character (identity) escapes.
   testIdentityEscapes();
   // Test that quotes are handled correctly.
   testQuotes();