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

Test language/get_set_syntax_test fails on dart2js #12805

Closed
sgjesse opened this issue Aug 27, 2013 · 8 comments
Closed

Test language/get_set_syntax_test fails on dart2js #12805

sgjesse opened this issue Aug 27, 2013 · 8 comments
Assignees

Comments

@sgjesse
Copy link
Contributor

sgjesse commented Aug 27, 2013

The following parts of the multi-test fails:

  get_set_syntax_test/00
  get_set_syntax_test/01
  get_set_syntax_test/02
  get_set_syntax_test/03
  get_set_syntax_test/04
  get_set_syntax_test/05
  get_set_syntax_test/06
  get_set_syntax_test/07
  get_set_syntax_test/13
  get_set_syntax_test/14

@sgjesse
Copy link
Contributor Author

sgjesse commented Aug 27, 2013

Missed

  get_set_syntax_test/15

The related dart2dart issue is issue #12806.

@sgjesse
Copy link
Contributor Author

sgjesse commented Aug 28, 2013

When running with minification the following tests pass

  get_set_syntax_test/05
  get_set_syntax_test/06
  get_set_syntax_test/07
  get_set_syntax_test/13
  get_set_syntax_test/14
  get_set_syntax_test/15

@sgjesse
Copy link
Contributor Author

sgjesse commented Aug 28, 2013

The current status is that the following tests are failing

get_set_syntax_test/00: Fail # Issue #12805
get_set_syntax_test/01: Fail # Issue #12805
get_set_syntax_test/02: Fail # Issue #12805
get_set_syntax_test/03: Fail # Issue #12805
get_set_syntax_test/04: Fail # Issue #12805

(not depending on whether --minified is used or not)

@peter-ahe-google
Copy link
Contributor

Set owner to @peter-ahe-google.
Added Started label.

@peter-ahe-google
Copy link
Contributor

Marked this as blocking #5519.

@peter-ahe-google
Copy link
Contributor

Issue #12806 has been merged into this issue.

@peter-ahe-google
Copy link
Contributor

@peter-ahe-google
Copy link
Contributor

Fixed in r27016.


Added Fixed label.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants