Add Pyreverse option to exclude standalone nodes

[nickdrozd]

Type of Changes

	Type
✓	✨ New feature

Description

Closes #8476

[nickdrozd]

From manual test, this seems to work fine. I have no idea how the Pyreverse tests work.

[DudeNr33]

From manual test, this seems to work fine. I have no idea how the Pyreverse tests work.

For class diagrams you can use the functional tests similar to how the regular test_functional.py works for messages:

• choose or create a directory under tests/pyreverse/functional/class_diagrams
• Create a Python file with sample code, e. g. no_standalone.py
• In your case which requires special command line options: create a no_standalone.rc, and add command_line_args=<your args>
• Create a corresponding expected output file (default format is MermaidJS) with the same name, i.e. no_standalone.mmd

Unfortunately we don't have a similar concept yet for package diagrams. You can take a look at test_writer.py, which creates class and package diagrams from the files located under tests/data, and add a suitable fixture for a config including the new option there. As far as I see there should already be suitable classes and "packages" (modules also count as packages here) present that should be omitted with --no-standalone.

[codecov]

Codecov Report

Merging #8520 (80299c1) into main (e9894d3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8520   +/-   ##
=======================================
  Coverage   95.90%   95.90%           
=======================================
  Files         174      174           
  Lines       18357    18362    +5     
=======================================
+ Hits        17605    17610    +5     
  Misses        752      752           

Impacted Files	Coverage Δ
pylint/pyreverse/main.py	93.87% <ø> (ø)
pylint/pyreverse/writer.py	100.00% <100.00%> (ø)
pylint/testutils/pyreverse.py	98.21% <100.00%> (+0.03%)	⬆️

[nickdrozd]

@DudeNr33 I keep getting a FileNotFoundError in the package test. Any ideas? There must be some obvious mistake somewhere.

[DudeNr33]

You have to specify name here like above in setup_colorized_dot. Also you have to rename the files from no_standalone_classes.dot to classes_no_standalone.dot (same for package diagram).

[github-actions]

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit 80299c1

[DudeNr33]

Looks good to me, thank you for the addition! 👍