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

pkg/crypto/test/sha256_test flaky on dart2js safari #12502

Closed
DartBot opened this issue Aug 16, 2013 · 11 comments
Closed

pkg/crypto/test/sha256_test flaky on dart2js safari #12502

DartBot opened this issue Aug 16, 2013 · 11 comments
Labels
P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js

Comments

@DartBot
Copy link

DartBot commented Aug 16, 2013

This issue was originally filed by benwells@chromium.org


Test failed here: http://build.chromium.org/p/client.dart/builders/dart2js-safari-mac10.7-1-3/builds/5625

Output:

@@@BUILD_STEP dart2js tests failures@@@

FAILED: dart2js-safari release_ia32 pkg/crypto/test/sha256_test
Expected: pass slow
Actual: fail
CommandOutput[dart2js]:
CommandOutput[safari]:

stdout:

  <h1> Running /Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/pkg/crypto/test/sha256_test.dart </h1>
  <script type="text/javascript" src="/root_dart/pkg/unittest/lib/test_controller.js">
  </script>
  <script type="text/javascript" src="/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js" onerror="externalError(null)">
  </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>Expect.equals(at index 0: Expected &lt;ada6b2683a885f5fef65...&gt;, Found: &lt;622a4607c2f5824c9207...&gt;, '228') fails.</pre><pre>Expect__fail@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:1667
Expect_equals@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:1581
test@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:1685
main@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:1730
@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:2037
dartMainRunner@http://127.0.0.1:60661/root_dart/pkg/unittest/lib/test_controller.js:162
@http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js:2034
[native code]</pre><pre>Error: Some tests failed.</pre><pre>FAIL</pre>

To retest, run: /Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/tools/testing/bin/macos/dart /Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/tools/testing/dart/http_server.dart -p 60661 -c 60662 --build-directory=/Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/xcodebuild/ReleaseIA32 --runtime=safari

Command[dart2js]: xcodebuild/ReleaseIA32/dart-sdk/bin/dart2js --package-root=xcodebuild/ReleaseIA32/packages/ --out=/Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/xcodebuild/ReleaseIA32/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.js /Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/pkg/crypto/test/sha256_test.dart
Took 0:00:08.205000

Command[safari]: /Volumes/data/b/build/slave/dart2js-safari-mac10.7-1-3/build/dart/tools/testing/bin/macos/dart tools/testing/dart/launch_browser.dart safari http://127.0.0.1:60661/root_build/generated_tests/dart2js-safari/pkg_crypto_test_sha256_test/test.html?crossOriginPort=60662
Took 0:00:00.791336

Short reproduction command (experimental):
    python tools/test.py -mrelease --clear_safari_cache -t60 -rsafari --use_browser_controller -cdart2js --use-sdk pkg/crypto/test/sha256_test

===
=== 1 test failed
===

--- Total time: 10:34 ---
0:00:24.721000 - dart2js - dart2js-safari release_ia32/pkg/mdv/test/template_element_test
0:00:24.647000 - dart2js - dart2js-safari release_ia32/pkg/mdv/test/element_bindings_test
0:00:24.117000 - dart2js - dart2js-safari release_ia32/pkg/polymer_expressions/test/syntax_test
0:00:17.135000 - dart2js - dart2js-safari release_ia32/pkg/analyzer_experimental/test/generated/parser_test
0:00:13.655000 - dart2js - dart2js-safari release_ia32/pkg/unittest/test/mock_test
0:00:13.385000 - dart2js - dart2js-safari release_ia32/pkg/unittest/test/mock_regexp_negative_test
0:00:13.372169 - safari - dart2js-safari release_ia32/isolate/isolate_stress_test
0:00:13.211000 - dart2js - dart2js-safari release_ia32/pkg/intl/test/date_time_format_http_request_test
0:00:13.125000 - dart2js - dart2js-safari release_ia32/lib/mirrors/library_uri_package_test
0:00:12.805000 - dart2js - dart2js-safari release_ia32/pkg/unittest/test/mirror_matchers_test
0:00:12.376000 - dart2js - dart2js-safari release_ia32/samples/lib/view/view_test
0:00:12.199000 - dart2js - dart2js-safari release_ia32/lib/mirrors/static_test
0:00:11.964000 - dart2js - dart2js-safari release_ia32/lib/mirrors/operator_test
0:00:11.873000 - dart2js - dart2js-safari release_ia32/pkg/third_party/html5lib/test/dom_test
0:00:11.826000 - dart2js - dart2js-safari release_ia32/lib/mirrors/method_mirror_name_test
0:00:11.785000 - dart2js - dart2js-safari release_ia32/lib/mirrors/reflect_runtime_type_test
0:00:11.693000 - dart2js - dart2js-safari release_ia32/html/element_test/_ElementList, dart2js-safari release_ia32/html/element_test/constructors, dart2js-safari release_ia32/html/element_test/functional, dart2js-safari release_ia32/html/element_test/position, dart2js-safari release_ia32/html/element_test/queryAll, dart2js-safari release_ia32/html/element_test/matches, dart2js-safari release_ia32/html/element_test/children, dart2js-safari release_ia32/html/element_test/attributes, dart2js-safari release_ia32/html/element_test/click, dart2js-safari release_ia32/html/element_test/eventDelegation, dart2js-safari release_ia32/html/element_test/eventListening
0:00:11.606000 - dart2js - dart2js-safari release_ia32/lib/mirrors/invoke_test
0:00:11.369000 - dart2js - dart2js-safari release_ia32/html/history_test/supported_state, dart2js-safari release_ia32/html/history_test/supported_HashChangeEvent, dart2js-safari release_ia32/html/history_test/history
0:00:11.334000 - dart2js - dart2js-safari release_ia32/lib/mirrors/fake_function_test
Running: "/usr/bin/python" "./tools/test.py" "--step_name=dart2js tests" "--mode=release" "--compiler=dart2js" "--runtime=safari" "--time" "--use-sdk" "--report" "--write-debug-log" "--progress=buildbot" "-v" "--use_browser_controller" "--clear_safari_cache" "--shards=3" "--shard=1"
@@@STEP_FAILURE@@@


started: Thu Aug 15 16:45:05 2013
ended: Thu Aug 15 16:45:05 2013
duration: 0 secs

This has also failed recently here: http://build.chromium.org/p/client.dart/builders/dart2js-safari-mac10.7-1-3/builds/5616

@ricowind
Copy link
Contributor

cc @peter-ahe-google.
Removed Priority-Unassigned label.
Added Priority-Medium, Area-Dart2JS, Triaged labels.
Changed the title to: "pkg/crypto/test/sha256_test flaky on dart2js safari".

@ricowind
Copy link
Contributor

ricowind commented Sep 5, 2013

Another crypto related flaky failure on the mac
http://build.chromium.org/p/client.dart/builders/dart2js-safari-mac10.7-1-3/builds/6348/steps/dart2js%20tests%20failures/logs/stdio

I think this is slightly concerning, we are calculating wrong numbers

@peter-ahe-google
Copy link
Contributor

Unfortunately, it seems like there is a bug in Safari. I'm not sure we can do much about it except reporting it to Apple. In order to file a bug report to Apple we need a self-contained repro we can attach to the bug report.

@ricowind
Copy link
Contributor

Also flaking pkg/crypto/test/sha256_test


cc @mkustermann.

@kasperl
Copy link

kasperl commented Sep 18, 2013

Added this to the M7 milestone.

@kasperl
Copy link

kasperl commented Oct 2, 2013

Removed this from the M7 milestone.
Added this to the M8 milestone.

@kasperl
Copy link

kasperl commented Jun 4, 2014

Removed this from the M8 milestone.
Added this to the 1.6 milestone.

@kasperl
Copy link

kasperl commented Jul 10, 2014

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

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-1.6 label.

@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed triaged labels Feb 29, 2016
@bkonyi
Copy link
Contributor

bkonyi commented Jun 21, 2018

Is this still a valid issue?

@sigmundch
Copy link
Member

Thanks for checking @bkonyi - I'd like to automatically close any issue that is redundant with an entry in a .status file. It was probably used to notify us when someone flip the status, but I honestly don't think we need to keep an issue open for those.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) web-dart2js
Projects
None yet
Development

No branches or pull requests

7 participants