Skip to content

Commit 7124839

Browse files
committed
Add descriptions to new tests
Also adding the steel materials to the input objects for obejcts to be pushed in the DuplicateObjects_EnsureAllOutputHaveIds test. They should have been there in the first palce, but got missed when writing the test.
1 parent 6a02360 commit 7124839

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

.ci/unit-tests/BHoM_Adapter_Tests/PushTests.cs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -518,6 +518,8 @@ public void Preprocess_PanelLoadsDuplicateIds()
518518
}
519519

520520
[Test]
521+
[Description("Tests that objects being pushed are correctly 'merged' by calls to CopyProperties modules. \n" +
522+
"Two nodes pushed in the same location, one with a support and one without, the adapter should make sure the final node being sent for creation should contain the support.")]
521523
public void CopyProperties_NodesReplaced()
522524
{
523525
//Create bar from line. Nodes will have null-constraints on the Bar
@@ -539,6 +541,7 @@ public void CopyProperties_NodesReplaced()
539541
}
540542

541543
[Test]
544+
[Description("Tests that all objects sent to the push have AdapterIds assigned, even though some have been identified as duplicates and hence culled out.")]
542545
public void DuplicateObjects_EnsureAllOutputHaveIds()
543546
{
544547
//Create duplicate elements
@@ -563,7 +566,7 @@ public void DuplicateObjects_EnsureAllOutputHaveIds()
563566
Node node2 = new Node { Position = line.End };
564567

565568
//Push duplicates
566-
List<IBHoMObject> inputObjs = new List<IBHoMObject> { bar1, bar2, node1, node2, section1, section2 };
569+
List<IBHoMObject> inputObjs = new List<IBHoMObject> { bar1, bar2, node1, node2, section1, section2, steel1, steel2 };
567570
List<IBHoMObject> pushed = sa.Push(inputObjs).OfType<IBHoMObject>().ToList();
568571

569572
//Make sure correct number of items has been created to ensure comparers work.

0 commit comments

Comments
 (0)