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

(MODULES-7024) Add 20-octet MAC addresses #905

Merged
merged 1 commit into from
Oct 11, 2018

Conversation

ananace
Copy link
Contributor

@ananace ananace commented Apr 24, 2018

Not quite sure if the best thing is to extend the MAC matcher, but since the 20-octet hardware address is the exact same format it feels like a good start.

@pmcmaw
Copy link
Contributor

pmcmaw commented Aug 14, 2018

Closing and reopening to kick tests from an updated Travis test configuration that is now in puppetlabs-stdlib master.

@pmcmaw pmcmaw closed this Aug 14, 2018
@pmcmaw pmcmaw reopened this Aug 14, 2018
@pmcmaw
Copy link
Contributor

pmcmaw commented Aug 14, 2018

Would it be possible to add a test to confirm that this functionality works as expected. An example of testing this can be found here: https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/spec/acceptance/is_mac_address_spec.rb so if you extend them to include a 20-octet MAC address.

We would also like to thank you for taking time to contribute, it is much appreciated.

@ananace
Copy link
Contributor Author

ananace commented Aug 15, 2018

Tests have been added, though I didn't see anywhere that actively tested the Puppet type itself, only the is_mac_address function - which uses its own regex.

The AppVeyor build seems to have had some issues with testing fqdn_rotate though

@ananace
Copy link
Contributor Author

ananace commented Oct 1, 2018

Just to throw a ping at this, are there any examples for testing an actual type - and not a function? Or is it just a case of it being enough to test the is_mac_address function?

@david22swan david22swan merged commit e180d71 into puppetlabs:master Oct 11, 2018
@ananace ananace deleted the modules-7024 branch October 11, 2018 15:55
@pegasd
Copy link
Contributor

pegasd commented Oct 14, 2018

@ananace There are! Check out spec/type_aliases directory - this is the best approach for testing custom types imo.

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

Successfully merging this pull request may close these issues.

4 participants