pyreverse does not show inheritance relation when using a flat folder

[DudeNr33]

Bug description

See this SO question.

Foo.py, Bar.py and __init__.py are all located in the same folder.

Foo.py:

class Foo():
    def print_foo(self):
        print("hi")

Bar.py:

from Foo import Foo

class Bar(Foo):
    pass

pyreverse is executed from within the same folder.

Configuration

No response

Command used

pyreverse -ASmy -o png .

Pylint output

The resulting image shows both classes `Foo` and `Bar`, but not the inheritance relationship between them.

Expected behavior

It could be argued that when using an __init__.py and therefore a package relative or absolute imports should be used, the code provided does run fine when just running e.g. python -c "import Bar" from inside the directory.

Therefore pyreverse should pick up the relationships between the two classes as well.

Pylint version

pylint 2.14.5
astroid 2.12.10
Python 3.10.5 (main, Jun  6 2022, 18:49:26) [GCC 12.1.0]

OS / Environment

No response

Additional dependencies

No response

[nhadziosma1]

Are there any updates on this issue?

[Pierre-Sassoulas]

No one worked on it yet, do you want to contribute @nhadziosma1 ? :)

[atomic-light]

Same issue here now. Are there any solutions? I can look try to contribute, but I'm not a pro though.

[Pierre-Sassoulas]

Hey @atomic-light sure we can answer your question if you have some. Imo the first step is to add the test for that. Also do not hesitate to open a draft merge request early it's easier to talk about the issue when seeing the code directly.

[DudeNr33]

The way I would go at this would probably be:

1. Create a new functional test by creating a new folder beneath tests/pyreverse/functional/class_diagrams and check if the problem stated in the opening post or linked SO question can be reproduced
2. Debug the test to see what is going on - I could imagine that especially the visit_importfrom method could be worthwhile to investigate.

   pylint/pylint/pyreverse/inspector.py

   Line 240 in 3f93f1e

   def visit_importfrom(self, node: nodes.ImportFrom) -> None: