fix unsound cast failures in tests

pkg/dev_compiler/test/browser/language_tests.js

 // Copyright (c) 2015, 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.
 
 define(['dart_sdk', 'async_helper', 'expect', 'unittest', 'is', 'require'],
       function(dart_sdk, async_helper, expect, unittest, is, require) {
   'use strict';
 
   async_helper = async_helper.async_helper;
   let minitest = expect.minitest;
@@ skipping 10 matching lines @@
   // Test attributes are a list of strings, or a string for a single
   // attribute. Valid attributes are:
   //
   //   'pass' - test passes (default)
   //   'skip' - don't run the test
   //   'fail' - test fails
   //   'timeout' - test times out
   //   'slow' - use 5s timeout instead of default 2s.
   //   'helper'  - not a test, used by other tests.
   //   'unittest' - run separately as a unittest test.
-  //   'whitelist' - run with whitelisted type errors allowed
   //
   // Common combinations:
   const pass = 'pass';
   const fail = 'fail';
   const skip_timeout = ['skip', 'timeout'];
 
   // Browsers
   const firefox_fail = is.firefox() ? fail : pass;
   const chrome_fail = is.chrome() ? fail : pass;
 
   // These are typically tests with asynchronous exceptions that our
   // test framework doesn't always catch.
   const flaky = 'skip';
-  const whitelist = 'whitelist';
 
   // Tests marked with this are still using the deprecated unittest package
   // because they rely on its support for futures and asynchronous tests, which
   // expect and minitest do not handle.
   // TODO(rnystrom): Move all of these away from using the async test API so
   // they can stop using unittest.
   const async_unittest = ['unittest', 'skip', 'fail'];
 
   // The number of expected unittest errors should be zero but unfortunately
   // there are a lot of broken html unittests.
@@ skipping 19 matching lines @@
 
       // TODO(vsm): Re-enable (https://github.com/dart-lang/sdk/issues/28319)
       'async_star_test_none_multi': async_unittest,
       'async_star_test_01_multi': async_unittest,
       'async_star_test_02_multi': async_unittest,
       'async_star_test_03_multi': async_unittest,
       'async_star_test_04_multi': async_unittest,
       'async_star_test_05_multi': async_unittest,
 
       'async_switch_test': fail,
-      'async_test': whitelist,
-      'async_this_bound_test': whitelist,
       'asyncstar_throw_in_catch_test': ['skip', 'fail'],
       'await_future_test': skip_timeout,
       'await_for_test': 'slow',
       'bit_operations_test_none_multi': fail,  // DDC/dart2js canonicalize bitop results to unsigned
       'branch_canonicalization_test': fail,  // JS bit operations truncate to 32 bits.
       'call_closurization_test': fail, // Functions do not expose a "call" method.
       'canonical_const2_test': fail,
       'canonical_const_test': fail,
       'compile_time_constant10_test_none_multi': fail,
       'compile_time_constant_a_test': fail,
@@ skipping 64 matching lines @@
       'issue23244_test': fail,
       'invocation_mirror_test': fail, // dcall does not correctly detect named arguments
       'invocation_mirror2_test': fail, // JsInstanceMirror.delegate unimplemented
       'lazy_static3_test': fail,
       'least_upper_bound_expansive_test_none_multi': fail,
       'left_shift_test': fail,
       'list_is_test': fail,
       'list_literal3_test': fail,
       'main_test_03_multi': fail,
       'many_generic_instanceof_test': fail,
-      'many_named_arguments_test': whitelist,
       'map_literal10_test': fail,
       'map_literal7_test': fail,
       'memory_swap_test': skip_timeout,
       'mint_arithmetic_test': fail,
       'modulo_test': fail,
       'named_parameter_clash_test': fail,
       'named_parameters_passing_falsy_test': is.firefox('<=50') ? fail : pass,
       'nan_identical_test': fail,
       'nested_switch_label_test': fail,
       'number_identifier_test_05_multi': fail,
       'number_identity2_test': fail,
       'numbers_test': fail,
-      'reg_exp_test': whitelist,
       'regress_16640_test': fail,
       'regress_22445_test': fail,
       'regress_22666_test': fail,
       'regress_22777_test': flaky,
       'stack_overflow_stacktrace_test': fail,
       'stack_overflow_test': fail,
       'stacktrace_test': chrome_fail,
       'switch_label2_test': fail,
       'switch_label_test': fail,
       'switch_try_catch_test': fail,
@@ skipping 62 matching lines @@
       'hash_map2_test': skip_timeout,
       'hash_set_test_01_multi': fail,
       'int_modulo_arith_test_bignum_multi': fail,
       'int_modulo_arith_test_modPow_multi': fail,
       'int_modulo_arith_test_none_multi': fail,
       'int_parse_radix_test_01_multi': fail, // JS implementations disagree on U+0085 being whitespace.
       'int_parse_radix_test_02_multi': ['fail', 'timeout', 'skip'], // No bigints.
       'int_parse_radix_test_none_multi': ['slow'],
       'integer_to_radix_string_test': fail,
       'integer_to_string_test_01_multi': fail,
-      'iterable_empty_test': whitelist,
-      'iterable_join_test': whitelist,
       'iterable_return_type_test_02_multi': fail,
       'json_map_test': fail,
       'list_fill_range_test': fail,
-      'list_insert_all_test': whitelist,
       'list_replace_range_test': fail,
       'list_set_all_test': fail,
-      'list_sort_test': fail, // runtime strong mode error
       'list_test_01_multi': fail,
       'list_test_none_multi': fail,
       'main_test': fail,
       'map_keys2_test': fail,
       'map_from_iterable_test': is.firefox('<=50') ? fail : pass,
       'nan_infinity_test_01_multi': fail,
       'null_nosuchmethod_test': fail,
-      'reg_exp_all_matches_test': whitelist,
-      'reg_exp_start_end_test': whitelist,
       'regress_r21715_test': fail,
-      'sort_test': whitelist,
       'splay_tree_from_iterable_test': is.firefox('<=50') ? fail : pass,
       'string_case_test_01_multi': firefox_fail,
       'string_fromcharcodes_test': skip_timeout,
       'string_operations_with_null_test': fail,
-      'string_split_test': whitelist,
       'string_trimlr_test_01_multi': is.chrome('<=58') ? fail : pass,
       'string_trimlr_test_none_multi': is.chrome('<=58') ? fail : pass,
       'symbol_operator_test_03_multi': fail,
       'symbol_operator_test_none_multi': fail,
       'symbol_reserved_word_test_06_multi': fail,
       'symbol_reserved_word_test_09_multi': fail,
       'symbol_reserved_word_test_12_multi': fail,
       'symbol_test_none_multi': fail,
       'unicode_test': firefox_fail,
       'uri_parameters_all_test': is.firefox('<=50') ? fail : pass,
@@ skipping 54 matching lines @@
       'timer_isActive_test': async_unittest,
       'timer_not_available_test': fail,
       'timer_repeat_test': async_unittest,
       'timer_test': async_unittest,
       'zone_bind_callback_test': fail,
       'zone_error_callback_test': fail,
       'zone_register_callback_test': fail,
       'zone_run_unary_test': fail,
     },
 
-    'lib/collection': {
-      'linked_list_test': whitelist,
-    },
+    'lib/collection': {},
 
     'lib/convert': {
       'base64_test_01_multi': 'slow',
-      'chunked_conversion_utf82_test': whitelist,
-      'chunked_conversion_utf83_test': whitelist,
-      'chunked_conversion_utf85_test': ['whitelist', 'slow'],
-      'chunked_conversion_utf86_test': whitelist,
-      'chunked_conversion_utf87_test': whitelist,
+      'chunked_conversion_utf85_test': 'slow',
 
       'encoding_test': skip_timeout,
 
       'json_utf8_chunk_test': skip_timeout,
       'latin1_test': skip_timeout,
 
-      'streamed_conversion_json_decode1_test': whitelist,
       'streamed_conversion_json_encode1_test': skip_timeout,
       'streamed_conversion_json_utf8_decode_test': skip_timeout,
       'streamed_conversion_json_utf8_encode_test': skip_timeout,
       'streamed_conversion_utf8_decode_test': skip_timeout,
       'streamed_conversion_utf8_encode_test': skip_timeout,
-      'utf82_test': whitelist,
       'utf85_test': skip_timeout,
-      'utf8_encode_test': whitelist,
-      'utf8_test': whitelist,
     },
 
     'lib/html': {
       'async_spawnuri_test': async_unittest,
       'async_test': async_unittest,
 
        // was https://github.com/dart-lang/sdk/issues/27578, needs triage
       'audiocontext_test': is.chrome('<=54') ? fail : pass,
 
       'canvas_test': ['unittest'],
@@ skipping 119 matching lines @@
       'pi_test': skip_timeout,
       'random_big_test': fail,
     },
 
     'lib/typed_data': {
       // No bigint or int64 support
       'int32x4_bigint_test': fail,
       'int64_list_load_store_test': fail,
       'typed_data_hierarchy_int64_test': fail,
       'typed_data_list_test': fail,
-      'typed_list_iterable_test': whitelist,
     },
 
     'lib/mirrors': {
       'abstract_class_test_none_multi': fail,
       'accessor_cache_overflow_test': fail,
       'basic_types_in_dart_core_test': fail,
       'class_mirror_location_test': fail,
       'class_mirror_type_variables_test': fail,
       'closurization_equivalence_test': fail,
       'constructor_kinds_test_01_multi': fail,
@@ skipping 229 matching lines @@
         //    timeout.
         //
         // TODO(vsm): This currently doesn't handle tests that trigger multiple
         // asynchronous exceptions.
 
         let mainLibrary = require(module)[libraryName(name)];
         let negative = /negative_test/.test(name) ||
             mainLibrary._expectRuntimeError;
         let fail = has('fail');
 
-        let whitelist = has('whitelist');
-        dart_sdk.dart.ignoreWhitelistedErrors(whitelist);
-
         function finish(error) {
           // If the test left any lingering detritus in the DOM, blow it away
           // so it doesn't interfere with later tests.
           if (fail) {
             if (negative) {
               if (error) {
                 error = new Error(
                   "negative test marked as 'fail' " +
                   "but passed by throwing:\n" + error);
               }
             } else if (error) {
               error = null
             } else {
               error = new Error("test marked as 'fail' but passed");
             }
           } else if (negative) {
             if (!error) {
               error = new Error("test marked as 'negative' but did not throw");
             } else {
               error = null;
             }
           }
           minitest.finishTests();
           document.body.innerHTML = '';
           console.log("cleared");
           if (error && !(error instanceof Error)) error = new Error(error);
-          dart_sdk.dart.ignoreWhitelistedErrors(false);
           done(error);
         }
 
         // Intercept uncaught exceptions
         window.onerror = function(message, url, line, column, error) {
           console.warn('Asynchronous error in ' + name + ': ' + message);
           if (!error) {
             error = new Error(message);
           }
           finish(error);
@@ skipping 88 matching lines @@
 
     for (let action of unittest_tests) {
       try {
         action();
       } catch (e) {
         console.error("Caught error tying to setup test:", e);
       }
     }
   });
 });