Skip to content

Feature/73 migrate azdo unit tests to lunit#123

Merged
svelderrainruiz merged 7 commits into
ni:feature/73-migrate-azdo-unit-tests-to-lunitfrom
crossrulz:feature/73-migrate-azdo-unit-tests-to-lunit
Apr 25, 2025
Merged

Feature/73 migrate azdo unit tests to lunit#123
svelderrainruiz merged 7 commits into
ni:feature/73-migrate-azdo-unit-tests-to-lunitfrom
crossrulz:feature/73-migrate-azdo-unit-tests-to-lunit

Conversation

@crossrulz
Copy link
Copy Markdown

GitHub Issue for the Pull Request

Resolved #96
Resolved #97
Partially Resolved #108

Checklists

Completing these checklists ensures that your PR meets our build server requirements on the first attempt. If you're unable to complete any of the following checks, please submit your PR as a draft to the best of your ability. We will provide any clarification you may need after posting it.

  • [X ] I do not require assistance from NI to complete any of the following checks.
  • [X ] The changes in this PR are based on the appropriate NI-repo feature branch
  • [X ] I am submitting the changes in this PR to the appropriate NI-repo feature branch
  • I built a VI Package using the Powershell build tool.
  • I installed the VI Package produced by the Powershell build tool and tested my change.
  • I tested my changes after installing the VI package.
  • [X ] NI has my contributor license agreement.

Summary of Changes

Converted the tests already in the project to use LUnit
Converted AzDo tests to use LUnit
Tests added to Actor Framework Core.lvproj

Reason for Change

LUnit is the standard unit test framework for NI open source initiatives

Testing

All unit tests were passing after running inside of the project

Moved established unit tests into libraries to keep with support classes
Removed tests that were ported from Icon Editor
Tests include Basic Messaging, Priority Queue, and Self Messaging
Also implemented Batch Messaging test
@svelderrainruiz svelderrainruiz merged commit aafd2ac into ni:feature/73-migrate-azdo-unit-tests-to-lunit Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants