Validate is over-complicating things, not necessary

lib/puppet/type/resourceref.rb

@@ -79,18 +79,6 @@
     end
   end
 
-
-  # Validate that we're not using the same target as resource
-  validate do 
-    @resource.catalog.resources.select { |res|
-      # Skip resource if we're not interested in it...
-      next unless referenceable_resources.include?(res.type)
-
-      # Match on resource name...
-      raise Puppet::Error, 'Referenced resource has matching target.' if res[:name] == self[:name] and res[:target] == self[:target]
-    }
-  end
-
   # Autorequire the user running command
   autorequire(:user) do
     self[:user]

spec/unit/puppet/type/resourceref_spec.rb

@@ -268,7 +268,7 @@
         )
       end
 
-      # Need to stub jdbcresource type and provider.
+      # Need to stub jdbcresource provider.
       let :jdbcresourceprovider do
         Puppet::Type.type(:jdbcresource).provide(:fake_jdbcresource_provider) { mk_resource_methods }
       end