Depends on D1748
- Maniphest Tasks
- T3193: Add a plugin system
- rPTVca40c6bb5fc2: pluginmanager: Write and read activated plugins from the configuration file
if plugin_info not in self.plugins: self.warning("Plugin missing: %s", plugin_name) continue self.engine.load_plugin(plugin_info)
"""Handles the addition of a window to the application."""
The comment does not make sense. Storing an empty list is a valid use-case. The default value could be very well a non-empty list. Maybe extend the unit test with this case?
I would just call the signal handler directly from the test. It's too much noise to have a custom App class, and it does not add much. I would remove it, but your choice.