roslyn-sdk
roslyn-sdk copied to clipboard
Update Testing README to add best practice for caching ReferenceAssemblies
Add a "best practice" and example of caching ReferenceAssemblies between tests. New users of the testing infrastructure often set the reference assemblies in a constructor, but don't use static or another method to share the instance between tests resulting in a lot of re-resolution and slow tests. As best I can tell this guidance isn't documented anywhere, but all the tests seem to do it.
@sharwell, this PR has some documentation changes that are also related to #1162.
Feel free to either take this PR as-is, or if you want the test framework stuff as part of your change I can rebase away that part and make this reference assembly specific.
Edit: I rebased away the XUnit stuff since that's already merged.
Hi @MattKotsenas,
Can you review #1176 to see if it meets your needs in a (hopefully) simpler fashion?
Thanks!
@MattKotsenas I believe this should no longer be necessary (per the previous comment). Please feel free to reopen if you find otherwise.
Sorry, this fell down my TODO list. This works for me! Thanks!