Fix bug in group queries related to node/instance fields
authorIustin Pop <iustin@google.com>
Mon, 25 Feb 2013 16:18:36 +0000 (17:18 +0100)
committerIustin Pop <iustin@google.com>
Mon, 4 Mar 2013 16:09:17 +0000 (17:09 +0100)
commite7124835a33a053ada8a78407d8b548324e59002
tree763a74cd268e795f6a398af9907415544a4f0812
parentd5b2753abf43fc7f3cc26d7542d43bcb7ddf3a2c
Fix bug in group queries related to node/instance fields

Since we use the primitive string type for group UUIDs, the group
fields have a bug where we pass the group name as filter for node
tests, whereas the nodes themselves use the group UUID. This results
in zero node count, empty node list, and no instances being reported
as assigned to groups.

The patch fixes this and adds a test for the node count. It does some
test generation improvement, and also cleans up whitespace issues in
Test/G/Q/Query.hs (the functions case_queryNode_allfields,
prop_queryGroup_noUnknown and case_queryGroup_allfields are unchanged
but simply have indentation fixed).

Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com>
src/Ganeti/Query/Group.hs
test/hs/Test/Ganeti/Objects.hs
test/hs/Test/Ganeti/Query/Query.hs