Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#4214 Add in automated testing for 32/64 bit nuget package #4369

Merged
merged 4 commits into from
Jul 23, 2021

Conversation

tijcolem
Copy link
Collaborator

@tijcolem tijcolem commented Jul 1, 2021

Pull request overview

This adds a simple csharp project that uses a local nuget package to compile and test using a windows .bat test_nuget.bat script. Here is an example of executing the .bat script.

set NUGET_PATH=D:/OSN/build&&^
set NUGET_NAME=OpenStudio&&^
set NUGET_VERSION=3.2.1-alpha&&^
set NUGET_SOURCE_DIR=D:/jenkins/nuget_packages&&^
test_nuget.bat

This tests both x86 and x64 nuget packages by compiling for each platform and then running each compiled .exe.

Ideally, this would be all rolled up into a CMake project. CMake does support nuget packages by using VS_PACKAGE_REFERENCES but this assumes that packages are available via nuget.org which get installed by msbuild during the configure steps. I couldn't find a way to install local .nupkg packages, like the test_nuget.bat script does.

The actual test runs will only be enabled for OpenStudio nightly builds. Incremental builds have this option to turned off to help speed up the builds which are already too slow. Here is a nightly build run that runs the test_nuget.bat script.

https://ci.commercialbuildings.dev/job/openstudio-develop-nightly/1153/execution/node/298/log/

This will get enabled when this PR is merged.

@tijcolem tijcolem requested a review from kbenne July 1, 2021 18:50
@tijcolem tijcolem added this to the OpenStudio SDK 3.3.0 milestone Jul 1, 2021
@tijcolem tijcolem merged commit 9198b9e into develop Jul 23, 2021
@tijcolem tijcolem deleted the nuget_test branch July 23, 2021 14:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add in automated testing for 32/64 bit nuget package
1 participant