LGTM, thanks
On Tue, Mar 18, 2014 at 2:27 PM, Petr Pudlak <[email protected]> wrote: > .. because instead of a job ID and a lock file, we now pass a single, > client ID object. > > Signed-off-by: Petr Pudlak <[email protected]> > --- > test/py/cmdlib/testsupport/wconfd_mock.py | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/test/py/cmdlib/testsupport/wconfd_mock.py > b/test/py/cmdlib/testsupport/wconfd_mock.py > index b2ea69d..6823083 100644 > --- a/test/py/cmdlib/testsupport/wconfd_mock.py > +++ b/test/py/cmdlib/testsupport/wconfd_mock.py > @@ -29,7 +29,7 @@ class MockClient(object): > def __init__(self, wconfdmock): > self.wconfdmock = wconfdmock > > - def TryUpdateLocks(self, _jid, _lockfile, req): > + def TryUpdateLocks(self, _cid, req): > for lockrq in req: > if lockrq[1] == "release": > if lockrq[0] in self.wconfdmock.mylocks: > @@ -44,13 +44,13 @@ class MockClient(object): > result.append([lock, self.wconfdmock.mylocks[lock]]) > return result > > - def FreeLocksLevel(self, _jid, _lockfile, level): > + def FreeLocksLevel(self, _cid, level): > locks = self.wconfdmock.mylocks.keys() > for lock in locks: > if lock.startswith(level + "/"): > del self.wconfdmock.mylocks[lock] > > - def OpportunisticLockUnion(self, _jid, _lockfile, req): > + def OpportunisticLockUnion(self, _cid, req): > for lockrq in req: > self.wconfdmock.mylocks[lockrq[0]] = lockrq[1] > return [lockrq[0] for lockrq in req] > -- > 1.9.0.279.gdc9e3eb > > -- -- Helga Velroyen | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
