Refactored brittle dereferenceable specs.

Author: jdantonio

spec/concurrent/dereferenceable_shared.rb

@@ -117,15 +117,6 @@
     subject.value.should be_nil
   end
 
-  it 'locks when all options are false' do
-    subject = dereferenceable_subject(0)
-    mutex = double('mutex')
-    subject.stub(:mutex).and_return(mutex)
-    mutex.should_receive(:lock).at_least(:once)
-    mutex.should_receive(:unlock).at_least(:once)
-    subject.value
-  end
-
   it 'supports dereference flags with observers', :brittle, :refactored do
 
     if dereferenceable_subject(0).respond_to?(:add_observer)

spec/concurrent/scheduled_task_spec.rb

@@ -39,7 +39,10 @@ module Concurrent
       # dereferenceable
 
       def dereferenceable_subject(value, opts = {})
-        ScheduledTask.execute(0.1, opts){ value }.tap{ sleep(0.2) }
+        latch = Concurrent::CountDownLatch.new(1)
+        task = ScheduledTask.execute(0.1, opts){ value }.tap{ latch.count_down }
+        latch.wait(1)
+        task
       end
 
       def dereferenceable_observable(opts = {})