-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added base NUnitTest class with LoadAssembly and Note reporting functionality #422
Added base NUnitTest class with LoadAssembly and Note reporting functionality #422
Conversation
To be removed from this PR, committed so they can be recovered in separate PR.
0b39f9e
to
4de4823
Compare
For clarity. This test project has two types of tests: - Tests against incorrectly referenced assemblies (e.g. wrong copy local) - Tests against test methods that depend on assemblies to be dynamically loaded, but are not referenced in the Test project.
@FraserGreenroyd to confirm, the following actions are now queued:
|
@FraserGreenroyd fix requested for copyright headers. The errors with the copyright headers on the CS ( I will apply the fixes to every case detailed on the checks tab. If you want to perform the fixes in a different manner please resolve this manually and rerun the check. Each CS ( If you are happy for me to go ahead and perform this action, please reply with:
|
@BHoMBot fix copyright headers ref. 11853368994 |
@FraserGreenroyd I have queued up your request to fix copyright headers. There are 0 requests in the queue ahead of you. |
@FraserGreenroyd I am now going to fix the copyright compliance in accordance with the annotations previously made. |
@FraserGreenroyd to confirm I have now resolved the copyright compliance and pushed a commit to this Pull Request. |
@BHoMBot check null-handling |
@FraserGreenroyd to confirm, the following actions are now queued:
|
@FraserGreenroyd to confirm, the following actions are now queued:
|
@FraserGreenroyd fix requested for copyright headers. The errors with the copyright headers on the CS ( I will apply the fixes to every case detailed on the checks tab. If you want to perform the fixes in a different manner please resolve this manually and rerun the check. Each CS ( If you are happy for me to go ahead and perform this action, please reply with:
|
@BHoMBot fix copyright headers ref. 11853831998 |
@FraserGreenroyd I have queued up your request to fix copyright headers. There are 0 requests in the queue ahead of you. |
@FraserGreenroyd I am now going to fix the copyright compliance in accordance with the annotations previously made. |
@FraserGreenroyd to confirm I have now resolved the copyright compliance and pushed a commit to this Pull Request. |
@FraserGreenroyd to confirm, the following actions are now queued:
|
FAO: @FraserGreenroyd The check they wish to have dispensation on is project-compliance. If you are providing dispensation on this occasion, please reply with:
|
@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 11854079970 |
@FraserGreenroyd I have now provided a passing check on reference |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issues addressed by this PR
Closes #420
This also lays the foundation for #421, however the resolution of #421 requires some refactoring as the required information to be plotted is stored in
testResult.InnerXml
(as opposed to the currently usedtestResult.OuterXml
). This means we need to change the Convert methods. Will be targeted by separate issue/PR.Test files
See
.ci/unit-test
. All tests must pass.Note that the
.ci
folder contains 3 separate testing projects, as required to appropriately test the added functionality.