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 run summary is wrong #11381

Closed
brson opened this issue Jan 7, 2014 · 5 comments · Fixed by #11601
Closed

Test run summary is wrong #11381

brson opened this issue Jan 7, 2014 · 5 comments · Fixed by #11601
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@brson
Copy link
Contributor

brson commented Jan 7, 2014

http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2536/steps/test/logs/stdio

The final summary at the end says we ran 28 tests when we ran thousands.

@eastwood
Copy link
Contributor

Interesting, is the code for buildbot in src/test?

@brson
Copy link
Contributor Author

brson commented Jan 11, 2014

@b1nd This code is part of the build system. mk/tests.mk is the place to start looking. The code for recording the number of tests is probably in extra::test.

@eastwood
Copy link
Contributor

Thanks brson,
You were right, the code is in extra::test. Do I execute the tests through:

make check

https://github.com/mozilla/rust/wiki/Note-testsuite

@dguenther
Copy link
Contributor

I'll pick this up, unless it's still in-progress or has been fixed already.

@eastwood
Copy link
Contributor

Go for it, I'd love to see what you come up with!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants