Entry Point Warnings

.coveragerc

@@ -1,5 +1,6 @@
 [run]
 source = featuretools
+omit = */featuretools_plugin/*
 
 [report]
 exclude_lines =

docs/source/changelog.rst

@@ -3,14 +3,16 @@
 Changelog
 ---------
 **Future Release**
+    * Changes
+        * Added Plugin Warnings at Entry Point (:pr:`850`)
     * Documentation Changes
         * Add links to primitives.featurelabs.com (:pr:`860`)
         * Add source code links to API reference (:pr:`862`)
     * Testing Changes
         * Miscellaneous changes (:pr:`861`)
 
     Thanks to the following people for contributing to this release:
-    :user:`rwedge`, :user:`thehomebrewnerd`
+    :user:`rwedge`, :user:`thehomebrewnerd`, :user:`jeff-hernandez`
 
 **v0.13.3 Feb 28, 2020**
     * Fixes

featuretools/__init__.py

@@ -1,5 +1,4 @@
 # flake8: noqa
-import warnings
 from .config_init import config
 from . import variable_types
 from .entityset.api import *
@@ -15,8 +14,13 @@
 from . import feature_base
 from .feature_base import AggregationFeature, DirectFeature, Feature, FeatureBase, IdentityFeature, TransformFeature, GroupByTransformFeature, save_features, load_features
 
+import logging
 import pkg_resources
 import sys
+import traceback
+
+logger = logging.getLogger('featuretools')
+
 # Call functions registered by other libraries when featuretools is imported
 for entry_point in pkg_resources.iter_entry_points('featuretools_initialize'):
     try:
@@ -31,4 +35,7 @@
     try:
         sys.modules["featuretools." + entry_point.name] = entry_point.load()
     except Exception:
-        pass
+        message = "Featuretools failed to load plugin {} from library {}. "
+        message += "For a full stack trace, set logging to debug."
+        logger.warn(message.format(entry_point.name, entry_point.module_name))
+        logger.debug(traceback.format_exc())

featuretools/tests/plugin_tests/featuretools_plugin/featuretools_plugin/__init__.py

@@ -0,0 +1 @@
+raise NotImplementedError('plugin not implemented')

featuretools/tests/plugin_tests/featuretools_plugin/setup.py

@@ -0,0 +1,11 @@
+from setuptools import setup
+
+setup(
+    name='featuretools_plugin',
+    packages=['featuretools_plugin'],
+    entry_points={
+        'featuretools_plugin': [
+            'module = featuretools_plugin',
+        ],
+    },
+)

featuretools/tests/plugin_tests/test_plugin.py

@@ -0,0 +1,20 @@
+from featuretools.tests.plugin_tests.utils import (
+    import_featuretools,
+    install_featuretools_plugin,
+    uninstall_featuretools_plugin
+)
+
+
+def test_plugin_warning():
+    install_featuretools_plugin()
+    warning = import_featuretools('warning').stdout.decode()
+    debug = import_featuretools('debug').stdout.decode()
+    uninstall_featuretools_plugin()
+
+    message = 'Featuretools failed to load plugin module from library featuretools_plugin'
+    traceback = 'NotImplementedError: plugin not implemented'
+
+    assert message in warning
+    assert traceback not in warning
+    assert message in debug
+    assert traceback in debug

featuretools/tests/plugin_tests/utils.py

@@ -0,0 +1,28 @@
+import os
+import subprocess
+import sys
+
+
+def import_featuretools(level=None):
+    c = ''
+    if level:
+        c += 'import os;'
+        c += 'os.environ["FEATURETOOLS_LOG_LEVEL"] = "%s";' % level
+
+    c += 'import featuretools;'
+    return python('-c', c)
+
+
+def install_featuretools_plugin():
+    pwd = os.path.dirname(__file__)
+    os.chdir(os.path.join(pwd, 'featuretools_plugin'))
+    return python('-m', 'pip', 'install', '-e', '.')
+
+
+def python(*args):
+    command = [sys.executable, *args]
+    return subprocess.run(command, stdout=subprocess.PIPE)
+
+
+def uninstall_featuretools_plugin():
+    return python('-m', 'pip', 'uninstall', 'featuretools_plugin', '-y')

setup.cfg

@@ -1,7 +1,7 @@
 [metadata]
 description-file = README.md
 [tool:pytest]
-addopts = --doctest-modules
+addopts = --doctest-modules --ignore=featuretools/tests/plugin_tests/featuretools_plugin
 python_files = featuretools/tests/*
 filterwarnings =
     ignore::DeprecationWarning