BHoM_Engine_Tests: Align with Test_Toolkit changes

[alelom]

NOTE: Depends on

BHoM/Test_Toolkit#422 (already merged)
BHoM/Test_Toolkit#445 (already merged)

Issues addressed by this PR

Closes #3014
Closes #3059

Existing unit test now have their class inheriting from the Test_Toolkit NUnitTest abstract class, which provides them with dynamic assembly loading and reporting of event log into the results of the test.

Additionally, a new Structure_Engine_Test project was added with a specific test on a method that requires dynamic assembly loading in order to work (it uses an extension method). This was added mainly to verify that the functionality added via the NUnitTest class works, but it is also a useful test in itself which justifies keeping it in. @FraserGreenroyd

Test files

Unit tests must all pass, both from Test Explorer in Visual Studio and from an NUnit TestRunner environment (BHoMBot).

Changelog

Additional comments

[alelom]

@BHoMBot check unit-tests

[bhombot-ci]

@alelom to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting force checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I am now running force checks on the checks requested. These will be queued and actioned ASAP.

[bhombot-ci]

@FraserGreenroyd I have provided force checks to the checks requested.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting force checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I am now running force checks on the checks requested. These will be queued and actioned ASAP.

[bhombot-ci]

@FraserGreenroyd I have provided force checks to the checks requested.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting force checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I am now running force checks on the checks requested. These will be queued and actioned ASAP.

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting force checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I am now running force checks on the checks requested. These will be queued and actioned ASAP.

[bhombot-ci]

@FraserGreenroyd I have provided force checks to the checks requested.

[FraserGreenroyd]

@BHoMBot check unit-tests

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check unit-tests

There are 19 requests in the queue ahead of you.

[FraserGreenroyd]

@alelom I see no more reason to hold up this PR so I'm going to go for merge - the failures are from Data Driven unit tests, and we've previously tested this heavily - so I think we're good to go for merging this one and continuing developments of tests.

[FraserGreenroyd]

@BHoMBot check compliance

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

There are 34 requests in the queue ahead of you.

[FraserGreenroyd]

@alelom I've removed that this PR takes care of #3018 because this PR isn't doing that anymore so we'll need to tackle #3018 later 😄

[FraserGreenroyd]

@BHoMBot check project-compliance

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check project-compliance

There are 2 requests in the queue ahead of you.

[FraserGreenroyd]

@BHoMBot check copyright-compliance

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check copyright-compliance

There are 2 requests in the queue ahead of you.

[FraserGreenroyd]

@BHoMBot check copyright-compliance
@BHoMBot check project-compliance

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check copyright-compliance
• check project-compliance

There are 1 requests in the queue ahead of you.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: versioning, unit-tests, serialisation, ready-to-merge, null-handling, installer, documentation-compliance, dataset-compliance, core, code-compliance

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.

[FraserGreenroyd]

Reviewed over time with @alelom - some things to resolve later but this PR is in a good enough state to close out and merge in for future testing work.