Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add validation for type values #7

Closed
adrienthebo opened this issue Oct 22, 2012 · 2 comments
Closed

Add validation for type values #7

adrienthebo opened this issue Oct 22, 2012 · 2 comments
Assignees

Comments

@adrienthebo
Copy link
Member

Pending:
  Puppet::Type::Network_config when validating the attribute options should be a descendant of the KeyValue property
    # on conversion to specific type
    # ./spec/unit/type/network_config_spec.rb:46
  Puppet::Type::Network_config when validating the attribute value ipaddress should fail if a malformed address is used
    # implementation of IP address validation
    # ./spec/unit/type/network_config_spec.rb:81
  Puppet::Type::Network_config when validating the attribute value ipaddress using the inet family should fail when passed an IPv6 address
    # implementation of IP address validation
    # ./spec/unit/type/network_config_spec.rb:65
  Puppet::Type::Network_config when validating the attribute value ipaddress using the inet6 family should fail when passed an IPv4 address
    # implementation of IP address validation
    # ./spec/unit/type/network_config_spec.rb:75
  Puppet::Type::Network_config when validating the attribute value netmask should validate a CIDR netmask
    # Not yet implemented
    # ./spec/unit/type/network_config_spec.rb:88
  Puppet::Type::Network_config when validating the attribute value netmask should fail if an invalid CIDR netmask is used
    # implementation of IP address validation
    # ./spec/unit/type/network_config_spec.rb:89

This should be not pending.

@jyaworski
Copy link
Member

Hello:

Is this still a problem?

@jyaworski
Copy link
Member

Closing due to lack of response. Please reopen if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants