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.

Sign up for GitHub

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

Jump to bottom

Added cmake tests that prevents people from changing message datatypes #337

Merged

yash-ni merged 23 commits into ni:master from yash-ni:capi-changes

Jan 18, 2024

Collaborator

yash-ni commented Jan 9, 2024 •

edited

Loading

Added a new python script that uses pytest to check whether any message metadata structure has been changed or not.
Added another script that checks for changes in exported function signatures as well.
Created a new custom target in cmakelists to run the tests and made it a dependency for server and generator dll build process.
Updated github workflows to run the tests.

yash-ni added 5 commits

January 9, 2024 17:59


          Added cmake tests for Message Structures

87d4682


          Updated github workflows

c9a9f24


          Corrected wrong meesage_metadata path

b0ab240


          Checking github workflows by changing structures

e25bb37


          Reverting changes

9e84f67

yash-ni requested review from CPattar-NI, ni-sujain and chethanravindranath

January 10, 2024 06:05

yash-ni added 6 commits

January 10, 2024 15:09


          Added MessageElementMetadata

4932e01


          Updated Building.md

b7dc903


          Added exported function checks

3dcd66e


          Added functionality to throw warning if user adds a new exported func…

a7c476b

…tion


          Merge branch 'master' of https://github.com/ni/grpc-labview into capi…

af2bad1

…-changes


          Updated exported function list

a50fc6e

ni-sujain reviewed

View reviewed changes

.github/workflows/windows_x86_build.yml Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

tests/CMakeTests/requirements.txt Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

tests/CMakeTests/tests/exported_functions_test.py Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

tests/CMakeTests/tests/exported_functions_test.py Outdated Show resolved Hide resolved

ni-sujain reviewed

View reviewed changes

tests/CMakeTests/tests/message_structure_test.py Outdated Show resolved Hide resolved

ni-sujain approved these changes

View reviewed changes

Collaborator

ni-sujain left a comment

do some testing on getexported functions and then submit

yash-ni added 7 commits

January 17, 2024 15:48


          Renamed some variables and replaced os.path with pathlib

ba1324f


          Fixed pathlib error

58bb332


          Probably fixed weird subprocess bug

6624d5c


          Added python virtual environment

96d9b91


          Fixed /bin/sh bug

fa8152f


          Updated github windows workflows to print CMakeTests logs

4352bff


          Removed extra new lines from logs

eb74d3d

yash-ni and others added 5 commits

January 18, 2024 14:15


          Modified github workflows to print CMakeTest logs

82c423f


          Updated linux print logs to exit instead of throwing error

74c891b


          Updated folder in linux workflows name

91fa9eb


          Corrected formatting for linux

6c2a3e3


          Merge branch 'master' into capi-changes

9f82f46

yash-ni merged commit a5226b7 into ni:master

5 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet