Skip to content

Commit

Permalink
[Truffle] Adding monitor to the stdlib.
Browse files Browse the repository at this point in the history
@bjfish
bjfish committed Apr 18, 2015
1 parent 3735705 commit 5e5168f
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/monitor.rb
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../stdlib/monitor'

3 comments on commit 5e5168f

@bjfish
Copy link
Contributor Author

@bjfish bjfish commented on 5e5168f Apr 18, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only 1 failing MRI test:

Running tests:

TestMonitor#test_cond = 0.01 s = .
TestMonitor#test_enter = 0.00 s = .
TestMonitor#test_killed_thread_in_synchronize = 0.02 s = F
TestMonitor#test_synchronize = 0.00 s = .
TestMonitor#test_timedwait = 0.11 s = .
TestMonitor#test_try_enter = 0.00 s = .

Finished tests in 0.196000s, 30.6122 tests/s, 153.0612 assertions/s.

  1. Failure:
    TestMonitor#test_killed_thread_in_synchronize [/Users/brandonfish/Documents/jruby-mine/test/mri/monitor/test_monitor.rb:80]:
    <[:main]> expected but was
    <[:main, :t1, :t2]>.

6 tests, 30 assertions, 1 failures, 0 errors, 0 skips

@chrisseaton
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does that still need your workarounds, though?

@bjfish
Copy link
Contributor Author

@bjfish bjfish commented on 5e5168f Apr 18, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chrisseaton Yes, I still need workarounds for the 2 issues with "for MRI Tests" in the their titles to be able to run the test runner.

Please sign in to comment.