Refactor voltage level topology implementation

[geofjamg]

Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

Does this PR already have an issue describing the problem?

No

What kind of change does this PR introduce?

Refactoring

What is the current behavior?

The 2 topology models bus/breaker and node/breaker are implemented by 2 inherited classes from AbstractVoltageLevel.

What is the new behavior (if this is a feature change)?
A new TopologyModel concept has been added so that we only have one voltage level implementation class delegating every specific code to a topology model to a dedicated class.
Main advantages:

• cleaner design promoting composition over weak inheritance.
• for next devs, easy topology model convertion (switch from bus/breaker to node/breaker or inverse)

Does this PR introduce a breaking change or deprecate an API?

• Yes
• No

If yes, please check if the following requirements are fulfilled

• The Breaking Change or Deprecated label has been added
• The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

Noticeable changes:

• Abstract class AbstractVoltageLevel was replaced by non-abstract class VoltageLevelImpl.
• BusBreakerVoltageLevel and NodeBreakerVoltageLevel were removed. Their topology management methods were respectively moved in new classes BusBreakerTopologyModel and NodeBreakerTopologyModel, implementing a new ModelTopology interface.
• In the network index, voltage levels were mapped to either BusBreakerVoltageLevel.class or NodeBreakerVoltageLevel.class. They are now all mapped to VoltageLevelImpl.class.
• The following methods from VoltageLevelExt were removed. These operations can be performed on the TopologyModel, accessible via voltageLevel.getTopologyModel():
  • attach(TerminalExt, boolean)
  • detach(TerminalExt)
  • connect(TerminalExt, Predicate<? super SwitchImpl>)
  • disconnect(TerminalExt, Predicate<? super SwitchImpl>)
  • invalidateCache()
  • invalidateCache(boolean)

[sonarqubecloud]

Quality Gate passed

Issues
17 New issues
0 Accepted issues

Measures
0 Security Hotspots
83.4% Coverage on New Code
0.1% Duplication on New Code

See analysis details on SonarCloud