diff --git a/sphinx/ext/autosummary/__init__.py b/sphinx/ext/autosummary/__init__.py index ed49c56df..0cdd88feb 100644 --- a/sphinx/ext/autosummary/__init__.py +++ b/sphinx/ext/autosummary/__init__.py @@ -302,8 +302,7 @@ class Autosummary(SphinxDirective): with mock(self.config.autosummary_mock_imports): real_name, obj, parent, modname = import_by_name(name, prefixes=prefixes) except ImportError: - logger.warning(__('failed to import %s'), name) - items.append((name, '', '', name)) + logger.warning(__('autosummary: failed to import %s'), name) continue self.bridge.result = StringList() # initialize for each documenter diff --git a/tests/test_ext_autosummary.py b/tests/test_ext_autosummary.py index 365a5a53c..aa075a9e6 100644 --- a/tests/test_ext_autosummary.py +++ b/tests/test_ext_autosummary.py @@ -203,17 +203,16 @@ def test_autosummary_generate(app, status, warning): [autosummary_table, nodes.table, nodes.tgroup, (nodes.colspec, nodes.colspec, [nodes.tbody, (nodes.row, - nodes.row, nodes.row, nodes.row, nodes.row)])]) assert_node(doctree[4][0], addnodes.toctree, caption="An autosummary") + assert len(doctree[3][0][0][2]) == 4 assert doctree[3][0][0][2][0].astext() == 'autosummary_dummy_module\n\n' assert doctree[3][0][0][2][1].astext() == 'autosummary_dummy_module.Foo()\n\n' assert doctree[3][0][0][2][2].astext() == 'autosummary_dummy_module.Foo.Bar\n\n' assert doctree[3][0][0][2][3].astext() == 'autosummary_dummy_module.bar(x[, y])\n\n' - assert doctree[3][0][0][2][4].astext() == 'autosummary_importfail\n\n' module = (app.srcdir / 'generated' / 'autosummary_dummy_module.rst').read_text() assert (' .. autosummary::\n' @@ -397,7 +396,7 @@ def test_autosummary_template(app): confoverrides={'autosummary_generate': []}) def test_empty_autosummary_generate(app, status, warning): app.build() - assert ("WARNING: autosummary: stub file not found 'autosummary_importfail'" + assert ("WARNING: autosummary: failed to import autosummary_importfail" in warning.getvalue())