In the following code:
354 + try: 355 + remove_fsevents_daemon(authRef) 356 + except Exception, e: 357 + logger.exception("Problem removing running daemon: %r" % e) 358 + 359 + AuthorizationFree(authRef, kAuthorizationFlagDestroyRights)
shouldn't the Free be called in a finally clause?
In the InstallDaemonTestCase you can move the following to the setup instead of calling it in each of the tests:
469 + self._patch_and_track(utils.darwin, 470 + [('get_authorization', 'Fake AuthRef'), 471 + ('remove_fsevents_daemon', None), 472 + ('install_fsevents_daemon', None), 473 + ('AuthorizationFree', None)])
« Back to merge proposal
In the following code:
354 + try: fsevents_ daemon( authRef) exception( "Problem removing running daemon: %r" % e) ee(authRef, kAuthorizationF lagDestroyRight s)
355 + remove_
356 + except Exception, e:
357 + logger.
358 +
359 + AuthorizationFr
shouldn't the Free be called in a finally clause?
In the InstallDaemonTe stCase you can move the following to the setup instead of calling it in each of the tests:
469 + self._patch_ and_track( utils.darwin, authorization' , 'Fake AuthRef'), fsevents_ daemon' , None), fsevents_ daemon' , None), Free', None)])
470 + [('get_
471 + ('remove_
472 + ('install_
473 + ('Authorization