Skip to content

pub test failure on 'banana' #36

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

Closed
DartBot opened this issue Jun 5, 2015 · 6 comments
Closed

pub test failure on 'banana' #36

DartBot opened this issue Jun 5, 2015 · 6 comments
Assignees
Labels
type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@DartBot
Copy link

DartBot commented Jun 5, 2015

Originally opened as dart-lang/sdk#2729

This issue was originally filed by [email protected]


I'm seeing a pub test failure about banana:

dart$ tools/test.py -mrelease
Test configuration: none_vm_release_ia32
[00:06 | 3% | + 149 | - 0]
FAILED: none-vm release_ia32 pub/pub_tests
Expected: pass
Actual: fail

stderr:
Unhandled exception:
Expect.fail('Output line 1 was: banana
expected: apple')
 0. Function: 'Expect._fail@127eafe4' url: 'bootstrap' line:1514 col:5
 1. Function: 'Expect.fail' url: 'bootstrap' line:1301 col:10
 2. Function: '::_validateOutput@e8053a5' url: 'file:///usr/local/google/home/mattsh/dart-code1/dart/utils/tests/pub/test_pub.dart' line:116 col:18
 3. Function: '::function' url: 'file:///usr/local/google/home/mattsh/dart-code1/dart/utils/tests/pub/test_pub.dart' line:60 col:22
 4. Function: 'FutureImpl._complete@924b4b8' url: 'bootstrap_impl' line:3086 col:17
 5. Function: 'FutureImpl._setValue@924b4b8' url: 'bootstrap_impl' line:3100 col:14
 6. Function: 'CompleterImpl.complete' url: 'bootstrap_impl' line:3169 col:26
 7. Function: 'FutureImpl.function' url: 'bootstrap_impl' line:3152 col:44
 8. Function: 'FutureImpl._complete@924b4b8' url: 'bootstrap_impl' line:3086 col:17
 9. Function: 'FutureImpl._setValue@924b4b8' url: 'bootstrap_impl' line:3100 col:14
 10. Function: 'CompleterImpl.complete' url: 'bootstrap_impl' line:3169 col:26
 11. Function: '::checkComplete' url: 'file:///usr/local/google/home/mattsh/dart-code1/dart/utils/pub/io.dart' line:179 col:23
 12. Function: '::function' url: 'file:///usr/local/google/home/mattsh/dart-code1/dart/utils/pub/io.dart' line:193 col:18
 13. Function: '[email protected]' url: 'dart:io' line:6849 col:18
 14. Function: '[email protected]' url: 'dart:io' line:6855 col:53
 15. Function: '_SocketBase@14117cc4._multiplex@14117cc4' url: 'dart:io' line:7301 col:25
 16. Function: '[email protected]' url: 'dart:io' line:7414 col:59
 17. Function: '_ReceivePortImpl@6be832b._handleMessage@6be832b' url: 'dart:isolate' line:288 col:22

@DartBot DartBot added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) Fixed labels Jun 5, 2015
@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


Added Accepted label.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/8633293?v=3" align="left" width="48" height="48"hspace="10"> Comment by a-siva


Can you update the status file to skip this test until it is fixed.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


This doesn't break the buildbots, and I haven't heard anyone else having problems with it. I will fix it, but unless you're running into this too I'd rather not skip the test. Is that OK?

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/8633293?v=3" align="left" width="48" height="48"hspace="10"> Comment by a-siva


I spent an hour thinking my change introduced the regression before I realized it was already broken. The buildbots seem to run the tests differently and hence are not running into it.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


Oh, I'm very sorry. Matt was the first one to bring this to my attention. Until he did that, I thought everything was fine. I'll skip the test.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


Added Fixed label.

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

2 participants