Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

web_2/dummy_compiler_test is failing on all web backends #47257

Closed
nshahan opened this issue Sep 20, 2021 · 4 comments
Closed

web_2/dummy_compiler_test is failing on all web backends #47257

nshahan opened this issue Sep 20, 2021 · 4 comments
Assignees
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js web-dev-compiler web-triage-0 repro is available

Comments

@nshahan
Copy link
Contributor

nshahan commented Sep 20, 2021

Compile time error:

org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:16:8: Error: Error when reading 'org-dartlang-app:/tests/web_2/mock_libraries.dart': File not found
import 'mock_libraries.dart';
       ^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:20:12: Error: Undefined name 'DEFAULT_PLATFORM_CONFIG'.
    return DEFAULT_PLATFORM_CONFIG;
           ^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:23:31: Error: Undefined name 'DEFAULT_CORE_LIBRARY'.
    return buildLibrarySource(DEFAULT_CORE_LIBRARY);
                              ^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:23:12: Error: Method not found: 'buildLibrarySource'.
    return buildLibrarySource(DEFAULT_CORE_LIBRARY);
           ^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:25:12: Error: Undefined name 'DEFAULT_PATCH_CORE_SOURCE'.
    return DEFAULT_PATCH_CORE_SOURCE;
           ^^^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:27:31: Error: Undefined name 'DEFAULT_INTERNAL_LIBRARY'.
    return buildLibrarySource(DEFAULT_INTERNAL_LIBRARY);
                              ^^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:27:12: Error: Method not found: 'buildLibrarySource'.
    return buildLibrarySource(DEFAULT_INTERNAL_LIBRARY);
           ^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:29:31: Error: Undefined name 'DEFAULT_INTERCEPTORS_LIBRARY'.
    return buildLibrarySource(DEFAULT_INTERCEPTORS_LIBRARY);
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:29:12: Error: Method not found: 'buildLibrarySource'.
    return buildLibrarySource(DEFAULT_INTERCEPTORS_LIBRARY);
           ^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:31:31: Error: Undefined name 'DEFAULT_JS_HELPER_LIBRARY'.
    return buildLibrarySource(DEFAULT_JS_HELPER_LIBRARY);
                              ^^^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:31:12: Error: Method not found: 'buildLibrarySource'.
    return buildLibrarySource(DEFAULT_JS_HELPER_LIBRARY);
           ^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:33:31: Error: Undefined name 'DEFAULT_ASYNC_LIBRARY'.
    return buildLibrarySource(DEFAULT_ASYNC_LIBRARY);
                              ^^^^^^^^^^^^^^^^^^^^^
org-dartlang-app:/tests/web_2/dummy_compiler_test.dart:33:12: Error: Method not found: 'buildLibrarySource'.
    return buildLibrarySource(DEFAULT_ASYNC_LIBRARY);
           ^^^^^^^^^^^^^^^^^^
@nshahan nshahan added the area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. label Sep 20, 2021
@nshahan
Copy link
Contributor Author

nshahan commented Oct 6, 2021

web/dummy_compiler_test as well.

@nshahan nshahan added the web-triage-0 repro is available label Oct 6, 2021
@fishythefish fishythefish added web-dart2js web-dev-compiler type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) labels Oct 12, 2021
@nshahan
Copy link
Contributor Author

nshahan commented Nov 3, 2021

@sigmundch I'm trying to remember did you say that this test was obsolete and should be deleted?

@sigmundch
Copy link
Member

/cc @joshualitt - I recall we had trouble with this test in the past, but I don't recall the reasons.

Seems the import to mock_libraries.dart broke when we shuffled files around in 05ca544, so we could try and see, if fixing the import would make the test pass again, then we can keep it. If not, I'm inclined to delete it since ensuring that we can run the compiler on the browser is not a goal at the moment.

@sigmundch
Copy link
Member

Just checked, adding the import helps some, but not entirely. The test can be compiled, but fails at runtime because the inputs are very outdated and flags are not well configured. For example, it still depends on .platform (a predecessor to the libraries.json file) and fails trying to detect the location of the input .dill files.

We could try to fix those issues, but there may be more downstream. I don't feel it carries it's weight at the moment.

I am sending out https://dart-review.googlesource.com/c/sdk/+/219124 to delete it.

@joshualitt @rakudrama - let me know if you disagree :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js web-dev-compiler web-triage-0 repro is available
Projects
None yet
Development

No branches or pull requests

3 participants