Skip to content

Add initial test#60

Merged
sharwell merged 18 commits into
tunnelvisionlabs:masterfrom
jnm2:add_initial_test
Dec 30, 2019
Merged

Add initial test#60
sharwell merged 18 commits into
tunnelvisionlabs:masterfrom
jnm2:add_initial_test

Conversation

@jnm2

@jnm2 jnm2 commented Dec 29, 2019

Copy link
Copy Markdown
Contributor

Fixes #51

I did this really fast. Feel free to modify or replace.

@jnm2

jnm2 commented Dec 30, 2019

Copy link
Copy Markdown
Contributor Author

Looks like the build hangs on AppVeyor. Wonder what the difference is between that environment and my machine.

@sharwell

Copy link
Copy Markdown
Member

@jnm2 I'm going to push to this branch with some updates 👍

@jnm2

jnm2 commented Dec 30, 2019

Copy link
Copy Markdown
Contributor Author

Dotnet msbuild is not going to work unless you add the new netfx reference assembly NuGet package reference.

@sharwell
sharwell force-pushed the add_initial_test branch 7 times, most recently from 30c285f to cdc1dff Compare December 30, 2019 07:14
@sharwell
sharwell force-pushed the add_initial_test branch 3 times, most recently from a9dc875 to 405e1ce Compare December 30, 2019 07:33
@sharwell

Copy link
Copy Markdown
Member

@jnm2 I'll finish this in the morning and merge it. 😀

@sharwell
sharwell merged commit 232c3af into tunnelvisionlabs:master Dec 30, 2019
@jnm2
jnm2 deleted the add_initial_test branch December 30, 2019 15:00

<ItemGroup>
<!-- https://github.com/dotnet/core-sdk/issues/2022 -->
<PackageReference Condition="'$(TargetFramework)' == 'net35'" Include="jnm2.ReferenceAssemblies.net35" Version="1.0.0" PrivateAssets="all" />

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh you found it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did! 😄

Comment thread appveyor.yml
Comment on lines +14 to +49
test_script:
- git clean -dxf tests
- msbuild /restore tests/MultiTFM /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=net35 /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=net48 /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=netstandard1.6 /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=netstandard2.0 /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=netcoreapp1.0 /p:DisableImplicitNuGetFallbackFolder=true /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=netcoreapp1.1 /p:DisableImplicitNuGetFallbackFolder=true /warnaserror /v:m

- git clean -dxf tests
- dotnet msbuild -restore tests/MultiTFM -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=net35 -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=net48 -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=netstandard1.6 -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=netstandard2.0 -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=netcoreapp1.0 -p:DisableImplicitNuGetFallbackFolder=true -warnaserror -v:m
- taskkill /im dotnet.exe /f
- git clean -dxf tests
- dotnet msbuild -restore tests/SingleTFM -p:TestFramework=netcoreapp1.1 -p:DisableImplicitNuGetFallbackFolder=true -warnaserror -v:m

@jnm2 jnm2 Dec 31, 2019

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sharwell

I was looking for something I could queue up from the command line. Do you mind if I add back a .ps1 or .cmd? git clean -fxd test was something I was thinking about.

/nodeReuse:false or env MSBUILDDISABLENODEREUSE=1 can be used rather than taskkill IIRC.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are also recommended for CI:

But since it's important to tell people with CI servers and so on to disable node reuse, we should have clear documentation in the usual place.

dotnet/msbuild#2916

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But since it's important to tell people with CI servers and so on to disable node reuse, we should have clear documentation in the usual place.

This only applies to build servers that share state across builds. No CI server should ever be doing this (and neither AppVeyor nor Travis have this problem).

Comment thread appveyor.yml
- git clean -dxf tests
- msbuild /restore tests/MultiTFM /warnaserror /v:m
- git clean -dxf tests
- msbuild /restore tests/SingleTFM /p:TestFramework=net35 /warnaserror /v:m

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sharwell What would happen if this was /p:TargetFramework=net35 and <TargetFramework>$(TestFramework)</TargetFramework> was removed from the csproj?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No idea 🤷‍♂

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.

Builds should include tests

2 participants