Correct behavior of the Part remove feature in the test.

diff --git a/src/ModelAPI/ModelAPI_AttributeSelectionList.h b/src/ModelAPI/ModelAPI_AttributeSelectionList.h
index 7070744b6aa2a7dcf5402d029368f320806f9c16..454d9e34cf41874c6c3f14347f4fa4b4c5f955bb 100644 (file)
--- a/src/ModelAPI/ModelAPI_AttributeSelectionList.h
+++ b/src/ModelAPI/ModelAPI_AttributeSelectionList.h
@@ -81,7 +81,7 @@ class ModelAPI_AttributeSelectionList : public ModelAPI_Attribute
   /// Returns the attribute selection by the index (zero based)
   virtual std::shared_ptr<ModelAPI_AttributeSelection> value(const int theIndex) = 0;
 
-  /// Returns all attributes
+  /// Removes all attributes of the list.
   virtual void clear() = 0;
 
   /// Starts or stops cashing of the values in the attribute (the cash may become invalid

diff --git a/src/PartSetAPI/Test/TestRemoveDuplicate.py b/src/PartSetAPI/Test/TestRemoveDuplicate.py
index da4c94f8b82af8f63a61d5da244136b49360ec02..b66e18113d6d6380f261045cf6124de59843a699 100644 (file)
--- a/src/PartSetAPI/Test/TestRemoveDuplicate.py
+++ b/src/PartSetAPI/Test/TestRemoveDuplicate.py
@@ -47,6 +47,8 @@ for n in ["Part_1", "Part_3", "Part_2", "Part_4"]:
 
 # Remove the middle Part
 model.removePart(Copy_2)
+# move the history line to the end, because remove makes the current feature up
+PartSet.setCurrentFeature(Copy_3.feature(), False)
 model.do()
 
 # check the intermediate result
@@ -58,6 +60,8 @@ for n in ["Part_1", "Part_2", "Part_4"]:
 
 # Remove a first copy
 model.removePart(Copy_1)
+# move the history line to the end, because remove makes the current feature up
+PartSet.setCurrentFeature(Copy_3.feature(), False)
 model.end()
 
 # Undo the last remove