From: Ken Barber Date: Thu, 28 Jul 2011 20:10:33 +0000 (+0100) Subject: Remove is_valid_netmask instead of unstubbing. Doesn't seem like a sensible function... X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=1a7bd1ae8321bfc3d2ae87f1d5184828abf56916;p=puppet-stdlib.git Remove is_valid_netmask instead of unstubbing. Doesn't seem like a sensible function on its own. --- diff --git a/lib/puppet/parser/functions/is_valid_netmask.rb b/lib/puppet/parser/functions/is_valid_netmask.rb deleted file mode 100644 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 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