-
Notifications
You must be signed in to change notification settings - Fork 4
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
add tests #6
add tests #6
Conversation
|
||
|
||
[SetUpFixture] | ||
public class Setup |
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.
this code is stolen from @pinzart90's example in DynamoSamples repo.
<Nullable>enable</Nullable> | ||
|
||
<IsPackable>false</IsPackable> | ||
<IsTestProject>true</IsTestProject> |
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.
IsTestProject is set automatically by consuming the microsoft test.sdk nuget
Not sure why you need to specify IsPackable false
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.
these were set by VS when creating a test project from nunit template. Do you want me to remove them?
</PropertyGroup> | ||
|
||
<ItemGroup> | ||
<PackageReference Include="DynamoVisualProgramming.Tests" Version="3.1.0-beta3874" /> |
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.
Curious why the DynamoCoreRuntime used is 3.0.2 whereas the references are 3.1.
add test project
pull dynamo core from s3 and run tests.
move to net8