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

Add nuget package testing. #1080

Merged
merged 1 commit into from
Feb 15, 2018
Merged

Conversation

robward-ms
Copy link
Contributor

Description

This intention is to provide a small test project that can ingest the latest NuPkg's
from the build, restore them, build the project which results in a testable Dll. These
tests are very limited in scope, i.e. they only test the Dlls ability to load and
perform basic functions; the functional tests will test the full functionality.

Checklist (Uncheck if it is not completed)

  • Test cases added
  • Build and test with one-click build and test script passed

Additional work necessary

An additional build task will need to be added to update/compile/test this code.

<WarningLevel>4</WarningLevel>
</PropertyGroup>
<ItemGroup>
<Reference Include="Microsoft.OData.Client, Version=7.4.0.20111, Culture=neutral, PublicKeyToken=31bf3856ad364e35">
Copy link
Contributor

Choose a reason for hiding this comment

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

CR note: the hardcode here will be updated during automation

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct. However, it makes sense to checkin a lower version, say 7.0.0. Automation will update it but never checkin the update. I'll make a change here...

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed, that would be cleaner.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed. Core, Edm and Spatial point to 7.0.0, Client points to 7.4.0 since that is the earliest 7.x RTM version of the client.

This intention is to provide a small test project that can ingest the latest NuPkg's
from the build, restore them, build the project which results in a testable Dll. These
tests are very limited in scope, i.e. they only test the Dlls ability to load and
perform basic functions; the functional tests will test the full functionality.
@robward-ms robward-ms merged commit c145644 into OData:master Feb 15, 2018
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.

2 participants