Fix UnidentifiedShape I/O
This MR fixes the I/O of the GeoUnidentifiedShape. In particular, it fixes the restore, which was broken in two points:
- the UnidentifiedShape
name
parameter was read from the DB instead of theasciiData
, because the name was read twice (the index of the vector was not modified correctly) - the UnidentifiedShape parameters were used in the wrong order and the shape was constructed in the wrong way: the constructor, in fact, was called as
GeoUnidentifiedShape(asciiData, name)
instead of the correctGeoUnidentifiedShape(name, asciiData)
This MR also adds a new CTest unit test to test the write and restore of the GeoUnidentifiedShape node.
Edited by Riccardo Maria Bianchi
Merge request reports
Activity
added GeoModelIO Tests labels
enabled an automatic merge when the pipeline for 4a1936d0 succeeds
added 5 commits
-
1cd8d90f...c570e7bf - 2 commits from branch
main
- 31dc04a7 - Add unit test for GeoUnidentifiedShape
- e332ee67 - Fix load of UnidentifiedShape parameters from DB and their order in the constructor call
- 971cb6be - Update unit test for UnidentifiedShape
Toggle commit list-
1cd8d90f...c570e7bf - 2 commits from branch
Please register or sign in to reply