Re: [Libvirt-cim] [PATCH] [TEST] add a try..except to get_value_xpath()

Deepti B Kalakeri wrote:

...

def get_value_xpath(self, xpathStr): - node = self.get_node(xpathStr) + try: + node = self.get_node(xpathStr) + except LoopUpError:

I think this should be *LookUpError.*

...

+ raise Exception('Zero or multiple node found')

And can we use logger.error instead, I guess using raise statement does not proceed with the execution of the next stmt after it, right ?? In that case the return None wont be of much use. ** Agree. Logger is better.

-- - Zhengang