[PATCH 7/7] PuSH: add test for config value

Matt Domsch Matt_Domsch at dell.com
Wed Jun 16 05:31:35 EST 2010


On Tue, Jun 15, 2010 at 12:54:39PM -0500, Matt Domsch wrote:
> From 7441fce8c4100be061dd014619005e32cefe85f6 Mon Sep 17 00:00:00 2001
> From: Matt Domsch <Matt_Domsch at dell.com>
> Date: Tue, 15 Jun 2010 12:44:33 -0500
> Subject: [PATCH 7/7] PuSH: add test for config value

I'd argue that this test is useless - testing whether config.foo()
does the same thing for this config value vs another similar isn't
interesting; feel free to ignore it.

What I haven't figured out is how to parse the resulting XML files and
ensure they have the fields expected.  Is that test_reconstitute and
related tests/data/reconstitute/* ?  The other fields under <channel>
aren't tested for ATM, which I was hoping to crib from.

Thanks,
Matt

-- 
Matt Domsch
Technology Strategist
Dell | Office of the CTO


More information about the devel mailing list