Merge pull request #8 from shekibobo/feature/handle-null-hrefs

Don't add invalid link relations to HAL output

lib/oat/adapters/hal.rb

@@ -3,7 +3,7 @@ module Oat
   module Adapters
     class HAL < Oat::Adapter
       def link(rel, opts = {})
-        data[:_links][rel] = opts
+        data[:_links][rel] = opts if opts[:href]
       end
 
       def properties(&block)

spec/adapters/hal_spec.rb

@@ -17,6 +17,8 @@
         h[:controller_name].should == 'some_controller'
         # links
         h[:_links][:self][:href].should == "http://foo.bar.com/#{user.id}"
+        # HAL Spec says href is REQUIRED
+        h[:_links].should_not include(:empty)
         # embedded manager
         h[:_embedded][:manager].tap do |m|
           m[:id].should == manager.id

spec/fixtures.rb

@@ -12,6 +12,7 @@ def self.included(base)
         schema do
           type 'user'
           link :self, href: url_for(item.id)
+          link :empty, href: nil
 
           property :id, item.id
           map_properties :name, :age