+1, but with comments below.

On Oct 11, 2008, at 1:00 PM, Brice Figureau wrote:
> diff --git a/test/data/snippets/multipleclass.pp b/test/data/ 
> snippets/multipleclass.pp
> new file mode 100644
> index 0000000..fb363a1
> --- /dev/null
> +++ b/test/data/snippets/multipleclass.pp
> @@ -0,0 +1,9 @@
> +class one {
> +notice('class one')
> +}
> +
> +class one {
> +notice('second class one')
> +}
> +
> +include one


Shouldn't there be a test somewhere for this?  Snippets without an  
associated test method in test/language/snippets are ignored.

That whole file should be migrated to spec/integration at some point,  
of course.

-- 
The salesman asked me what size I wore, I told him extra-medium.
     -- Stephen Wright
---------------------------------------------------------------------
Luke Kanies | http://reductivelabs.com | http://madstop.com


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to