Skip to content

Move tests to Newtonsoft.Json v7.0.1 #74

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

Merged
merged 1 commit into from
Oct 20, 2017
Merged

Conversation

dougbu
Copy link
Contributor

@dougbu dougbu commented Oct 19, 2017

@dnfclas
Copy link

dnfclas commented Oct 19, 2017

@dougbu,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

- #72 and also related to #65
- v7.0.1 is the next major version after what the product code is built against (v6.0.4)
  - this version corrects test failures I saw on a machine with .NET 4.7 (#72)
- add binding redirects and include the app.config files in test projects
- avoid new `TypeLoadException`s for Newtonsoft.Json that break numerous tests
  - consistently disable Copy Local (aka `%(Private)` metadata) in `src` projects
  - add Newtonsoft.Json references in test projects that bring it in transitively
    - otherwise, product code is last to mention Newtonsoft.Json assembly
@dougbu dougbu force-pushed the dougbu/test.failures.72 branch from 5b521c1 to e4c8e01 Compare October 20, 2017 04:05
@dougbu dougbu merged commit e4c8e01 into master Oct 20, 2017
@dougbu
Copy link
Contributor Author

dougbu commented Oct 20, 2017

e4c8e01

@dougbu dougbu deleted the dougbu/test.failures.72 branch October 20, 2017 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants