[Libvirt-cim] [PATCH] [TEST] #7 Update RAFP.01 for LXC support

[Kaitlin Rupert]

> +   
> +    for item in data:
> +        if item['InstanceID'] == rasd[cn]:
> +            logger.info("%s InstanceID match - expect %s, got %s" \
> +                        % (cn, rasd[cn], item['InstanceID']))
> +            return PASS 
> +        else:
> +            logger.error("RASD instance with InstanceID %s not found" % rasd[cn])
> +            return FAIL
> +               

You don't want an else statement in the for loop.  If the first element 
in data doesn't match, you'll return an error from the function. 
Instead, you'll want return a failure after the loop is complete:

     for item in data:
         if item['InstanceID'] == rasd[cn]:
             logger.info("%s InstanceID match - expect %s, got %s" \
                         % (cn, rasd[cn], item['InstanceID']))
             return PASS

     logger.error("RASD instance with InstanceID %s not found" % rasd[cn])
     return FAIL

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com