Skip to content
This repository
Browse code

Remove legacy_rack? tests.

It was mocking and testing a single implementation based on how we think it works, and as we saw, actually caused a false positive.
  • Loading branch information...
commit b6681ca1219eb75d4a8ce47873028f5ef55cec93 1 parent 0a3627f
Andri Möll authored November 08, 2012

Showing 1 changed file with 0 additions and 12 deletions. Show diff stats Hide diff stats

  1. 12  spec/dependencies_spec.rb
12  spec/dependencies_spec.rb
@@ -311,17 +311,5 @@ module Rails
311 311
         end
312 312
       end
313 313
     end
314  
-
315  
-    describe "legacy_rack?" do
316  
-      it "should return false if Rack::Server exists" do
317  
-        Rack.stub(:constants).and_return([:Server])
318  
-        Jasmine::Dependencies.legacy_rack?.should be_false
319  
-      end
320  
-      it "should return true if Rack::Server does not exist" do
321  
-        Rack.stub(:constants).and_return([])
322  
-        Jasmine::Dependencies.legacy_rack?.should be_true
323  
-      end
324  
-    end
325 314
   end
326  
-
327 315
 end

0 notes on commit b6681ca

Please sign in to comment.
Something went wrong with that request. Please try again.