← Back to team overview

zeitgeist team mailing list archive

[Bug 695297] [NEW] unused helper ZeitgeistRemoteDataSourceRegistryTest._register_data_source_python_api() is broken

 

Public bug reported:

I'm talking about

	def _register_data_source_python_api(self, *args):
		mainloop = gobject.MainLoop()
		self.client.register_data_source(*args,
			reply_handler=lambda: mainloop.quit(),
			error_handler=lambda: fail("Error registering data-source"))
		mainloop.run()

* In error_handler it should be   self.fail(...)
* register_data_source() has no reply_handler/error_handler arguments

On the other hand, this method isn't used anywhere

@Siegfried, I think you added this code recently, can you please fix it.
Thanks so much ;)

** Affects: zeitgeist
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Zeitgeist
Framework Team, which is subscribed to Zeitgeist Framework.
https://bugs.launchpad.net/bugs/695297

Title:
  unused helper ZeitgeistRemoteDataSourceRegistryTest._register_data_source_python_api() is broken

Status in Zeitgeist Framework:
  New

Bug description:
  I'm talking about 

	def _register_data_source_python_api(self, *args):
		mainloop = gobject.MainLoop()
		self.client.register_data_source(*args,
			reply_handler=lambda: mainloop.quit(),
			error_handler=lambda: fail("Error registering data-source"))
		mainloop.run()

* In error_handler it should be   self.fail(...)
* register_data_source() has no reply_handler/error_handler arguments

On the other hand, this method isn't used anywhere

@Siegfried, I think you added this code recently, can you please fix it. Thanks so much ;)





Follow ups

References