Feature request: #24

Reviewing now.

Thanks for the considered review! I think I've addressed the file comments. I need a bit of time to sort out the unit tests. I don't have TFS available to test against but I'll try to follow the existing tests.

I've added the unit tests along with the other requested changes. Will you please review the updates?

Thanks for the considered review! I think I've addressed the file comments. I need a bit of time to sort out the unit tests. I don't have TFS available to test against but I'll try to follow the existing tests.

I really appreciate your support on the project.

I will send you a DM and give you access to my TFS 2017 and 2018 test machines. What is your twitter handle?

TFS2017 supports variable groups with version 3.2-preview.1. I added to Set-VSTeamAPIVersion but I think the shape of the JSON might be different. I am willing to give you access to my 2017 and 2018. You can email me if you are not on twitter. I am dbrown at microsoft.

I sent you an email earlier in the week but I suspect it was caught in a spam filter so I've just now sent another with the title "VSTeam PullRequest 170". Meanwhile, I've updated the unit tests to use JSON samples that I collected using Invoke-VSTeamRequest calls. If possible would you please add a similar sample from VSTS 2017? If so, I'll update the unit test to use the new file and then fix any failing code as necessary.

Thanks for the access to the test system. I think I've addressed the TFS2017 differences with this last round of commits. I'll look forward to the next review.

Thanks for the access to the test system. I think I've addressed the TFS2017 differences with this last round of commits. I'll look forward to the next review.

Did you run the integration tests? None of them pass for the variable groups. If you open 000_team.Tests.ps1 file it will tell you which environment variables to set to run the tests. I will email you a Azure DevOps organization you can use to run these tests. DO NOT RUN on your own accounts as they delete everything! Expect email soon.

Thanks for your email with instructions on performing the integration tests, much time was saved. The integration tests on all three target types complete successfully now.

Well done! Your code is on it way to the PowerShell Gallery right now. Are you on Twitter? If so I can tag you in the announcement of the new version.

Thanks you very much! I'm not on Twitter but I am excited to put the changes to work.