Skip to content
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

XML Updater unit tests - BugFix #145

Merged
merged 40 commits into from
Dec 8, 2021
Merged

Conversation

DStavilaNI
Copy link
Collaborator

What does this Pull Request accomplish?

  • Fixes an issue with incorrectly saved "setup" and "teardown" VIs, for the XML Updater unit tests

Why should this Pull Request be merged?

  • BugFix -> incorrect saving

What testing has been done?

  • affected unit tests can be loaded and are passing.

- Refactored Help File API
- Updated the XML Configuration file
 - Engine library refactoring
 - PPL Build Spec for the Engine
 - XML File update
…cluster array was set to fixed size when creating the subVI
…ntly available, in line with the future target support policy

- Updated the XML file
 - Code might still require refactoring and some additional error handling
…he RT Driver paths are now removed at Load time ( ActionVIOnLoad.vi .... if the paths exist) and are set dynamically before deployment (ActionVIOnCompile)

- Refactor of the "ActionVIOnLoad" to simplify the code.
- Updated the FPGA  Addon Configuration XML
# Conflicts:
#	Source/Addon/Addon Engine/RT Driver VI.vi
- Removed build support for LabVIEW 2018
- Replaced Linux cRIO with a Linux PXI x64 target
…tests for the 3.3.1 version of the FPGA Addon custom device
# Conflicts:
#	Source/FPGA Addon.lvproj
Karl-G1 and others added 10 commits November 30, 2021 13:54
* Fix LabVIEW package dependencies in control files for scripting
* Fix LabVIEW bitness install paths for 64-bit
* Update Jenkinsfile to prevent build failures due to diff call
- Added error handling API
- Some small refactoring for the API
- Renamed the test class
# Conflicts:
#	Jenkinsfile
# Conflicts:
#	Jenkinsfile
#	Source/Utilities/.LVLIBP CD XML Updater.vi
#	Source/Utilities/.LVLIBP CD XML Updater/.LVLIBP CD XML Updater.lvlib
#	Source/Utilities/.LVLIBP CD XML Updater/Project XML Editor/Destroy XML Editor.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Project XML Editor/Get Path Node VI.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Project XML Editor/Identify XML Descriptor.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Project XML Editor/Project XML Editor.lvclass
#	Source/Utilities/.LVLIBP CD XML Updater/Utilities/Copy .LVLIBP to NI VeriStand dir.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Utilities/Find LVLIBP Path.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Utilities/Find XML Config Path.vi
#	Source/Utilities/.LVLIBP CD XML Updater/Utilities/Get BuildSpec Type.vi
# Conflicts:
#	Source/Tests/Unit Tests/LVLIBP CD XML Updater/LVLIBP CD XML Updater/LVLIBP CD XML Updater Unit Tests.lvclass
#	Source/Tests/Unit Tests/LVLIBP CD XML Updater/LVLIBP CD XML Updater/setUp.vi
#	Source/Tests/Unit Tests/LVLIBP CD XML Updater/LVLIBP CD XML Updater/tearDown.vi
@DStavilaNI DStavilaNI requested a review from PaulDanH as a code owner December 8, 2021 10:52
@DStavilaNI DStavilaNI merged commit efaed20 into ni:ppl-migration Dec 8, 2021
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.

3 participants