The changes support the fact the CF does not have a CurrentDirectory,

CF does not have a CurrentDirectory, and consequently, will not allow
relative paths or file names in any file or directory method.  These
fixes solve that problem.
The problem is that, if you remember, I use a totally different format (a more modern one).  So I have to use a program to reformat to the NUnit standard.  No formatting program will align using statements, because it is just not done anywhere :(  So I will have to go back and fix every source file by hand :(

 

From: CharliePoole [mailto:notifications@github.com] 
Sent: Saturday, November 22, 2014 9:09 AM
To: nunit/nunit
Cc: oznetmaster
Subject: Re: [nunit] The changes support the fact the CF does not have a CurrentDirectory, (#376)

 

In src/NUnitFramework/tests/Assertions/FileAssertTests.cs:

> @@ -57,88 +71,91 @@ public void AreEqualPassesWithSameStream()
>          [Test]
>          public void AreEqualPassesWithEqualStreams()
>          {
> -            using (new TestFile("Test1.jpg", "TestImage1.jpg"))
> -            using (new TestFile("Test2.jpg", "TestImage1.jpg"))
> -            using (FileStream expected = File.OpenRead("Test1.jpg"))
> -            using (FileStream actual = File.OpenRead("Test2.jpg"))
> -            {
> -                FileAssert.AreEqual(expected, actual);
> -            }
> +            using (new TestFile(TF("Test1.jpg"), "TestImage1.jpg"))
> +                using (new TestFile(TF("Test2.jpg"), "TestImage1.jpg"))
> +                    using (FileStream expected = File.OpenRead(TF("Test1.jpg")))
> +                        using (FileStream actual = File.OpenRead(TF("Test2.jpg")))

In any case, please don't do unrelated reformatting when you make a change. It makes the PR hard to review.

—
Reply to this email directly or view it on GitHub <https://github.com/nunit/nunit/pull/376/files#r20752571> .  <https://github.com/notifications/beacon/AFh64Q0G78tbBVX3ZpnEhxtoIIVGbM1Kks5nP9mUgaJpZM4C-3B0.gif>