uri objects now return nil for ssl? instead of true/false in case the…

… scheme is missing
magec · Mar 17, 2010 · 83528a1 · 83528a1
1 parent 80d4c8e
commit 83528a1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/savon/core_ext/uri.rb b/lib/savon/core_ext/uri.rb
@@ -3,7 +3,7 @@ class Generic
 
     # Returns whether the URI hints to SSL.
     def ssl?
-      @scheme.starts_with? "https"
+      !@scheme ? nil : @scheme.starts_with?("https")
     end
 
   end

diff --git a/spec/savon/core_ext/uri_spec.rb b/spec/savon/core_ext/uri_spec.rb
@@ -10,6 +10,10 @@
     it "returns false for non-https URI's" do
       URI("http://example.com").ssl?.should be_false
     end
+
+    it "returns nil for invalid URI's without a scheme" do
+      URI("example").ssl?.should be_nil
+    end
   end
 
 end