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

Dartium tests don't propagate an error when main has the wrong signature. #14478

Closed
vsmenon opened this issue Oct 25, 2013 · 2 comments
Closed
Labels
type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@vsmenon
Copy link
Member

vsmenon commented Oct 25, 2013

This test isn't failing properly on Dartium:

FAILED: none-dartium-checked release_ia32 co19/Language/14_Libraries_and_Scripts/4_Scripts_A03_t03
Expected: MissingRuntimeError Ok
Actual: Pass
Runtime error expected.
CommandOutput[dartium]:

stdout:

  <h1> Running /usr/local/google/vsm/dartium/src/dart/tests/co19/src/Language/14_Libraries_and_Scripts/4_Scripts_A03_t03.dart </h1>
  <script type="text/javascript" src="/root_dart/pkg/unittest/lib/test_controller.js">
  </script>
  <script type="application/dart" src="/root_build/generated_tests/none-dartium-checked/tests_co19_src_Language_14_Libraries_and_Scripts_4_Scripts_A03_t03/test.dart" onerror="externalError(null)" defer="">
  </script>
  <script type="text/javascript" src="/root_dart/pkg/browser/lib/dart.js"></script>
  <script type="text/javascript" src="/root_dart/pkg/browser/lib/interop.js"></script>

<pre>Error:

http://127.0.0.1:47934/root_build/generated_tests/none-dartium-checked/tests_co19_src_Language_14_Libraries_and_Scripts_4_Scripts_A03_t03/test.dart:0:
Exception: No top-level method 'main' with matching arguments declared.

NoSuchMethodError: incorrect number of arguments passed to method named 'main'
Receiver: top-level
Tried calling: main(...)
Found: main(dynamic)

</pre>
<pre>FAIL</pre>
<pre>Calling notifyDone()</pre>

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed priority-unassigned labels Feb 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

4 participants