Adjust expectations of library_env_test to take into account noopt, precompiled and product mode.

[email protected]

Review URL: https://codereview.chromium.org/1748153003 .

Author: rmacnak-google

tests/language/language.status

@@ -181,14 +181,20 @@ regress_22443_test: Skip
 [ $runtime == vm && $mode == product ]
 vm/type_vm_test: Fail,OK  # Expects exact type name.
 
-[ $compiler == none && $browser ]
+[ ($compiler == none || $compiler == precompiler || $compiler == dart2app) && $browser ]
 # The following tests are supposed to fail.
 library_env_test/has_io_support: RuntimeError, OK
 library_env_test/has_no_html_support: RuntimeError, OK
-library_env_test/has_no_mirror_support: RuntimeError, OK
 
-[ $compiler == none && $browser != true ]
+[ ($compiler == none || $compiler == precompiler || $compiler == dart2app) && $browser != true ]
 # The following tests are supposed to fail.
 library_env_test/has_html_support: RuntimeError, OK
 library_env_test/has_no_io_support: RuntimeError, OK
+
+[ $compiler == none && $noopt == false && $mode != product ]
+# The following tests are supposed to fail.
 library_env_test/has_no_mirror_support: RuntimeError, OK
+
+[ $noopt || $compiler == precompiler || $mode == product ]
+# The following tests are supposed to fail.
+library_env_test/has_mirror_support: RuntimeError, OK

tests/language/library_env_test.dart

@@ -5,7 +5,7 @@
 import 'package:expect/expect.dart';
 
 main() {
-  const NOT_PRESENT = null;
+  const NOT_PRESENT = false;
 
   Expect.isTrue(const bool.fromEnvironment("dart.library.async"));
   Expect.isTrue(const bool.fromEnvironment("dart.library.collection"));