crush: unittest remove useless c->create()

Signed-off-by: Loic Dachary <loic@dachary.org>
crush: unittest CrushWrapper::check_item_loc

Signed-off-by: Loic Dachary <loic@dachary.org>
crush: unittest use const instead of define

And reduce the depth of the hierarchy because three levels of buckets
capture the same cases as four levels.

Signed-off-by: Loic Dachary <loic@dachary.org>
crush: unittest s/std::string/string/

Signed-off-by: Loic Dachary <loic@dachary.org>
crush: unittest CrushWrapper::update_item

Signed-off-by: Loic Dachary <loic@dachary.org>
crush: unittest CrushWrapper::get_immediate_parent

Signed-off-by: Loic Dachary <loic@dachary.org>
Merge pull request #909 from dachary/wip-crush-unittest

more CrushWrapper unittest