Improve test on voltage level detail display #621
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce?
Bug fix
What is the current behavior?
In the
TextNodeTest
class, the customgetLabelProvider
function, defined in theTextNodeTest
class returns thelabelProvider
variable if this variable is notnull
or a customizedLabelProvider
if thelabelProvider
variable is null :In the
TextNodeTest
>testSubstationDescription
test, thelabelProvider
variable was instantiated with anew DefaultLabelProvider(...)
and thus notnull
.Because of that, the customized
LabelProvider
, with its customizedgetVoltageLevelDetails
method is not used.It seems like it should be, in order to test the
setVoltageLevelDetails(true)
configuration.What is the new behavior (if this is a feature change)?
The instantiation of the
labelProvider
variable is removed so that the customizedLabelProvider
is used.The test resource file is modified (voltage level details now appear in the text box).
Does this PR introduce a breaking change or deprecate an API?