Update unit test framework to NUnit #6

Merged
merged 13 commits into from Jan 25, 2013

Projects

None yet

2 participants

Contributor
myagley commented Jan 25, 2013

Updating to NUnit was required to get the client to build and test under mono.

@maddenpj maddenpj commented on the diff Jan 25, 2013
Client/TempoClient.csproj
@@ -37,7 +37,7 @@
<HintPath>..\packages\Newtonsoft.Json.4.5.7\lib\net40\Newtonsoft.Json.dll</HintPath>
</Reference>
<Reference Include="RestSharp">
- <HintPath>..\packages\RestSharp.103.1\lib\net4\RestSharp.dll</HintPath>
+ <HintPath>..\packages\RestSharp.104.1\lib\net4\RestSharp.dll</HintPath>
maddenpj
maddenpj Jan 25, 2013 Contributor

We should remember to indicate the rest sharp version bump when we deploy the package to nuget. Nuget page still shows 103

myagley
myagley Jan 25, 2013 Contributor

Yeah, I had to do that because the client and tests were using different versions of RestSharp and it wouldn't compile in mono. I could downgrade to 103.1 for the tests.

maddenpj
maddenpj Jan 25, 2013 Contributor

Was the client using 103 or 104? We might as well bump it up

myagley
myagley Jan 25, 2013 Contributor

Yeah, it was using 103, the tests were on 104.

maddenpj
maddenpj Jan 25, 2013 Contributor

Let's bump it up and see if anything breaks, I don't think 103 -> 104 is a breaking change

myagley
myagley Jan 25, 2013 Contributor

Ok, we can handle it when the release is cut.

@maddenpj maddenpj merged commit fbfe1bb into master Jan 25, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment