[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 Ralf Habacker changed: What|Removed |Added Version Fixed In|2.29.90 (KDE Applications |2.29.80 (KDE Applications |19.11.90) |19.11.80) -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #11 from Ralf Habacker --- Git commit bdda692fca25310dcafff7c9eb50ddfe92e947df by Ralf Habacker. Committed on 16/11/2019 at 00:03. Pushed by habacker into branch 'release/19.12'. Fixup of commit 5f2e7863aeeab89b9277c94072c8af31f950756c Subordinates references m_List not m_objects. M +1-1umbrello/uml1model/package.cpp https://commits.kde.org/umbrello/bdda692fca25310dcafff7c9eb50ddfe92e947df --- Comment #10 from Ralf Habacker --- Git commit 3c0be5a0eb85002ae1d4bc47828c7b6a8bb4dc34 by Ralf Habacker. Committed on 16/11/2019 at 00:03. Pushed by habacker into branch 'release/19.12'. UMLPackage is required by UMLClassifier so adding a unit case is useful M +7-0unittests/CMakeLists.txt A +65 -0unittests/testpackage.cpp [License: GPL (v2/3)] A +48 -0unittests/testpackage.h [License: GPL (v2/3)] https://commits.kde.org/umbrello/3c0be5a0eb85002ae1d4bc47828c7b6a8bb4dc34 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #10 from Ralf Habacker --- Git commit 3c0be5a0eb85002ae1d4bc47828c7b6a8bb4dc34 by Ralf Habacker. Committed on 16/11/2019 at 00:03. Pushed by habacker into branch 'release/19.12'. UMLPackage is required by UMLClassifier so adding a unit case is useful M +7-0unittests/CMakeLists.txt A +65 -0unittests/testpackage.cpp [License: GPL (v2/3)] A +48 -0unittests/testpackage.h [License: GPL (v2/3)] https://commits.kde.org/umbrello/3c0be5a0eb85002ae1d4bc47828c7b6a8bb4dc34 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #9 from Ralf Habacker --- Git commit edb241a4c1d0661361042a6217c69c4fb85cf39e by Ralf Habacker. Committed on 15/11/2019 at 18:03. Pushed by habacker into branch 'release/19.12'. Cleanup is...() methods for uml classifier and object unit tests Most of them are located in UMLObject. M +6-11 unittests/testclassifier.cpp M +0-2unittests/testclassifier.h M +66 -0unittests/testumlobject.cpp M +1-0unittests/testumlobject.h https://commits.kde.org/umbrello/edb241a4c1d0661361042a6217c69c4fb85cf39e -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #8 from Ralf Habacker --- Git commit aa24a8a0046738002dfd6b2b2fe4cf65b371d6a1 by Ralf Habacker. Committed on 14/11/2019 at 23:40. Pushed by habacker into branch 'release/19.12'. Add load and save unit test for UMLClassifier M +20 -0unittests/testclassifier.cpp M +1-0unittests/testclassifier.h https://commits.kde.org/umbrello/aa24a8a0046738002dfd6b2b2fe4cf65b371d6a1 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044
--- Comment #7 from Ralf Habacker ---
Git commit 5f2e7863aeeab89b9277c94072c8af31f950756c by Ralf Habacker.
Committed on 14/11/2019 at 09:15.
Pushed by habacker into branch 'release/19.12'.
Avoid to use non reentrant iteration as mentioned by Oliver in
UMLClassifier::resolveRef()
/
For reference, here is the non-reentrant iteration scheme -
NOT USE THIS !
for (UMLObject *obj = subordinates().first(); obj; obj = subordinates().next())
{ }
/
M +2-4umbrello/uml1model/entity.cpp
M +2-4umbrello/uml1model/folder.cpp
M +3-6umbrello/uml1model/package.cpp
M +6-17 umbrello/uml1model/umlcanvasobject.cpp
https://commits.kde.org/umbrello/5f2e7863aeeab89b9277c94072c8af31f950756c
--
You are receiving this mail because:
You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #6 from Ralf Habacker --- Git commit c083902ef178d592af4f7c4b54c484d8f57e263c by Ralf Habacker. Committed on 13/11/2019 at 16:08. Pushed by habacker into branch 'release/19.12'. Add super/sub class related tests to classifier unit tests M +19 -2unittests/testclassifier.cpp https://commits.kde.org/umbrello/c083902ef178d592af4f7c4b54c484d8f57e263c -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #5 from Ralf Habacker --- Git commit 7b422fc22f1f2a4873ab1ce7d624677d7aca6e3f by Ralf Habacker. Committed on 13/11/2019 at 15:48. Pushed by habacker into branch 'release/19.12'. Add unit test for class UMLCanvasObject Since UMLCanvasObject is a base class of UMLClassifier, some tests for UMLClassifier belong here. M +7-0unittests/CMakeLists.txt A +172 -0unittests/testumlcanvasobject.cpp [License: GPL (v2/3)] A +43 -0unittests/testumlcanvasobject.h [License: GPL (v2/3)] https://commits.kde.org/umbrello/7b422fc22f1f2a4873ab1ce7d624677d7aca6e3f -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #4 from Ralf Habacker --- Git commit e8927446120a64b49b7f69ccdebb5de1a20c5430 by Ralf Habacker. Committed on 13/11/2019 at 07:26. Pushed by habacker into branch 'release/19.12'. Use static objects instead of dynamic to avoid memory leaks M +11 -10 unittests/testclassifier.cpp https://commits.kde.org/umbrello/e8927446120a64b49b7f69ccdebb5de1a20c5430 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #3 from Ralf Habacker --- Git commit 331546aac0dcdde20e947a6768d05f4618e37a6d by Ralf Habacker. Committed on 11/11/2019 at 23:05. Pushed by habacker into branch 'release/19.12'. testclassifier: add tests for finding operation(s) M +2-1unittests/testbase.cpp M +69 -3unittests/testclassifier.cpp https://commits.kde.org/umbrello/331546aac0dcdde20e947a6768d05f4618e37a6d --- Comment #2 from Ralf Habacker --- Git commit fd39f64bb3967f710a20dc741a326ad99b620df5 by Ralf Habacker. Committed on 11/11/2019 at 23:02. Pushed by habacker into branch 'release/19.12'. Cleanup classifier unit test 1. Remove unused code 2. Show simple debug message for tests already performed by another unit test instead of showing a skipped message. M +2-32 unittests/testclassifier.cpp https://commits.kde.org/umbrello/fd39f64bb3967f710a20dc741a326ad99b620df5 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 --- Comment #2 from Ralf Habacker --- Git commit fd39f64bb3967f710a20dc741a326ad99b620df5 by Ralf Habacker. Committed on 11/11/2019 at 23:02. Pushed by habacker into branch 'release/19.12'. Cleanup classifier unit test 1. Remove unused code 2. Show simple debug message for tests already performed by another unit test instead of showing a skipped message. M +2-32 unittests/testclassifier.cpp https://commits.kde.org/umbrello/fd39f64bb3967f710a20dc741a326ad99b620df5 -- You are receiving this mail because: You are watching all bug changes.
[umbrello] [Bug 414044] Classifier unit case not enabled
https://bugs.kde.org/show_bug.cgi?id=414044 Ralf Habacker changed: What|Removed |Added Version Fixed In||2.29.90 (KDE Applications ||19.11.90) Status|REPORTED|RESOLVED Latest Commit||https://commits.kde.org/umb ||rello/9001925a05953d1284177 ||56780c461c656e9de2e Resolution|--- |FIXED --- Comment #1 from Ralf Habacker --- Git commit 9001925a05953d128417756780c461c656e9de2e by Ralf Habacker. Committed on 11/11/2019 at 21:46. Pushed by habacker into branch 'release/19.12'. Fix 'Classifier unit case not enabled' One problem was that comparing the list address of contained objects in UMLCanvasObject is incorrect. Instead, each object in this list must be compared. Also, the comparison of UMLClassifier pointers does not seem to give the desired result. Unimplemented tests are now skipped to avoid aborting the test. FIXED-IN:2.29.90 (KDE Applications 19.11.90) M +5-2umbrello/uml1model/umlcanvasobject.cpp M +6-6unittests/CMakeLists.txt M +85 -48 unittests/testclassifier.cpp https://commits.kde.org/umbrello/9001925a05953d128417756780c461c656e9de2e -- You are receiving this mail because: You are watching all bug changes.
