Remove is_valid_netmask instead of unstubbing. Doesn't seem like a sensible function...

diff --git a/lib/puppet/parser/functions/is_valid_netmask.rb b/lib/puppet/parser/functions/is_valid_netmask.rb
deleted file mode 100644 (file)
index 41e4843..0000000
--- a/lib/puppet/parser/functions/is_valid_netmask.rb
+++ /dev/null
@@ -1,18 +0,0 @@
-#
-# is_valid_netmask.rb
-#
-
-module Puppet::Parser::Functions
-  newfunction(:is_valid_netmask, :type => :rvalue, :doc => <<-EOS
-    EOS
-  ) do |arguments|
-
-    if (arguments.size != 1) then
-      raise(Puppet::ParseError, "is_valid_netmask(): Wrong number of arguments "+
-        "given #{arguments.size} for 1")
-    end
-
-  end
-end
-
-# vim: set ts=2 sw=2 et :

diff --git a/spec/unit/parser/functions/is_valid_netmask_spec.rb b/spec/unit/parser/functions/is_valid_netmask_spec.rb
deleted file mode 100644 (file)
index 97cbb7c..0000000
--- a/spec/unit/parser/functions/is_valid_netmask_spec.rb
+++ /dev/null
@@ -1,21 +0,0 @@
-#!/usr/bin/env rspec
-require 'spec_helper'
-
-describe "the is_valid_netmask function" do
-  before :all do
-    Puppet::Parser::Functions.autoloader.loadall
-  end
-
-  before :each do
-    @scope = Puppet::Parser::Scope.new
-  end
-
-  it "should exist" do
-    Puppet::Parser::Functions.function("is_valid_netmask").should == "function_is_valid_netmask"
-  end
-
-  it "should raise a ParseError if there is less than 1 arguments" do
-    lambda { @scope.function_is_valid_netmask([]) }.should( raise_error(Puppet::ParseError))
-  end
-
-end