Merge pull request #230 from mottosso/master

Fixes #229

pyblish/plugin.py

@@ -1088,9 +1088,16 @@ def discover(type=None, regex=None, paths=None):
                 continue
 
             module = types.ModuleType(mod_name)
+            module.__file__ = abspath
 
             try:
                 execfile(abspath, module.__dict__)
+
+                # Store reference to original module, to avoid
+                # garbage collection from collecting it's global
+                # imports, such as `import os`.
+                sys.modules[mod_name] = module
+
             except Exception as err:
                 log.debug("Skipped: \"%s\" (%s)", mod_name, err)
                 continue
@@ -1100,11 +1107,6 @@ def discover(type=None, regex=None, paths=None):
                     log.debug("Duplicate plug-in found: %s", plugin)
                     continue
 
-                # Store reference to original module, to avoid
-                # garbage collection from collecting it's global
-                # imports, such as `import os`.
-                plugin.__parent__ = module
-
                 plugins[plugin.id] = plugin
 
     # Include plug-ins from registration.

tests/test_plugin.py

@@ -452,3 +452,16 @@ def e():
 
     assert pyblish.api.Plugin.process.__code__.co_code == e.__code__.co_code
     assert pyblish.api.Plugin.repair.__code__.co_code == e.__code__.co_code
+
+
+def test_plugin_source_path():
+    """Plugins discovered carry a source path"""
+
+    import sys
+    plugin = pyblish.plugin.discover()[0]
+    module = sys.modules[plugin.__module__]
+    assert hasattr(module, "__file__")
+
+    # Also works with inspect.getfile
+    import inspect
+    assert inspect.getfile(plugin) == module.__file__