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.

Sign up for GitHub

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

Jump to bottom

[Truffle] Add MRI test profile to pom files. #2862

Merged
merged 1 commit into from Apr 22, 2015
Merged

[Truffle] Add MRI test profile to pom files. #2862

merged 1 commit into from Apr 22, 2015

Conversation

bjfish
Copy link
Contributor

@bjfish bjfish commented Apr 21, 2015

With workarounds (after current MRI test issues are fixed) mvn -Ptruffle-mri-test completes with:

     [exec] Finished tests in 43.266000s, 38.5291 tests/s, 3727.0836 assertions/s.
     [exec] 
     [exec] 1667 tests, 161256 assertions, 0 failures, 0 errors, 1

@chrisseaton
Copy link
Contributor

This is great. Do you know off the top of your head how many assertions there are? Is 161k 0.1%? 10%

@bjfish
Copy link
Contributor Author

bjfish commented Apr 21, 2015

@chrisseaton
161256 / 1099625 = 14.664635671 %
jruby has the following

6517 tests, 1099625 assertions, 0 failures, 0 errors, 16 skips

@bjfish
Copy link
Contributor Author

bjfish commented Apr 22, 2015

@eregon I updated this to use ruby from the system path to run tool/jt.rb.

eregon added a commit that referenced this pull request Apr 22, 2015
@eregon
Merge pull request #2862 from bjfish/truffle_mri_tests_mvn
b80323e 
[Truffle] Add MRI test profile to pom files.
@eregon eregon merged commit b80323e into jruby:master Apr 22, 2015
@eregon eregon added this to the truffle-dev milestone Apr 22, 2015
@eregon eregon added the truffle label Apr 22, 2015
@enebo enebo added this to the Non-Release milestone Dec 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Non-Release
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@bjfish @chrisseaton @enebo @eregon